Hi,

        Various stuff...

        Jean
diff -u -p linux/net/irda/irlap_event.d0.c linux/net/irda/irlap_event.c
--- linux/net/irda/irlap_event.d0.c     Fri Sep 28 11:51:16 2001
+++ linux/net/irda/irlap_event.c        Fri Sep 28 11:51:33 2001
@@ -701,7 +701,21 @@ static int irlap_state_conn(struct irlap
                 * We are allowed to send two frames, but this may increase
                 * the connect latency, so lets not do it for now.
                 */
-               /* What the hell is this ? - Jean II */
+               /* This is full of good intentions, but doesn't work in
+                * practice.
+                * After sending the first UA response, we switch the
+                * dongle to the negociated speed, which is usually
+                * different than 9600 kb/s.
+                * From there, there is two solutions :
+                * 1) The other end has received the first UA response :
+                * it will set up the connection, move to state LAP_NRM_P,
+                * and will ignore and drop the second UA response.
+                * Actually, it's even worse : the other side will almost
+                * immediately send a RR that will likely collide with the
+                * UA response (depending on negociated turnaround).
+                * 2) The other end has not received the first UA response,
+                * will stay at 9600 and will never see the second UA response.
+                * Jean II */
                irlap_send_ua_response_frame(self, &self->qos_rx);
 #endif
 
diff -u -p linux/net/irda/irias_object.d0.c linux/net/irda/irias_object.c
--- linux/net/irda/irias_object.d0.c    Fri Sep 28 11:41:36 2001
+++ linux/net/irda/irias_object.c       Fri Sep 28 11:42:09 2001
@@ -435,8 +435,6 @@ struct ias_value *irias_new_integer_valu
 struct ias_value *irias_new_string_value(char *string)
 {
        struct ias_value *value;
-       int len;
-       char *new_str;
 
        value = kmalloc(sizeof(struct ias_value), GFP_ATOMIC);
        if (value == NULL) {
diff -u -p linux/drivers/net/irda/nsc-ircc.d0.c linux/drivers/net/irda/nsc-ircc.c
--- linux/drivers/net/irda/nsc-ircc.d0.c        Fri Sep 28 11:43:42 2001
+++ linux/drivers/net/irda/nsc-ircc.c   Fri Sep 28 11:44:26 2001
@@ -112,7 +112,7 @@ static char *dongle_types[] = {
        "Reserved",
        "Reserved",
        "HP HSDL-1100/HSDL-2100",
-       "HP HSDL-1100/HSDL-2100"
+       "HP HSDL-1100/HSDL-2100",
        "Supports SIR Mode only",
        "No dongle connected",
 };
diff -u -p linux/drivers/net/irda/smc-ircc.d0.c linux/drivers/net/irda/smc-ircc.c
--- linux/drivers/net/irda/smc-ircc.d0.c        Fri Sep 28 11:45:17 2001
+++ linux/drivers/net/irda/smc-ircc.c   Fri Sep 28 11:46:47 2001
@@ -99,7 +99,7 @@ static int  ircc_pmproc(struct pm_dev *d
 #define        SERx4   8       /* SuperIO Chip supports 115,2 KBaud * 4=460,8 KBaud */
 
 /* These are the currently known SMC SuperIO chipsets */
-static const smc_chip_t __init fdc_chips_flat[]=
+static smc_chip_t __initdata fdc_chips_flat[]=
 {
        /* Base address 0x3f0 or 0x370 */
        { "37C44",      KEY55_1|NoIRDA,         0x00, 0x00 }, /* This chip can not 
detected */
@@ -113,7 +113,7 @@ static const smc_chip_t __init fdc_chips
        { NULL }
 };
 
-static const smc_chip_t __init fdc_chips_paged[]=
+static smc_chip_t __initdata fdc_chips_paged[]=
 {
        /* Base address 0x3f0 or 0x370 */
        { "37B72X",     KEY55_1|SIR|SERx4,      0x4c, 0x00 },
@@ -132,7 +132,7 @@ static const smc_chip_t __init fdc_chips
        { NULL }
 };
 
-static const smc_chip_t __init lpc_chips_flat[]=
+static smc_chip_t __initdata lpc_chips_flat[]=
 {
        /* Base address 0x2E or 0x4E */
        { "47N227",     KEY55_1|FIR|SERx4,      0x5a, 0x00 },
@@ -140,7 +140,7 @@ static const smc_chip_t __init lpc_chips
        { NULL }
 };
 
-static const smc_chip_t __init lpc_chips_paged[]=
+static smc_chip_t __initdata lpc_chips_paged[]=
 {
        /* Base address 0x2E or 0x4E */
        { "47B27X",     KEY55_1|SIR|SERx4,      0x51, 0x00 },

Reply via email to