From: Arnd Bergmann Date: Wed, 16 Oct 2024 11:15:16 +0000 (+0000) Subject: staging: gpib: avoid unused const variables X-Git-Tag: v6.13-rc1~30^2~97 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b8989f45d1ec0b5e1aac5da2a915f8002015eb39;p=thirdparty%2Fkernel%2Flinux.git staging: gpib: avoid unused const variables Variables that are 'static const' but not used anywhere cause a warning with "gcc -Wunused-const-variable", which we may want to enable by default in the future. The gpib code already has a mix of 'enum' and 'static const' variables for named constants, so convert the ones that are causing problems to enums as well, or move them closer to the only users where possible. Signed-off-by: Arnd Bergmann Link: https://lore.kernel.org/r/20241016111521.1143191-3-arnd@kernel.org Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/gpib/common/gpib_os.c b/drivers/staging/gpib/common/gpib_os.c index 01efe99adeb36..e9dd7b2d1569a 100644 --- a/drivers/staging/gpib/common/gpib_os.c +++ b/drivers/staging/gpib/common/gpib_os.c @@ -523,6 +523,8 @@ int serial_poll_all(gpib_board_t *board, unsigned int usec_timeout) * SPD and UNT are sent at the completion of the poll. */ +static const int gpib_addr_max = 30; /* max address for primary/secondary gpib addresses */ + int dvrsp(gpib_board_t *board, unsigned int pad, int sad, unsigned int usec_timeout, uint8_t *result) { diff --git a/drivers/staging/gpib/eastwood/fluke_gpib.h b/drivers/staging/gpib/eastwood/fluke_gpib.h index d6c5b0124c7e2..fcbd42f8f9af6 100644 --- a/drivers/staging/gpib/eastwood/fluke_gpib.h +++ b/drivers/staging/gpib/eastwood/fluke_gpib.h @@ -136,6 +136,8 @@ enum cb7210_aux_cmds { AUX_HI_SPEED = 0x41, }; -static const int fluke_reg_offset = 4; -static const int fluke_num_regs = 8; -static const unsigned int write_transfer_counter_mask = 0x7ff; +enum { + fluke_reg_offset = 4, + fluke_num_regs = 8, + write_transfer_counter_mask = 0x7ff, +}; diff --git a/drivers/staging/gpib/include/nec7210_registers.h b/drivers/staging/gpib/include/nec7210_registers.h index 2ad512c372c4d..888803dd97f9e 100644 --- a/drivers/staging/gpib/include/nec7210_registers.h +++ b/drivers/staging/gpib/include/nec7210_registers.h @@ -17,9 +17,6 @@ enum nec7210_chipset { TNT5004, // NI (minor differences to TNT4882) }; -// nec7210 has 8 registers -static const int nec7210_num_registers = 8; - /* nec7210 register numbers (might need to be multiplied by * a board-dependent offset to get actually io address offset) */ @@ -33,6 +30,9 @@ enum nec7210_write_regs { AUXMR, // auxiliary mode ADR, // address EOSR, // end-of-string + + // nec7210 has 8 registers + nec7210_num_registers = 8, }; // read registers diff --git a/drivers/staging/gpib/include/tms9914.h b/drivers/staging/gpib/include/tms9914.h index 1cbba02c35815..456b488212d2b 100644 --- a/drivers/staging/gpib/include/tms9914.h +++ b/drivers/staging/gpib/include/tms9914.h @@ -132,7 +132,9 @@ irqreturn_t tms9914_interrupt_have_status(gpib_board_t *board, struct tms9914_pr int status1, int status2); // tms9914 has 8 registers -static const int tms9914_num_registers = 8; +enum { + ms9914_num_registers = 8, +}; /* tms9914 register numbers (might need to be multiplied by * a board-dependent offset to get actually io address offset) diff --git a/drivers/staging/gpib/include/tnt4882_registers.h b/drivers/staging/gpib/include/tnt4882_registers.h index f0a719772f413..1b1441cd03d50 100644 --- a/drivers/staging/gpib/include/tnt4882_registers.h +++ b/drivers/staging/gpib/include/tnt4882_registers.h @@ -40,7 +40,9 @@ enum { BSR = BCR, }; -static const int tnt_pagein_offset = 0x11; +enum { + tnt_pagein_offset = 0x11, +}; /*============================================================*/ diff --git a/drivers/staging/gpib/ines/ines.h b/drivers/staging/gpib/ines/ines.h index ae7d042e9f040..7e8302619998f 100644 --- a/drivers/staging/gpib/ines/ines.h +++ b/drivers/staging/gpib/ines/ines.h @@ -212,7 +212,4 @@ enum ines_auxd_bits { INES_T6_50us = 0x10, }; -static const int ines_isa_iosize = 0x20; -static const int ines_pcmcia_iosize = 0x20; - #endif // _INES_GPIB_H diff --git a/drivers/staging/gpib/ines/ines_gpib.c b/drivers/staging/gpib/ines/ines_gpib.c index 0b9a5c70e53fa..e98a114a9570f 100644 --- a/drivers/staging/gpib/ines/ines_gpib.c +++ b/drivers/staging/gpib/ines/ines_gpib.c @@ -88,8 +88,6 @@ unsigned int ines_t1_delay(gpib_board_t *board, unsigned int nano_sec) return retval; } -static const int in_fifo_size = 0xff; - static inline unsigned short num_in_fifo_bytes(struct ines_priv *ines_priv) { return ines_inb(ines_priv, IN_FIFO_COUNT); @@ -885,6 +883,8 @@ int ines_pci_accel_attach(gpib_board_t *board, const gpib_board_config_t *config return 0; } +static const int ines_isa_iosize = 0x20; + int ines_isa_attach(gpib_board_t *board, const gpib_board_config_t *config) { struct ines_priv *ines_priv; @@ -995,6 +995,8 @@ static int pc_debug = PCMCIA_DEBUG; #define DEBUG(args...) #endif +static const int ines_pcmcia_iosize = 0x20; + /* The event() function is this driver's Card Services event handler. * It will be called by Card Services when an appropriate card status * event is received. The config() and release() entry points are diff --git a/drivers/staging/gpib/tms9914/tms9914.c b/drivers/staging/gpib/tms9914/tms9914.c index 4772a1ce8bc9c..b55340ee25341 100644 --- a/drivers/staging/gpib/tms9914/tms9914.c +++ b/drivers/staging/gpib/tms9914/tms9914.c @@ -816,9 +816,6 @@ irqreturn_t tms9914_interrupt_have_status(gpib_board_t *board, struct tms9914_pr } EXPORT_SYMBOL(tms9914_interrupt_have_status); -// size of modbus pci memory io region -static const int iomem_size = 0x2000; - void tms9914_board_reset(struct tms9914_priv *priv) { /* chip reset */ diff --git a/drivers/staging/gpib/tnt4882/tnt4882_gpib.c b/drivers/staging/gpib/tnt4882/tnt4882_gpib.c index caf25c2a6e1f9..4d702c4452e8f 100644 --- a/drivers/staging/gpib/tnt4882/tnt4882_gpib.c +++ b/drivers/staging/gpib/tnt4882/tnt4882_gpib.c @@ -100,7 +100,6 @@ static const int atgpib_reg_offset = 2; // number of ioports used static const int atgpib_iosize = 32; -static const int pcmcia_gpib_iosize = 32; /* paged io */ static inline unsigned int tnt_paged_readb(struct tnt4882_priv *priv, unsigned long offset) @@ -1795,6 +1794,8 @@ void __exit exit_ni_gpib_cs(void) pcmcia_unregister_driver(&ni_gpib_cs_driver); } +static const int pcmcia_gpib_iosize = 32; + int ni_pcmcia_attach(gpib_board_t *board, const gpib_board_config_t *config) { struct local_info_t *info; diff --git a/drivers/staging/gpib/uapi/gpib_user.h b/drivers/staging/gpib/uapi/gpib_user.h index 4348bb69c9334..0896a55a758fe 100644 --- a/drivers/staging/gpib/uapi/gpib_user.h +++ b/drivers/staging/gpib/uapi/gpib_user.h @@ -46,12 +46,12 @@ enum ibsta_bits { SRQI = (1 << SRQI_NUM), /* SRQ is asserted */ END = (1 << END_NUM), /* EOI or EOS encountered */ TIMO = (1 << TIMO_NUM), /* Time limit on I/O or wait function exceeded */ - ERR = (1 << ERR_NUM) /* Function call terminated on error */ -}; + ERR = (1 << ERR_NUM), /* Function call terminated on error */ -static const int device_status_mask = ERR | TIMO | END | CMPL | RQS; -static const int board_status_mask = ERR | TIMO | END | CMPL | SPOLL | - EVENT | LOK | REM | CIC | ATN | TACS | LACS | DTAS | DCAS | SRQI; + device_status_mask = ERR | TIMO | END | CMPL | RQS, + board_status_mask = ERR | TIMO | END | CMPL | SPOLL | + EVENT | LOK | REM | CIC | ATN | TACS | LACS | DTAS | DCAS | SRQI, +}; /* IBERR error codes */ enum iberr_code { @@ -209,7 +209,9 @@ static inline uint8_t CFGn(unsigned int meters) } /* mask of bits that actually matter in a command byte */ -static const uint8_t gpib_command_mask = 0x7f; +enum { + gpib_command_mask = 0x7f, +}; static inline int is_PPE(uint8_t command) { @@ -261,8 +263,6 @@ static inline int gpib_address_equal(unsigned int pad1, int sad1, unsigned int p return 0; } -static const int gpib_addr_max = 30; /* max address for primary/secondary gpib addresses */ - enum ibask_option { IbaPAD = 0x1, IbaSAD = 0x2, @@ -341,7 +341,9 @@ enum t1_delays { T1_DELAY_350ns = 3 }; -static const int request_service_bit = 0x40; +enum { + request_service_bit = 0x40, +}; enum gpib_events { EventNone = 0,