Seems like "the right thing" to me

Reviewed-by: Jeremy Huddleston <[email protected]>

On Oct 29, 2011, at 09:40, Anssi Hannula wrote:

> On 20.04.2011 14:34, Anssi Hannula wrote:
>> From 003e32786053eb01e2a84697487aa134214f140c Mon Sep 17 00:00:00 2001
>> From: Anssi Hannula <[email protected]>
>> Date: Sun, 17 Apr 2011 22:04:29 +0300
>> Subject: [PATCH] xfree86: add nouveau as the first automatic driver for 
>> NVIDIA hardware
>> 
>> Add nouveau as the first driver on linux for NVIDIA hardware when
>> driver autoconfiguration is done, as it is more capable than nv.
>> 
>> nv is also kept in the list as it is more widely supported and because
>> some old cards are not supported by nouveau.
>> 
>> Signed-off-by: Anssi Hannula <[email protected]>
> 
> 
> Ping? The patch is still not applied, and it got no NAKs :)
> 
> 
>> ---
>> hw/xfree86/common/xf86pciBus.c |   10 +++++++++-
>> 1 files changed, 9 insertions(+), 1 deletions(-)
>> 
>> diff --git a/hw/xfree86/common/xf86pciBus.c b/hw/xfree86/common/xf86pciBus.c
>> index 447b192..8b7e298 100644
>> --- a/hw/xfree86/common/xf86pciBus.c
>> +++ b/hw/xfree86/common/xf86pciBus.c
>> @@ -1123,7 +1123,15 @@ videoPtrToDriverList(struct pci_device *dev,
>>          break;
>>      case 0x102b:                driverList[0] = "mga";      break;
>>      case 0x10c8:                driverList[0] = "neomagic"; break;
>> -    case 0x10de: case 0x12d2:   driverList[0] = "nv";       break;
>> +    case 0x10de: case 0x12d2:
>> +    {
>> +        int idx = 0;
>> +#ifdef __linux__
>> +        driverList[idx++] = "nouveau";
>> +#endif
>> +        driverList[idx++] = "nv";
>> +        break;
>> +    }
>>      case 0x1106:                driverList[0] = "openchrome"; break;
>>         case 0x1b36:             driverList[0] = "qxl"; break;
>>      case 0x1163:                driverList[0] = "rendition"; break;
>> -- 1.7.3
> 
> 
> -- 
> Anssi Hannula
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to