Re: [PATCH] drm/msm/dpu: Use the encoder for default CRC source

2023-10-08 Thread Rob Clark
On Sun, Oct 8, 2023 at 4:21 PM Helen Koike wrote: > > > > On 08/10/2023 16:59, Dmitry Baryshkov wrote: > > On Sun, 8 Oct 2023 at 20:56, Rob Clark wrote: > >> > >> From: Rob Clark > >> > >> i-g-t expects the CRC to reflect any applied CTM. But the layer mixer > >> source is upstream of the DSPP,

Re: [PATCH] drm/msm/dpu: Use the encoder for default CRC source

2023-10-08 Thread Helen Koike
On 08/10/2023 16:59, Dmitry Baryshkov wrote: On Sun, 8 Oct 2023 at 20:56, Rob Clark wrote: From: Rob Clark i-g-t expects the CRC to reflect any applied CTM. But the layer mixer source is upstream of the DSPP, so it is before the CTM is applied. Switch the default source to 'encoder' ins

Re: [PATCH] drm/msm/dpu: Use the encoder for default CRC source

2023-10-08 Thread Dmitry Baryshkov
On Sun, 8 Oct 2023 at 20:56, Rob Clark wrote: > > From: Rob Clark > > i-g-t expects the CRC to reflect any applied CTM. But the layer mixer > source is upstream of the DSPP, so it is before the CTM is applied. > > Switch the default source to 'encoder' instead so that the CRC is > captured downs