On 2/21/19 5:05 PM, David Rivshin wrote:
From: David Rivshin <drivs...@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 <drivs...@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

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to