On Wed, May 08, 2013 at 11:51:24AM +0100, Mark Brown wrote: > On Wed, May 08, 2013 at 09:14:18AM +0200, Fabio Baltieri wrote: > > Drop pinctrl default/sleep state switching code, as it was breaking the > > capture interface by putting the I2S pins in hi-z mode regardless of its > > usage status, and not giving any real benefit. > > > > Pinctrl default mode configuration is already managed automatically by a > > specific pinctrl hog. > > I tend to agree with Lee that this looks like a bad approach - there's a > whole bunch of other code in there which I'd guess is probably equally > broken but only the pinctrl code is being removed. Why not just fix it > (or better yet simplify all this stuff)?
Sorry I did not get the subject, are you referring to other broken code in the ux500 driver or to pinctrl and shared pins in generals? Fabio -- Fabio Baltieri -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
