diff mbox

ath9k_htc: Fix ethtool reporting

Message ID 19875.14815.278751.401499@gargle.gargle.HOWL (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Sujith Manoharan April 11, 2011, 5:26 p.m. UTC
From: Sujith Manoharan <Sujith.Manoharan@atheros.com>

Pass the correct module name and device interface so that
ethtool can display the proper values.

The firmware version will be fixed later on when the FW
can actually report a version. :)

Reported-by: Richard Farina <sidhayn@gmail.com>
Signed-off-by: Sujith Manoharan <Sujith.Manoharan@atheros.com>
---
 drivers/net/wireless/ath/ath9k/hif_usb.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Sid Hayn April 11, 2011, 5:50 p.m. UTC | #1
On 04/11/11 13:26, Sujith wrote:
> From: Sujith Manoharan<Sujith.Manoharan@atheros.com>
>
> Pass the correct module name and device interface so that
> ethtool can display the proper values.
>
> The firmware version will be fixed later on when the FW
> can actually report a version. :)
>
> Reported-by: Richard Farina<sidhayn@gmail.com>
> Signed-off-by: Sujith Manoharan<Sujith.Manoharan@atheros.com>
Tested-by: Richard Farina <sidhayn@gmail.com>

Sujith,

Thanks a lot for this patch, it does exactly what I need until the new 
firmware stuff is official.

All,

Since ethtool reports the driver incorrectly without this patch is it 
too much to ask for a stable push? At least into .39, further if anyone 
thinks it appropriate. This seems like a pretty minor patch to me but 
fixes a real problem (well at least it was a real problem for me).

Thanks,
Rick Farina
> ---
>   drivers/net/wireless/ath/ath9k/hif_usb.c |    4 ++--
>   1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
> index f1b8af6..2d10239 100644
> --- a/drivers/net/wireless/ath/ath9k/hif_usb.c
> +++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
> @@ -1040,7 +1040,7 @@ static int ath9k_hif_usb_probe(struct usb_interface *interface,
>   	}
>
>   	ret = ath9k_htc_hw_init(hif_dev->htc_handle,
> -				&hif_dev->udev->dev, hif_dev->device_id,
> +				&interface->dev, hif_dev->device_id,
>   				hif_dev->udev->product, id->driver_info);
>   	if (ret) {
>   		ret = -EINVAL;
> @@ -1158,7 +1158,7 @@ fail_resume:
>   #endif
>
>   static struct usb_driver ath9k_hif_usb_driver = {
> -	.name = "ath9k_hif_usb",
> +	.name = KBUILD_MODNAME,
>   	.probe = ath9k_hif_usb_probe,
>   	.disconnect = ath9k_hif_usb_disconnect,
>   #ifdef CONFIG_PM
> -- 1.7.4.4

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/wireless/ath/ath9k/hif_usb.c b/drivers/net/wireless/ath/ath9k/hif_usb.c
index f1b8af6..2d10239 100644
--- a/drivers/net/wireless/ath/ath9k/hif_usb.c
+++ b/drivers/net/wireless/ath/ath9k/hif_usb.c
@@ -1040,7 +1040,7 @@  static int ath9k_hif_usb_probe(struct usb_interface *interface,
 	}
 
 	ret = ath9k_htc_hw_init(hif_dev->htc_handle,
-				&hif_dev->udev->dev, hif_dev->device_id,
+				&interface->dev, hif_dev->device_id,
 				hif_dev->udev->product, id->driver_info);
 	if (ret) {
 		ret = -EINVAL;
@@ -1158,7 +1158,7 @@  fail_resume:
 #endif
 
 static struct usb_driver ath9k_hif_usb_driver = {
-	.name = "ath9k_hif_usb",
+	.name = KBUILD_MODNAME,
 	.probe = ath9k_hif_usb_probe,
 	.disconnect = ath9k_hif_usb_disconnect,
 #ifdef CONFIG_PM