[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v2] console: Correct computation of bytes per
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-trivial] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel |
Date: |
Wed, 22 Aug 2012 17:32:13 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-08-22 17:19, BALATON Zoltan wrote:
> Division with round up is the correct way to compute this even if the
> only case where division with round down gives incorrect result is
> probably 15 bpp. This case was explicitely patched up in one of these
> functions but was unhandled in the other. (I'm not sure about setting
> 16 bpp for the 15bpp case either but I left that there for now.)
>
> Signed-off-by: BALATON Zoltan <address@hidden>
> ---
> console.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> v2: Use DIV_ROUND_UP and extended commit message
>
> diff --git a/console.c b/console.c
> index 4525cc7..9df1701 100644
> --- a/console.c
> +++ b/console.c
> @@ -1612,7 +1612,7 @@ PixelFormat
> qemu_different_endianness_pixelformat(int bpp)
> memset(&pf, 0x00, sizeof(PixelFormat));
>
> pf.bits_per_pixel = bpp;
> - pf.bytes_per_pixel = bpp / 8;
> + pf.bytes_per_pixel = DIV_ROUND_UP(bpp, 8);
> pf.depth = bpp == 32 ? 24 : bpp;
>
> switch (bpp) {
> @@ -1661,13 +1661,12 @@ PixelFormat qemu_default_pixelformat(int bpp)
> memset(&pf, 0x00, sizeof(PixelFormat));
>
> pf.bits_per_pixel = bpp;
> - pf.bytes_per_pixel = bpp / 8;
> + pf.bytes_per_pixel = DIV_ROUND_UP(bpp, 8);
> pf.depth = bpp == 32 ? 24 : bpp;
>
> switch (bpp) {
> case 15:
> pf.bits_per_pixel = 16;
> - pf.bytes_per_pixel = 2;
Removed unintentionally?
Jan
--
Siemens AG, Corporate Technology, CT RTC ITP SDP-DE
Corporate Competence Center Embedded Linux
- [Qemu-trivial] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, BALATON Zoltan, 2012/08/22
- Re: [Qemu-trivial] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel,
Jan Kiszka <=
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Stefan Weil, 2012/08/22
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Jan Kiszka, 2012/08/22
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Jan Kiszka, 2012/08/22
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Stefan Hajnoczi, 2012/08/24
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Jan Kiszka, 2012/08/24
- Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Stefan Hajnoczi, 2012/08/24
Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel, Stefan Hajnoczi, 2012/08/24