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