On Tue, Aug 27, 2013 at 7:04 PM, Ian Molton <[email protected]> wrote:
> Signed-off-by: Ian Molton <[email protected]> > --- > drivers/gpio/gpio-em.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpio/gpio-em.c b/drivers/gpio/gpio-em.c > index 5cba855..279821e 100644 > --- a/drivers/gpio/gpio-em.c > +++ b/drivers/gpio/gpio-em.c > @@ -303,6 +303,7 @@ static int em_gio_probe(struct platform_device *pdev) > } > > gpio_chip = &p->gpio_chip; > + gpio_chip->of_node = pdev->dev.of_node; This doesn't work on non-DT compiles. Look at struct gpio_chip: #if defined(CONFIG_OF_GPIO) /* * If CONFIG_OF is enabled, then all GPIO controllers described in the * device tree automatically may have an OF translation */ struct device_node *of_node; int of_gpio_n_cells; int (*of_xlate)(struct gpio_chip *gc, const struct of_phandle_args *gpiospec, u32 *flags); #endif Either the statement needs to be inside #ifdef CONFIG_OF_GPIO or the Emma driver need to select OF_GPIO in its Kconfig entry. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
