[irda-users] Cleanup of donauboe

[Pavel M. <pa...@uc...>]

Hi!

This patch cleans up worst case of non-CodingStyle code in
donauboe. Still *much* is left to clean up there. Please apply,

							Pavel

Index: linux/drivers/net/irda/donauboe.c
===================================================================
--- linux.orig/drivers/net/irda/donauboe.c	2003-06-24 11:54:37.000000000 +0200
+++ linux/drivers/net/irda/donauboe.c	2003-06-15 22:46:24.000000000 +0200
@@ -165,8 +165,6 @@
 
 #include <net/irda/wrapper.h>
 #include <net/irda/irda.h>
-//#include <net/irda/irmod.h>
-//#include <net/irda/irlap_frame.h>
 #include <net/irda/irda_device.h>
 #include <net/irda/crc.h>
 
@@ -467,16 +465,15 @@
       config0l = OBOE_CONFIG0L_ENSIR;
       if (self->async)
         {
-          /*Set to character based reception */
-          /*System will lock if MAXLEN=0 */
-          /*so have to be careful */
+          /* Set to character based reception
+	     System will lock if MAXLEN=0, so be careful */
           OUTB (0x01, OBOE_MAXLENH);
           OUTB (0x01, OBOE_MAXLENL);
           OUTB (0x00, OBOE_MAXLENH);
         }
       else
         {
-          /*Set to packet based reception */
+          /* Set to packet based reception */
           config0l |= OBOE_CONFIG0L_ENSIRF;
           OUTB (RX_LEN >> 8, OBOE_MAXLENH);
           OUTB (RX_LEN & 0xff, OBOE_MAXLENL);
@@ -484,9 +481,9 @@
       break;
 
 #ifdef USE_MIR
-      /* MIR mode */
-      /* Set for 16 bit CRC and enable MIR */
-      /* Preamble now handled by the chip */
+      /* MIR mode
+	 Set for 16 bit CRC and enable MIR
+	 Preamble now handled by the chip */
     case 1152000:
       pconfig |= 0 << OBOE_PCONFIG_BAUDSHIFT;
       pconfig |= 8 << OBOE_PCONFIG_WIDTHSHIFT;
@@ -494,9 +491,9 @@
       config0l = OBOE_CONFIG0L_CRC16 | OBOE_CONFIG0L_ENMIR;
       break;
 #endif
-      /* FIR mode */
-      /* Set for 32 bit CRC and enable FIR */
-      /* Preamble handled by the chip */
+      /* FIR mode
+	 Set for 32 bit CRC and enable FIR
+	 Preamble handled by the chip */
     case 4000000:
       pconfig |= 0 << OBOE_PCONFIG_BAUDSHIFT;
       /* Documentation says 14, but toshiba use 15 in their drivers */


-- 
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]