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

[Daniel Vetter <daniel@ffwll.ch>]

On Wed, Mar 25, 2015 at 01:24:01PM +0100, Linus Walleij wrote:
> On Tue, Mar 24, 2015 at 11:53 AM, Daniel Vetter <daniel@ffwll.ch> wrote:
> > On Tue, Mar 24, 2015 at 11:16 AM, Linus Walleij
> > <linus.walleij@linaro.org> wrote:
> 
> > So summary:
> > - Reusing the dynamic gpio lookup stuff would be nice, and might be
> > interesting as a new crazy use-case (or maybe not). But not a
> > requirement since we have the component framework to handroll
> > something.
> 
> OK I guess you have me convinced, I will apply the patch from
> Shobhit. If it turns out ugly we can always revert it. If you believe
> in it, it's worth a try.
> 
> Also as you say else it will be reinvented, let's go this way as it
> is likely the lesser of two evils.

Thanks for reconsidering.

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?

Wrt merging the 4.1 window on the drm side is pretty much closed so I
think I'll have to postpone the i915 side to 4.2 anyway. Luckily there's
no direct depencies because we look up everything dynamically, so patches
can go in in any order.

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