xf86-video-ati randr-1.2 branch on PowerBook5,4 with RV350

[Alex Deucher]

On 8/30/07, Sjoerd Simons <sjoerd at luon.net> wrote:
> On Thu, Aug 30, 2007 at 08:06:50AM +0200, Michel Dänzer wrote:
> > On Wed, 2007-08-29 at 23:16 -0400, Alex Deucher wrote:
> > >
> > > Can you grab the latest code from git and add:
> > >
> > > Option "MacModel" "powerbook"
> > >
> > > to the device section of your config and verify that everything works
> > > correctly?
> >
> > I suspect it should rather be "powerbook-duallink" - it's one of the
> > later PowerBook models.
>
> I've got the same card in my powerbook and MacModel powerbook works best.
>
> But first i had to apply the following patch to get it to work:
> diff --git a/src/radeon_driver.c b/src/radeon_driver.c
> index 5c2efd3..eb7fb23 100644
> --- a/src/radeon_driver.c
> +++ b/src/radeon_driver.c
> @@ -189,7 +189,7 @@ static const OptionInfoRec RADEONOptions[] = {
>      { OPTION_DEFAULT_CONNECTOR_TABLE, "DefaultConnectorTable", OPTV_BOOLEAN, {0}, FALSE },
>      { OPTION_DEFAULT_TMDS_PLL, "DefaultTMDSPLL", OPTV_BOOLEAN, {0}, FALSE },
>  #if defined(__powerpc__)
> -    { OPTION_CONNECTORTABLE, "MacModel",         OPTV_STRING,  {0}, FALSE },
> +    { OPTION_MAC_MODEL,      "MacModel",         OPTV_STRING,  {0}, FALSE },
>  #endif
>      { -1,                    NULL,               OPTV_NONE,    {0}, FALSE }
>  };
>
>
> Michel is right though. These models have an external TDMS for DVI, but it
> seems the VGA parts of the DVI connector are run of the internal one (does that
> make sense?).
>
> When using "powerbook-duallink" my DVI->VGA does work, but when
> running xrandr without options or starting an app using XV the external screen
> blanks momentarily.. Which is not the case when using "powerbook"

It should be the same either way.  both variants drive the analog
portion with the primary dac.  The only difference is whether they use
internal or external tmds.  The screen blinks when it does load
detection on the dacs.  this is unavoidable unless we disable load
detection all together.

Alex