Re: [PATCH] ath5k: adding LED support for AR5BXB63 cards

["John W. Linville" <linville@tuxdriver.com>]

On Tue, Dec 22, 2009 at 06:10:01PM +0100, Luca Verdesca wrote:
> With following patch, LED should now work with LiteOn AR5BXB63 mini
> pci-e cards.
> 
> Signed-off-by: Luca Verdesca <magooz@salug.it>
> Acked-by: Bob Copeland <me@bobcopeland.com>
> Cc: John Linville <linville@tuxdriver.com>
> ---
>  home/magooz/led.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> --- /usr/src/linux/drivers/net/wireless/ath/ath5k/led.c	2009-12-22 16:37:42.000000000 +0100
> +++ /home/magooz/led.c	2009-12-22 16:38:14.000000000 +0100
> @@ -75,6 +75,8 @@ static const struct pci_device_id ath5k_
>  	{ ATH_SDEVICE(PCI_VENDOR_ID_HP, 0x0137a), ATH_LED(3, 1) },
>  	/* HP Compaq C700 (nitrousnrg@gmail.com) */
>  	{ ATH_SDEVICE(PCI_VENDOR_ID_HP, 0x0137b), ATH_LED(3, 1) },
> +	/* LiteOn AR5BXB63 (magooz@salug.it) */
> +	{ PCI_SDEVICE(ATHEROS, 0x3067), ATH_LED(3, 0) },
>  	/* IBM-specific AR5212 (all others) */
>  	{ PCI_VDEVICE(ATHEROS, PCI_DEVICE_ID_ATHEROS_AR5212_IBM), ATH_LED(0, 0) },
>  	{ }

Did you compile this?  Perhaps you meant to use ATH_SDEVICE instead?
Of course, it still wouldn't be quite right AFAICT...

Also, please make sure that your patch can be applied with "patch -p1".

Thanks,

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.