[PATCH] video: add PXA3xx accelerator ID [v2]
Daniel Mack
daniel at caiaq.de
Fri Sep 11 11:20:19 EDT 2009
On Thu, Sep 10, 2009 at 03:33:27PM +0200, Daniel Mack wrote:
> On Thu, Sep 10, 2009 at 03:31:34PM +0200, Guennadi Liakhovetski wrote:
> > > + 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:-))
>
> Don't know. I left it in there intentionally, but I'm fine to remove it.
> Or Eric does it when applying ;) Eric?
What I was considering are cases where a kernel binary that works on
both PXA3xx and other PXAs shares the same pxafb_mach_info struct across
different board support code. The accel flag would then be set for wrong
platforms.
But this might be academic, that's why I don't have a strong opinion on
that :)
Daniel
More information about the linux-arm-kernel
mailing list