diff options
| author | V. ANANDA KRISHNAN <mansarov@us.ibm.com> | 2005-07-27 14:43:49 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-07-27 19:25:52 -0400 |
| commit | c223695634fb360ed65e5a811161853a05e46962 (patch) | |
| tree | 0339c37cb05b028dbc0c2cfd9224afb158df423a | |
| parent | 9539c1d495c8d92837e7b6382a1219ac275b94b2 (diff) | |
[PATCH] jsm: warning fixes
- updates the version
- fix mixing of declarations and code. The mixing of declarations and
code displays warnings when used against RedHat RHEL4.0 distro (compiler
version is 3.4.3-22.1) and hence I separated them out.
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | drivers/serial/jsm/jsm.h | 2 | ||||
| -rw-r--r-- | drivers/serial/jsm/jsm_driver.c | 1 | ||||
| -rw-r--r-- | drivers/serial/jsm/jsm_neo.c | 30 |
3 files changed, 20 insertions, 13 deletions
diff --git a/drivers/serial/jsm/jsm.h b/drivers/serial/jsm/jsm.h index 5bf3c45521f4..18753193f59b 100644 --- a/drivers/serial/jsm/jsm.h +++ b/drivers/serial/jsm/jsm.h | |||
| @@ -89,7 +89,7 @@ enum { | |||
| 89 | #define WRITEBUFLEN ((4096) + 4) | 89 | #define WRITEBUFLEN ((4096) + 4) |
| 90 | #define MYFLIPLEN N_TTY_BUF_SIZE | 90 | #define MYFLIPLEN N_TTY_BUF_SIZE |
| 91 | 91 | ||
| 92 | #define JSM_VERSION "jsm: 1.1-1-INKERNEL" | 92 | #define JSM_VERSION "jsm: 1.2-1-INKERNEL" |
| 93 | #define JSM_PARTNUM "40002438_A-INKERNEL" | 93 | #define JSM_PARTNUM "40002438_A-INKERNEL" |
| 94 | 94 | ||
| 95 | struct jsm_board; | 95 | struct jsm_board; |
diff --git a/drivers/serial/jsm/jsm_driver.c b/drivers/serial/jsm/jsm_driver.c index 963ebaa37fb1..7e56c7824194 100644 --- a/drivers/serial/jsm/jsm_driver.c +++ b/drivers/serial/jsm/jsm_driver.c | |||
| @@ -22,6 +22,7 @@ | |||
| 22 | * Scott H Kilau <Scott_Kilau@digi.com> | 22 | * Scott H Kilau <Scott_Kilau@digi.com> |
| 23 | * Wendy Xiong <wendyx@us.ltcfwd.linux.ibm.com> | 23 | * Wendy Xiong <wendyx@us.ltcfwd.linux.ibm.com> |
| 24 | * | 24 | * |
| 25 | * | ||
| 25 | ***********************************************************************/ | 26 | ***********************************************************************/ |
| 26 | #include <linux/moduleparam.h> | 27 | #include <linux/moduleparam.h> |
| 27 | #include <linux/pci.h> | 28 | #include <linux/pci.h> |
diff --git a/drivers/serial/jsm/jsm_neo.c b/drivers/serial/jsm/jsm_neo.c index 3a11a69feb44..6f22b42d9337 100644 --- a/drivers/serial/jsm/jsm_neo.c +++ b/drivers/serial/jsm/jsm_neo.c | |||
| @@ -48,8 +48,9 @@ static inline void neo_pci_posting_flush(struct jsm_board *bd) | |||
| 48 | 48 | ||
| 49 | static void neo_set_cts_flow_control(struct jsm_channel *ch) | 49 | static void neo_set_cts_flow_control(struct jsm_channel *ch) |
| 50 | { | 50 | { |
| 51 | u8 ier = readb(&ch->ch_neo_uart->ier); | 51 | u8 ier, efr; |
| 52 | u8 efr = readb(&ch->ch_neo_uart->efr); | 52 | ier = readb(&ch->ch_neo_uart->ier); |
| 53 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 53 | 54 | ||
| 54 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting CTSFLOW\n"); | 55 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting CTSFLOW\n"); |
| 55 | 56 | ||
| @@ -78,8 +79,9 @@ static void neo_set_cts_flow_control(struct jsm_channel *ch) | |||
| 78 | 79 | ||
| 79 | static void neo_set_rts_flow_control(struct jsm_channel *ch) | 80 | static void neo_set_rts_flow_control(struct jsm_channel *ch) |
| 80 | { | 81 | { |
| 81 | u8 ier = readb(&ch->ch_neo_uart->ier); | 82 | u8 ier, efr; |
| 82 | u8 efr = readb(&ch->ch_neo_uart->efr); | 83 | ier = readb(&ch->ch_neo_uart->ier); |
| 84 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 83 | 85 | ||
| 84 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting RTSFLOW\n"); | 86 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting RTSFLOW\n"); |
| 85 | 87 | ||
| @@ -117,8 +119,9 @@ static void neo_set_rts_flow_control(struct jsm_channel *ch) | |||
| 117 | 119 | ||
| 118 | static void neo_set_ixon_flow_control(struct jsm_channel *ch) | 120 | static void neo_set_ixon_flow_control(struct jsm_channel *ch) |
| 119 | { | 121 | { |
| 120 | u8 ier = readb(&ch->ch_neo_uart->ier); | 122 | u8 ier, efr; |
| 121 | u8 efr = readb(&ch->ch_neo_uart->efr); | 123 | ier = readb(&ch->ch_neo_uart->ier); |
| 124 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 122 | 125 | ||
| 123 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting IXON FLOW\n"); | 126 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting IXON FLOW\n"); |
| 124 | 127 | ||
| @@ -153,8 +156,9 @@ static void neo_set_ixon_flow_control(struct jsm_channel *ch) | |||
| 153 | 156 | ||
| 154 | static void neo_set_ixoff_flow_control(struct jsm_channel *ch) | 157 | static void neo_set_ixoff_flow_control(struct jsm_channel *ch) |
| 155 | { | 158 | { |
| 156 | u8 ier = readb(&ch->ch_neo_uart->ier); | 159 | u8 ier, efr; |
| 157 | u8 efr = readb(&ch->ch_neo_uart->efr); | 160 | ier = readb(&ch->ch_neo_uart->ier); |
| 161 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 158 | 162 | ||
| 159 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting IXOFF FLOW\n"); | 163 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Setting IXOFF FLOW\n"); |
| 160 | 164 | ||
| @@ -190,8 +194,9 @@ static void neo_set_ixoff_flow_control(struct jsm_channel *ch) | |||
| 190 | 194 | ||
| 191 | static void neo_set_no_input_flow_control(struct jsm_channel *ch) | 195 | static void neo_set_no_input_flow_control(struct jsm_channel *ch) |
| 192 | { | 196 | { |
| 193 | u8 ier = readb(&ch->ch_neo_uart->ier); | 197 | u8 ier, efr; |
| 194 | u8 efr = readb(&ch->ch_neo_uart->efr); | 198 | ier = readb(&ch->ch_neo_uart->ier); |
| 199 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 195 | 200 | ||
| 196 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Unsetting Input FLOW\n"); | 201 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Unsetting Input FLOW\n"); |
| 197 | 202 | ||
| @@ -228,8 +233,9 @@ static void neo_set_no_input_flow_control(struct jsm_channel *ch) | |||
| 228 | 233 | ||
| 229 | static void neo_set_no_output_flow_control(struct jsm_channel *ch) | 234 | static void neo_set_no_output_flow_control(struct jsm_channel *ch) |
| 230 | { | 235 | { |
| 231 | u8 ier = readb(&ch->ch_neo_uart->ier); | 236 | u8 ier, efr; |
| 232 | u8 efr = readb(&ch->ch_neo_uart->efr); | 237 | ier = readb(&ch->ch_neo_uart->ier); |
| 238 | efr = readb(&ch->ch_neo_uart->efr); | ||
| 233 | 239 | ||
| 234 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Unsetting Output FLOW\n"); | 240 | jsm_printk(PARAM, INFO, &ch->ch_bd->pci_dev, "Unsetting Output FLOW\n"); |
| 235 | 241 | ||
