linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: mbizon@freebox.fr
Cc: Richard Cochran <richardcochran@gmail.com>,
	Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"ksummit-2013-discuss@lists.linuxfoundation.org" 
	<ksummit-2013-discuss@lists.linuxfoundation.org>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Ian Campbell <ian.campbell@citrix.com>,
	Pawel Moll <Pawel.Moll@arm.com>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Domenico Andreoli <cavokz@gmail.com>,
	"rob.herring@calxeda.com" <rob.herring@calxeda.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Jason Gunthorpe <jgunthorpe@obsidianresearch.com>,
	Olof Johansson <olof@lixom.net>, Mark Brown <broonie@kernel.org>,
	Arend van Spriel <arend@broadcom.com>,
	Dave P Martin <Dave.Martin@arm.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [Ksummit-2013-discuss] DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?]
Date: Wed, 31 Jul 2013 12:59:59 +0200	[thread overview]
Message-ID: <1479410.BSGDrOcRvP@thinkpad> (raw)
In-Reply-To: <1375267057.20042.52.camel@sakura.staff.proxad.net>

On Wednesday 31 of July 2013 12:37:37 Maxime Bizon wrote:
> On Sat, 2013-07-27 at 11:51 -0700, Tomasz Figa wrote:
> > Well, it depends on how we use the DT. There are (at least) two possible
> > 
> > usage scenarios:
> >  a) using DT as direct replacement for board files - this means that you
> >  
> >     are free to say that DTSes are strictly coupled with kernel version
> >     and you are free to modify the bindings - see the analogy to board
> >     files, where you could modify the platform data structures and could
> >     not directly copy board file from one kernel version to another,
> 
> I'm shocked to see this as a possible option.
> 
> Board files are C code anyone has the skill to edit/understand/refactor.
> Moving to DT and keep them in tree tightly coupled with the kernel
> version just adds another layer of indirection for *no purpose*.

Well, I agree that it's not the most fortunate scenario, but it is not that 
bad as it might seem. It still has the benefit of separating hardware data 
(including board-specific data) from kernel code and has all the benefits of DT, 
e.g. easy way to specify relations between devices (phandles), no need to 
manually register any resources, platform data or platform devices, in other 
words, when describing hardware you just care about the hardware, not how the 
kernel works.

> The fact that we loose compiler syntax/type checking (as highlighted
> somewhere else in this thread) even looks like a regression.

Syntax checking is already there. We don't have semantic checking yet, but we 
are working on this.

> >  b) using DT as an ABI - this is the original way, i.e. define stable
> >  
> >     bindings and make sure that anu DTB built for older kernel will
> > 
> > work, with equal or greater set of functionality on newer kernels.
> 
> Linus started the whole thing some years ago by refusing to pull ARM
> tree [1]. Reread his post, what he wants is clearly b).
> 
> Going a) does not solve any problem. You are just moving churn to
> somewhere else. We had board files churn, then defconfigs churn, DTS
> files (and associated drivers) will be next.
> 
> DT is self inflicted pain. It has to be for the greater good.

It has several benefits over board files that I mentioned above, possible 
without fully separating them from kernel tree.

> Going b) *might* allow what some people here dream about, a kernel free
> of hardware knowledge. A new board could boot a kernel compiled before
> it was even designed.

This is very unlikely to happen. You can already see problems with defining 
bindings for existing hardware and what to say about hardware that is not even 
designed yet? Unless all the board uses is a set of already supported 
components, that have bindings defined and drivers merged, you would still need 
to provide remaining drivers and bindings for them.

> Now since I do "embedded" stuff everyday, I don't think b) can apply to
> the whole ARM world. There is just to much hardware peculiarity.

That's why I think we need a hybrid solution. Staging bindings could use a) 
and those stable one would use b). Staging bindings would be fixed over time if 
needed and if they turn out to be fine, they can be stabilized and move to b).

So basically, you can get all the stable functionality with one DTB on any 
kernel released after the functionality got stable, but during development of 
some functionality you might need to change the DTB to test the development.

Best regards,
Tomasz
 
> [1] https://lkml.org/lkml/2011/3/30/525

  reply	other threads:[~2013-07-31 10:59 UTC|newest]

Thread overview: 142+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-25 16:09 DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?] Olof Johansson
2013-07-25 17:57 ` Mark Rutland
2013-07-25 18:05   ` Stephen Warren
2013-07-25 18:25     ` Olof Johansson
2013-07-25 18:50       ` Stephen Warren
2013-07-25 19:30       ` [Ksummit-2013-discuss] " Mark Brown
2013-07-25 18:29     ` Mark Rutland
2013-07-25 18:48       ` Richard Cochran
2013-07-25 18:53         ` Stephen Warren
2013-07-25 21:41           ` Domenico Andreoli
2013-07-26  4:43           ` Richard Cochran
2013-07-25 21:37         ` Jason Gunthorpe
2013-07-26  4:54           ` Richard Cochran
2013-07-26 13:42             ` Rob Herring
2013-07-26 17:15             ` Jason Gunthorpe
2013-07-27  8:48               ` Richard Cochran
2013-07-27 10:40                 ` [Ksummit-2013-discuss] " Mark Brown
2013-07-27 17:37                   ` Richard Cochran
2013-07-27 17:57                     ` David Lang
2013-07-27 18:17                       ` Richard Cochran
2013-07-28 11:27                     ` Mark Brown
2013-07-29 17:54                 ` Jason Gunthorpe
2013-07-29 18:16                   ` Russell King - ARM Linux
2013-07-29 18:40                     ` Jason Gunthorpe
2013-07-27  8:53               ` Richard Cochran
2013-07-27 10:20                 ` Tomasz Figa
2013-07-27 11:36                 ` [Ksummit-2013-discuss] " Mark Brown
2013-07-27 18:07                   ` Richard Cochran
2013-07-28 12:10                     ` Mark Brown
2013-07-26  8:01           ` Richard Cochran
2013-07-26  9:42             ` [Ksummit-2013-discuss] " David Woodhouse
2013-07-26 13:09               ` Richard Cochran
2013-07-26 13:14                 ` Russell King - ARM Linux
2013-07-27 15:28                   ` David Gibson
2013-07-26 13:27                 ` Jason Cooper
2013-07-26 13:38                   ` Russell King - ARM Linux
2013-07-26 13:45                     ` Jason Cooper
2013-07-26 13:50                       ` David Woodhouse
2013-07-26 13:59                         ` Jason Cooper
2013-07-26 13:41                   ` David Woodhouse
2013-07-26 13:47                     ` Jason Cooper
2013-07-26 14:14                     ` jonsmirl
2013-07-26 14:16                       ` David Woodhouse
2013-07-26 14:21                       ` Russell King - ARM Linux
2013-07-26 14:39                         ` jonsmirl
2013-07-26 14:40                         ` Mark Brown
2013-07-29 15:32                           ` Matt Porter
2013-07-27 15:19                         ` David Gibson
2013-07-27 19:37                         ` Grant Likely
2013-07-27  9:16                       ` Ming Lei
2013-07-26 14:10                 ` Mark Brown
2013-07-26 15:49                   ` Olof Johansson
2013-07-26 16:36                     ` Rob Herring
2013-07-27  4:57                       ` Richard Cochran
2013-07-27  5:04                     ` Richard Cochran
2013-07-27  9:51                       ` Tomasz Figa
2013-07-27 10:24                         ` Arend van Spriel
2013-07-27 10:34                           ` Arend van Spriel
2013-07-27 11:09                             ` Russell King - ARM Linux
2013-07-27 10:36                           ` Tomasz Figa
2013-07-27 10:59                             ` Arend van Spriel
2013-07-27 19:45                               ` Grant Likely
2013-07-27 20:01                                 ` jonsmirl
2013-07-27 20:03                                   ` David Woodhouse
2013-07-27 20:25                                   ` Grant Likely
2013-07-28  3:28                                     ` Grant Likely
2013-07-28  5:11                                       ` James Bottomley
2013-07-28  5:13                                         ` Grant Likely
2013-07-30  1:44                                         ` David Gibson
2013-07-30  2:15                                           ` jonsmirl
2013-07-30  3:29                                             ` David Gibson
2013-07-30  4:35                                               ` Grant Likely
2013-07-30  8:39                                                 ` Maxime Ripard
2013-07-30 16:30                                             ` Stephen Warren
2013-07-30 17:27                                               ` John W. Linville
2013-07-30 16:40                                           ` Stephen Warren
2013-07-29  9:19                                   ` Arend van Spriel
2013-07-29 10:16                                     ` Arend van Spriel
2013-07-27 18:31                           ` Richard Cochran
2013-07-27 18:51                             ` Tomasz Figa
2013-07-27 19:22                               ` jonsmirl
2013-07-28  8:56                               ` Richard Cochran
2013-07-28  9:12                                 ` Tomasz Figa
2013-07-28 13:19                                   ` Richard Cochran
2013-07-28 13:39                                     ` Tomasz Figa
2013-07-28 14:09                                       ` jonsmirl
2013-07-28 15:35                                         ` Richard Cochran
2013-07-28 15:50                                           ` jonsmirl
2013-07-28 17:41                                             ` Mark Brown
2013-07-28 21:46                                           ` David Gibson
2013-07-28 15:30                                       ` Richard Cochran
2013-07-29  7:31                                     ` Maxime Ripard
2013-07-29 18:38                                       ` Richard Cochran
2013-07-29 18:51                                         ` Richard Cochran
2013-07-29 18:05                               ` Jason Gunthorpe
2013-07-29 18:27                                 ` Russell King - ARM Linux
2013-07-29 22:20                                 ` David Gibson
2013-07-29 23:14                                   ` Jason Gunthorpe
2013-07-29 23:49                                     ` David Gibson
2013-07-31 10:37                               ` Maxime Bizon
2013-07-31 10:59                                 ` Tomasz Figa [this message]
2013-07-31 15:07                                   ` Richard Cochran
2013-07-31 15:23                                     ` Tomasz Figa
2013-07-31 19:12                                       ` Richard Cochran
2013-07-31 19:29                                         ` Tomasz Figa
2013-07-31 20:00                                           ` Richard Cochran
2013-07-31 20:14                                             ` Russell King - ARM Linux
2013-07-31 20:37                                               ` jonsmirl
2013-07-31 20:48                                                 ` Russell King - ARM Linux
2013-07-31 21:26                                                   ` jonsmirl
2013-08-01  9:57                                                     ` Arend van Spriel
2013-08-01 10:09                                                       ` Mark Brown
2013-08-01 10:18                                                     ` David Woodhouse
2013-08-01 13:34                                                       ` jonsmirl
2013-08-01 13:43                                                         ` jonsmirl
2013-08-01 13:48                                                       ` Rob Herring
2013-08-02  3:50                                                     ` David Gibson
2013-08-13 23:32                                                       ` H. Peter Anvin
2013-08-14  1:49                                                         ` Guenter Roeck
2013-08-14 14:46                                                           ` H. Peter Anvin
2013-08-02  8:49                                               ` Tony Lindgren
2013-07-29 15:45                     ` Matt Porter
2013-07-31 12:58   ` Ian Campbell
2013-07-25 17:59 ` Stephen Warren
2013-07-25 19:11 ` Rob Herring
2013-07-25 19:31   ` Jason Cooper
2013-07-25 20:04     ` [Ksummit-2013-discuss] " Mark Brown
2013-07-25 20:08       ` Jason Cooper
2013-07-25 20:16     ` Rob Herring
2013-07-25 20:32       ` Jason Cooper
2013-07-25 21:53         ` [Ksummit-2013-discuss] " Ben Hutchings
2013-07-26  0:36           ` Stephen Warren
2013-07-26 12:14           ` Jason Cooper
2013-07-26  0:34       ` Stephen Warren
2013-07-26 22:11         ` Rob Herring
2013-07-25 23:18     ` Russell King - ARM Linux
2013-07-26  0:27       ` jonsmirl
2013-07-26 11:38         ` Jason Cooper
2013-07-26 14:08           ` [Ksummit-2013-discuss] " David Woodhouse
2013-07-26 13:57       ` Mark Brown
2013-07-26 10:55   ` Mark Brown
2013-07-28  4:39 ` Grant Likely

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1479410.BSGDrOcRvP@thinkpad \
    --to=tomasz.figa@gmail.com \
    --cc=Dave.Martin@arm.com \
    --cc=Pawel.Moll@arm.com \
    --cc=arend@broadcom.com \
    --cc=broonie@kernel.org \
    --cc=cavokz@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=ian.campbell@citrix.com \
    --cc=jgunthorpe@obsidianresearch.com \
    --cc=ksummit-2013-discuss@lists.linuxfoundation.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=mbizon@freebox.fr \
    --cc=olof@lixom.net \
    --cc=richardcochran@gmail.com \
    --cc=rob.herring@calxeda.com \
    --cc=swarren@wwwdotorg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).