Re: [PATCH RFC 2/2] gpio: samsung: Remove OF support for Exynos4/5

[Sylwester Nawrocki]

On 04/17/2013 06:35 PM, Linus Walleij wrote:
> On Wed, Apr 17, 2013 at 1:06 PM, Sylwester Nawrocki
> <s.nawrocki@xxxxxxxxxxx>  wrote:
>
> > -#endif
> > +        * All Samsung platforms that support device tree will use the pinctrl
> > +        * driver, hence we skip initialization of this driver if the OF tree
> > +        * is available.
> > +        */
> > +       if (of_have_populated_dt())
> > +               return -ENODEV;
> >
> >          samsung_gpiolib_set_cfg(samsung_gpio_cfgs, ARRAY_SIZE(samsung_gpio_cfgs));
>
> Oh is that how you do it? It looks fragile.
>
> Can't you check if you have the pinctrl node atleast?

But this driver won't help if there is no pinctrl node found anyway, since
any OF support is removed from it.

This way there is no need for any additional DT parsing, just to skip 
probing
of this legacy GPIO driver.

There might be something that could have gone wrong and I am missing though.

Thanks,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/