Re: Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-24 Thread Jernej Škrabec
Dne nedelja, 22. maj 2022 ob 12:48:27 CEST je benly...@gmail.com napisal(a): > Hi Jernej! > > > Which SDK? All SDKs that I have or found on internet don't mention YUV nor RGB > > scanline limit. That doesn't mean there is none, I'm just unable to verify > > your claim. Did you test this by yourse

Re: Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-24 Thread Jernej Škrabec
Dne torek, 24. maj 2022 ob 19:07:27 CEST je Samuel Holland napisal(a): > On 5/23/22 8:14 AM, Icenowy Zheng wrote: > > 在 2022-05-22星期日的 10:36 +0200,Jernej Škrabec写道: > >> Hi! > >> > >> Dne sobota, 21. maj 2022 ob 15:34:43 CEST je Genfu Pan napisal(a): > >>> Accrording the SDK from Allwinner, the sca

Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-24 Thread Samuel Holland
On 5/23/22 8:14 AM, Icenowy Zheng wrote: > 在 2022-05-22星期日的 10:36 +0200,Jernej Škrabec写道: >> Hi! >> >> Dne sobota, 21. maj 2022 ob 15:34:43 CEST je Genfu Pan napisal(a): >>> Accrording the SDK from Allwinner, the scanline value of yuv and >>> rgb for >>> V3s are both 1024. >> >> s/scanline value/sc

Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-23 Thread Icenowy Zheng
在 2022-05-22星期日的 10:36 +0200,Jernej Škrabec写道: > Hi! > > Dne sobota, 21. maj 2022 ob 15:34:43 CEST je Genfu Pan napisal(a): > > Accrording the SDK from Allwinner, the scanline value of yuv and > > rgb for > > V3s are both 1024. > > s/scanline value/scanline length/ > > Which SDK? All SDKs that I

[PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-22 Thread Genfu Pan
Accrording the SDK from Allwinner, the scanline value of yuv and rgb for V3s are both 1024. The is also the same for mixer 1 of D1. Currently the scanline value of rgb is hardcoded to 2048 for all SOCs. Change the scanline_yuv property of V3s to 1024. Add the scanline_rgb property to the mixer con

Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-22 Thread benly...@gmail.com
Hi Jernej! > Which SDK? All SDKs that I have or found on internet don't mention YUV nor RGB > scanline limit. That doesn't mean there is none, I'm just unable to verify > your claim. Did you test this by yourself? Also, please make YUV scanline > change separate patch with fixes tag. Here is the

Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-22 Thread benly...@gmail.com
Hi Jernej! > Which SDK? All SDKs that I have or found on internet don't mention YUV nor RGB > scanline limit. That doesn't mean there is none, I'm just unable to verify > your claim. Did you test this by yourself? Also, please make YUV scanline > change separate patch with fixes tag. Here is the

Re: [PATCH] drm/sun4i: mixer: fix scanline for V3s and D1

2022-05-22 Thread Jernej Škrabec
Hi! Dne sobota, 21. maj 2022 ob 15:34:43 CEST je Genfu Pan napisal(a): > Accrording the SDK from Allwinner, the scanline value of yuv and rgb for > V3s are both 1024. s/scanline value/scanline length/ Which SDK? All SDKs that I have or found on internet don't mention YUV nor RGB scanline limit.