Re: [PATCH 2/2] drm/fourcc: Fill out all block sizes for P210

2019-11-28 Thread Daniel Vetter
On Tue, Nov 26, 2019 at 11:46:54AM +, Liviu Dudau wrote: > On Tue, Nov 26, 2019 at 10:14:14AM +0100, Daniel Vetter wrote: > > 0 means 1 as the default, but it's mighty confusing if the block size > > for the first plane is spelled out explicitly, but not for the 2nd > > plane. > > > > No cc:

Re: [PATCH 2/2] drm/fourcc: Fill out all block sizes for P210

2019-11-26 Thread Liviu Dudau
On Tue, Nov 26, 2019 at 10:14:14AM +0100, Daniel Vetter wrote: > 0 means 1 as the default, but it's mighty confusing if the block size > for the first plane is spelled out explicitly, but not for the 2nd > plane. > > No cc: stable because this is just confusion, but 0 functional issue. Agree! >

[PATCH 2/2] drm/fourcc: Fill out all block sizes for P210

2019-11-26 Thread Daniel Vetter
0 means 1 as the default, but it's mighty confusing if the block size for the first plane is spelled out explicitly, but not for the 2nd plane. No cc: stable because this is just confusion, but 0 functional issue. Fixes: 7ba0fee247ee ("drm/fourcc: Add AFBC yuv fourccs for Mali") Cc: Brian