Re: [PATCH 00/71] drm/i915/chv: Add Cherryview support

S, Deepak
2014-04-09T16:27:06Z

Thanks Ville. I agree patches need polishing :). I might rewrite some of
the patches based on testing :)

Most of the BDW Rc6/Rps patches are in. I guess some of the interrupt
related patch are pending.

Thanks
Deepak

On 4/9/2014 8:35 PM, Ville Syrjälä wrote:
> On Wed, Apr 09, 2014 at 02:30:52PM +0000, S, Deepak wrote:
>> Hi Ville,
>>
>> I am Ok with cleaning up and pushing the Code. Can you please tell me when we need to start pushing the code and branch to use (drm-intel-next)?
>
> Well you can consider it pushed now that it's in the open. The patches
> just need a bit of extra polish I think. Well, unless you're planning
> a full blown rewrite of the code ;)
>
> I guess you need to take into consideration whatever bdw rc6/rps patches
> are still in flight, but since you've been doing some review there I
> think you have a better idea than I do how things are progressing.
>
> I always work on top of nightly, so I guess that's a good choice :)
>
>>
>> Thanks
>> Deepak
>>
>> -----Original Message-----
>> From: Ville Syrjälä [mailto:ville.syrjala@linux.intel.com]
>> Sent: Wednesday, April 9, 2014 6:55 PM
>> To: intel-gfx@lists.freedesktop.org
>> Cc: S, Deepak; Deak, Imre
>> Subject: Re: [PATCH 00/71] drm/i915/chv: Add Cherryview support
>>
>> As you may have noticed some of this stuf may need a bit of work still to fit in better with the more recent upstream changes.
>>
>> So I think we need a few people to do some work here to the the patches into a really good shape. So some division of labor would in order. I'm proposing the following:
>> rc6/turbo -> Deepak as he's written the code anyway interrupts -> Imre since he worked on the VLV interrupts recently
>>
>> The rest of the patches should be fairly OK I think. So for the rest we probably just need reviews mostly, and maybe we want to squash some of the more trivial patches.
>>
>> --
>> Ville Syrjälä
>> Intel OTC
>