[PATCH] drm/panel:Adjust sync values for Feixin K101-IM2BYL02 panel

[PATCH] drm/panel:Adjust sync values for Feixin K101-IM2BYL02 panel

[Dan Johansen]

This adjusts sync values according to the datasheet

Fixes: 	1c243751c095bb95e2795f076ea7a0bcdd60a93a ("drm/panel: ilitek-ili9881c: add support for Feixin K101-IM2BYL02 panel")
Co-developed-by: Marius Gripsgard <marius@ubports.com>
Signed-off-by: Dan Johansen <strit@manjaro.org>
---
 drivers/gpu/drm/panel/panel-ilitek-ili9881c.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
index 0145129d7c66..534dd7414d42 100644
--- a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
+++ b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
@@ -590,14 +590,14 @@ static const struct drm_display_mode k101_im2byl02_default_mode = {
 	.clock		= 69700,
 
 	.hdisplay	= 800,
-	.hsync_start	= 800 + 6,
-	.hsync_end	= 800 + 6 + 15,
-	.htotal		= 800 + 6 + 15 + 16,
+	.hsync_start	= 800 + 52,
+	.hsync_end	= 800 + 52 + 8,
+	.htotal		= 800 + 52 + 8 + 48,
 
 	.vdisplay	= 1280,
-	.vsync_start	= 1280 + 8,
-	.vsync_end	= 1280 + 8 + 48,
-	.vtotal		= 1280 + 8 + 48 + 52,
+	.vsync_start	= 1280 + 16,
+	.vsync_end	= 1280 + 16 + 6,
+	.vtotal		= 1280 + 16 + 6 + 15,
 
 	.width_mm	= 135,
 	.height_mm	= 217,
-- 
2.32.0

Re: [PATCH] drm/panel:Adjust sync values for Feixin K101-IM2BYL02 panel

[Sam Ravnborg]

Hi Dan,

On Wed, Aug 18, 2021 at 11:48:18PM +0200, Dan Johansen wrote:
> This adjusts sync values according to the datasheet
> 
> Fixes: 	1c243751c095bb95e2795f076ea7a0bcdd60a93a ("drm/panel: ilitek-ili9881c: add support for Feixin K101-IM2BYL02 panel")
> Co-developed-by: Marius Gripsgard <marius@ubports.com>
> Signed-off-by: Dan Johansen <strit@manjaro.org>

Sorry for not picking this up sooner - I have been away for a while.

The patch is now applied to drm-misc-fixes and will show up in the
upstream kernel in 1-2 week from now.
I updated the subject and the fixes tag while applying.

The Co-developed-by: is supposed to be paired with a Signed-off-by:
by the same person. Something to do right next time.

	Sam

> ---
>  drivers/gpu/drm/panel/panel-ilitek-ili9881c.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> index 0145129d7c66..534dd7414d42 100644
> --- a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
> @@ -590,14 +590,14 @@ static const struct drm_display_mode k101_im2byl02_default_mode = {
>  	.clock		= 69700,
>  
>  	.hdisplay	= 800,
> -	.hsync_start	= 800 + 6,
> -	.hsync_end	= 800 + 6 + 15,
> -	.htotal		= 800 + 6 + 15 + 16,
> +	.hsync_start	= 800 + 52,
> +	.hsync_end	= 800 + 52 + 8,
> +	.htotal		= 800 + 52 + 8 + 48,
>  
>  	.vdisplay	= 1280,
> -	.vsync_start	= 1280 + 8,
> -	.vsync_end	= 1280 + 8 + 48,
> -	.vtotal		= 1280 + 8 + 48 + 52,
> +	.vsync_start	= 1280 + 16,
> +	.vsync_end	= 1280 + 16 + 6,
> +	.vtotal		= 1280 + 16 + 6 + 15,
>  
>  	.width_mm	= 135,
>  	.height_mm	= 217,
> -- 
> 2.32.0

Re: [PATCH] drm/panel:Adjust sync values for Feixin K101-IM2BYL02 panel

[Dan Johansen]

[-- Attachment #1: Type: text/plain, Size: 2015 bytes --]

Thanks for applying.

Den 17.10.2021 kl. 19.24 skrev Sam Ravnborg:
> Hi Dan,
>
> On Wed, Aug 18, 2021 at 11:48:18PM +0200, Dan Johansen wrote:
>> This adjusts sync values according to the datasheet
>>
>> Fixes: 	1c243751c095bb95e2795f076ea7a0bcdd60a93a ("drm/panel: ilitek-ili9881c: add support for Feixin K101-IM2BYL02 panel")
>> Co-developed-by: Marius Gripsgard<marius@ubports.com>
>> Signed-off-by: Dan Johansen<strit@manjaro.org>
> Sorry for not picking this up sooner - I have been away for a while.
>
> The patch is now applied to drm-misc-fixes and will show up in the
> upstream kernel in 1-2 week from now.
> I updated the subject and the fixes tag while applying.
Thanks.
>
> The Co-developed-by: is supposed to be paired with a Signed-off-by:
> by the same person. Something to do right next time.
Didn't know that. Thanks, will remember for next time!
>
> 	Sam
>
>> ---
>>   drivers/gpu/drm/panel/panel-ilitek-ili9881c.c | 12 ++++++------
>>   1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
>> index 0145129d7c66..534dd7414d42 100644
>> --- a/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
>> +++ b/drivers/gpu/drm/panel/panel-ilitek-ili9881c.c
>> @@ -590,14 +590,14 @@ static const struct drm_display_mode k101_im2byl02_default_mode = {
>>   	.clock		= 69700,
>>   
>>   	.hdisplay	= 800,
>> -	.hsync_start	= 800 + 6,
>> -	.hsync_end	= 800 + 6 + 15,
>> -	.htotal		= 800 + 6 + 15 + 16,
>> +	.hsync_start	= 800 + 52,
>> +	.hsync_end	= 800 + 52 + 8,
>> +	.htotal		= 800 + 52 + 8 + 48,
>>   
>>   	.vdisplay	= 1280,
>> -	.vsync_start	= 1280 + 8,
>> -	.vsync_end	= 1280 + 8 + 48,
>> -	.vtotal		= 1280 + 8 + 48 + 52,
>> +	.vsync_start	= 1280 + 16,
>> +	.vsync_end	= 1280 + 16 + 6,
>> +	.vtotal		= 1280 + 16 + 6 + 15,
>>   
>>   	.width_mm	= 135,
>>   	.height_mm	= 217,
>> -- 
>> 2.32.0
-- 
Kind regards
*Dan Johansen*
Project lead of the *Manjaro ARM* project
Manjaro-ARM <https://manjaro.org>

[-- Attachment #2: Type: text/html, Size: 3003 bytes --]