<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">ChangeSet 1.985.10.3, 2003/03/25 11:33:39-08:00, baldrick@wanadoo.fr

[PATCH] USB speedtouch: trivial cleanups


 drivers/usb/misc/speedtch.c |  110 ++++++++++++++++++++------------------------
 1 files changed, 51 insertions(+), 59 deletions(-)


diff -Nru a/drivers/usb/misc/speedtch.c b/drivers/usb/misc/speedtch.c
--- a/drivers/usb/misc/speedtch.c	Tue Mar 25 16:46:55 2003
+++ b/drivers/usb/misc/speedtch.c	Tue Mar 25 16:46:55 2003
@@ -148,19 +148,16 @@
 
 #define UDSL_SKB(x)		((struct udsl_control *)(x)-&gt;cb)
 
-struct atmsar_vcc_data {
-	struct atmsar_vcc_data *next;
-
-	/* connection specific non-atmsar data */
-	struct atm_vcc *vcc;
-	unsigned short mtu;	/* max is actually  65k for AAL5... */
-
-	/* cell data */
-	unsigned int vp;
-	unsigned int vc;
+struct udsl_vcc_data {
+	/* vpi/vci lookup */
+	struct udsl_vcc_data *next;
+	unsigned int vpi;
+	unsigned int vci;
 	unsigned long atmHeader;
+	struct atm_vcc *vcc;
 
 	/* raw cell reassembly */
+	unsigned short mtu;
 	struct sk_buff *reasBuffer;
 };
 
@@ -178,7 +175,7 @@
 
 	/* atm device part */
 	struct atm_dev *atm_dev;
-	struct atmsar_vcc_data *atmsar_vcc_list;
+	struct udsl_vcc_data *vcc_list;
 
 	/* receiving */
 	struct udsl_receiver all_receivers [UDSL_NUMBER_RCV_URBS];
@@ -252,24 +249,23 @@
 
 #define ATM_HDR_VPVC_MASK		(ATM_HDR_VPI_MASK | ATM_HDR_VCI_MASK)
 
-struct sk_buff *atmsar_decode_rawcell (struct atmsar_vcc_data *list, struct sk_buff *skb,
-				       struct atmsar_vcc_data **ctx)
+static struct sk_buff *udsl_decode_rawcell (struct udsl_vcc_data *list, struct sk_buff *skb, struct udsl_vcc_data **ctx)
 {
+	if (!list || !skb || !ctx)
+		return NULL;
+	if (!skb-&gt;data || !skb-&gt;tail)
+		return NULL;
+
 	while (skb-&gt;len) {
 		unsigned char *cell = skb-&gt;data;
 		unsigned char *cell_payload;
-		struct atmsar_vcc_data *vcc = list;
+		struct udsl_vcc_data *vcc = list;
 		unsigned long atmHeader =
 		    ((unsigned long) (cell[0]) &lt;&lt; 24) | ((unsigned long) (cell[1]) &lt;&lt; 16) |
 		    ((unsigned long) (cell[2]) &lt;&lt; 8) | (cell[3] &amp; 0xff);
 
-		dbg ("atmsar_decode_rawcell (0x%p, 0x%p, 0x%p) called", list, skb, ctx);
-		dbg ("atmsar_decode_rawcell skb-&gt;data %p, skb-&gt;tail %p", skb-&gt;data, skb-&gt;tail);
-
-		if (!list || !skb || !ctx)
-			return NULL;
-		if (!skb-&gt;data || !skb-&gt;tail)
-			return NULL;
+		dbg ("udsl_decode_rawcell (0x%p, 0x%p, 0x%p) called", list, skb, ctx);
+		dbg ("udsl_decode_rawcell skb-&gt;data %p, skb-&gt;tail %p", skb-&gt;data, skb-&gt;tail);
 
 		/* here should the header CRC check be... */
 
@@ -279,7 +275,7 @@
 		     &amp;&amp; ((vcc-&gt;atmHeader &amp; ATM_HDR_VPVC_MASK) != (atmHeader &amp; ATM_HDR_VPVC_MASK));
 		     vcc = vcc-&gt;next);
 
-		dbg ("atmsar_decode_rawcell found vcc %p for packet on vp %d, vc %d", vcc,
+		dbg ("udsl_decode_rawcell found vcc %p for packet on vpi %d, vci %d", vcc,
 			(int) ((atmHeader &amp; ATM_HDR_VPI_MASK) &gt;&gt; ATM_HDR_VPI_SHIFT),
 			(int) ((atmHeader &amp; ATM_HDR_VCI_MASK) &gt;&gt; ATM_HDR_VCI_SHIFT));
 
@@ -313,9 +309,7 @@
 					tmp = vcc-&gt;reasBuffer;
 					vcc-&gt;reasBuffer = NULL;
 
-					dbg
-					    ("atmsar_decode_rawcell returns ATM_AAL5 pdu 0x%p with length %d",
-					     tmp, tmp-&gt;len);
+					dbg ("udsl_decode_rawcell returns ATM_AAL5 pdu 0x%p with length %d", tmp, tmp-&gt;len);
 					return tmp;
 				}
 			}
@@ -331,14 +325,14 @@
 	}
 
 	return NULL;
-};
+}
 
-struct sk_buff *atmsar_decode_aal5 (struct atmsar_vcc_data *ctx, struct sk_buff *skb)
+static struct sk_buff *udsl_decode_aal5 (struct udsl_vcc_data *ctx, struct sk_buff *skb)
 {
 	uint crc = 0xffffffff;
 	uint length, pdu_crc, pdu_length;
 
-	dbg ("atmsar_decode_aal5 (0x%p, 0x%p) called", ctx, skb);
+	dbg ("udsl_decode_aal5 (0x%p, 0x%p) called", ctx, skb);
 
 	if (skb-&gt;len &amp;&amp; (skb-&gt;len % 48))
 		return NULL;
@@ -348,7 +342,7 @@
 	    (skb-&gt;tail[-4] &lt;&lt; 24) + (skb-&gt;tail[-3] &lt;&lt; 16) + (skb-&gt;tail[-2] &lt;&lt; 8) + skb-&gt;tail[-1];
 	pdu_length = ((length + 47 + 8) / 48) * 48;
 
-	dbg ("atmsar_decode_aal5: skb-&gt;len = %d, length = %d, pdu_crc = 0x%x, pdu_length = %d",
+	dbg ("udsl_decode_aal5: skb-&gt;len = %d, length = %d, pdu_crc = 0x%x, pdu_length = %d",
 		skb-&gt;len, length, pdu_crc, pdu_length);
 
 	/* is skb long enough ? */
@@ -360,7 +354,7 @@
 
 	/* is skb too long ? */
 	if (skb-&gt;len &gt; pdu_length) {
-		dbg ("atmsar_decode_aal5: Warning: readjusting illeagl size %d -&gt; %d",
+		dbg ("udsl_decode_aal5: Warning: readjusting illegal size %d -&gt; %d",
 			skb-&gt;len, pdu_length);
 		/* buffer is too long. we can try to recover
 		 * if we discard the first part of the skb.
@@ -373,7 +367,7 @@
 
 	/* check crc */
 	if (pdu_crc != crc) {
-		dbg ("atmsar_decode_aal5: crc check failed!");
+		dbg ("udsl_decode_aal5: crc check failed!");
 		if (ctx-&gt;vcc-&gt;stats)
 			atomic_inc (&amp;ctx-&gt;vcc-&gt;stats-&gt;rx_err);
 		return NULL;
@@ -386,16 +380,17 @@
 	if (ctx-&gt;vcc-&gt;stats)
 		atomic_inc (&amp;ctx-&gt;vcc-&gt;stats-&gt;rx);
 
-	dbg ("atmsar_decode_aal5 returns pdu 0x%p with length %d", skb, skb-&gt;len);
+	dbg ("udsl_decode_aal5 returns pdu 0x%p with length %d", skb, skb-&gt;len);
 	return skb;
-};
+}
 
 
 /*************
 **  encode  **
 *************/
 
-static void udsl_groom_skb (struct atm_vcc *vcc, struct sk_buff *skb) {
+static void udsl_groom_skb (struct atm_vcc *vcc, struct sk_buff *skb)
+{
 	struct udsl_control *ctrl = UDSL_SKB (skb);
 	unsigned int i, zero_padding;
 	unsigned char zero = 0;
@@ -435,7 +430,8 @@
 	ctrl-&gt;aal5_trailer [7] = crc;
 }
 
-unsigned int udsl_write_cells (unsigned int howmany, struct sk_buff *skb, unsigned char **target_p) {
+static unsigned int udsl_write_cells (unsigned int howmany, struct sk_buff *skb, unsigned char **target_p)
+{
 	struct udsl_control *ctrl = UDSL_SKB (skb);
 	unsigned char *target = *target_p;
 	unsigned int nc, ne, i;
@@ -524,7 +520,7 @@
 	unsigned char *data_start;
 	struct sk_buff *skb;
 	struct urb *urb;
-	struct atmsar_vcc_data *atmsar_vcc = NULL;
+	struct udsl_vcc_data *atmsar_vcc = NULL;
 	struct sk_buff *new = NULL, *tmp = NULL;
 	int err;
 
@@ -552,13 +548,11 @@
 			dbg ("skb-&gt;len = %d", skb-&gt;len);
 			PACKETDEBUG (skb-&gt;data, skb-&gt;len);
 
-			while ((new =
-				atmsar_decode_rawcell (instance-&gt;atmsar_vcc_list, skb,
-						       &amp;atmsar_vcc)) != NULL) {
+			while ((new = udsl_decode_rawcell (instance-&gt;vcc_list, skb, &amp;atmsar_vcc))) {
 				dbg ("(after cell processing)skb-&gt;len = %d", new-&gt;len);
 
 				tmp = new;
-				new = atmsar_decode_aal5 (atmsar_vcc, new);
+				new = udsl_decode_aal5 (atmsar_vcc, new);
 
 				/* we can't send NULL skbs upstream, the ATM layer would try to close the vcc... */
 				if (new) {
@@ -574,7 +568,7 @@
 						dev_kfree_skb (new);
 					}
 				} else {
-					dbg ("atmsar_decode_aal5 returned NULL!");
+					dbg ("udsl_decode_aal5 returned NULL!");
 					dev_kfree_skb (tmp);
 				}
 			}
@@ -917,12 +911,10 @@
 	return 0;
 }
 
-#define ATMSAR_SET_PTI		0x2L
-
 static int udsl_atm_open (struct atm_vcc *vcc, short vpi, int vci)
 {
 	struct udsl_instance_data *instance = vcc-&gt;dev-&gt;dev_data;
-	struct atmsar_vcc_data *new;
+	struct udsl_vcc_data *new;
 
 	dbg ("udsl_atm_open called");
 
@@ -931,19 +923,19 @@
 		return -ENODEV;
 	}
 
-	/* at the moment only AAL5 support */
+	/* only support AAL5 */
 	if (vcc-&gt;qos.aal != ATM_AAL5)
 		return -EINVAL;
 
-	if (!(new = kmalloc (sizeof (struct atmsar_vcc_data), GFP_KERNEL)))
+	if (!(new = kmalloc (sizeof (struct udsl_vcc_data), GFP_KERNEL)))
 		return -ENOMEM;
 
 	MOD_INC_USE_COUNT;
 
-	memset (new, 0, sizeof (struct atmsar_vcc_data));
+	memset (new, 0, sizeof (struct udsl_vcc_data));
 	new-&gt;vcc = vcc;
-	new-&gt;vp = vpi;
-	new-&gt;vc = vci;
+	new-&gt;vpi = vpi;
+	new-&gt;vci = vci;
 
 	new-&gt;mtu = UDSL_MAX_AAL5_MRU;
 
@@ -951,10 +943,10 @@
 			 ((unsigned long) vci &lt;&lt; ATM_HDR_VCI_SHIFT);
 	new-&gt;reasBuffer = NULL;
 
-	new-&gt;next = instance-&gt;atmsar_vcc_list;
-	instance-&gt;atmsar_vcc_list = new;
+	new-&gt;next = instance-&gt;vcc_list;
+	instance-&gt;vcc_list = new;
 
-	dbg ("Allocated new SARLib vcc 0x%p with vp %d vc %d", new, vpi, vci);
+	dbg ("Allocated new SARLib vcc 0x%p with vpi %d vci %d", new, vpi, vci);
 
 	vcc-&gt;dev_data = new;
 
@@ -974,7 +966,7 @@
 static void udsl_atm_close (struct atm_vcc *vcc)
 {
 	struct udsl_instance_data *instance = vcc-&gt;dev-&gt;dev_data;
-	struct atmsar_vcc_data *work;
+	struct udsl_vcc_data *work;
 
 	dbg ("udsl_atm_close called");
 
@@ -987,10 +979,10 @@
 	/* cancel all sends on this vcc */
 	udsl_cancel_send (instance, vcc);
 
-	if (instance-&gt;atmsar_vcc_list == vcc-&gt;dev_data) {
-		instance-&gt;atmsar_vcc_list = instance-&gt;atmsar_vcc_list-&gt;next;
+	if (instance-&gt;vcc_list == vcc-&gt;dev_data) {
+		instance-&gt;vcc_list = instance-&gt;vcc_list-&gt;next;
 	} else {
-		for (work = instance-&gt;atmsar_vcc_list; work &amp;&amp; work-&gt;next &amp;&amp; (work-&gt;next != vcc-&gt;dev_data); work = work-&gt;next);
+		for (work = instance-&gt;vcc_list; work &amp;&amp; work-&gt;next &amp;&amp; (work-&gt;next != vcc-&gt;dev_data); work = work-&gt;next);
 
 		/* return if not found */
 		if (work-&gt;next != vcc-&gt;dev_data)
@@ -999,11 +991,11 @@
 		work-&gt;next = work-&gt;next-&gt;next;
 	}
 
-	if (((struct atmsar_vcc_data *)vcc-&gt;dev_data)-&gt;reasBuffer) {
-		dev_kfree_skb (((struct atmsar_vcc_data *)vcc-&gt;dev_data)-&gt;reasBuffer);
+	if (((struct udsl_vcc_data *)vcc-&gt;dev_data)-&gt;reasBuffer) {
+		dev_kfree_skb (((struct udsl_vcc_data *)vcc-&gt;dev_data)-&gt;reasBuffer);
 	}
 
-	dbg ("Deallocated SARLib vcc 0x%p with vp %d vc %d", vcc-&gt;dev_data, vcc-&gt;dev_data-&gt;vp, vcc-&gt;dev_data-&gt;vc);
+	dbg ("Deallocated SARLib vcc 0x%p with vpi %d vci %d", vcc-&gt;dev_data, vcc-&gt;dev_data-&gt;vpi, vcc-&gt;dev_data-&gt;vci);
 
 	kfree (vcc-&gt;dev_data);
 
</pre></body></html>