[Xorg-driver-geode] [PATCH 00/21] Patch series available for a pull

[Huang, FrankR]

I gave a look at this file. It has "DriverRec Z4L" definition which is similar like "DriverRec RADEON", "DriverRec GEODE". So from my understanding, it can be seen as an individual module that has relationship with geode graphics driver. And It will use PutImage function to do the overlay like what Radeon does in this part. And from the Jordan's log, I can not find anymore information that is useful to prove my point. 
In my opinion, what not merge that part code into geode driver?

For the Lenovo's project, our team has co-operate on this before. But I do not know if that is launched by Lenovo itself. It is a project near 2008 with a v4l2 driver.


Thanks,
Frank

> -----Original Message-----
> From: m.e.racine at gmail.com [mailto:m.e.racine at gmail.com] On Behalf Of
> Martin-éric Racine
> Sent: 2011年11月23日 4:26
> To: Huang, FrankR
> Cc: Gaetan Nadon
> Subject: Re: [Xorg-driver-geode] [PATCH 00/21] Patch series available for
> a pull
> 
> > About the ZTV part, I have not had a chance to try this module, even for
> the Lenovo's project. But AFAIK, this part works find with VIP module with
> an external decoder to do the video overlay. So please keep this part as
> what it is now.
> 
> As Gaetan noticed, X server's driver detection doesn't doesn't know
> anything about "ztv" and X doesn't offer any method for manually
> loading the driver either. He's thus wondering how it's supposed to be
> used.
> 
> Also, he was wondering if the 'ztv' code should simply be linked with
> the normal display driver and the C file renamed to e.g.
> geode_v4l2_ztv.c to match the naming convention.
> 
> Finally, the TODO file hints at the possibility that the GX2 might
> also have VIP capabilities that have not been implemented yet, but I
> haven't been able to certify this.
> 
> Would you know anything about these things?
> 
> Martin-Éric
> 
> PS: in what Lenovo product was the LX recently used?