All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Arend van Spriel" <arend@broadcom.com>
To: "Tomasz Figa" <tomasz.figa@gmail.com>
Cc: "Richard Cochran" <richardcochran@gmail.com>,
	"Olof Johansson" <olof@lixom.net>,
	"Mark Brown" <broonie@kernel.org>,
	"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>,
	"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: Sat, 27 Jul 2013 12:24:24 +0200	[thread overview]
Message-ID: <51F39FD8.6080808@broadcom.com> (raw)
In-Reply-To: <2007664.vYsECFSKrV@flatron>

On 07/27/2013 11:51 AM, Tomasz Figa wrote:
> On Saturday 27 of July 2013 07:04:08 Richard Cochran wrote:
>> On Fri, Jul 26, 2013 at 08:49:43AM -0700, Olof Johansson wrote:
>>> Long term, final goal is likely to be close to what Russell is saying
>>
>> Why is this a long term goal? Start today.
>>
>>> -- nothing should go into the kernel tree unless the binding is in a
>>> fully stable state. However, we have a transitional period between now
>>> and then, and even when we're at the final state there will be need to
>>> have some sort of sandbox for development and test of future bindings.
>>
>> Why not just set up a git tree right away?
>>
>>> Dealing with all that, as well as the actual process for locking in
>>> bindings, is what needs to be sorted out.
>>>
>>> I think we're all in agreement that bindings that change over time are
>>> nothing but pain, but we're arguing that in circles anyway.
>>
>> No.
>>
>> I keep saying, the bindings must be stable ABI, *today*.
>>
>> You keep saying, maybe later, but until then we will make things up as
>> we go along.
>
> We have currently a lot of broken bindings, because people didn't know how
> to define ones and those they defined have not been properly reviewed. Do
> you really want such broken ABI in the kernel?
>
> Sure, there are many existing bindings that can be just made stable and
> well they probably are already de facto stable. This is mostly about
> subsystem bindings and whatever already has many users, both made them get
> more thought when designing and more review before merging.
>
> Still, a lot of device and platform-specific bindings are simply broken.
> Take max8925 backlight driver, that Olof started this whole discussion
> with, as an example. We need to sort them out before they can be
> stabilized.

That is a nice summary of how we got from null to now and Richard seems 
to be simply saying: let's stop mucking about and make this a project 
with a well-defined process of dealing with staging and stable bindings 
and keep stable bindings stable. Whether it should be within the kernel 
repo as a separate subsystem or in an entire different repo is a trivial 
decision, but still a decision that needs to be made.

Apart from stable DT bindings I would love to see a DT compiler that 
that next to DT syntax detects mistakes in properties used for the 
selfish reason that I spent hours debugging regulator code, because I 
typed vmmc_supply iso vmmc-supply. I did not go through all the 
bindings, but this may require a more formal description so it could be 
compiled/read in the DT compiler.

Regards,
Arend


WARNING: multiple messages have this Message-ID (diff)
From: arend@broadcom.com (Arend van Spriel)
To: linux-arm-kernel@lists.infradead.org
Subject: [Ksummit-2013-discuss] DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?]
Date: Sat, 27 Jul 2013 12:24:24 +0200	[thread overview]
Message-ID: <51F39FD8.6080808@broadcom.com> (raw)
In-Reply-To: <2007664.vYsECFSKrV@flatron>

On 07/27/2013 11:51 AM, Tomasz Figa wrote:
> On Saturday 27 of July 2013 07:04:08 Richard Cochran wrote:
>> On Fri, Jul 26, 2013 at 08:49:43AM -0700, Olof Johansson wrote:
>>> Long term, final goal is likely to be close to what Russell is saying
>>
>> Why is this a long term goal? Start today.
>>
>>> -- nothing should go into the kernel tree unless the binding is in a
>>> fully stable state. However, we have a transitional period between now
>>> and then, and even when we're at the final state there will be need to
>>> have some sort of sandbox for development and test of future bindings.
>>
>> Why not just set up a git tree right away?
>>
>>> Dealing with all that, as well as the actual process for locking in
>>> bindings, is what needs to be sorted out.
>>>
>>> I think we're all in agreement that bindings that change over time are
>>> nothing but pain, but we're arguing that in circles anyway.
>>
>> No.
>>
>> I keep saying, the bindings must be stable ABI, *today*.
>>
>> You keep saying, maybe later, but until then we will make things up as
>> we go along.
>
> We have currently a lot of broken bindings, because people didn't know how
> to define ones and those they defined have not been properly reviewed. Do
> you really want such broken ABI in the kernel?
>
> Sure, there are many existing bindings that can be just made stable and
> well they probably are already de facto stable. This is mostly about
> subsystem bindings and whatever already has many users, both made them get
> more thought when designing and more review before merging.
>
> Still, a lot of device and platform-specific bindings are simply broken.
> Take max8925 backlight driver, that Olof started this whole discussion
> with, as an example. We need to sort them out before they can be
> stabilized.

That is a nice summary of how we got from null to now and Richard seems 
to be simply saying: let's stop mucking about and make this a project 
with a well-defined process of dealing with staging and stable bindings 
and keep stable bindings stable. Whether it should be within the kernel 
repo as a separate subsystem or in an entire different repo is a trivial 
decision, but still a decision that needs to be made.

Apart from stable DT bindings I would love to see a DT compiler that 
that next to DT syntax detects mistakes in properties used for the 
selfish reason that I spent hours debugging regulator code, because I 
typed vmmc_supply iso vmmc-supply. I did not go through all the 
bindings, but this may require a more formal description so it could be 
compiled/read in the DT compiler.

Regards,
Arend

  reply	other threads:[~2013-07-27 10:24 UTC|newest]

Thread overview: 284+ 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 16:09 ` Olof Johansson
2013-07-25 17:57 ` Mark Rutland
2013-07-25 17:57   ` Mark Rutland
2013-07-25 18:05   ` Stephen Warren
2013-07-25 18:05     ` Stephen Warren
2013-07-25 18:25     ` Olof Johansson
2013-07-25 18:25       ` Olof Johansson
2013-07-25 18:50       ` Stephen Warren
2013-07-25 18:50         ` Stephen Warren
2013-07-25 19:30       ` [Ksummit-2013-discuss] " Mark Brown
2013-07-25 19:30         ` Mark Brown
2013-07-25 18:29     ` Mark Rutland
2013-07-25 18:29       ` Mark Rutland
2013-07-25 18:48       ` Richard Cochran
2013-07-25 18:48         ` Richard Cochran
2013-07-25 18:53         ` Stephen Warren
2013-07-25 18:53           ` Stephen Warren
2013-07-25 21:41           ` Domenico Andreoli
2013-07-25 21:41             ` Domenico Andreoli
2013-07-26  4:43           ` Richard Cochran
2013-07-26  4:43             ` Richard Cochran
2013-07-25 21:37         ` Jason Gunthorpe
2013-07-25 21:37           ` Jason Gunthorpe
2013-07-26  4:54           ` Richard Cochran
2013-07-26  4:54             ` Richard Cochran
2013-07-26 13:42             ` Rob Herring
2013-07-26 13:42               ` Rob Herring
2013-07-26 17:15             ` Jason Gunthorpe
2013-07-26 17:15               ` Jason Gunthorpe
2013-07-27  8:48               ` Richard Cochran
2013-07-27  8:48                 ` Richard Cochran
2013-07-27 10:40                 ` [Ksummit-2013-discuss] " Mark Brown
2013-07-27 10:40                   ` Mark Brown
2013-07-27 17:37                   ` Richard Cochran
2013-07-27 17:37                     ` Richard Cochran
2013-07-27 17:57                     ` David Lang
2013-07-27 17:57                       ` David Lang
2013-07-27 18:17                       ` Richard Cochran
2013-07-27 18:17                         ` Richard Cochran
2013-07-28 11:27                     ` Mark Brown
2013-07-28 11:27                       ` Mark Brown
2013-07-29 17:54                 ` Jason Gunthorpe
2013-07-29 17:54                   ` Jason Gunthorpe
2013-07-29 18:16                   ` Russell King - ARM Linux
2013-07-29 18:16                     ` Russell King - ARM Linux
2013-07-29 18:40                     ` Jason Gunthorpe
2013-07-29 18:40                       ` Jason Gunthorpe
2013-07-27  8:53               ` Richard Cochran
2013-07-27  8:53                 ` Richard Cochran
2013-07-27 10:20                 ` Tomasz Figa
2013-07-27 10:20                   ` Tomasz Figa
2013-07-27 11:36                 ` [Ksummit-2013-discuss] " Mark Brown
2013-07-27 11:36                   ` Mark Brown
2013-07-27 18:07                   ` Richard Cochran
2013-07-27 18:07                     ` Richard Cochran
2013-07-28 12:10                     ` Mark Brown
2013-07-28 12:10                       ` Mark Brown
2013-07-26  8:01           ` Richard Cochran
2013-07-26  8:01             ` Richard Cochran
2013-07-26  9:42             ` [Ksummit-2013-discuss] " David Woodhouse
2013-07-26  9:42               ` David Woodhouse
2013-07-26 13:09               ` Richard Cochran
2013-07-26 13:09                 ` Richard Cochran
2013-07-26 13:14                 ` Russell King - ARM Linux
2013-07-26 13:14                   ` Russell King - ARM Linux
2013-07-27 15:28                   ` David Gibson
2013-07-27 15:28                     ` David Gibson
2013-07-26 13:27                 ` Jason Cooper
2013-07-26 13:27                   ` Jason Cooper
2013-07-26 13:38                   ` Russell King - ARM Linux
2013-07-26 13:38                     ` Russell King - ARM Linux
2013-07-26 13:45                     ` Jason Cooper
2013-07-26 13:45                       ` Jason Cooper
2013-07-26 13:50                       ` David Woodhouse
2013-07-26 13:50                         ` David Woodhouse
2013-07-26 13:59                         ` Jason Cooper
2013-07-26 13:59                           ` Jason Cooper
2013-07-26 13:41                   ` David Woodhouse
2013-07-26 13:41                     ` David Woodhouse
2013-07-26 13:47                     ` Jason Cooper
2013-07-26 13:47                       ` Jason Cooper
2013-07-26 14:14                     ` jonsmirl
2013-07-26 14:14                       ` jonsmirl at gmail.com
2013-07-26 14:16                       ` David Woodhouse
2013-07-26 14:16                         ` David Woodhouse
2013-07-26 14:21                       ` Russell King - ARM Linux
2013-07-26 14:21                         ` Russell King - ARM Linux
2013-07-26 14:39                         ` jonsmirl
2013-07-26 14:39                           ` jonsmirl at gmail.com
2013-07-26 14:40                         ` Mark Brown
2013-07-26 14:40                           ` Mark Brown
2013-07-29 15:32                           ` Matt Porter
2013-07-29 15:32                             ` Matt Porter
2013-07-27 15:19                         ` David Gibson
2013-07-27 15:19                           ` David Gibson
2013-07-27 19:37                         ` Grant Likely
2013-07-27 19:37                           ` Grant Likely
2013-07-27  9:16                       ` Ming Lei
2013-07-27  9:16                         ` Ming Lei
2013-07-26 14:10                 ` Mark Brown
2013-07-26 14:10                   ` Mark Brown
2013-07-26 15:49                   ` Olof Johansson
2013-07-26 15:49                     ` Olof Johansson
2013-07-26 16:36                     ` Rob Herring
2013-07-26 16:36                       ` Rob Herring
2013-07-27  4:57                       ` Richard Cochran
2013-07-27  4:57                         ` Richard Cochran
2013-07-27  5:04                     ` Richard Cochran
2013-07-27  5:04                       ` Richard Cochran
2013-07-27  9:51                       ` Tomasz Figa
2013-07-27  9:51                         ` Tomasz Figa
2013-07-27 10:24                         ` Arend van Spriel [this message]
2013-07-27 10:24                           ` Arend van Spriel
2013-07-27 10:34                           ` Arend van Spriel
2013-07-27 10:34                             ` Arend van Spriel
2013-07-27 11:09                             ` Russell King - ARM Linux
2013-07-27 11:09                               ` Russell King - ARM Linux
2013-07-27 10:36                           ` Tomasz Figa
2013-07-27 10:36                             ` Tomasz Figa
2013-07-27 10:59                             ` Arend van Spriel
2013-07-27 10:59                               ` Arend van Spriel
2013-07-27 19:45                               ` Grant Likely
2013-07-27 19:45                                 ` Grant Likely
2013-07-27 20:01                                 ` jonsmirl
2013-07-27 20:01                                   ` jonsmirl at gmail.com
2013-07-27 20:03                                   ` David Woodhouse
2013-07-27 20:03                                     ` David Woodhouse
2013-07-27 20:25                                   ` Grant Likely
2013-07-27 20:25                                     ` Grant Likely
2013-07-28  3:28                                     ` Grant Likely
2013-07-28  3:28                                       ` Grant Likely
2013-07-28  5:11                                       ` James Bottomley
2013-07-28  5:11                                         ` James Bottomley
2013-07-28  5:13                                         ` Grant Likely
2013-07-28  5:13                                           ` Grant Likely
2013-07-30  1:44                                         ` David Gibson
2013-07-30  1:44                                           ` David Gibson
2013-07-30  2:15                                           ` jonsmirl
2013-07-30  2:15                                             ` jonsmirl at gmail.com
2013-07-30  3:29                                             ` David Gibson
2013-07-30  3:29                                               ` David Gibson
2013-07-30  4:35                                               ` Grant Likely
2013-07-30  4:35                                                 ` Grant Likely
2013-07-30  8:39                                                 ` Maxime Ripard
2013-07-30  8:39                                                   ` Maxime Ripard
2013-07-30 16:30                                             ` Stephen Warren
2013-07-30 16:30                                               ` Stephen Warren
2013-07-30 17:27                                               ` John W. Linville
2013-07-30 17:27                                                 ` John W. Linville
2013-07-30 16:40                                           ` Stephen Warren
2013-07-30 16:40                                             ` Stephen Warren
2013-07-29  9:19                                   ` Arend van Spriel
2013-07-29  9:19                                     ` Arend van Spriel
2013-07-29 10:16                                     ` Arend van Spriel
2013-07-29 10:16                                       ` Arend van Spriel
2013-07-27 18:31                           ` Richard Cochran
2013-07-27 18:31                             ` Richard Cochran
2013-07-27 18:51                             ` Tomasz Figa
2013-07-27 18:51                               ` Tomasz Figa
2013-07-27 19:22                               ` jonsmirl
2013-07-27 19:22                                 ` jonsmirl at gmail.com
2013-07-28  8:56                               ` Richard Cochran
2013-07-28  8:56                                 ` Richard Cochran
2013-07-28  9:12                                 ` Tomasz Figa
2013-07-28  9:12                                   ` Tomasz Figa
2013-07-28 13:19                                   ` Richard Cochran
2013-07-28 13:19                                     ` Richard Cochran
2013-07-28 13:39                                     ` Tomasz Figa
2013-07-28 13:39                                       ` Tomasz Figa
2013-07-28 14:09                                       ` jonsmirl
2013-07-28 14:09                                         ` jonsmirl at gmail.com
2013-07-28 15:35                                         ` Richard Cochran
2013-07-28 15:35                                           ` Richard Cochran
2013-07-28 15:50                                           ` jonsmirl
2013-07-28 15:50                                             ` jonsmirl at gmail.com
2013-07-28 17:41                                             ` Mark Brown
2013-07-28 17:41                                               ` Mark Brown
2013-07-28 21:46                                           ` David Gibson
2013-07-28 21:46                                             ` David Gibson
2013-07-28 15:30                                       ` Richard Cochran
2013-07-28 15:30                                         ` Richard Cochran
2013-07-29  7:31                                     ` Maxime Ripard
2013-07-29  7:31                                       ` Maxime Ripard
2013-07-29 18:38                                       ` Richard Cochran
2013-07-29 18:38                                         ` Richard Cochran
2013-07-29 18:51                                         ` Richard Cochran
2013-07-29 18:51                                           ` Richard Cochran
2013-07-29 18:05                               ` Jason Gunthorpe
2013-07-29 18:05                                 ` Jason Gunthorpe
2013-07-29 18:27                                 ` Russell King - ARM Linux
2013-07-29 18:27                                   ` Russell King - ARM Linux
2013-07-29 22:20                                 ` David Gibson
2013-07-29 22:20                                   ` David Gibson
2013-07-29 23:14                                   ` Jason Gunthorpe
2013-07-29 23:14                                     ` Jason Gunthorpe
2013-07-29 23:49                                     ` David Gibson
2013-07-29 23:49                                       ` David Gibson
2013-07-31 10:37                               ` Maxime Bizon
2013-07-31 10:37                                 ` Maxime Bizon
2013-07-31 10:59                                 ` Tomasz Figa
2013-07-31 10:59                                   ` Tomasz Figa
2013-07-31 15:07                                   ` Richard Cochran
2013-07-31 15:07                                     ` Richard Cochran
2013-07-31 15:23                                     ` Tomasz Figa
2013-07-31 15:23                                       ` Tomasz Figa
2013-07-31 19:12                                       ` Richard Cochran
2013-07-31 19:12                                         ` Richard Cochran
2013-07-31 19:29                                         ` Tomasz Figa
2013-07-31 19:29                                           ` Tomasz Figa
2013-07-31 20:00                                           ` Richard Cochran
2013-07-31 20:00                                             ` Richard Cochran
2013-07-31 20:14                                             ` Russell King - ARM Linux
2013-07-31 20:14                                               ` Russell King - ARM Linux
2013-07-31 20:37                                               ` jonsmirl
2013-07-31 20:37                                                 ` jonsmirl at gmail.com
2013-07-31 20:48                                                 ` Russell King - ARM Linux
2013-07-31 20:48                                                   ` Russell King - ARM Linux
2013-07-31 21:26                                                   ` jonsmirl
2013-07-31 21:26                                                     ` jonsmirl at gmail.com
2013-08-01  9:57                                                     ` Arend van Spriel
2013-08-01  9:57                                                       ` Arend van Spriel
2013-08-01 10:09                                                       ` Mark Brown
2013-08-01 10:09                                                         ` Mark Brown
2013-08-01 10:18                                                     ` David Woodhouse
2013-08-01 10:18                                                       ` David Woodhouse
2013-08-01 13:34                                                       ` jonsmirl
2013-08-01 13:34                                                         ` jonsmirl at gmail.com
2013-08-01 13:43                                                         ` jonsmirl
2013-08-01 13:43                                                           ` jonsmirl at gmail.com
2013-08-01 13:48                                                       ` Rob Herring
2013-08-01 13:48                                                         ` Rob Herring
2013-08-02  3:50                                                     ` David Gibson
2013-08-02  3:50                                                       ` David Gibson
2013-08-13 23:32                                                       ` H. Peter Anvin
2013-08-13 23:32                                                         ` H. Peter Anvin
2013-08-14  1:49                                                         ` Guenter Roeck
2013-08-14  1:49                                                           ` Guenter Roeck
2013-08-14 14:46                                                           ` H. Peter Anvin
2013-08-14 14:46                                                             ` H. Peter Anvin
2013-08-02  8:49                                               ` Tony Lindgren
2013-08-02  8:49                                                 ` Tony Lindgren
2013-07-29 15:45                     ` Matt Porter
2013-07-29 15:45                       ` Matt Porter
2013-07-31 12:58   ` Ian Campbell
2013-07-31 12:58     ` Ian Campbell
2013-07-25 17:59 ` Stephen Warren
2013-07-25 17:59   ` Stephen Warren
2013-07-25 19:11 ` Rob Herring
2013-07-25 19:11   ` Rob Herring
2013-07-25 19:31   ` Jason Cooper
2013-07-25 19:31     ` Jason Cooper
2013-07-25 20:04     ` [Ksummit-2013-discuss] " Mark Brown
2013-07-25 20:04       ` Mark Brown
2013-07-25 20:08       ` Jason Cooper
2013-07-25 20:08         ` Jason Cooper
2013-07-25 20:16     ` Rob Herring
2013-07-25 20:16       ` Rob Herring
2013-07-25 20:32       ` Jason Cooper
2013-07-25 20:32         ` Jason Cooper
2013-07-25 21:53         ` [Ksummit-2013-discuss] " Ben Hutchings
2013-07-25 21:53           ` Ben Hutchings
2013-07-26  0:36           ` Stephen Warren
2013-07-26  0:36             ` Stephen Warren
2013-07-26 12:14           ` Jason Cooper
2013-07-26 12:14             ` Jason Cooper
2013-07-26  0:34       ` Stephen Warren
2013-07-26  0:34         ` Stephen Warren
2013-07-26 22:11         ` Rob Herring
2013-07-26 22:11           ` Rob Herring
2013-07-25 23:18     ` Russell King - ARM Linux
2013-07-25 23:18       ` Russell King - ARM Linux
2013-07-26  0:27       ` jonsmirl
2013-07-26  0:27         ` jonsmirl at gmail.com
2013-07-26 11:38         ` Jason Cooper
2013-07-26 11:38           ` Jason Cooper
2013-07-26 14:08           ` [Ksummit-2013-discuss] " David Woodhouse
2013-07-26 14:08             ` David Woodhouse
2013-07-26 13:57       ` Mark Brown
2013-07-26 13:57         ` Mark Brown
2013-07-26 10:55   ` Mark Brown
2013-07-26 10:55     ` Mark Brown
2013-07-28  4:39 ` Grant Likely
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=51F39FD8.6080808@broadcom.com \
    --to=arend@broadcom.com \
    --cc=Dave.Martin@arm.com \
    --cc=Pawel.Moll@arm.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=olof@lixom.net \
    --cc=richardcochran@gmail.com \
    --cc=rob.herring@calxeda.com \
    --cc=swarren@wwwdotorg.org \
    --cc=tomasz.figa@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.