On 2016-11-08 22:11, Greg KH wrote:
> On Tue, Nov 08, 2016 at 04:34:50PM -0800, Stefan Agner wrote:
>> On 2016-11-08 16:32, Stefan Agner wrote:
>> > commit 533169d164c6b4c8571d0d48779f6ff6be593d72 upstream.
>> >
>> > Some SoC might load the GPIO driver after the I2C driver and
>> > using the I2C bus recovery mechanism via GPIOs. In this case
>> > it is crucial to defer probing if the GPIO request functions
>> > do so, otherwise the I2C driver gets loaded without recovery
>> > mechanisms enabled.
>> >
>> > Signed-off-by: Stefan Agner <stefan@xxxxxxxx>
>> > Acked-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
>> > Acked-by: Li Yang <leoyang.li@xxxxxxx>
>> > Signed-off-by: Wolfram Sang <wsa@xxxxxxxxxxxxx>
>> > Cc: stable@xxxxxxxxxx
>>
>> I forgot to mention, this should be applied to 4.4 only.
>
> Did you build the code there?
Oh, I built it on a branch where I had some other patches applied, sorry
about that.
>
>> Later versions made the return value void, and it would require a larger
>> rework. I am mainly concerned about LTS...
>
> I don't see this working for either 4.4 or 4.8-stable right now, do you?
No, the patch really doesn't work for 4.4 through 4.8. The patch relies
on commit fd8961c5ba9e6c ("i2c: imx: make bus recovery through pinctrl
optional"), which is not marked for stable (and probably also not
suitable actually).
The issue can only be observed on Vybrid SoC's with I2C bus recovery
enabled. The upstream device trees don't use I2C bus recovery so far. So
the patch is actually just enabling a feature we did not have so far...
Given all this circumstances, I think we should not further waste our
time and try to backport this patch for stable.
Sorry for the noise.
--
Stefan
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html