From: Brian Norris <computersforpeace@gmail.com> To: Josh Wu <josh.wu@atmel.com> Cc: devicetree@vger.kernel.org, sergei.shtylyov@cogentembedded.com, dedekind1@gmail.com, nicolas.ferre@atmel.com, linux-mtd@lists.infradead.org, plagnioj@jcrosoft.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Date: Mon, 5 Aug 2013 11:56:14 -0700 [thread overview] Message-ID: <CAN8TOE_QBpGNQsdxXNPEHN7L_xhQHJE30px3PeE-vFxJOVyaRA@mail.gmail.com> (raw) In-Reply-To: <51FF8D90.2060308@atmel.com> It seems Artem applied this to l2-mtd.git. One comment below. On Mon, Aug 5, 2013 at 4:33 AM, Josh Wu <josh.wu@atmel.com> wrote: > Add device tree mail list to CC. > > > On 8/5/2013 7:14 PM, Josh Wu wrote: >> >> In case that the nand device will support some features like Nand Flash >> Controller, we want to make the sub feature as a sub node of nand device. >> >> Use such organization it is easy to enable/disable feature, also it is >> back >> compatible and more readable. >> >> If the sub-node has a compatible property then it is a driver not >> partition. >> >> Signed-off-by: Josh Wu <josh.wu@atmel.com> >> Acked-by: Brian Norris <computersforpeace@gmail.com> >> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> >> --- >> v3 --> v4: >> - remove the 'len' parameter, just use NULL. >> - refine the commit message. >> - also add a NOTE in devicetree/bindings/mtd/partition.txt. >> >> .../devicetree/bindings/mtd/partition.txt | 1 + >> drivers/mtd/ofpart.c | 13 ++++++++++++- >> 2 files changed, 13 insertions(+), 1 deletion(-) >> ... >> diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c >> index 553d6d6..30fcea1 100644 >> --- a/drivers/mtd/ofpart.c >> +++ b/drivers/mtd/ofpart.c >> @@ -20,6 +20,10 @@ >> #include <linux/slab.h> >> #include <linux/mtd/partitions.h> >> +static bool node_has_compatible(struct device_node *pp) >> +{ >> + return of_get_property(pp, "compatible", NULL); >> +} Missing a trivial additional newline here. I amended the patch in l2-mtd.git. >> static int parse_ofpart_partitions(struct mtd_info *master, >> struct mtd_partition **pparts, >> struct mtd_part_parser_data *data) ... Brian
WARNING: multiple messages have this Message-ID (diff)
From: computersforpeace@gmail.com (Brian Norris) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Date: Mon, 5 Aug 2013 11:56:14 -0700 [thread overview] Message-ID: <CAN8TOE_QBpGNQsdxXNPEHN7L_xhQHJE30px3PeE-vFxJOVyaRA@mail.gmail.com> (raw) In-Reply-To: <51FF8D90.2060308@atmel.com> It seems Artem applied this to l2-mtd.git. One comment below. On Mon, Aug 5, 2013 at 4:33 AM, Josh Wu <josh.wu@atmel.com> wrote: > Add device tree mail list to CC. > > > On 8/5/2013 7:14 PM, Josh Wu wrote: >> >> In case that the nand device will support some features like Nand Flash >> Controller, we want to make the sub feature as a sub node of nand device. >> >> Use such organization it is easy to enable/disable feature, also it is >> back >> compatible and more readable. >> >> If the sub-node has a compatible property then it is a driver not >> partition. >> >> Signed-off-by: Josh Wu <josh.wu@atmel.com> >> Acked-by: Brian Norris <computersforpeace@gmail.com> >> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> >> --- >> v3 --> v4: >> - remove the 'len' parameter, just use NULL. >> - refine the commit message. >> - also add a NOTE in devicetree/bindings/mtd/partition.txt. >> >> .../devicetree/bindings/mtd/partition.txt | 1 + >> drivers/mtd/ofpart.c | 13 ++++++++++++- >> 2 files changed, 13 insertions(+), 1 deletion(-) >> ... >> diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c >> index 553d6d6..30fcea1 100644 >> --- a/drivers/mtd/ofpart.c >> +++ b/drivers/mtd/ofpart.c >> @@ -20,6 +20,10 @@ >> #include <linux/slab.h> >> #include <linux/mtd/partitions.h> >> +static bool node_has_compatible(struct device_node *pp) >> +{ >> + return of_get_property(pp, "compatible", NULL); >> +} Missing a trivial additional newline here. I amended the patch in l2-mtd.git. >> static int parse_ofpart_partitions(struct mtd_info *master, >> struct mtd_partition **pparts, >> struct mtd_part_parser_data *data) ... Brian
next prev parent reply other threads:[~2013-08-05 18:56 UTC|newest] Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-08-05 11:14 [PATCH v6 0/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) support Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 11:14 ` [PATCH 1/6] MTD: atmel_nand: use devm_xxx gpio kzalloc, gpio and ioremap Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 13:04 ` Artem Bityutskiy 2013-08-05 13:04 ` Artem Bityutskiy 2013-08-05 11:14 ` [PATCH 2/6] mtd: atmel_nand: replace pmecc enable code with one function Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-07 7:20 ` Brian Norris 2013-08-07 7:20 ` Brian Norris 2013-08-07 9:20 ` Josh Wu 2013-08-07 9:20 ` Josh Wu 2013-08-05 11:14 ` [PATCH 3/6] mtd: atmel_nand: add Nand Flash Controller (NFC) support Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 11:14 ` [PATCH 4/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) read data via sram Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 11:14 ` [PATCH 5/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) write " Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 11:14 ` [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Josh Wu 2013-08-05 11:14 ` Josh Wu 2013-08-05 11:33 ` Josh Wu 2013-08-05 11:33 ` Josh Wu 2013-08-05 11:33 ` Josh Wu 2013-08-05 18:56 ` Brian Norris [this message] 2013-08-05 18:56 ` Brian Norris -- strict thread matches above, loose matches on Subject: below -- 2013-07-03 9:50 [PATCH v5 0/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) support Josh Wu 2013-07-03 9:50 ` [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Josh Wu 2013-07-03 9:50 ` Josh Wu 2013-07-18 7:14 ` Brian Norris 2013-07-18 7:14 ` Brian Norris 2013-07-18 7:14 ` Brian Norris 2013-07-18 8:28 ` Jean-Christophe PLAGNIOL-VILLARD 2013-07-18 8:28 ` Jean-Christophe PLAGNIOL-VILLARD 2013-07-18 8:28 ` Jean-Christophe PLAGNIOL-VILLARD 2013-06-19 6:23 [PATCH v4 0/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) support Josh Wu 2013-06-19 6:23 ` [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Josh Wu 2013-06-19 6:23 ` Josh Wu 2013-06-10 10:26 [PATCH v3 0/6] mtd: atmel_nand: enable Nand Flash Controller (NFC) support Josh Wu 2013-06-10 10:26 ` [PATCH 6/6] mtd: ofpart: add compatible check for child nodes Josh Wu 2013-06-10 10:26 ` Josh Wu 2013-06-10 12:35 ` Sergei Shtylyov 2013-06-10 12:35 ` Sergei Shtylyov 2013-06-13 3:42 ` Josh Wu 2013-06-13 3:42 ` Josh Wu 2013-06-13 13:13 ` Sergei Shtylyov 2013-06-13 13:13 ` Sergei Shtylyov 2013-06-11 23:34 ` Brian Norris 2013-06-11 23:34 ` Brian Norris 2013-06-13 3:38 ` Josh Wu 2013-06-13 3:38 ` Josh Wu
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=CAN8TOE_QBpGNQsdxXNPEHN7L_xhQHJE30px3PeE-vFxJOVyaRA@mail.gmail.com \ --to=computersforpeace@gmail.com \ --cc=dedekind1@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=josh.wu@atmel.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-mtd@lists.infradead.org \ --cc=nicolas.ferre@atmel.com \ --cc=plagnioj@jcrosoft.com \ --cc=sergei.shtylyov@cogentembedded.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: linkBe 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.