public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] USB speedtouch: handle usb_string failure
@ 2003-02-28 12:31 Duncan Sands
  2003-02-28 21:32 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Duncan Sands @ 2003-02-28 12:31 UTC (permalink / raw)
  To: linux-usb-users; +Cc: Greg KH, linux-kernel

 speedtouch.c |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)


diff -Nru a/drivers/usb/misc/speedtouch.c b/drivers/usb/misc/speedtouch.c
--- a/drivers/usb/misc/speedtouch.c	Fri Feb 28 13:23:18 2003
+++ b/drivers/usb/misc/speedtouch.c	Fri Feb 28 13:23:18 2003
@@ -892,7 +892,6 @@
 	int ifnum = intf->altsetting->desc.bInterfaceNumber;
 	struct udsl_instance_data *instance;
 	unsigned char mac_str [13];
-	unsigned char mac [6];
 	int i, length;
 	char *buf;
 
@@ -995,13 +994,10 @@
 	instance->atm_dev->link_rate = 128 * 1000 / 424;
 
 	/* set MAC address, it is stored in the serial number */
-	usb_string (instance->usb_dev, instance->usb_dev->descriptor.iSerialNumber, mac_str, 13);
-	for (i = 0; i < 6; i++)
-		mac[i] = (hex2int (mac_str[i * 2]) * 16) + (hex2int (mac_str[i * 2 + 1]));
-
-	dbg ("MAC is %02x:%02x:%02x:%02x:%02x:%02x", mac[0], mac[1], mac[2], mac[3], mac[4], mac[5]);
-
-	memcpy (instance->atm_dev->esi, mac, 6);
+	memset (instance->atm_dev->esi, 0, sizeof (instance->atm_dev->esi));
+	if (usb_string (dev, dev->descriptor.iSerialNumber, mac_str, sizeof (mac_str)) == 12)
+		for (i = 0; i < 6; i++)
+			instance->atm_dev->esi[i] = (hex2int (mac_str[i * 2]) * 16) + (hex2int (mac_str[i * 2 + 1]));
 
 	/* device description */
 	buf = instance->description;


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] USB speedtouch: handle usb_string failure
  2003-02-28 12:31 [PATCH] USB speedtouch: handle usb_string failure Duncan Sands
@ 2003-02-28 21:32 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2003-02-28 21:32 UTC (permalink / raw)
  To: Duncan Sands; +Cc: linux-usb-users, linux-kernel

On Fri, Feb 28, 2003 at 01:31:31PM +0100, Duncan Sands wrote:
>  speedtouch.c |   12 ++++--------
>  1 files changed, 4 insertions(+), 8 deletions(-)

Applied, thanks.

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2003-02-28 21:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-02-28 12:31 [PATCH] USB speedtouch: handle usb_string failure Duncan Sands
2003-02-28 21:32 ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox