Re: [PATCH] i2c: s3c2410: check for NULL pinctrl handle

From: Tomasz Figa
Date: Sat Feb 23 2013 - 19:58:57 EST

On Sunday 24 of February 2013 01:47:49 Linus Walleij wrote:
> On Sun, Feb 24, 2013 at 1:38 AM, Tomasz Figa <tomasz.figa@xxxxxxxxx>
> > The driver must know whether pin control is available, because it has
> > to fall back to legacy GPIO-based pin configuration if it is not.
> > This means that we must either check for NULL (which probably is not
> > right, since returned handle is considered to be opaque) or pin
> > control core must return an error code specific to this situation,
> > e.g. -ENODEV.
> OK so pass a flag like a bool in your platform data from the
> machine like go into <linux/platform_data/i2c-s3c2410.h>
> and add:
> struct s3c2410_platform_i2c {
> bool use_that_old_gpio_interface;
> (...)
> };
> Instead of trying to semi-guess if the pinctrl framework is there?
> Surely you know this when setting up the pdata from your machine?

Cases 2) and 3) are both DT-enabled cases, where there is no pdata coming
from board-specific code.

In case of this particular driver, a check for presence of "gpios"
property will be enough, because this driver does not use GPIO pins
normally - they are just used for pin configuration, when legacy method
has to be used.

However it will not work in case of drivers that also need some GPIO pins
unrelated to the pins that have to be configured to dedicated functions.

Note that we are talking here about a temporary solution. The legacy DT-
based pin configuration will go away after all the DT-enabled platforms
using this driver get migrated to pin control and so will the need to
check if pin control is available.

Best regards,

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at