[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOMqctTbUNd0GtY07YJuxem9HkU1WZ-NtO556=Yg2QrsF49Xjw@mail.gmail.com>
Date: Sat, 5 Dec 2015 22:33:30 +0100
From: Michal Suchanek <hramrach@...il.com>
To: Jonas Gorski <jogo@...nwrt.org>
Cc: Brian Norris <computersforpeace@...il.com>,
Boris Brezillon <boris.brezillon@...e-electrons.com>,
Arnd Bergmann <arnd@...db.de>,
Geert Uytterhoeven <geert+renesas@...der.be>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
devicetree-spec@...r.kernel.org, Simon Arlott <simon@...e.lp0.eu>,
Linus Walleij <linus.walleij@...aro.org>,
Rafał Miłecki <zajec5@...il.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Jason Gunthorpe <jgunthorpe@...idianresearch.com>,
Rob Herring <robh+dt@...nel.org>,
MTD Maling List <linux-mtd@...ts.infradead.org>,
Hauke Mehrtens <hauke@...ke-m.de>
Subject: Re: [RFC PATCH 3/7] doc: dt: mtd: partition: add on-flash format binding
On 5 December 2015 at 12:39, Jonas Gorski <jogo@...nwrt.org> wrote:
> On Sat, Dec 5, 2015 at 6:19 AM, Brian Norris
> <computersforpeace@...il.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.
Also how do you plan to write partitioning schemes with parameters
like with non-zero offset of the partition table.
Thanks
Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists