From mboxrd@z Thu Jan 1 00:00:00 1970 From: Magnus Damm Date: Mon, 07 Sep 2015 16:04:46 +0000 Subject: Re: [PATCH 0/9] i2c: rcar: tackle race conditions in the driver Message-Id: List-Id: References: <1441311613-2681-1-git-send-email-wsa@the-dreams.de> <5464456.UnsMOS3MTx@avalon> <20150903204000.GB1574@katana> <20150905073143.GA1616@katana> In-Reply-To: <20150905073143.GA1616@katana> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Wolfram Sang Cc: Laurent Pinchart , Linux-I2C , SH-Linux , Simon Horman , Geert Uytterhoeven , Kuninori Morimoto , Yoshihiro Kaneko , Sergei Shtylyov Hi Wolfram, On Sat, Sep 5, 2015 at 4:31 PM, Wolfram Sang wrote: > On Fri, Sep 04, 2015 at 01:33:39PM +0900, Magnus Damm wrote: >> Hi Wolfram, >> >> On Fri, Sep 4, 2015 at 5:40 AM, Wolfram Sang wrote: >> > >> >> > So I refactored the driver to setup new messages in interrupt context, too. >> >> > This avoids the race for b) because we are now setting up the new message >> >> > before we release the i2c bus clock (before we released the clock and set up >> >> > the message in process context). >> >> >> >> Could this fix the HDMI EDID read issue on Koelsch ? >> > >> > I surely hope so. I can't test because I don't have Koelsch. >> >> FYI, I have enabled Koelsch remote access on port 8001 if you want to >> give it a shot! > > Thanks! Is there a HDMI monitor connected? Yes, now a HD-ready TV is hooked up. It should give you something over EDID. Let me know when you're done and I'll move things back. Cheers, / magnus From mboxrd@z Thu Jan 1 00:00:00 1970 From: Magnus Damm Subject: Re: [PATCH 0/9] i2c: rcar: tackle race conditions in the driver Date: Tue, 8 Sep 2015 01:04:46 +0900 Message-ID: References: <1441311613-2681-1-git-send-email-wsa@the-dreams.de> <5464456.UnsMOS3MTx@avalon> <20150903204000.GB1574@katana> <20150905073143.GA1616@katana> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20150905073143.GA1616@katana> Sender: linux-sh-owner@vger.kernel.org To: Wolfram Sang Cc: Laurent Pinchart , Linux-I2C , SH-Linux , Simon Horman , Geert Uytterhoeven , Kuninori Morimoto , Yoshihiro Kaneko , Sergei Shtylyov List-Id: linux-i2c@vger.kernel.org Hi Wolfram, On Sat, Sep 5, 2015 at 4:31 PM, Wolfram Sang wrote: > On Fri, Sep 04, 2015 at 01:33:39PM +0900, Magnus Damm wrote: >> Hi Wolfram, >> >> On Fri, Sep 4, 2015 at 5:40 AM, Wolfram Sang wrote: >> > >> >> > So I refactored the driver to setup new messages in interrupt context, too. >> >> > This avoids the race for b) because we are now setting up the new message >> >> > before we release the i2c bus clock (before we released the clock and set up >> >> > the message in process context). >> >> >> >> Could this fix the HDMI EDID read issue on Koelsch ? >> > >> > I surely hope so. I can't test because I don't have Koelsch. >> >> FYI, I have enabled Koelsch remote access on port 8001 if you want to >> give it a shot! > > Thanks! Is there a HDMI monitor connected? Yes, now a HD-ready TV is hooked up. It should give you something over EDID. Let me know when you're done and I'll move things back. Cheers, / magnus