From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Glauber Subject: Re: [PATCH v2 0/3] i2c: octeon: thunder: Fix i2c not working on Octeon Date: Tue, 22 Nov 2016 15:55:39 +0100 Message-ID: <20161122145539.GA7716@hardcore> References: <1479149445-4663-1-git-send-email-jglauber@cavium.com> <99500824-4c63-b769-ad66-c136529b14b2@cavium.com> <20161122120106.GB3993@katana> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <20161122120106.GB3993@katana> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: "Steven J. Hill" Cc: Wolfram Sang , linux-i2c@vger.kernel.org, linux-mips@linux-mips.org, Paul Burton , David Daney List-Id: linux-i2c@vger.kernel.org On Tue, Nov 22, 2016 at 01:01:06PM +0100, Wolfram Sang wrote: > On Mon, Nov 14, 2016 at 01:53:40PM -0600, Steven J. Hill wrote: > > On 11/14/2016 12:50 PM, Jan Glauber wrote: > > > > > > Since time is running out for 4.9 (or might have already if you're not > > > going to send another pull request) I'm going for the safe option > > > to fix the Octeon i2c problems, which is: > > > > > > 1. Reverting the readq_poll_timeout patch since it is broken > > > 2. Apply Patch #2 from Paul > > > 3. Add a small fix for the recovery that makes Paul's patch > > > work on ThunderX > > > > > > I'll try to come up with a better solution for 4.10. My plan is to get rid > > > of the polling-around-interrupt thing completely, but for that we need more > > > time to make it work on Octeon. > > > > > > Please consider for 4.9. > > > > > Hey Jan. > > > > This does not work on Octeon 71xx platforms. I will look at it more > > closely tomorrow. > > What's the outcome here? It seems we want a bugfix for 4.9 but this > report keeps me reluctant to apply the series. > Steven, did you have a chance to check which of the patches makes Octeon 71xx fail? --Jan