[U-Boot] [PATCH 4/7] EXYNOS: support EXYNOS framebuffer and FIMD display drivers.
Anatolij Gustschin
agust at denx.de
Thu Apr 5 14:28:14 CEST 2012
Hi,
Thank you for reworking this patch. It is mostly okay now and I can compile
the drivers, :-). Only two minor issues, please see comments below.
On Thu, 05 Apr 2012 15:29:27 +0900
Donghwa Lee <dh09.lee at samsung.com> wrote:
> This patch support EXYNOS FB and FIMD display drivers.
>
> Signed-off-by: Donghwa Lee <dh09.lee at samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> Signed-off-by: Inki Dae <inki.dae at samsung.com>
> ---
> arch/arm/include/asm/arch-exynos/fb.h | 446 +++++++++++++++++++++++++++++++++
> drivers/video/Makefile | 4 +
> drivers/video/exynos_fb.c | 129 ++++++++++
> drivers/video/exynos_fb.h | 61 +++++
> drivers/video/exynos_fimd.c | 354 ++++++++++++++++++++++++++
> 5 files changed, 994 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/include/asm/arch-exynos/fb.h
> create mode 100644 drivers/video/exynos_fb.c
> create mode 100644 drivers/video/exynos_fb.h
> create mode 100644 drivers/video/exynos_fimd.c
...
> diff --git a/drivers/video/Makefile b/drivers/video/Makefile
> index 6252f6a..4fad20d 100644
> --- a/drivers/video/Makefile
> +++ b/drivers/video/Makefile
> @@ -28,7 +28,11 @@ LIB := $(obj)libvideo.o
...
> COBJS-$(CONFIG_FSL_DIU_FB) += fsl_diu_fb.o videomodes.o
> +COBJS-$(CONFIG_S6E8AX0) += s6e8ax0.o
The s6e8ax0 driver is added in another patch, so please move
COBJS-$(CONFIG_S6E8AX0) += s6e8ax0.o to the patch which adds s6e8ax0
driver.
...
> diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c
> new file mode 100644
> index 0000000..c6aaecf
> --- /dev/null
> +++ b/drivers/video/exynos_fb.c
...
> +static void exynos_lcd_init_mem(void *lcdbase, vidinfo_t *vid)
> +{
> + unsigned long palette_size, palette_mem_size;
> + unsigned int fb_size;
> +
> + fb_size = vid->vl_row * vid->vl_col * (vid->vl_bpix >> 3);
> +
> + lcd_base = lcdbase;
> +
> + palette_size = NBITS(vid->vl_bpix) == 8 ? 256 : 16;
> + palette_mem_size = palette_size * sizeof(u32);
Now I can compile the driver and see the following warning:
exynos_fb.c: In function 'exynos_lcd_init_mem':
exynos_fb.c:52:30: warning: variable 'palette_mem_size' set but not used
[-Wunused-but-set-variable]
Please drop 'palette_mem_size' local variable here.
Thanks,
Anatolij
More information about the U-Boot
mailing list