[U-Boot] [PATCH 05/18] dm: video: Add a video uclass
Anatolij Gustschin
agust at denx.de
Sun Jan 17 19:35:54 CET 2016
Hi Simon,
Some minor comments below. Otherwise
Acked-by: Anatolij Gustschin <agust at denx.de>
On Tue, 5 Jan 2016 09:31:01 -0700
Simon Glass <sjg at chromium.org> wrote:
...
> +config VIDEO_BPP8
> + bool "Support 8-bit-per-pixel displays"
> + depends on DM_VIDEO
> + default y if DM_VIDEO
> + help
> + Support drawing text and bitmaps onto a 8-bit-per-pixel display.
> + Ebabling this will include code to support this display. Without
s/Ebabling/Enabling/
> +config VIDEO_BPP16
> + bool "Support 16-bit-per-pixel displays"
> + depends on DM_VIDEO
> + default y if DM_VIDEO
> + help
> + Support drawing text and bitmaps onto a 16-bit-per-pixel display.
> + Ebabling this will include code to support this display. Without
same typo here.
> +config VIDEO_BPP32
> + bool "Support 32-bit-per-pixel displays"
> + depends on DM_VIDEO
> + default y if DM_VIDEO
> + help
> + Support drawing text and bitmaps onto a 32-bit-per-pixel display.
> + Ebabling this will include code to support this display. Without
s/Ebabling/Enabling/
...
> +/**
> + * struct video_priv - Device information used by the video uclass
> + *
> + * @xsize: Number of pixel columns (e.g. 1366)
> + * @ysize: Number of pixels rows (e.g.. 768)
> + * @tor: Display rotation (0=none, 1=90 degrees clockwise, etc.)
> + * @bpix: Encoded bits per pixel
> + * @fb: Frame buffer
> + * @fb_size: Frame buffer size
> + * @fb_size: Frame buffer size
Please remove duplicated line here.
...
> +/**
> + * video_get_xsize() - Get the height of the display in pixels
s/_xsize/_ysize/
Thanks,
Anatolij
More information about the U-Boot
mailing list