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

From: Linus Walleij
Date: Sat Feb 23 2013 - 19:16:27 EST

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>


This is not the right solution for this driver.

It uses pinctrl in a very simplistic way, just grabbing the
default handler.

After commit
"drivers/pinctrl: grab default handles from device core"

The right solution is to simply revert commit
"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?

Linus Walleij
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