Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1290142

Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Thu, Dec 10, 2015 at 12:43:24PM -0800, Brian Norris wrote:
> On Mon, Dec 07, 2015 at 12:36:28PM +1100, David Gibson wrote:
> > On Sat, Dec 05, 2015 at 10:33:30PM +0100, Michal Suchanek wrote:
> > > On 5 December 2015 at 12:39, Jonas Gorski <jogo@openwrt.org> wrote:
> > > > On Sat, Dec 5, 2015 at 6:19 AM, Brian Norris
> > > > <computersforpeace@gmail.com> wrote:
> > > 
> > > >> +
> > > >> +Examples:
> > > >> +
> > > >> +flash@0 {
> > > >> +       partitions {
> > > >> +               compatible = "google,fmap";
> > > >> +       };
> > > >> +};
> > > >
> > > > I wonder if this wouldn't be better served in a separate binding doc
> > > > with its compatible name as the filename, like we do with
> > > > driver^Whardware blocks, especially if we want to add more parsers.
> > > 
> > > 
> > > I find that *very* counter productive for bindings that go to the same
> > > node. You have a description of a node, and then suddenly there you
> > > have another file with another description of the same node. Totally
> > > awesome.
> > 
> > I can't actually work out from that if you're agreeing with the
> > original post or the first reply.
> 
> Perhaps I'm biased, but I think he was agreeing with the first reply.
> (Particularly, "I find that *very* counter productive" uses the word
> "that" to refer to "separate binding doc[s]".)
> 
> > > Also how do you plan to write partitioning schemes with parameters
> > > like with non-zero offset of the partition table.
> 
> If you are directing this question at me: I don't have a specific plan
> for it. MTD parsers don't currently take external input for this; many
> scan the whole device, but some might also have conventions built into
> the parser itself too, so this just gets hooked based on "compatible".
> But if the need arose, I would hope we could work out a common binding.
> 
> > Presumably with properties in the patitions node.  Not seeing the
> > problem here.
> 
> I believe Michal is bringing up the (important, IMO) point that if
> distinct partition types are being described in the same node, then any
> use of additional properties *must* be closely coordinated. We can't
> have two parsers "foo" and "bar" defining conflicting uses of the same
> property in the same node, like this:
> 
> 	partitions {
> 		compatible = "foo", "bar";
> 		property-baz = ...; // e.g., reg = <...>;
> 	};
> 
> where if "foo" is not found, we fall back to "bar". But what if "foo"
> and "bar" use "property-baz" differently?

Ah.. that is an excellent point, and leads me to realise that using
compatible in this way is wrong.  The whole point of compatible is
that the node is, well, compatible with *all* the things in the list,
and therefore the things in the list are compatible with each other.

Using it for a list of entirely different things to attempt in order
is not correct.


> Having everything in one doc would help ensure that the entire
> "partitions" binding is considered as a whole when extending it, in my
> (and an in my interpretation of Michal's) opinion.
> 
> Brian

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Brian Norris <computersforpeace@gmail.com> - 2015-12-05 06:20 +0100
  Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Jonas Gorski <jogo@openwrt.org> - 2015-12-05 12:50 +0100
    Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Michal Suchanek <hramrach@gmail.com> - 2015-12-05 22:40 +0100
      Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding David Gibson <david@gibson.dropbear.id.au> - 2015-12-07 04:10 +0100
        Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding Brian Norris <computersforpeace@gmail.com> - 2015-12-10 21:50 +0100
          Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Michal Suchanek <hramrach@gmail.com> - 2015-12-11 17:00 +0100
          Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding David Gibson <david@gibson.dropbear.id.au> - 2015-12-12 09:50 +0100
            Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-14 11:30 +0100
              Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Michal Suchanek <hramrach@gmail.com> - 2015-12-14 13:30 +0100
              Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding David Gibson <david@gibson.dropbear.id.au> - 2015-12-15 11:00 +0100
                Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding Geert Uytterhoeven <geert@linux-m68k.org> - 2015-12-15 11:10 +0100
                Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding David Gibson <david@gibson.dropbear.id.au> - 2015-12-17 02:10 +0100
  Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format  binding Rob Herring <robh@kernel.org> - 2015-12-07 04:10 +0100

csiph-web