Fixed issues relating to prink message including: a. using __func__ to replace function's name b. using pr_warn to replace "printk(KERN_ERR" Issue found by checkpatch Signed-off-by: Piro Yang <piroyangg@xxxxxxxxx> --- drivers/staging/vme_user/vme.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c index 5960562dcd96..56c528628e7d 100644 --- a/drivers/staging/vme_user/vme.c +++ b/drivers/staging/vme_user/vme.c @@ -418,7 +418,7 @@ int vme_slave_get(struct vme_resource *resource, int *enabled, image = list_entry(resource->entry, struct vme_slave_resource, list); if (!bridge->slave_get) { - printk(KERN_ERR "vme_slave_get not supported\n"); + pr_warn("%s not supported\n", __func__); return -EINVAL; } @@ -568,7 +568,7 @@ int vme_master_set(struct vme_resource *resource, int enabled, image = list_entry(resource->entry, struct vme_master_resource, list); if (!bridge->master_set) { - printk(KERN_WARNING "vme_master_set not supported\n"); + pr_warn("%s not supported\n", __func__); return -EINVAL; } @@ -618,7 +618,7 @@ int vme_master_get(struct vme_resource *resource, int *enabled, image = list_entry(resource->entry, struct vme_master_resource, list); if (!bridge->master_get) { - printk(KERN_WARNING "%s not supported\n", __func__); + pr_warn("%s not supported\n", __func__); return -EINVAL; } @@ -1552,7 +1552,7 @@ int vme_lm_set(struct vme_resource *resource, unsigned long long lm_base, lm = list_entry(resource->entry, struct vme_lm_resource, list); if (!bridge->lm_set) { - printk(KERN_ERR "vme_lm_set not supported\n"); + pr_warn("%s not supported\n", __func__); return -EINVAL; } @@ -1588,7 +1588,7 @@ int vme_lm_get(struct vme_resource *resource, unsigned long long *lm_base, lm = list_entry(resource->entry, struct vme_lm_resource, list); if (!bridge->lm_get) { - printk(KERN_ERR "vme_lm_get not supported\n"); + printk(KERN_ERR "%s not supported\n", __func__); return -EINVAL; } @@ -1625,7 +1625,7 @@ int vme_lm_attach(struct vme_resource *resource, int monitor, lm = list_entry(resource->entry, struct vme_lm_resource, list); if (!bridge->lm_attach) { - printk(KERN_ERR "vme_lm_attach not supported\n"); + printk(KERN_ERR "%s not supported\n", __func__); return -EINVAL; } @@ -1658,7 +1658,7 @@ int vme_lm_detach(struct vme_resource *resource, int monitor) lm = list_entry(resource->entry, struct vme_lm_resource, list); if (!bridge->lm_detach) { - printk(KERN_ERR "vme_lm_detach not supported\n"); + printk(KERN_ERR "%s not supported\n", __func__); return -EINVAL; } @@ -1725,7 +1725,7 @@ int vme_slot_num(struct vme_dev *vdev) } if (!bridge->slot_get) { - printk(KERN_WARNING "vme_slot_num not supported\n"); + printk(KERN_WARNING "%s not supported\n", __func__); return -EINVAL; } -- 2.25.1