Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-29 Thread Wolfram Sang
Laurent, > RTPORC7791SEB00010S > KOELSCH SN.057 > > I'm not sure if that tells anything about the board revision. Can you try this patch I just sent out? "i2c: rcar: make sure clocks are on when doing hw init" We know that Koelsch boards have different bootloaders leaving the clocks in differe

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-25 Thread Wolfram Sang
Hi, so I did some testing regarding my setup. > > Best report ever: I simply don't get a picture :) No warnings, no > > messages. I think we want to make sure my HDMI->DVI converter is proper, > > though. > > Does it work with other HDMI sources ? I borrowed another HDMI monitor. There, my Lage

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Laurent Pinchart
Hi Wolfram, On Friday 23 October 2015 14:14:39 Wolfram Sang wrote: > > > :/ Let me know if I can send you debug output. > > > > What's the exact issue ? > > Best report ever: I simply don't get a picture :) No warnings, no > messages. I think we want to make sure my HDMI->DVI converter is proper

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Wolfram Sang
> > :/ Let me know if I can send you debug output. > > What's the exact issue ? Best report ever: I simply don't get a picture :) No warnings, no messages. I think we want to make sure my HDMI->DVI converter is proper, though. > > > .config attached. > > > > That one also probes for me... I onl

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Laurent Pinchart
Hi Wolfram, On Friday 23 October 2015 11:45:00 Wolfram Sang wrote: > > I had CONFIG_DRM_FBDEV_EMULATION disabled. I've then enabled it but also > > merged git://people.freedesktop.org/~airlied/linux next in my branch, > > which probably fixes the problem. > > So, your tree is not strictly renesas

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Wolfram Sang
> - error = true; > + return ret; Yup, did that, too :) signature.asc Description: Digital signature

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Geert Uytterhoeven
On Fri, Oct 23, 2015 at 10:06 AM, Wolfram Sang wrote: >> b9653e9c000dc2ebd9c8712442c659ccd1586e22 from Geert's drivers tree ? On my > > Does that build for you? > > drivers/gpu/drm/drm_fb_helper.c: In function 'restore_fbdev_mode': > drivers/gpu/drm/drm_fb_helper.c:448:5: error: 'error' undeclared

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Wolfram Sang
> I had CONFIG_DRM_FBDEV_EMULATION disabled. I've then enabled it but also > merged git://people.freedesktop.org/~airlied/linux next in my branch, which > probably fixes the problem. So, your tree is not strictly renesas-drivers-2015-10-13-v4.3-rc5? > > I fixed it locally and will see if I see

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-23 Thread Wolfram Sang
> Do you have a Koelsch board now ? Could you try Nope, I only have Lager. I'd be surprised if it really was a Koelsch only issue, though... > b9653e9c000dc2ebd9c8712442c659ccd1586e22 from Geert's drivers tree ? On my Does that build for you? drivers/gpu/drm/drm_fb_helper.c: In function 'res

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-22 Thread Laurent Pinchart
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 > > > > > > Setting up new messages was done in process context while h

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-22 Thread Wolfram Sang
On Thu, Oct 22, 2015 at 02:10:52AM +0300, Laurent Pinchart wrote: > Hi Wolfram, > > On Thursday 03 September 2015 22:20:09 Wolfram Sang wrote: > > From: Wolfram Sang > > > > Setting up new messages was done in process context while handling a > > message was in interrupt context. Because of the

Re: [PATCH 5/9] i2c: rcar: init new messages in irq

2015-10-21 Thread Laurent Pinchart
Hi Wolfram, On Thursday 03 September 2015 22:20:09 Wolfram Sang wrote: > From: Wolfram Sang > > 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 t

[PATCH 5/9] i2c: rcar: init new messages in irq

2015-09-03 Thread Wolfram Sang
From: Wolfram Sang 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, thi