linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Richard Cochran <richardcochran@gmail.com>
To: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
Cc: 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>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Pawel Moll <Pawel.Moll@arm.com>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Catalin Marinas <Catalin.Marinas@arm.com>,
	Domenico Andreoli <cavokz@gmail.com>,
	"rob.herring@calxeda.com" <rob.herring@calxeda.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Olof Johansson <olof@lixom.net>,
	Dave P Martin <Dave.Martin@arm.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Ian Campbell <ian.campbell@citrix.com>
Subject: Re: DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?]
Date: Sat, 27 Jul 2013 10:48:26 +0200	[thread overview]
Message-ID: <20130727084825.GA4707@netboy> (raw)
In-Reply-To: <20130726171524.GB28895@obsidianresearch.com>

On Fri, Jul 26, 2013 at 11:15:24AM -0600, Jason Gunthorpe wrote:
> 
> Further, every other kernel release tended to break the board.c files,
> just due to the usual kernel churn.

Yes...

> DT is much better, the stuff that can be described in DT is broader
> and more thought tends to have been put into DT bindings than was put
> into the old C methods. It has less churn, and what churn there is
> seems simpler to follow.

...
 
> Of course all this could have been done in C, but it wasn't/hasn't been..

You have identified the real issue: poor quality of the ARM board
support process within the kernel. Churn is still churn, whether in DT
or in platform devices, but the DT people promised to end the churn.

At least with the platform code, I knew where to look to resolve
issues. Thanks to DT, I now have three haystacks to search, namely
boot loader, DT, and kernel.
 
[ I disagree about the "more thought" part. The current discussion,
  coming years too late after the introduction of DT to ARM Linux, is
  contrary evidence enough. ]

> Why would Freescale PPC be any different from the IBM PPC we use? We'd
> use exactly the same techniques we use on ARM and PPC today and the
> churn to the DT wouldn't be an operational problem in the field.

I always suspected that the IBM powerpc Linux code was of higher
quality. Freescale has been just awful.

And that is just the point. No one is saying that DT cannot work. In
theory (and in your experience) it is really wonderful. However, all
this talk about "one day we will offer stable binding on ARM" already
tells me what kind of quality to expect.

> Why do you think our experiences are so different?

Because the "embedded mindset" was used to develop the code on the
platforms that I have been using.

> *shrug* For embedded I am being *very pragmatic*. I don't need/want an
> ABI boundary between the DT and the kernel. I don't need the DT to
> statically describe the hardware.

Yes, I know the kind of quality to expect from the embedded vendors.
But that doesn't mean that mainline Linux should also stink.

> Well, you know why. The DT schema used by the kernel changes over
> time. Kirkwood just went through a massive change in schema in the
> past few releases, and the same was true on our PPC platforms when
> that was new.

I find the very idea to be total unacceptable. This should never
happen in the stable kernel.

> I can't delay shipping while upstream sorts this out - so I know
> *absolutely* that the DT schema will change. This has been planned for
> and designed into the boot system and won't be a problem.
> 
> Why would anyone do embedded any other way?

Yep, that is the embedded way: ship it!

We can do better than that.

Thanks,
Richard

  reply	other threads:[~2013-07-27  8:48 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 [this message]
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
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=20130727084825.GA4707@netboy \
    --to=richardcochran@gmail.com \
    --cc=Catalin.Marinas@arm.com \
    --cc=Dave.Martin@arm.com \
    --cc=Pawel.Moll@arm.com \
    --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=rob.herring@calxeda.com \
    --cc=sameo@linux.intel.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).