Hi Alan, Sorry to quibble, but the subsystem label on the commit subject line should be "ath6kl:" it's a lower-case "L", not a one. On Wed, Oct 12, 2016 at 11:08 AM, Alan <alan@xxxxxxxxxxxxxxx> wrote: > From: Adam Williamson <adamw@xxxxxxxxxxxxxxxxx> > > SDIO ID 0271:0418 > > Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx> > Bugzilla-ID: https://bugzilla.kernel.org/show_bug.cgi?id=67921 > > diff --git a/drivers/net/wireless/ath/ath6kl/sdio.c b/drivers/net/wireless/ath/ath6kl/sdio.c > index eab0ab9..76eb336 100644 > --- 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> - Steve
