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

[Huang, FrankR]

Martin,

	It is pity that there is no GX2 document in my hand, so it is hardly to have the conclusion that VIP is only supported by LX.
	Actually Radeon does not have VIP module. It is specific for Geode LX I think to enhance the video overlay capacity. But the architecture of the overlay is nearly same in those two GPUs because they are following Xv rules.

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月24日 0:07
> To: Huang, FrankR
> Cc: Gaetan Nadon; Geode Mailing List
> Subject: Re: [Xorg-driver-geode] [PATCH 00/21] Patch series available for
> a pull
> 
> Frank,
> 
> Thank you for these precisions.  We'll look at how VIP is implemented
> in Radeon and see if we can learn from it.
> 
> Did you say that GX2 also has VIP capabilities or is this only
> applicable to the LX?
> 
> Martin-Éric
> 
> 2011/11/23 Huang, FrankR <FrankR.Huang at amd.com>:
> > 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?
> >
> >