Remove hard-coded strings by using the str_enabled_disabled() helper function. Use pr_debug() instead of printk(KERN_DEBUG) to silence a checkpatch warning. Suggested-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx> --- drivers/scsi/vmw_pvscsi.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c index 32242d86cf5b..4927c6a33bd8 100644 --- a/drivers/scsi/vmw_pvscsi.c +++ b/drivers/scsi/vmw_pvscsi.c @@ -25,6 +25,7 @@ #include <linux/slab.h> #include <linux/workqueue.h> #include <linux/pci.h> +#include <linux/string_choices.h> #include <scsi/scsi.h> #include <scsi/scsi_host.h> @@ -1508,8 +1509,8 @@ static int pvscsi_probe(struct pci_dev *pdev, const struct pci_device_id *id) goto out_reset_adapter; adapter->use_req_threshold = pvscsi_setup_req_threshold(adapter, true); - printk(KERN_DEBUG "vmw_pvscsi: driver-based request coalescing %sabled\n", - adapter->use_req_threshold ? "en" : "dis"); + pr_debug("vmw_pvscsi: driver-based request coalescing %s\n", + str_enabled_disabled(adapter->use_req_threshold)); if (adapter->dev->msix_enabled || adapter->dev->msi_enabled) { printk(KERN_INFO "vmw_pvscsi: using MSI%s\n", -- 2.47.1