[Pixman] [PATCH] Add support for 18bpp X14R6G6B6 format.
Soeren Sandmann
sandmann at daimi.au.dk
Sat Jul 31 03:38:53 PDT 2010
Marek Vasut <marek.vasut at gmail.com> writes:
> This format is used on PXA framebuffer with some boards.
Looks good in general; just a couple of comments, the main one being
that the format needs to be added to pixman_format_supported_source().
The format is 32bpp, not 18bpp as you say in the subject line.
See below for some style issues.
> Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> ---
> pixman/pixman-access.c | 68 +++++++++++++++++++++++++++++++++++++++++++++++-
> pixman/pixman.h | 1 +
> 2 files changed, 68 insertions(+), 1 deletions(-)
>
> diff --git a/pixman/pixman-access.c b/pixman/pixman-access.c
> index 80fa9e8..82a28de 100644
> --- a/pixman/pixman-access.c
> +++ b/pixman/pixman-access.c
> @@ -397,6 +397,31 @@ fetch_scanline_b8g8r8 (pixman_image_t *image,
> }
>
> static void
> +fetch_scanline_x14r6g6b6 (pixman_image_t *image,
> + int x,
> + int y,
> + int width,
> + uint32_t * buffer,
> + const uint32_t *mask,
> + uint32_t mask_bits)
Please make sure all the arguments are aligned, instead of just
leaving the indentation in place from where cutted-and-pasted. This
applies to the other functions as well.
> +{
> + const uint32_t *bits = image->bits.bits + y * image->bits.rowstride;
> + const uint32_t *pixel = (const uint32_t *)bits + x;
> + const uint32_t *end = pixel + width;
> +
> + while (pixel < end) {
Braces go on their own line.
Finally, the functions in pixman-access.c are generally in the same
order as the formats are listed in pixman.h.
Thanks,
Soren
More information about the Pixman
mailing list