On Thu, Dec 01, 2011 at 01:49:21PM -0700, Stephen Warren wrote:

> -     if (pdata && pdata->gpio_base)
> -             wm8903->gpio_chip.base = pdata->gpio_base;
> -     else
> -             wm8903->gpio_chip.base = -1;
> +     wm8903->gpio_chip.base = pdata->gpio_base;

This will break existing users in counjuntion with the previous patch.
Previously if the user provided platform data but left gpio_base as zero
we'd use -1 and let gpiolib pick for us.  Now instead the driver will
take that zero and pass it on to gpiolib, probably failing as the SoC
will have taken the low numbered GPIOs.
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss

Reply via email to