[PATCH 6/6] mtd: ofpart: add compatible check for child nodes

Brian Norris computersforpeace at gmail.com
Mon Aug 5 14:56:14 EDT 2013


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 at 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 at atmel.com>
>> Acked-by: Brian Norris <computersforpeace at gmail.com>
>> Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at 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



More information about the linux-mtd mailing list