[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support
From: |
Vasily Khoruzhick |
Subject: |
Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support |
Date: |
Wed, 8 Jun 2011 14:22:42 +0300 |
User-agent: |
KMail/1.13.7 (Linux/2.6.39-gentoo-anarsoul; KDE/4.6.3; i686; ; ) |
On Wednesday 08 June 2011 12:50:50 Peter Maydell wrote:
> On 3 June 2011 16:36, Vasily Khoruzhick <address@hidden> wrote:
> > Until now, pxa2xx_lcd only supported 90deg rotation, but
> > some machines (for example Zipit Z2) needs 270deg rotation.
> >
> > Signed-off-by: Vasily Khoruzhick <address@hidden>
>
> Still haven't tested this yet, but some very quick first
> pass comments.
>
> > +static void pxa2xx_lcdc_dma0_redraw_rot180(PXA2xxLCDState *s,
> > + target_phys_addr_t addr, int *miny, int *maxy)
> > +{
> > + int src_width, dest_width;
> > + drawfn fn = NULL;
> > + if (s->dest_width)
> > + fn = s->line_fn[s->transp][s->bpp];
> > + if (!fn)
> > + return;
>
> Braces (here and elsewhere); if we're adding whole new functions
> we might as well do them in accordance with the styl.
Oh, ok. I'll add patch to fix pxa2xx_lcd.c codestyle to the series, and then
fix this patch.
>
> > case QEMU_OPTION_portrait:
> > - graphic_rotate = 1;
> > + graphic_rotate = 90;
> > + break;
> > + case QEMU_OPTION_rotate:
> > + graphic_rotate = atoi(optarg);
> > + if (graphic_rotate != 0 && graphic_rotate != 90 &&
> > + graphic_rotate != 180 && graphic_rotate != 270) {
> > + fprintf(stderr,
> > + "qemu: only 90, 180, 270 deg rotation is
> > available\n"); + exit(1);
> > + }
>
> We seem to generally use strtol() for argument parsing rather
> than atoi, might be nicer to be consistent.
Ok
> The other bit of code that looks at graphic_rotate
> is hw/spitz.c:spitz_keyboard_register(). Does that require
> changes? It doesn't seem entirely right for us to assert
> 'tablet mode' for everything except 0deg rotation.
Well, -portrait argument is preserved, so all old code should work fine
(including scripts to start qemu). Passing -rotate 180 or -rotate 270 will
produce unpredictable result (but not a crash). I have no spitz image, so I
can't test what happens.
Regards
Vasily
- [Qemu-devel] [PATCH v2 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/01
- [Qemu-devel] [PATCH v2 2/2] Add support for Zipit Z2 machine, Vasily Khoruzhick, 2011/06/01
- Re: [Qemu-devel] [PATCH v2 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/03
- [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/03
- [Qemu-devel] [PATCH v3 2/2] Add support for Zipit Z2 machine, Vasily Khoruzhick, 2011/06/03
- Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/03
- Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/08
- Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support, Peter Maydell, 2011/06/08
- Re: [Qemu-devel] [PATCH v3 1/2] pxa2xx_lcd: add proper rotation support,
Vasily Khoruzhick <=
- [Qemu-devel] [PATCH v4 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/17
- [Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine, Vasily Khoruzhick, 2011/06/17
- Re: [Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine, Vasily Khoruzhick, 2011/06/21
- Re: [Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine, Peter Maydell, 2011/06/24
- Re: [Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine, Vasily Khoruzhick, 2011/06/24
- Re: [Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine, Peter Maydell, 2011/06/24
- Re: [Qemu-devel] [PATCH v4 1/2] pxa2xx_lcd: add proper rotation support, Vasily Khoruzhick, 2011/06/21