On 11/06/2009 09:16 AM, Bob Copeland wrote: > On Fri, Nov 6, 2009 at 8:00 AM, Carlo Parata <carlo_parata@xxxxxxxxxxx> wrote: >> With this patch, a Acer Aspire One D250 can turn on the wifi led. The array of >> compatible devices in ath5k driver now includes the hardware present in this >> computer, as well as the led pin and polarity. >> >> --- ../led.c 2009-11-06 06:16:39.000000000 +0100 >> +++ ./drivers/net/wireless/ath/ath5k/led.c 2009-11-06 12:37:44.000000000 +0100 >> @@ -73,6 +73,8 @@ >> { ATH_SDEVICE(PCI_VENDOR_ID_HP, 0x0137b), ATH_LED(3, 1) }, >> /* IBM-specific AR5212 (all others) */ >> { PCI_VDEVICE(ATHEROS, PCI_DEVICE_ID_ATHEROS_AR5212_IBM), ATH_LED(0, 0) }, >> + /* Acer Aspire One D250 AR5007EG */ >> + { PCI_VDEVICE(ATHEROS, 0x001c), ATH_LED(3, 1) }, > > No, this should be based on the subdevice instead > (there are lots of ath:001c with different LED settings). > > Also, please add your email to the appropriate stanza. It is more than just your E-mail address. Please see Documents/SubmittingPatches. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html