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

[Shobhit Kumar <shobhit.kumar@linux.intel.com>]

On 03/25/2015 06:43 PM, Daniel Vetter wrote:
> 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?

Will work on this.

Regards
Shobhit

> 
> 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
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx