Steve deRosier <deros...@gmail.com> 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.deros...@lairdtech.com>

Thanks.

-- 
Kalle Valo

Reply via email to