Hi Simon,

Some minor comments below. Otherwise

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


On Tue,  5 Jan 2016 09:31:01 -0700
Simon Glass <s...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to