On Fri, Apr 26, 2013 at 03:36:42PM +0300, Alexander Shishkin wrote: > In author's words, some controllers don't need it, so it's ok if it > fails. Printing warning in that case is not a good idea, so I agree with > Fabio. I'll apply it soon unless I hear from people involved with > chipidea on imx. Next thing we need to agree on if this bit belongs to > platform or core, Yes, we already have that supported in driver core. See commit ab78029 (drivers/pinctrl: grab default handles from device core). It seems the ci13xxx_imx just falls into the use case that the above commit targets as below. "So if you just want to put the pins for a certain device into the default state and be done with it, there is nothing you need to do besides providing the proper mapping table. The device core will take care of the rest." Shawn > which is not obvious, and the patch that introduced > this code doesn't clarify why exactly this is needed. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html