From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 4/4] drm/nouveau: use i2c encoder helper wrappers Date: Thu, 24 Jan 2013 15:46:08 +0100 Message-ID: <20130124144608.GE31306@phenom.ffwll.local> References: <1357704526-3447-1-git-send-email-robdclark@gmail.com> <1357704526-3447-5-git-send-email-robdclark@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-we0-f174.google.com (mail-we0-f174.google.com [74.125.82.174]) by gabe.freedesktop.org (Postfix) with ESMTP id A5DB6E5CFA for ; Thu, 24 Jan 2013 06:44:03 -0800 (PST) Received: by mail-we0-f174.google.com with SMTP id r6so1497587wey.33 for ; Thu, 24 Jan 2013 06:44:02 -0800 (PST) Content-Disposition: inline In-Reply-To: <1357704526-3447-5-git-send-email-robdclark@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Rob Clark Cc: dri-devel@lists.freedesktop.org, patches@linaro.org List-Id: dri-devel@lists.freedesktop.org On Tue, Jan 08, 2013 at 10:08:46PM -0600, Rob Clark wrote: > Signed-off-by: Rob Clark Yeah, this looks a lot better, and even fixes a small leak on module unload since the allocated func-table isn't freed apparently. Reviewed-by: Daniel Vetter -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch