On Thu, Dec 5, 2013 at 4:53 PM, delicious quinoa
<delicious.qui...@gmail.com> wrote:
> On Thu, Dec 5, 2013 at 10:54 AM, Jamie Iles <ja...@jamieiles.com> wrote:
>> Hi Alan,
>>
>> One minor comment below, otherwise looks great!
>>
>> Signed-off-by: Jamie Iles <ja...@jamieiles.com>
>>
>> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to