Hi Wolfram, On Thursday 22 October 2015 13:05:05 Wolfram Sang wrote: > On Thu, Oct 22, 2015 at 02:10:52AM +0300, Laurent Pinchart wrote: > > On Thursday 03 September 2015 22:20:09 Wolfram Sang wrote: > > > From: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> > > > > > > Setting up new messages was done in process context while handling a > > > message was in interrupt context. Because of the HW design, this IP core > > > is sensitive to timing, so the context switches were too expensive. Move > > > this setup to interrupt context as well. > > > > > > In my test setup, this fixed the occasional 'data byte sent twice' issue > > > which a number of people have seen. It also fixes to send REP_START > > > after a read message which was wrongly send as a STOP + START sequence > > > before. > > > > I'm afraid this patch has been found by git bisect to break HDMI on > > Koelsch > > > > :-( > > > > The regmap_read(adv7511->regmap, ADV7511_REG_CHIP_REVISION, &val) call in > > drivers/gpu/drm/i2c/adv7511.c returns -ENXIO. > > > > Reverting the patch on top of Geert's current drivers master branch fixes > > the problem. > > But HDMI worked on Koelsch in Dublin?? I know :-) Do you have a Koelsch board now ? Could you try b9653e9c000dc2ebd9c8712442c659ccd1586e22 from Geert's drivers tree ? On my board the adv7511 fails to probe completely due to the regmap_read() failure. -- Regards, Laurent Pinchart -- 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
