On Tue, 15 Jan 2019 17:17:52 +0100
Neil Armstrong narmstr...@baylibre.com wrote:

> This adds video output support for Amlogic GXBB/GXL/GXM chips.
> The supported ports are CVBS and HDMI (based on DW_HDMI).
> 
> When using HDMI, only DMT modes are supported.
> 
> There is support for simple-framebuffer (CONFIG_VIDEO_DT_SIMPLEFB)
> 
> Signed-off-by: Neil Armstrong <narmstr...@baylibre.com>
> Signed-off-by: Jorge Ramire-Ortiz <jrami...@baylibre.com>
> Signed-off-by: Maxime Jourdan <mjour...@baylibre.com>

Reviewed-by: Anatolij Gustschin <ag...@denx.de>

Please see minor comment below.

> diff --git a/drivers/video/meson/meson_canvas.c 
> b/drivers/video/meson/meson_canvas.c
...
> +/* DMC Registers */
> +#define DMC_CAV_LUT_DATAL    0x48 /* 0x12 offset in data sheet */
> +#define CANVAS_WIDTH_LBIT    29
> +#define CANVAS_WIDTH_LWID       3
> +#define DMC_CAV_LUT_DATAH    0x4c /* 0x13 offset in data sheet */
> +#define CANVAS_WIDTH_HBIT       0
> +#define CANVAS_HEIGHT_BIT       9
> +#define CANVAS_BLKMODE_BIT      24
> +#define DMC_CAV_LUT_ADDR     0x50 /* 0x14 offset in data sheet */
> +#define CANVAS_LUT_WR_EN        (0x2 << 8)
> +#define CANVAS_LUT_RD_EN        (0x1 << 8)

Some define values are not aligned by tab here.

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

Reply via email to