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 completely. 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 time. 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 configuration. I do not see another solution of this problem. Feel free to suggest anything better. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
