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

From: Tomasz Figa
Date: Sat Feb 23 2013 - 19:38:35 EST

Hi Linus,

On Sunday 24 of February 2013 01:16:21 Linus Walleij wrote:
> On Sat, Feb 23, 2013 at 6:57 PM, Heiko Stübner <heiko@xxxxxxxxx> wrote:
> > When pinctrl is not built the fallback functions fail silently
> > and emit either 0 error codes or NULL pinctrl handles.
> >
> > Therefore it's needed to also check for this NULL-handle when
> > falling back to parsing the i2c gpios from devicetree.
> >
> > Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx>
> NAK.
> This is not the right solution for this driver.
> It uses pinctrl in a very simplistic way, just grabbing the
> default handler.
> After commit
> ab78029ecc347debbd737f06688d788bd9d60c1d:
> "drivers/pinctrl: grab default handles from device core"
> The right solution is to simply revert commit
> 2693ac69880a33d4d9df6f128415b65e745f00ba
> "i2c: s3c2410: Add support for pinctrl"
> Tomasz are you OK with this, or will you add more
> fine-grained pinctrl (like runtime PM etc) to this driver?

Yes, I'm fine. However reverting this patch will not solve the problem

There are 3 methods of pin configuration that has to be supported by this
driver (and several other drivers):
1) cfg_gpio callback passed in platform data,
2) legacy Samsung GPIO bindings (to be dropped ASAP),
3) pin control.
Each supported platform will support only one of these methods at the same

The first one is already handled correctly because it is always used
wherever it is available. The problem is with the remaining two.

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.

Keep in mind that there is no way to check whether method 2) succeeded,
because all it does is parsing GPIOs from device tree, assuming that the
custom xlate function of the old Samsung GPIO driver would do all the

I do not see another solution of this problem. Feel free to suggest
anything better.

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