Skip to content

Commit 6853885

Browse files
Ranjan Kumarmartinkpetersen
authored andcommitted
scsi: mpi3mr: Drop unnecessary volatile from __iomem pointers
The volatile qualifier is redundant for __iomem pointers. Cleaned up usage in mpi3mr_writeq() and sysif_regs pointer as per Upstream compliance. Signed-off-by: Ranjan Kumar <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Martin K. Petersen <[email protected]>
1 parent e6327c4 commit 6853885

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

drivers/scsi/mpi3mr/mpi3mr.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1185,7 +1185,7 @@ struct mpi3mr_ioc {
11851185
char name[MPI3MR_NAME_LENGTH];
11861186
char driver_name[MPI3MR_NAME_LENGTH];
11871187

1188-
volatile struct mpi3_sysif_registers __iomem *sysif_regs;
1188+
struct mpi3_sysif_registers __iomem *sysif_regs;
11891189
resource_size_t sysif_regs_phys;
11901190
int bars;
11911191
u64 dma_mask;

drivers/scsi/mpi3mr/mpi3mr_fw.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,12 @@ module_param(poll_queues, int, 0444);
2323
MODULE_PARM_DESC(poll_queues, "Number of queues for io_uring poll mode. (Range 1 - 126)");
2424

2525
#if defined(writeq) && defined(CONFIG_64BIT)
26-
static inline void mpi3mr_writeq(__u64 b, volatile void __iomem *addr)
26+
static inline void mpi3mr_writeq(__u64 b, void __iomem *addr)
2727
{
2828
writeq(b, addr);
2929
}
3030
#else
31-
static inline void mpi3mr_writeq(__u64 b, volatile void __iomem *addr)
31+
static inline void mpi3mr_writeq(__u64 b, void __iomem *addr)
3232
{
3333
__u64 data_out = b;
3434

0 commit comments

Comments
 (0)