Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-14 Thread Mark Yacoub
hmm I see, thanks for the heads up, I'll double check why it uses google email for sending. wrt the assignment in the if clauses, are those typically frowned upon? Thanks! On Fri, Jun 11, 2021 at 4:17 PM Alex Deucher wrote: > > Just a heads up, your sender email and your signed-off-by don't

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-14 Thread Alex Deucher
On Mon, Jun 14, 2021 at 10:49 AM Mark Yacoub wrote: > > hmm I see, thanks for the heads up, I'll double check why it uses > google email for sending. > wrt the assignment in the if clauses, are those typically frowned upon? Yes, checkpatch complains about them. Alex > Thanks! > > On Fri, Jun

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-11 Thread Alex Deucher
Just a heads up, your sender email and your signed-off-by don't match and you had some assignments in if clauses. I've fixed those up. Alex On Fri, Jun 11, 2021 at 1:35 PM Alex Deucher wrote: > > Applied. Thanks! > > On Thu, Jun 10, 2021 at 5:14 PM Harry Wentland wrote: > > > > > > > > On

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-11 Thread Alex Deucher
Applied. Thanks! On Thu, Jun 10, 2021 at 5:14 PM Harry Wentland wrote: > > > > On 2021-06-07 10:53 a.m., Mark Yacoub wrote: > > On Fri, Jun 4, 2021 at 4:17 PM Harry Wentland > > wrote: > >> > >> > >> > >> On 2021-06-04 1:01 p.m., Mark Yacoub wrote: > >>> From: Mark Yacoub > >>> > >>> For

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-10 Thread Harry Wentland
On 2021-06-07 10:53 a.m., Mark Yacoub wrote: > On Fri, Jun 4, 2021 at 4:17 PM Harry Wentland wrote: >> >> >> >> On 2021-06-04 1:01 p.m., Mark Yacoub wrote: >>> From: Mark Yacoub >>> >>> For each CRTC state, check the size of Gamma and Degamma LUTs so >>> unexpected and larger sizes wouldn't

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-07 Thread Mark Yacoub
On Fri, Jun 4, 2021 at 4:17 PM Harry Wentland wrote: > > > > On 2021-06-04 1:01 p.m., Mark Yacoub wrote: > > From: Mark Yacoub > > > > For each CRTC state, check the size of Gamma and Degamma LUTs so > > unexpected and larger sizes wouldn't slip through. > > > > TEST:

Re: [PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-04 Thread Harry Wentland
On 2021-06-04 1:01 p.m., Mark Yacoub wrote: > From: Mark Yacoub > > For each CRTC state, check the size of Gamma and Degamma LUTs so > unexpected and larger sizes wouldn't slip through. > > TEST: IGT:kms_color::pipe-invalid-gamma-lut-sizes > > Signed-off-by: Mark Yacoub > Change-Id:

[PATCH] drm/amd/display: Verify Gamma & Degamma LUT sizes in amdgpu_dm_atomic_check

2021-06-04 Thread Mark Yacoub
From: Mark Yacoub For each CRTC state, check the size of Gamma and Degamma LUTs so unexpected and larger sizes wouldn't slip through. TEST: IGT:kms_color::pipe-invalid-gamma-lut-sizes Signed-off-by: Mark Yacoub Change-Id: I9d513a38e8ac2af1b4bf802e1feb1a4d726fba4c ---