[PATCH 00/76] DAL Patches Nov 21, 2016

Harry Wentland harry.wentland at amd.com
Wed Nov 23 02:31:46 UTC 2016


Hi Jerome,

I understand the frustration but the intention of this set isn't to 
review the entirety of dal. Alex has been pulling our patches into his 
amd-staging-4.7 branch and dal has been used in our PRO driver stack. 
This is just making these patches a bit more public and inviting 
feedback instead of pulling them quietly into some public tree.

Cheers,
Harry

On 2016-11-22 07:06 PM, Jerome Glisse wrote:
> I hate to shime in but from review point of view what i don't undersntand is why
> isn't there a single reviewable patchset for DAL ?
>
> What i mean by that is a patchset in which you collapse all fixes and cleanup so
> that there is an easy way to review thing. Yes this means loosing history and
> major work to just spawn a new patchset but quite frankly in this state this thing
> is just unreviewable. The only sane way today is to look at git tree with the whole
> thing applied and avoid looking at individual patch. That not a sane way to do
> review in my mind.
>
> That just my opinion other might disagree but i thought i would say that outloud.
>
> Cheers,
> Jérôme



More information about the amd-gfx mailing list