Use memdup_user() helper instead of open-coding to simplify the code.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/block/cciss.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
index c
Use offset_in_page() macro instead of open-coding.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/xen-scsifront.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/xen-scsifront.c b/drivers/scsi/xen-scsifront.c
index 9aa1fe1..a6a8b60
Use offset_in_page() macro instead of open-coding.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/gdth.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/gdth.c b/drivers/scsi/gdth.c
index d020a13..557ce73 100644
--- a/drivers/scsi/
Use sg_virt() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/qla2xxx/qla_isr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
index 3203367..9610d85
Use kobj_to_dev() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/qla2xxx/qla_attr.c | 42 ++---
drivers/scsi/qla4xxx/ql4_attr.c | 6 ++
2 files changed, 16 insertions(+), 32 deletions(-)
diff
Use module_pci_driver() helper to simplify the code.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/dmx3191d.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/scsi/dmx3191d.c b/drivers/scsi/dmx3191d.c
index 3aa4657..6af3394
Use module_pci_driver() helper to simplify the code.
Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
drivers/scsi/a100u2w.c | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/drivers/scsi/a100u2w.c b/drivers/scsi/a100u2w.c
index 8086bd0..b2942ec
Use to_delayed_work() instead of open-coding it.
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/scsi/mvsas/mv_sas.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/mvsas/mv_sas.c b/drivers/scsi/mvsas/mv_sas.c
index e712fe7..78bf557
Use offset_in_page macro instead of (addr & ~PAGE_MASK).
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/target/target_core_user.c | 2 +-
drivers/target/tcm_fc/tfc_io.c| 8
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/driv
s/regsiter/register/
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/scsi/isci/registers.h | 4 ++--
drivers/scsi/mpt2sas/mpt2sas_base.h | 2 +-
drivers/scsi/mpt3sas/mpt3sas_base.h | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/scs
) For CONFIG_BUG and !HAVE_ARCH_BUG_ON cases, BUG_ON() contains an
unlikely compiler flag already.
Hence, we can drop unlikely behind BUG_ON().
[1] http://lkml.iu.edu/hypermail/linux/kernel/1101.3/02289.html
Signed-off-by: Geliang Tang <geliangt...@163.com>
Reviewed-by: Bart Van Assche <ba
Just fix two typos in the code comment.
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/scsi/isci/request.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c
index cfd0084..8fe106f 100644
--- a/d
BUG_ON() already contain an unlikely compiler flag. Drop it.
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/scsi/scsi_lib.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
index f570b48..3b5faab
IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it.
Signed-off-by: Geliang Tang <geliangt...@163.com>
---
drivers/scsi/cxlflash/superpipe.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/cxlflash/superpipe.c
b/drivers/scsi/cx
14 matches
Mail list logo