Re: [PATCH 3/9] drm/i915: Use the CRC gpio for panel enable/disable

From: Daniel Vetter <daniel@ffwll.ch>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Alexandre Courbot <gnurou@gmail.com>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Mark Brown <broonie@linaro.org>,
	Shobhit Kumar <shobhit.kumar@intel.com>,
	intel-gfx <intel-gfx@lists.freedesktop.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	Daniel Vetter <daniel.vetter@intel.com>
Subject: Re: [PATCH 3/9] drm/i915: Use the CRC gpio for panel enable/disable
Date: Wed, 25 Mar 2015 16:45:32 +0100	[thread overview]
Message-ID: <20150325154532.GK1349@phenom.ffwll.local> (raw)
In-Reply-To: <CACRpkdbTxk8U1MyLz_O5k4e3LummX+zc4bhNkrtxLcqrSDy31A@mail.gmail.com>

On Wed, Mar 25, 2015 at 03:55:43PM +0100, Linus Walleij wrote:
> On Wed, Mar 25, 2015 at 2:13 PM, Daniel Vetter <daniel@ffwll.ch> wrote:
> 
> > I quickly checked out your linux-gpio and it only has patch 2 to implement
> > the gpio. We also need patch 1 (but with the leak Thierry spotted fixed).
> > Should we drop Shobit's patch until that's done?
> 
> Nah I trust that Lee will merge it to MFD in due time, there is no
> compile-time dependence anyway. I Reviewed/ACKed the patch
> anyway, probably should have pointed out to fix the leak
> pointed out by Thierry but I count on you folks to fix it.
> 
> Can I add you ACK tag on the GPIO patch?

Sure. And yes Shobit is already working on the leak fix, so either it'll
be a fixup or new patch version. Shobit, to make sure that your patch will
apply please base the series on top of linux-next and not
drm-intel-nightly for the next iteration.

https://www.kernel.org/doc/man-pages/linux-next.html

Cheers, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx