ip2: cleanup globals
- do not init .bss zeroed data to zero again (by memset or explicit assignment) - use char [] instead of char * for string constants Signed-off-by: Jiri Slaby <jirislaby@gmail.com> Signed-off-by: Alan Cox <alan@redhat.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
47babd4c6a
commit
cf176bc302
@ -150,12 +150,12 @@ static int ip2_read_proc(char *, char **, off_t, int, int *, void * );
|
|||||||
/*************/
|
/*************/
|
||||||
|
|
||||||
/* String constants to identify ourselves */
|
/* String constants to identify ourselves */
|
||||||
static char *pcName = "Computone IntelliPort Plus multiport driver";
|
static const char pcName[] = "Computone IntelliPort Plus multiport driver";
|
||||||
static char *pcVersion = "1.2.14";
|
static const char pcVersion[] = "1.2.14";
|
||||||
|
|
||||||
/* String constants for port names */
|
/* String constants for port names */
|
||||||
static char *pcDriver_name = "ip2";
|
static const char pcDriver_name[] = "ip2";
|
||||||
static char *pcIpl = "ip2ipl";
|
static const char pcIpl[] = "ip2ipl";
|
||||||
|
|
||||||
/***********************/
|
/***********************/
|
||||||
/* Function Prototypes */
|
/* Function Prototypes */
|
||||||
@ -237,8 +237,8 @@ static const struct file_operations ip2_ipl = {
|
|||||||
.open = ip2_ipl_open,
|
.open = ip2_ipl_open,
|
||||||
};
|
};
|
||||||
|
|
||||||
static unsigned long irq_counter = 0;
|
static unsigned long irq_counter;
|
||||||
static unsigned long bh_counter = 0;
|
static unsigned long bh_counter;
|
||||||
|
|
||||||
// Use immediate queue to service interrupts
|
// Use immediate queue to service interrupts
|
||||||
#define USE_IQI
|
#define USE_IQI
|
||||||
@ -286,7 +286,7 @@ MODULE_AUTHOR("Doug McNash");
|
|||||||
MODULE_DESCRIPTION("Computone IntelliPort Plus Driver");
|
MODULE_DESCRIPTION("Computone IntelliPort Plus Driver");
|
||||||
MODULE_LICENSE("GPL");
|
MODULE_LICENSE("GPL");
|
||||||
|
|
||||||
static int poll_only = 0;
|
static int poll_only;
|
||||||
|
|
||||||
static int Eisa_irq;
|
static int Eisa_irq;
|
||||||
static int Eisa_slot;
|
static int Eisa_slot;
|
||||||
@ -615,10 +615,6 @@ static int ip2_loadmain(void)
|
|||||||
/* Initialise the iiEllis subsystem. */
|
/* Initialise the iiEllis subsystem. */
|
||||||
iiEllisInit();
|
iiEllisInit();
|
||||||
|
|
||||||
/* Initialize arrays. */
|
|
||||||
memset( i2BoardPtrTable, 0, sizeof i2BoardPtrTable );
|
|
||||||
memset( DevTable, 0, sizeof DevTable );
|
|
||||||
|
|
||||||
/* Initialise all the boards we can find (up to the maximum). */
|
/* Initialise all the boards we can find (up to the maximum). */
|
||||||
for ( i = 0; i < IP2_MAX_BOARDS; ++i ) {
|
for ( i = 0; i < IP2_MAX_BOARDS; ++i ) {
|
||||||
switch ( ip2config.addr[i] ) {
|
switch ( ip2config.addr[i] ) {
|
||||||
|
Loading…
Reference in New Issue
Block a user