From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756142Ab2CRSVY (ORCPT ); Sun, 18 Mar 2012 14:21:24 -0400 Received: from mail-wi0-f178.google.com ([209.85.212.178]:53066 "EHLO mail-wi0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755288Ab2CRSVX (ORCPT ); Sun, 18 Mar 2012 14:21:23 -0400 Date: Sun, 18 Mar 2012 19:22:05 +0100 From: Daniel Vetter To: Daniel Kurtz Cc: Keith Packard , David Airlie , Daniel Vetter , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Benson Leung , Yufeng Shen , Sameer Nanda Subject: Re: [PATCH 00/10 v2] fix gmbus writes and related issues Message-ID: <20120318182204.GH4286@phenom.ffwll.local> Mail-Followup-To: Daniel Kurtz , Keith Packard , David Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Benson Leung , Yufeng Shen , Sameer Nanda References: <1331318904-20577-1-git-send-email-djkurtz@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1331318904-20577-1-git-send-email-djkurtz@chromium.org> X-Operating-System: Linux phenom 3.2.0-1-amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Mar 10, 2012 at 02:48:14AM +0800, Daniel Kurtz wrote: > This patchset addresses a couple of issues with the i915 gmbus implementation: > * fixes misassigned pin port pair for HDMI-D > * fixes write transactions when they are the only transaction requested > (including large >4-byte writes) by terminating every transaction with a > WAIT cycle. > * returns -ENXIO and -ETIMEDOUT as appropriate so upper layers can handled > i2c transaction failures > * optimizes the typical read transaction case by using the INDEX cycle > * turns on the GMBUS interrupt whenever possible to dramatically improve > throughput by eliminating the very slow polling loop. > > The patchset should apply cleanly onto linus/master. > It is inspired by, but completely supercedes, a similar patch submitted > recently by Benson Leung (bleung@chromium.org). > I'd be happy to rebase on a different tree if necessary. > > v2 of the patchset address review comments from Chris Wilson (thanks for the > review, Chris!), and adds the interrupt patch. There weren't any review > comments for patches 5, 7, or 8 of the first set. Hopefully they will get > more love the second time around :). Oops, still catching up - I've commented on the first thread. Please resubmit on top of drm-intel-next-queued and I'll have a look. Also, please cc me so I don't miss them. Thanks, Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48