[U-Boot] [PATCH 06/10] video: exynos: fimd: always use 16bpp display mode.

[Donghwa Lee]

On 10 Dec 2013 17:15, Przemyslaw Marczak wrote:
> Hello Donghwa,
>
> On 12/10/2013 06:47 AM, Donghwa Lee wrote:
>> Hi,
>>
>> On 3 Dec 2013 18:03, Przemyslaw wrote:
>>> This change updates exynos board files too.
>>> 16 bpp mode is required by LCD console mode.
>>>
>>> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
>>> ---
>>>   board/samsung/trats/trats.c              |    2 +-
>>>   board/samsung/trats2/trats2.c            |    2 +-
>>>   board/samsung/universal_c210/universal.c |    2 +-
>>>   drivers/video/exynos_fimd.c              |   10 ++--------
>>>   include/configs/s5pc210_universal.h      |    2 +-
>>>   include/configs/trats.h                  |    2 +-
>>>   include/configs/trats2.h                 |    2 +-
>>>   7 files changed, 8 insertions(+), 14 deletions(-)
>>>
>>> diff --git a/board/samsung/trats/trats.c b/board/samsung/trats/trats.c
>>> index ce4b41f..db527c7 100644
>>> --- a/board/samsung/trats/trats.c
>>> +++ b/board/samsung/trats/trats.c
>>> @@ -739,7 +739,7 @@ vidinfo_t panel_info = {
>>>   	.vl_hsp		= CONFIG_SYS_LOW,
>>>   	.vl_vsp		= CONFIG_SYS_LOW,
>>>   	.vl_dp		= CONFIG_SYS_LOW,
>>> -	.vl_bpix	= 5,	/* Bits per pixel, 2^5 = 32 */
>>> +	.vl_bpix	= 4,	/* Bits per pixel, 2^4 = 16 */
>>>   
>>>   	/* s6e8ax0 Panel infomation */
>>>   	.vl_hspw	= 5,
>>> diff --git a/board/samsung/trats2/trats2.c b/board/samsung/trats2/trats2.c
>>> index 73b8cc1..6fa02c9 100644
>>> --- a/board/samsung/trats2/trats2.c
>>> +++ b/board/samsung/trats2/trats2.c
>>> @@ -542,7 +542,7 @@ vidinfo_t panel_info = {
>>>   	.vl_hsp		= CONFIG_SYS_LOW,
>>>   	.vl_vsp		= CONFIG_SYS_LOW,
>>>   	.vl_dp		= CONFIG_SYS_LOW,
>>> -	.vl_bpix	= 5,	/* Bits per pixel, 2^5 = 32 */
>>> +	.vl_bpix	= 4,	/* Bits per pixel, 2^4 = 16 */
>>>   
>>>   	/* s6e8ax0 Panel infomation */
>>>   	.vl_hspw	= 5,
>>> diff --git a/board/samsung/universal_c210/universal.c
>>> b/board/samsung/universal_c210/universal.c
>>> index 166d5ee..1ebea0f 100644
>>> --- a/board/samsung/universal_c210/universal.c
>>> +++ b/board/samsung/universal_c210/universal.c
>>> @@ -446,7 +446,7 @@ vidinfo_t panel_info = {
>>>   	.vl_vsp		= CONFIG_SYS_HIGH,
>>>   	.vl_dp		= CONFIG_SYS_HIGH,
>>>   
>>> -	.vl_bpix	= 5,	/* Bits per pixel */
>>> +	.vl_bpix	= 4,	/* Bits per pixel */
>>>   
>>>   	/* LD9040 LCD Panel */
>>>   	.vl_hspw	= 2,
>>> diff --git a/drivers/video/exynos_fimd.c b/drivers/video/exynos_fimd.c
>>> index f962c4f..847fbe8 100644
>>> --- a/drivers/video/exynos_fimd.c
>>> +++ b/drivers/video/exynos_fimd.c
>>> @@ -73,18 +73,12 @@ static void exynos_fimd_set_par(unsigned int win_id)
>>>   	/* DATAPATH is DMA */
>>>   	cfg |= EXYNOS_WINCON_DATAPATH_DMA;
>>>   
>>> -	if (pvid->logo_on) /* To get proprietary LOGO */
>>> -		cfg |= EXYNOS_WINCON_WSWP_ENABLE;
>>> -	else /* To get output console on LCD */
>>> -		cfg |= EXYNOS_WINCON_HAWSWP_ENABLE;
>>> +	cfg |= EXYNOS_WINCON_HAWSWP_ENABLE;
>>>   
>>>   	/* dma burst is 16 */
>>>   	cfg |= EXYNOS_WINCON_BURSTLEN_16WORD;
>>>   
>>> -	if (pvid->logo_on) /* To get proprietary LOGO */
>>> -		cfg |= EXYNOS_WINCON_BPPMODE_24BPP_888;
>>> -	else /* To get output console on LCD */
>>> -		cfg |= EXYNOS_WINCON_BPPMODE_16BPP_565;
>>> +	cfg |= EXYNOS_WINCON_BPPMODE_16BPP_565;
>>>   
>> Why does exynos fimd always use 16bpp display mode?
> Good question. I switched fimd to 16BPP because of LCD framework which
> supports in max 16BPP mode for LCD console. Leaving fimd 32BPP mode and
> 16BPP console is possible but it causes wrong setup of lcd console row
> and columns, it also displays wrong size fonts.
>
>> It could be used as other bpp modes in exynos.
>> How do you use above vl_bpix variable in panel_info structure to check bpp mode?
> Actually vl_bpix was never used to check BPP mode in fimd driver, it was
> only used to set proper memory space. Fimd BPP mode was set by checking
> logo_on in panel_info structure.
> This is good reason to use vl_bpix in fimd driver to set proper BPP. I
> will change this to something like this:
>
> if (pvid->vl_bpix == 4)
> 	cfg |= EXYNOS_WINCON_BPPMODE_16BPP_565;
> else
> 	cfg |= EXYNOS_WINCON_BPPMODE_24BPP_888;
>
> Do you agree?

There is many others bpp modes, but in most cases, we usually use 16bpp or 24bpp modes.
To support many bpp modes more than two, it looks good to use 'switch' like below.

switch(pvid->vl_bpix) {
case 0;
...
case 1:
...
...
case 5:
...
}

BR,
Donghwa Lee.