[U-Boot] [PATCH] am335x-fb: don't override lcd_get_size()

Hannes Schmelzer hannes at schmelzer.or.at
Mon Feb 25 11:54:57 UTC 2019


On 2/21/19 5:05 PM, David Rivshin wrote:
> From: David Rivshin <DRivshin at allworx.com>
>
> am335x-fb overrides the default lcd_get_size() to add an extra 32 bytes
> compared to the normal calculation. While the gd->fb_base is an extra 32
> bytes larger than the logical framebuffer size, the first 32 bytes is
> always skipped. Adding that extra 32 bytes in lcd_get_size() can cause
> check_cache_range() (called from lcd_sync(), via flush_dcache_range()) to
> output a warning because the end address is not cacheline aligned, even
> if the start and real framebuffer size are.
>
> When removing that "+ 0x20" the am335x-fb lcd_get_size() is identical
> to the default one, so just delete it altogether.
>
> Signed-off-by: David Rivshin <drivshin at allworx.com>
> ---
>   drivers/video/am335x-fb.c | 6 ------
>   1 file changed, 6 deletions(-)
>
> diff --git a/drivers/video/am335x-fb.c b/drivers/video/am335x-fb.c
> index 51c1af587f..3a510cf91c 100644
> --- a/drivers/video/am335x-fb.c
> +++ b/drivers/video/am335x-fb.c
> @@ -103,12 +103,6 @@ static struct am335x_lcdhw *lcdhw = (void *)LCD_CNTL_BASE;
>   
>   DECLARE_GLOBAL_DATA_PTR;
>   
> -int lcd_get_size(int *line_length)
> -{
> -	*line_length = (panel_info.vl_col * NBITS(panel_info.vl_bpix)) / 8;
> -	return *line_length * panel_info.vl_row + 0x20;
> -}
> -
>   int am335xfb_init(struct am335x_lcdpanel *panel)
>   {
>   	u32 raster_ctrl = 0;
>
> base-commit: d3689267f92c5956e09cc7d1baa4700141662bff
I'll look into.
But it's not easy as it looks like.

Just downloaded latest TRM, looks like documentation had become better.

cheers,
Hannes



More information about the U-Boot mailing list