[PATCH] video: add PXA3xx accelerator ID [v2]
Guennadi Liakhovetski
g.liakhovetski at gmx.de
Thu Sep 10 09:31:34 EDT 2009
On Thu, 10 Sep 2009, Daniel Mack wrote:
> Ok, here goes a new patch as just discussed.
> Does that look good to everyone?
>
> Thanks,
> Daniel
>
> >From 32b1fa824cd78f9ba6169b90c5d01870de05c86f Mon Sep 17 00:00:00 2001
> From: Daniel Mack <daniel at caiaq.de>
> Date: Mon, 20 Jul 2009 11:18:43 +0200
> Subject: [PATCH] video: add PXA3xx accelerator ID
>
> Add ID 99 for PXA3xx frame buffers and report it in the pxa frame buffer
> conditionally, depending on a new flag in struct pxafb_mach_info.
>
> Signed-off-by: Daniel Mack <daniel at caiaq.de>
> Cc: Eric Miao <eric.miao at marvell.com>
> Cc: linux-fbdev-devel at lists.sourceforge.net
> Cc: Dennis Oliver Kropp <dok at directfb.org>
> Cc: Sven Neumann <s.neumann at raumfeld.com>
> ---
> arch/arm/mach-pxa/include/mach/pxafb.h | 3 ++-
> drivers/video/pxafb.c | 3 +++
> include/linux/fb.h | 1 +
> 3 files changed, 6 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-pxa/include/mach/pxafb.h b/arch/arm/mach-pxa/include/mach/pxafb.h
> index 6932720..f73061c 100644
> --- a/arch/arm/mach-pxa/include/mach/pxafb.h
> +++ b/arch/arm/mach-pxa/include/mach/pxafb.h
> @@ -118,7 +118,8 @@ struct pxafb_mach_info {
> u_int fixed_modes:1,
> cmap_inverse:1,
> cmap_static:1,
> - unused:29;
> + acceleration_enabled:1,
> + unused:28;
>
> /* The following should be defined in LCCR0
> * LCCR0_Act or LCCR0_Pas Active or Passive
> diff --git a/drivers/video/pxafb.c b/drivers/video/pxafb.c
> index 3951cc5..b65cc2d 100644
> --- a/drivers/video/pxafb.c
> +++ b/drivers/video/pxafb.c
> @@ -2083,6 +2083,9 @@ static int __devinit pxafb_probe(struct platform_device *dev)
> goto failed;
> }
>
> + if (cpu_is_pxa3xx() && inf->acceleration_enabled)
> + fbi->fb.fix.accel = FB_ACCEL_PXA3XX;
> +
Ok, now remove 'cpu_is_pxa3xx() && ' above and you should be good to go:-)
What if tomorrow pxa4xx comes (ok, Eric will disagree, but wtill:-))
> fbi->backlight_power = inf->pxafb_backlight_power;
> fbi->lcd_power = inf->pxafb_lcd_power;
>
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index f847df9..a34bdf5 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -133,6 +133,7 @@ struct dentry;
> #define FB_ACCEL_NEOMAGIC_NM2230 96 /* NeoMagic NM2230 */
> #define FB_ACCEL_NEOMAGIC_NM2360 97 /* NeoMagic NM2360 */
> #define FB_ACCEL_NEOMAGIC_NM2380 98 /* NeoMagic NM2380 */
> +#define FB_ACCEL_PXA3XX 99 /* PXA3xx */
>
> #define FB_ACCEL_SAVAGE4 0x80 /* S3 Savage4 */
> #define FB_ACCEL_SAVAGE3D 0x81 /* S3 Savage3D */
> --
> 1.6.3.3
>
Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
More information about the linux-arm-kernel
mailing list