[RFC/PATCH 2/2] gpio-rcar: Add DT support
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Sat May 18 16:30:23 EST 2013
Hi Guennadi,
On Friday 17 May 2013 14:22:22 Guennadi Liakhovetski wrote:
> On Fri, 10 May 2013, Laurent Pinchart wrote:
> > Add DT bindings for the gpio-rcar driver and read the device
> > configuration from the DT node at probe time if available.
> >
> > Cc: devicetree-discuss at lists.ozlabs.org
> > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > ---
> >
> > .../devicetree/bindings/gpio/renesas,gpio-rcar.txt | 48 +++++++++++++++++
> > drivers/gpio/gpio-rcar.c | 63 +++++++++++++----
> > 2 files changed, 101 insertions(+), 10 deletions(-)
> > create mode 100644
> > Documentation/devicetree/bindings/gpio/renesas,gpio-rcar.txt
> Testing this patch reveals, that (at least) one thing is missing here:
>
> diff --git a/drivers/gpio/gpio-rcar.c b/drivers/gpio/gpio-rcar.c
> index a499ea6..e233778 100644
> --- a/drivers/gpio/gpio-rcar.c
> +++ b/drivers/gpio/gpio-rcar.c
> @@ -342,6 +342,7 @@ static int gpio_rcar_probe(struct platform_device *pdev)
> gpio_chip->owner = THIS_MODULE;
> gpio_chip->base = p->config.gpio_base;
> gpio_chip->ngpio = p->config.number_of_pins;
> + gpio_chip->dev = &pdev->dev;
>
> irq_chip = &p->irq_chip;
> irq_chip->name = name;
Indeed, thank you. I wonder how I managed not to notice this. I'll integrate
the modification for v2.
--
Regards,
Laurent Pinchart
More information about the devicetree-discuss
mailing list