On Thu, Dec 5, 2013 at 4:53 PM, delicious quinoa
<delicious.quinoa@xxxxxxxxx> wrote:
> On Thu, Dec 5, 2013 at 10:54 AM, Jamie Iles <jamie@xxxxxxxxxxxxx> wrote:
>> Hi Alan,
>>
>> One minor comment below, otherwise looks great!
>>
>> Signed-off-by: Jamie Iles <jamie@xxxxxxxxxxxxx>
>>
>> Thanks,
>>
>> Jamie
>>
>> On Tue, Dec 03, 2013 at 10:41:16AM -0600, Alan Tull wrote:
>>> diff --git a/drivers/gpio/gpio-dwapb.c b/drivers/gpio/gpio-dwapb.c
>>> new file mode 100644
>>> index 0000000..e6dc61f
>>> --- /dev/null
>>> +++ b/drivers/gpio/gpio-dwapb.c
>> ...
>>> +static unsigned int dwapb_gpio_nr_ports(struct device_node *of_node)
>>> +{
>>> + unsigned int nr_ports = 0;
>>> + struct device_node *np;
>>> +
>>> + for_each_child_of_node(of_node, np)
>>> + ++nr_ports;
>>> +
>>> + return nr_ports;
>>> +}
>>
>> I notice that we now have of_get_child_count() which didn't exist when I
>> first wrote the driver, we could use that now.
>
> Thanks Jamie.
>
> I'll include that in v8. I'll post that within a few days.
>
> Alan
er, I mean v9.
Alan
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
