On Thu, 2003-09-18 at 13:39, Chris Chiappa wrote:
> I made the "obvious" changes to
> debian/patches/030b_radeon_rv280_support.diff (included) but I don't know
> how to generate the right case clause for the switch () statement in
> ati_chip.c. I forced the switch to default to rv280 with this nasty hack:
>
> default:
> /*
> * I'd say it's a Rage128 or a Radeon here, except that I don't
> * support them.
> */
> return ATI_CHIP_RV280;
> return ATI_CHIP_Mach64;
> }
>
> and the X server (in about 5 minutes of use) seems fine. Happy to provide
> any other info or debugging.
I think you need this instead:
> -- xc/programs/Xserver/hw/xfree86/drivers/ati/atichip.c.2_rv280 Mon Mar
> 3 23:13:18 2003
> +++ xc/programs/Xserver/hw/xfree86/drivers/ati/atichip.c Tue Mar 4
> 04:22:23 2003
> @@ -675,6 +675,18 @@
> case NewChipID('L', 'g'):
> return ATI_CHIP_RADEONMOBILITY9;
>
> + case NewChipID('Y', '\''):
> + case NewChipID('Y', 'a'):
> + case NewChipID('I', 'b'):
> + case NewChipID('I', 'c'):
The last two should be replaced by
+ case NewChipID('Y', 'b'):
+ case NewChipID('Y', 'c'):
+ case NewChipID('Y', 'd'):
> + return ATI_CHIP_RV280;
> +
> + case NewChipID('Y', 'h'):
> + case NewChipID('Y', 'i'):
> + case NewChipID('Y', 'j'):
> + case NewChipID('Y', 'k'):
> + return ATI_CHIP_RADEONMOBILITY9PLUS;
> +
> case NewChipID('A', 'D'):
> case NewChipID('A', 'E'):
> case NewChipID('A', 'F'):
> -- xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_version.h.2_rv280
> Fri Mar 7 06:56:11 2003
> +++ xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_version.h Fri Mar
> 7 06:56:31 2003
> @@ -39,6 +39,7 @@
> #define RADEON_DRIVER_NAME "radeon"
> #define R200_DRIVER_NAME "r200"
> #define RV250_DRIVER_NAME "r200"
> +#define RV280_DRIVER_NAME "r200"
>
> #define RADEON_VERSION_MAJOR 4
> #define RADEON_VERSION_MINOR 0
BTW, this is as superfluous as RV250_DRIVER_NAME, just use
R200_DRIVER_NAME.
--
Earthling Michel Dänzer \ Debian (powerpc), XFree86 and DRI developer
Software libre enthusiast \ http://svcs.affero.net/rm.php?r=daenzer
