Search Linux Wireless

Re: [PATCH] ath6k1: add Dell OEM SDIO I/O for the Venue 8 Pro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Steve deRosier <derosier@xxxxxxxxx> writes:

> Sorry to quibble, but the subsystem label on the commit subject line
> should be "ath6kl:"  it's a lower-case "L", not a one.

Heh, I missed that :) I can fix it before I commit.

>> --- a/drivers/net/wireless/ath/ath6kl/sdio.c
>> +++ b/drivers/net/wireless/ath/ath6kl/sdio.c
>> @@ -1401,6 +1401,7 @@ static const struct sdio_device_id ath6kl_sdio_devices[] = {
>>         {SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x0))},
>>         {SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x1))},
>>         {SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x2))},
>> +       {SDIO_DEVICE(MANUFACTURER_CODE, (MANUFACTURER_ID_AR6004_BASE | 0x18))},
>>         {},
>>  };
>>
>>
>
> I see nothing wrong with this if the chip does indeed identify itself
> this way.  So please fix the subject and you can add:
>
> Reviewed-by: Steve deRosier <steve.derosier@xxxxxxxxxxxxx>

Thanks.

-- 
Kalle Valo



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux