Re: [PATCH v2 0/2] pinctrl: Intel Cherryview/Braswell support

Rafael J. Wysocki
2014-11-05T20:43:54Z

On Tuesday, November 04, 2014 05:18:47 PM Timur Tabi wrote:
> On 11/04/2014 04:56 PM, Timur Tabi wrote:
> >
> > > I'm
> >> not sure how much of that code can be shared between DT and ACPI ATM, but
> >> it looks like at least some of it can be shared.
> >
> > So ACPI does not support the concept if variable length properties?
>
> Ah, I see that it does:
>
> int acpi_dev_prop_read_array(struct acpi_device *adev, const char *propname,
> enum dev_prop_type proptype, void *val,
> size_t nval)
> {
> ...
> if (!val)
> return obj->package.count;
>
>
> If I pass NULL for 'val', it returns the number of items.

Yes, it does that.

If you pass NULL for val to device_property_read_*_array(), it will also
return the number of items in the property. Including the string one.

We're missing a couple of calls that you'd need for pinctrl, but I'd rather
add those along with some users.

Rafael