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
"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?
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/