[PATCH] ath9k_htc: Add new devices into AR7010

[PATCH] ath9k_htc: Add new devices into AR7010

[Rajkumar Manoharan]

Treat PIDs (0xA704, 0x1200) as AR7010 devices.

Signed-off-by: Rajkumar Manoharan <rmanoharan@atheros.com>
---
 drivers/net/wireless/ath/ath9k/hif_usb.c      |    4 ++++
 drivers/net/wireless/ath/ath9k/htc_drv_init.c |    2 ++
 drivers/net/wireless/ath/ath9k/reg.h          |    4 +++-
 3 files changed, 9 insertions(+), 1 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
index 0e12277..dfb6560 100644
--- a/drivers/net/wireless/ath/ath9k/hif_usb.c
+++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
@@ -811,6 +811,8 @@ static int ath9k_hif_usb_download_fw(struct hif_device_usb *hif_dev)
 	case 0x7010:
 	case 0x7015:
 	case 0x9018:
+	case 0xA704:
+	case 0x1200:
 		firm_offset = AR7010_FIRMWARE_TEXT;
 		break;
 	default:
@@ -933,6 +935,8 @@ static int ath9k_hif_usb_probe(struct usb_interface *interface,
 	case 0x7010:
 	case 0x7015:
 	case 0x9018:
+	case 0xA704:
+	case 0x1200:
 		if (le16_to_cpu(udev->descriptor.bcdDevice) == 0x0202)
 			hif_dev->fw_name = FIRMWARE_AR7010_1_1;
 		else
diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_init.c b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
index 3d7b97f..7c8a38d 100644
--- a/drivers/net/wireless/ath/ath9k/htc_drv_init.c
+++ b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
@@ -249,6 +249,8 @@ static int ath9k_init_htc_services(struct ath9k_htc_priv *priv, u16 devid)
 	case 0x7010:
 	case 0x7015:
 	case 0x9018:
+	case 0xA704:
+	case 0x1200:
 		priv->htc->credits = 45;
 		break;
 	default:
diff --git a/drivers/net/wireless/ath/ath9k/reg.h b/drivers/net/wireless/ath/ath9k/reg.h
index fa05b71..52ee82d 100644
--- a/drivers/net/wireless/ath/ath9k/reg.h
+++ b/drivers/net/wireless/ath/ath9k/reg.h
@@ -866,7 +866,9 @@
 #define AR_DEVID_7010(_ah) \
 	(((_ah)->hw_version.devid == 0x7010) || \
 	 ((_ah)->hw_version.devid == 0x7015) || \
-	 ((_ah)->hw_version.devid == 0x9018))
+	 ((_ah)->hw_version.devid == 0x9018) || \
+	 ((_ah)->hw_version.devid == 0xA704) || \
+	 ((_ah)->hw_version.devid == 0x1200))
 
 #define AR_RADIO_SREV_MAJOR                   0xf0
 #define AR_RAD5133_SREV_MAJOR                 0xc0
-- 
1.7.3.2

Re: [PATCH] ath9k_htc: Add new devices into AR7010

[Christian Lamparter]

On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> Treat PIDs (0xA704, 0x1200) as AR7010 devices.

just a though, but instead of relying only on the pid and
hope for no collisions. you could put a @driver_info flag
for all AR7010 devices into the usb_device_id table and
remove the per-device "cases". 

Best Regards,
	Chr

> ---
> diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
> index 0e12277..dfb6560 100644
> --- a/drivers/net/wireless/ath/ath9k/hif_usb.c
> +++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
> @@ -811,6 +811,8 @@ static int ath9k_hif_usb_download_fw(struct hif_device_usb *hif_dev)
>  	case 0x7010:
>  	case 0x7015:
>  	case 0x9018:
> +	case 0xA704: <--
> +	case 0x1200: <--
>  		firm_offset = AR7010_FIRMWARE_TEXT;
>  		break;
>  	default:
> @@ -933,6 +935,8 @@ static int ath9k_hif_usb_probe(struct usb_interface *interface,
>  	case 0x7010:
>  	case 0x7015:
>  	case 0x9018:
> +	case 0xA704:
> +	case 0x1200:
>  		if (le16_to_cpu(udev->descriptor.bcdDevice) == 0x0202)
>  			hif_dev->fw_name = FIRMWARE_AR7010_1_1;
>  		else
> diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_init.c b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> index 3d7b97f..7c8a38d 100644
> --- a/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> +++ b/drivers/net/wireless/ath/ath9k/htc_drv_init.c
> @@ -249,6 +249,8 @@ static int ath9k_init_htc_services(struct ath9k_htc_priv *priv, u16 devid)
>  	case 0x7010:
>  	case 0x7015:
>  	case 0x9018:
> +	case 0xA704:
> +	case 0x1200:
>  		priv->htc->credits = 45;
>  		break;
>  	default:
> diff --git a/drivers/net/wireless/ath/ath9k/reg.h b/drivers/net/wireless/ath/ath9k/reg.h
> index fa05b71..52ee82d 100644
> --- a/drivers/net/wireless/ath/ath9k/reg.h
> +++ b/drivers/net/wireless/ath/ath9k/reg.h
> @@ -866,7 +866,9 @@
>  #define AR_DEVID_7010(_ah) \
>  	(((_ah)->hw_version.devid == 0x7010) || \
>  	 ((_ah)->hw_version.devid == 0x7015) || \
> -	 ((_ah)->hw_version.devid == 0x9018))
> +	 ((_ah)->hw_version.devid == 0x9018) || \
> +	 ((_ah)->hw_version.devid == 0xA704) || \
> +	 ((_ah)->hw_version.devid == 0x1200))
>  
>  #define AR_RADIO_SREV_MAJOR                   0xf0
>  #define AR_RAD5133_SREV_MAJOR                 0xc0

Re: [PATCH] ath9k_htc: Add new devices into AR7010

[Rajkumar Manoharan]

On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> 
> just a though, but instead of relying only on the pid and
> hope for no collisions. you could put a @driver_info flag
> for all AR7010 devices into the usb_device_id table and
> remove the per-device "cases". 
> 
Make sense. Thanks for your review. I'll resend the patch.

--
Rajkumar

Re: [PATCH] ath9k_htc: Add new devices into AR7010

[Rajkumar Manoharan]

On Wed, Nov 10, 2010 at 06:43:57PM +0530, Rajkumar Manoharan wrote:
> On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> > On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > > Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> > 
> > just a though, but instead of relying only on the pid and
> > hope for no collisions. you could put a @driver_info flag
> > for all AR7010 devices into the usb_device_id table and
> > remove the per-device "cases". 
> > 
> Make sense. Thanks for your review. I'll resend the patch.
>
After adding @driver_info to replace per-device cases,
the patch looks like cleanup one which is not compatible for stable kernel.
So let the previous patches to be integrated, due to minimal changes.
And I'll submit the cleanup patch separately. Is that fine ?

--
Rajkumar

Re: [PATCH] ath9k_htc: Add new devices into AR7010

[Christian Lamparter]

On Thursday 11 November 2010 14:23:41 Rajkumar Manoharan wrote:
> On Wed, Nov 10, 2010 at 06:43:57PM +0530, Rajkumar Manoharan wrote:
> > On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> > > On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > > > Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> > > 
> > > just a though, but instead of relying only on the pid and
> > > hope for no collisions. you could put a @driver_info flag
> > > for all AR7010 devices into the usb_device_id table and
> > > remove the per-device "cases". 
> > > 
> > Make sense. Thanks for your review. I'll resend the patch.
> >
> After adding @driver_info to replace per-device cases,
> the patch looks like cleanup one which is not compatible for stable kernel.
> So let the previous patches to be integrated, due to minimal changes.
> And I'll submit the cleanup patch separately. Is that fine?
sure?

Or is this a question directed to John?

RE: [PATCH] ath9k_htc: Add new devices into AR7010

[Rajkumar Manoharan]

>On Thursday 11 November 2010 14:23:41 Rajkumar Manoharan wrote:
>> On Wed, Nov 10, 2010 at 06:43:57PM +0530, Rajkumar Manoharan wrote:
> >> On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> > >> On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > > >> Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> > >>
> > > >just a though, but instead of relying only on the pid and
>> > > hope for no collisions. you could put a @driver_info flag
>> > > for all AR7010 devices into the usb_device_id table and
> >> > remove the per-device "cases".
> > >>
>> > Make sense. Thanks for your review. I'll resend the patch.
>> >
>> After adding @driver_info to replace per-device cases,
>> the patch looks like cleanup one which is not compatible for stable kernel.
>> So let the previous patches to be integrated, due to minimal changes.
>> And I'll submit the cleanup patch separately. Is that fine?
>sure?
>
> Or is this a question directed to John?

Both you and John. Since these patches are stable and enable support to new devices,
I prefer to go with minimal changes. I will submit the cleanup patches to wireless-testing.

Any comments John?

--
Rajkumar

Re: [PATCH] ath9k_htc: Add new devices into AR7010

[John W. Linville]

On Thu, Nov 11, 2010 at 08:51:37PM +0530, Rajkumar Manoharan wrote:
> >On Thursday 11 November 2010 14:23:41 Rajkumar Manoharan wrote:
> >> On Wed, Nov 10, 2010 at 06:43:57PM +0530, Rajkumar Manoharan wrote:
> > >> On Wed, Nov 10, 2010 at 06:05:14PM +0530, Christian Lamparter wrote:
> > > >> On Wednesday 10 November 2010 12:37:21 Rajkumar Manoharan wrote:
> > > > >> Treat PIDs (0xA704, 0x1200) as AR7010 devices.
> > > >>
> > > > >just a though, but instead of relying only on the pid and
> >> > > hope for no collisions. you could put a @driver_info flag
> >> > > for all AR7010 devices into the usb_device_id table and
> > >> > remove the per-device "cases".
> > > >>
> >> > Make sense. Thanks for your review. I'll resend the patch.
> >> >
> >> After adding @driver_info to replace per-device cases,
> >> the patch looks like cleanup one which is not compatible for stable kernel.
> >> So let the previous patches to be integrated, due to minimal changes.
> >> And I'll submit the cleanup patch separately. Is that fine?
> >sure?
> >
> > Or is this a question directed to John?
> 
> Both you and John. Since these patches are stable and enable support to new devices,
> I prefer to go with minimal changes. I will submit the cleanup patches to wireless-testing.
> 
> Any comments John?

Sounds fine.

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