* [PATCH v2 0/2] staging: vme_user: Replace printk's & clean up log messages
@ 2023-10-19 7:20 Soumya Negi
2023-10-19 7:20 ` [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*() Soumya Negi
2023-10-19 7:20 ` [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name Soumya Negi
0 siblings, 2 replies; 11+ messages in thread
From: Soumya Negi @ 2023-10-19 7:20 UTC (permalink / raw)
To: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall
Cc: outreachy, linux-kernel, linux-staging, Soumya Negi
Staging driver vme_user has a bunch of printk() calls in vme.c which
triggers checkpatch warnings. Remove all printk's and change them to
the appropriate logging mechanism i.e pr_err()/pr_warn(), or
dev_err()/dev_warn().
Also, clean up the messages further by using __func__ in the string
instead of function names.
This patchset fixes all checkpatch warnings like:
WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then
dev_err(dev, ... then pr_err(... to printk(KERN_ERR ...
&
WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
this function's name, in a string
Patches must be applied in order.
Changes in v2:
1. Correct "cleanup" to "clean up" in cover letter. (as per feedback
from julia.lawall@inria.fr)
2. Patch 1: Change the pr_*() calls used to replace printk's in v1 to
dev_*() wherever possible, as it adds more context to the messages.
(as per feedback from julia.lawall@inria.fr &
gregkh@linuxfoundation.org)
Soumya Negi (2):
staging: vme_user: Replace printk() with pr_*(),dev_*()
staging: vme_user: Use __func__ instead of function name
drivers/staging/vme_user/vme.c | 193 +++++++++++++++++++--------------
1 file changed, 109 insertions(+), 84 deletions(-)
--
2.42.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*()
2023-10-19 7:20 [PATCH v2 0/2] staging: vme_user: Replace printk's & clean up log messages Soumya Negi
@ 2023-10-19 7:20 ` Soumya Negi
2023-10-19 10:45 ` Karolina Stolarek
2023-10-19 15:42 ` Andi Shyti
2023-10-19 7:20 ` [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name Soumya Negi
1 sibling, 2 replies; 11+ messages in thread
From: Soumya Negi @ 2023-10-19 7:20 UTC (permalink / raw)
To: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall
Cc: outreachy, linux-kernel, linux-staging, Soumya Negi
vme.c uses printk() to log messages. To improve and standardize message
formatting, use logging mechanisms pr_err()/pr_warn() and
dev_err()/dev_warn() instead. Retain the printk log levels of the
messages during replacement.
Issue found by checkpatch.pl
Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
---
Changes in v2:
* Change the pr_*() calls used to replace printk's in v1 to dev_*()
wherever possible, as it adds more context to the message.
(as per feedback from julia.lawall@inria.fr &
gregkh@linuxfoundation.org)
drivers/staging/vme_user/vme.c | 193 +++++++++++++++++++--------------
1 file changed, 109 insertions(+), 84 deletions(-)
diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
index 6519a7c994a0..640b2dda3ac6 100644
--- a/drivers/staging/vme_user/vme.c
+++ b/drivers/staging/vme_user/vme.c
@@ -9,6 +9,8 @@
* Copyright 2004 Motorola Inc.
*/
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
#include <linux/init.h>
#include <linux/export.h>
#include <linux/mm.h>
@@ -62,7 +64,7 @@ static struct vme_bridge *find_bridge(struct vme_resource *resource)
return list_entry(resource->entry, struct vme_lm_resource,
list)->parent;
default:
- printk(KERN_ERR "Unknown resource type\n");
+ pr_err("Unknown resource type\n");
return NULL;
}
}
@@ -84,24 +86,25 @@ void *vme_alloc_consistent(struct vme_resource *resource, size_t size,
struct vme_bridge *bridge;
if (!resource) {
- printk(KERN_ERR "No resource\n");
+ pr_err("No resource\n");
return NULL;
}
bridge = find_bridge(resource);
if (!bridge) {
- printk(KERN_ERR "Can't find bridge\n");
+ pr_err("Can't find bridge\n");
return NULL;
}
if (!bridge->parent) {
- printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
+ pr_err("Dev entry NULL for bridge %s\n", bridge->name);
return NULL;
}
if (!bridge->alloc_consistent) {
- printk(KERN_ERR "alloc_consistent not supported by bridge %s\n",
- bridge->name);
+ dev_err(bridge->parent,
+ "alloc_consistent not supported by bridge %s\n",
+ bridge->name);
return NULL;
}
@@ -124,24 +127,25 @@ void vme_free_consistent(struct vme_resource *resource, size_t size,
struct vme_bridge *bridge;
if (!resource) {
- printk(KERN_ERR "No resource\n");
+ pr_err("No resource\n");
return;
}
bridge = find_bridge(resource);
if (!bridge) {
- printk(KERN_ERR "Can't find bridge\n");
+ pr_err("Can't find bridge\n");
return;
}
if (!bridge->parent) {
- printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
+ pr_err("Dev entry NULL for bridge %s\n", bridge->name);
return;
}
if (!bridge->free_consistent) {
- printk(KERN_ERR "free_consistent not supported by bridge %s\n",
- bridge->name);
+ dev_err(bridge->parent,
+ "free_consistent not supported by bridge %s\n",
+ bridge->name);
return;
}
@@ -161,6 +165,7 @@ EXPORT_SYMBOL(vme_free_consistent);
*/
size_t vme_get_size(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
int enabled, retval;
unsigned long long base, size;
dma_addr_t buf_base;
@@ -184,7 +189,7 @@ size_t vme_get_size(struct vme_resource *resource)
case VME_DMA:
return 0;
default:
- printk(KERN_ERR "Unknown resource type\n");
+ dev_err(bridge->parent, "Unknown resource type\n");
return 0;
}
}
@@ -225,7 +230,7 @@ int vme_check_window(u32 aspace, unsigned long long vme_base,
/* User Defined */
break;
default:
- printk(KERN_ERR "Invalid address space\n");
+ pr_err("Invalid address space\n");
retval = -EINVAL;
break;
}
@@ -288,14 +293,15 @@ struct vme_resource *vme_slave_request(struct vme_dev *vdev, u32 address,
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
goto err_bus;
}
/* Loop through slave resources */
list_for_each_entry(slave_image, &bridge->slave_resources, list) {
if (!slave_image) {
- printk(KERN_ERR "Registered NULL Slave resource\n");
+ dev_err(bridge->parent,
+ "Registered NULL Slave resource\n");
continue;
}
@@ -362,20 +368,20 @@ int vme_slave_set(struct vme_resource *resource, int enabled,
int retval;
if (resource->type != VME_SLAVE) {
- printk(KERN_ERR "Not a slave resource\n");
+ dev_err(bridge->parent, "Not a slave resource\n");
return -EINVAL;
}
image = list_entry(resource->entry, struct vme_slave_resource, list);
if (!bridge->slave_set) {
- printk(KERN_ERR "Function not supported\n");
+ dev_err(bridge->parent, "Function not supported\n");
return -ENOSYS;
}
if (!(((image->address_attr & aspace) == aspace) &&
((image->cycle_attr & cycle) == cycle))) {
- printk(KERN_ERR "Invalid attributes\n");
+ dev_err(bridge->parent, "Invalid attributes\n");
return -EINVAL;
}
@@ -411,14 +417,14 @@ int vme_slave_get(struct vme_resource *resource, int *enabled,
struct vme_slave_resource *image;
if (resource->type != VME_SLAVE) {
- printk(KERN_ERR "Not a slave resource\n");
+ dev_err(bridge->parent, "Not a slave resource\n");
return -EINVAL;
}
image = list_entry(resource->entry, struct vme_slave_resource, list);
if (!bridge->slave_get) {
- printk(KERN_ERR "vme_slave_get not supported\n");
+ dev_err(bridge->parent, "vme_slave_get not supported\n");
return -EINVAL;
}
@@ -435,24 +441,25 @@ EXPORT_SYMBOL(vme_slave_get);
*/
void vme_slave_free(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_slave_resource *slave_image;
if (resource->type != VME_SLAVE) {
- printk(KERN_ERR "Not a slave resource\n");
+ dev_err(bridge->parent, "Not a slave resource\n");
return;
}
slave_image = list_entry(resource->entry, struct vme_slave_resource,
list);
if (!slave_image) {
- printk(KERN_ERR "Can't find slave resource\n");
+ dev_err(bridge->parent, "Can't find slave resource\n");
return;
}
/* Unlock image */
mutex_lock(&slave_image->mtx);
if (slave_image->locked == 0)
- printk(KERN_ERR "Image is already free\n");
+ dev_err(bridge->parent, "Image is already free\n");
slave_image->locked = 0;
mutex_unlock(&slave_image->mtx);
@@ -484,14 +491,15 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
goto err_bus;
}
/* Loop through master resources */
list_for_each_entry(master_image, &bridge->master_resources, list) {
if (!master_image) {
- printk(KERN_WARNING "Registered NULL master resource\n");
+ dev_warn(bridge->parent,
+ "Registered NULL master resource\n");
continue;
}
@@ -511,7 +519,7 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
/* Check to see if we found a resource */
if (!allocated_image) {
- printk(KERN_ERR "Can't find a suitable resource\n");
+ dev_err(&vdev->dev, "Can't find a suitable resource\n");
goto err_image;
}
@@ -561,21 +569,21 @@ int vme_master_set(struct vme_resource *resource, int enabled,
int retval;
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
image = list_entry(resource->entry, struct vme_master_resource, list);
if (!bridge->master_set) {
- printk(KERN_WARNING "vme_master_set not supported\n");
+ dev_warn(bridge->parent, "vme_master_set not supported\n");
return -EINVAL;
}
if (!(((image->address_attr & aspace) == aspace) &&
((image->cycle_attr & cycle) == cycle) &&
((image->width_attr & dwidth) == dwidth))) {
- printk(KERN_WARNING "Invalid attributes\n");
+ dev_warn(bridge->parent, "Invalid attributes\n");
return -EINVAL;
}
@@ -611,14 +619,14 @@ int vme_master_get(struct vme_resource *resource, int *enabled,
struct vme_master_resource *image;
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
image = list_entry(resource->entry, struct vme_master_resource, list);
if (!bridge->master_get) {
- printk(KERN_WARNING "%s not supported\n", __func__);
+ dev_warn(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -650,12 +658,13 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
size_t length;
if (!bridge->master_read) {
- printk(KERN_WARNING "Reading from resource not supported\n");
+ dev_warn(bridge->parent,
+ "Reading from resource not supported\n");
return -EINVAL;
}
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
@@ -664,7 +673,7 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
length = vme_get_size(resource);
if (offset > length) {
- printk(KERN_WARNING "Invalid Offset\n");
+ dev_warn(bridge->parent, "Invalid Offset\n");
return -EFAULT;
}
@@ -698,12 +707,12 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
size_t length;
if (!bridge->master_write) {
- printk(KERN_WARNING "Writing to resource not supported\n");
+ dev_warn(bridge->parent, "Writing to resource not supported\n");
return -EINVAL;
}
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
@@ -712,7 +721,7 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
length = vme_get_size(resource);
if (offset > length) {
- printk(KERN_WARNING "Invalid Offset\n");
+ dev_warn(bridge->parent, "Invalid Offset\n");
return -EFAULT;
}
@@ -749,12 +758,12 @@ unsigned int vme_master_rmw(struct vme_resource *resource, unsigned int mask,
struct vme_master_resource *image;
if (!bridge->master_rmw) {
- printk(KERN_WARNING "Writing to resource not supported\n");
+ dev_warn(bridge->parent, "Writing to resource not supported\n");
return -EINVAL;
}
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
@@ -777,12 +786,13 @@ EXPORT_SYMBOL(vme_master_rmw);
*/
int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_master_resource *image;
phys_addr_t phys_addr;
unsigned long vma_size;
if (resource->type != VME_MASTER) {
- pr_err("Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return -EINVAL;
}
@@ -791,7 +801,7 @@ int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
vma_size = vma->vm_end - vma->vm_start;
if (phys_addr + vma_size > image->bus_resource.end + 1) {
- pr_err("Map size cannot exceed the window size\n");
+ dev_err(bridge->parent, "Map size cannot exceed the window size\n");
return -EFAULT;
}
@@ -809,24 +819,25 @@ EXPORT_SYMBOL(vme_master_mmap);
*/
void vme_master_free(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_master_resource *master_image;
if (resource->type != VME_MASTER) {
- printk(KERN_ERR "Not a master resource\n");
+ dev_err(bridge->parent, "Not a master resource\n");
return;
}
master_image = list_entry(resource->entry, struct vme_master_resource,
list);
if (!master_image) {
- printk(KERN_ERR "Can't find master resource\n");
+ dev_err(bridge->parent, "Can't find master resource\n");
return;
}
/* Unlock image */
spin_lock(&master_image->lock);
if (master_image->locked == 0)
- printk(KERN_ERR "Image is already free\n");
+ dev_err(bridge->parent, "Image is already free\n");
master_image->locked = 0;
spin_unlock(&master_image->lock);
@@ -854,18 +865,19 @@ struct vme_resource *vme_dma_request(struct vme_dev *vdev, u32 route)
struct vme_resource *resource = NULL;
/* XXX Not checking resource attributes */
- printk(KERN_ERR "No VME resource Attribute tests done\n");
+ dev_err(&vdev->dev, "No VME resource Attribute tests done\n");
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
goto err_bus;
}
/* Loop through DMA resources */
list_for_each_entry(dma_ctrlr, &bridge->dma_resources, list) {
if (!dma_ctrlr) {
- printk(KERN_ERR "Registered NULL DMA resource\n");
+ dev_err(bridge->parent,
+ "Registered NULL DMA resource\n");
continue;
}
@@ -917,10 +929,11 @@ EXPORT_SYMBOL(vme_dma_request);
*/
struct vme_dma_list *vme_new_dma_list(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_dma_list *dma_list;
if (resource->type != VME_DMA) {
- printk(KERN_ERR "Not a DMA resource\n");
+ dev_err(bridge->parent, "Not a DMA resource\n");
return NULL;
}
@@ -1098,12 +1111,13 @@ int vme_dma_list_add(struct vme_dma_list *list, struct vme_dma_attr *src,
int retval;
if (!bridge->dma_list_add) {
- printk(KERN_WARNING "Link List DMA generation not supported\n");
+ dev_warn(bridge->parent,
+ "Link List DMA generation not supported\n");
return -EINVAL;
}
if (!mutex_trylock(&list->mtx)) {
- printk(KERN_ERR "Link List already submitted\n");
+ dev_err(bridge->parent, "Link List already submitted\n");
return -EINVAL;
}
@@ -1131,7 +1145,8 @@ int vme_dma_list_exec(struct vme_dma_list *list)
int retval;
if (!bridge->dma_list_exec) {
- printk(KERN_ERR "Link List DMA execution not supported\n");
+ dev_err(bridge->parent,
+ "Link List DMA execution not supported\n");
return -EINVAL;
}
@@ -1160,12 +1175,13 @@ int vme_dma_list_free(struct vme_dma_list *list)
int retval;
if (!bridge->dma_list_empty) {
- printk(KERN_WARNING "Emptying of Link Lists not supported\n");
+ dev_warn(bridge->parent,
+ "Emptying of Link Lists not supported\n");
return -EINVAL;
}
if (!mutex_trylock(&list->mtx)) {
- printk(KERN_ERR "Link List in use\n");
+ dev_err(bridge->parent, "Link List in use\n");
return -EBUSY;
}
@@ -1175,7 +1191,7 @@ int vme_dma_list_free(struct vme_dma_list *list)
*/
retval = bridge->dma_list_empty(list);
if (retval) {
- printk(KERN_ERR "Unable to empty link-list entries\n");
+ dev_err(bridge->parent, "Unable to empty link-list entries\n");
mutex_unlock(&list->mtx);
return retval;
}
@@ -1197,22 +1213,24 @@ EXPORT_SYMBOL(vme_dma_list_free);
*/
int vme_dma_free(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_dma_resource *ctrlr;
if (resource->type != VME_DMA) {
- printk(KERN_ERR "Not a DMA resource\n");
+ dev_err(bridge->parent, "Not a DMA resource\n");
return -EINVAL;
}
ctrlr = list_entry(resource->entry, struct vme_dma_resource, list);
if (!mutex_trylock(&ctrlr->mtx)) {
- printk(KERN_ERR "Resource busy, can't free\n");
+ dev_err(bridge->parent, "Resource busy, can't free\n");
return -EBUSY;
}
if (!(list_empty(&ctrlr->pending) && list_empty(&ctrlr->running))) {
- printk(KERN_WARNING "Resource still processing transfers\n");
+ dev_warn(bridge->parent,
+ "Resource still processing transfers\n");
mutex_unlock(&ctrlr->mtx);
return -EBUSY;
}
@@ -1290,8 +1308,9 @@ void vme_irq_handler(struct vme_bridge *bridge, int level, int statid)
if (call)
call(level, statid, priv_data);
else
- printk(KERN_WARNING "Spurious VME interrupt, level:%x, vector:%x\n",
- level, statid);
+ dev_warn(bridge->parent,
+ "Spurious VME interrupt, level:%x, vector:%x\n", level,
+ statid);
}
EXPORT_SYMBOL(vme_irq_handler);
@@ -1319,17 +1338,18 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
return -EINVAL;
}
if ((level < 1) || (level > 7)) {
- printk(KERN_ERR "Invalid interrupt level\n");
+ dev_err(bridge->parent, "Invalid interrupt level\n");
return -EINVAL;
}
if (!bridge->irq_set) {
- printk(KERN_ERR "Configuring interrupts not supported\n");
+ dev_err(bridge->parent,
+ "Configuring interrupts not supported\n");
return -EINVAL;
}
@@ -1337,7 +1357,7 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
if (bridge->irq[level - 1].callback[statid].func) {
mutex_unlock(&bridge->irq_mtx);
- printk(KERN_WARNING "VME Interrupt already taken\n");
+ dev_warn(bridge->parent, "VME Interrupt already taken\n");
return -EBUSY;
}
@@ -1368,17 +1388,18 @@ void vme_irq_free(struct vme_dev *vdev, int level, int statid)
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
return;
}
if ((level < 1) || (level > 7)) {
- printk(KERN_ERR "Invalid interrupt level\n");
+ dev_err(bridge->parent, "Invalid interrupt level\n");
return;
}
if (!bridge->irq_set) {
- printk(KERN_ERR "Configuring interrupts not supported\n");
+ dev_err(bridge->parent,
+ "Configuring interrupts not supported\n");
return;
}
@@ -1415,17 +1436,18 @@ int vme_irq_generate(struct vme_dev *vdev, int level, int statid)
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
return -EINVAL;
}
if ((level < 1) || (level > 7)) {
- printk(KERN_WARNING "Invalid interrupt level\n");
+ dev_warn(bridge->parent, "Invalid interrupt level\n");
return -EINVAL;
}
if (!bridge->irq_generate) {
- printk(KERN_WARNING "Interrupt generation not supported\n");
+ dev_warn(bridge->parent,
+ "Interrupt generation not supported\n");
return -EINVAL;
}
@@ -1452,14 +1474,15 @@ struct vme_resource *vme_lm_request(struct vme_dev *vdev)
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
goto err_bus;
}
/* Loop through LM resources */
list_for_each_entry(lm, &bridge->lm_resources, list) {
if (!lm) {
- printk(KERN_ERR "Registered NULL Location Monitor resource\n");
+ dev_err(bridge->parent,
+ "Registered NULL Location Monitor resource\n");
continue;
}
@@ -1511,10 +1534,11 @@ EXPORT_SYMBOL(vme_lm_request);
*/
int vme_lm_count(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return -EINVAL;
}
@@ -1545,14 +1569,14 @@ int vme_lm_set(struct vme_resource *resource, unsigned long long lm_base,
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return -EINVAL;
}
lm = list_entry(resource->entry, struct vme_lm_resource, list);
if (!bridge->lm_set) {
- printk(KERN_ERR "vme_lm_set not supported\n");
+ dev_err(bridge->parent, "vme_lm_set not supported\n");
return -EINVAL;
}
@@ -1581,14 +1605,14 @@ int vme_lm_get(struct vme_resource *resource, unsigned long long *lm_base,
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return -EINVAL;
}
lm = list_entry(resource->entry, struct vme_lm_resource, list);
if (!bridge->lm_get) {
- printk(KERN_ERR "vme_lm_get not supported\n");
+ dev_err(bridge->parent, "vme_lm_get not supported\n");
return -EINVAL;
}
@@ -1618,14 +1642,14 @@ int vme_lm_attach(struct vme_resource *resource, int monitor,
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return -EINVAL;
}
lm = list_entry(resource->entry, struct vme_lm_resource, list);
if (!bridge->lm_attach) {
- printk(KERN_ERR "vme_lm_attach not supported\n");
+ dev_err(bridge->parent, "vme_lm_attach not supported\n");
return -EINVAL;
}
@@ -1651,14 +1675,14 @@ int vme_lm_detach(struct vme_resource *resource, int monitor)
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return -EINVAL;
}
lm = list_entry(resource->entry, struct vme_lm_resource, list);
if (!bridge->lm_detach) {
- printk(KERN_ERR "vme_lm_detach not supported\n");
+ dev_err(bridge->parent, "vme_lm_detach not supported\n");
return -EINVAL;
}
@@ -1680,10 +1704,11 @@ EXPORT_SYMBOL(vme_lm_detach);
*/
void vme_lm_free(struct vme_resource *resource)
{
+ struct vme_bridge *bridge = find_bridge(resource);
struct vme_lm_resource *lm;
if (resource->type != VME_LM) {
- printk(KERN_ERR "Not a Location Monitor resource\n");
+ dev_err(bridge->parent, "Not a Location Monitor resource\n");
return;
}
@@ -1720,12 +1745,12 @@ int vme_slot_num(struct vme_dev *vdev)
bridge = vdev->bridge;
if (!bridge) {
- printk(KERN_ERR "Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
return -EINVAL;
}
if (!bridge->slot_get) {
- printk(KERN_WARNING "vme_slot_num not supported\n");
+ dev_warn(bridge->parent, "vme_slot_num not supported\n");
return -EINVAL;
}
@@ -1748,7 +1773,7 @@ int vme_bus_num(struct vme_dev *vdev)
bridge = vdev->bridge;
if (!bridge) {
- pr_err("Can't find VME bus\n");
+ dev_err(&vdev->dev, "Can't find VME bus\n");
return -EINVAL;
}
--
2.42.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name
2023-10-19 7:20 [PATCH v2 0/2] staging: vme_user: Replace printk's & clean up log messages Soumya Negi
2023-10-19 7:20 ` [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*() Soumya Negi
@ 2023-10-19 7:20 ` Soumya Negi
2023-10-19 15:41 ` Andi Shyti
1 sibling, 1 reply; 11+ messages in thread
From: Soumya Negi @ 2023-10-19 7:20 UTC (permalink / raw)
To: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall
Cc: outreachy, linux-kernel, linux-staging, Soumya Negi
Replace function names in message strings with __func__ to fix
all checkpatch warnings like:
WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
this function's name, in a string
Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
---
Changes in v2:
* None
drivers/staging/vme_user/vme.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
index 640b2dda3ac6..e533cce8e54e 100644
--- a/drivers/staging/vme_user/vme.c
+++ b/drivers/staging/vme_user/vme.c
@@ -424,7 +424,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) {
- dev_err(bridge->parent, "vme_slave_get not supported\n");
+ dev_err(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -576,7 +576,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) {
- dev_warn(bridge->parent, "vme_master_set not supported\n");
+ dev_warn(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -1576,7 +1576,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) {
- dev_err(bridge->parent, "vme_lm_set not supported\n");
+ dev_err(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -1612,7 +1612,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) {
- dev_err(bridge->parent, "vme_lm_get not supported\n");
+ dev_err(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -1649,7 +1649,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) {
- dev_err(bridge->parent, "vme_lm_attach not supported\n");
+ dev_err(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -1682,7 +1682,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) {
- dev_err(bridge->parent, "vme_lm_detach not supported\n");
+ dev_err(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
@@ -1750,7 +1750,7 @@ int vme_slot_num(struct vme_dev *vdev)
}
if (!bridge->slot_get) {
- dev_warn(bridge->parent, "vme_slot_num not supported\n");
+ dev_warn(bridge->parent, "%s not supported\n", __func__);
return -EINVAL;
}
--
2.42.0
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*()
2023-10-19 7:20 ` [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*() Soumya Negi
@ 2023-10-19 10:45 ` Karolina Stolarek
2023-10-19 15:31 ` Greg Kroah-Hartman
2023-10-20 2:06 ` Soumya Negi
2023-10-19 15:42 ` Andi Shyti
1 sibling, 2 replies; 11+ messages in thread
From: Karolina Stolarek @ 2023-10-19 10:45 UTC (permalink / raw)
To: Soumya Negi
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall, outreachy, linux-kernel, linux-staging
On 19.10.2023 09:20, Soumya Negi wrote:
> vme.c uses printk() to log messages. To improve and standardize message
> formatting, use logging mechanisms pr_err()/pr_warn() and
> dev_err()/dev_warn() instead. Retain the printk log levels of the
> messages during replacement.
>
> Issue found by checkpatch.pl
>
> Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
> ---
> Changes in v2:
> * Change the pr_*() calls used to replace printk's in v1 to dev_*()
> wherever possible, as it adds more context to the message.
> (as per feedback from julia.lawall@inria.fr &
> gregkh@linuxfoundation.org)
You don't have to include the change log here, as it's already in the
cover letter. This space is usually used when a patch is not sent as a
part of the series.
All the best,
Karolina
>
> drivers/staging/vme_user/vme.c | 193 +++++++++++++++++++--------------
> 1 file changed, 109 insertions(+), 84 deletions(-)
>
> diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
> index 6519a7c994a0..640b2dda3ac6 100644
> --- a/drivers/staging/vme_user/vme.c
> +++ b/drivers/staging/vme_user/vme.c
> @@ -9,6 +9,8 @@
> * Copyright 2004 Motorola Inc.
> */
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> #include <linux/init.h>
> #include <linux/export.h>
> #include <linux/mm.h>
> @@ -62,7 +64,7 @@ static struct vme_bridge *find_bridge(struct vme_resource *resource)
> return list_entry(resource->entry, struct vme_lm_resource,
> list)->parent;
> default:
> - printk(KERN_ERR "Unknown resource type\n");
> + pr_err("Unknown resource type\n");
> return NULL;
> }
> }
> @@ -84,24 +86,25 @@ void *vme_alloc_consistent(struct vme_resource *resource, size_t size,
> struct vme_bridge *bridge;
>
> if (!resource) {
> - printk(KERN_ERR "No resource\n");
> + pr_err("No resource\n");
> return NULL;
> }
>
> bridge = find_bridge(resource);
> if (!bridge) {
> - printk(KERN_ERR "Can't find bridge\n");
> + pr_err("Can't find bridge\n");
> return NULL;
> }
>
> if (!bridge->parent) {
> - printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
> + pr_err("Dev entry NULL for bridge %s\n", bridge->name);
> return NULL;
> }
>
> if (!bridge->alloc_consistent) {
> - printk(KERN_ERR "alloc_consistent not supported by bridge %s\n",
> - bridge->name);
> + dev_err(bridge->parent,
> + "alloc_consistent not supported by bridge %s\n",
> + bridge->name);
> return NULL;
> }
>
> @@ -124,24 +127,25 @@ void vme_free_consistent(struct vme_resource *resource, size_t size,
> struct vme_bridge *bridge;
>
> if (!resource) {
> - printk(KERN_ERR "No resource\n");
> + pr_err("No resource\n");
> return;
> }
>
> bridge = find_bridge(resource);
> if (!bridge) {
> - printk(KERN_ERR "Can't find bridge\n");
> + pr_err("Can't find bridge\n");
> return;
> }
>
> if (!bridge->parent) {
> - printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
> + pr_err("Dev entry NULL for bridge %s\n", bridge->name);
> return;
> }
>
> if (!bridge->free_consistent) {
> - printk(KERN_ERR "free_consistent not supported by bridge %s\n",
> - bridge->name);
> + dev_err(bridge->parent,
> + "free_consistent not supported by bridge %s\n",
> + bridge->name);
> return;
> }
>
> @@ -161,6 +165,7 @@ EXPORT_SYMBOL(vme_free_consistent);
> */
> size_t vme_get_size(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> int enabled, retval;
> unsigned long long base, size;
> dma_addr_t buf_base;
> @@ -184,7 +189,7 @@ size_t vme_get_size(struct vme_resource *resource)
> case VME_DMA:
> return 0;
> default:
> - printk(KERN_ERR "Unknown resource type\n");
> + dev_err(bridge->parent, "Unknown resource type\n");
> return 0;
> }
> }
> @@ -225,7 +230,7 @@ int vme_check_window(u32 aspace, unsigned long long vme_base,
> /* User Defined */
> break;
> default:
> - printk(KERN_ERR "Invalid address space\n");
> + pr_err("Invalid address space\n");
> retval = -EINVAL;
> break;
> }
> @@ -288,14 +293,15 @@ struct vme_resource *vme_slave_request(struct vme_dev *vdev, u32 address,
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> goto err_bus;
> }
>
> /* Loop through slave resources */
> list_for_each_entry(slave_image, &bridge->slave_resources, list) {
> if (!slave_image) {
> - printk(KERN_ERR "Registered NULL Slave resource\n");
> + dev_err(bridge->parent,
> + "Registered NULL Slave resource\n");
> continue;
> }
>
> @@ -362,20 +368,20 @@ int vme_slave_set(struct vme_resource *resource, int enabled,
> int retval;
>
> if (resource->type != VME_SLAVE) {
> - printk(KERN_ERR "Not a slave resource\n");
> + dev_err(bridge->parent, "Not a slave resource\n");
> return -EINVAL;
> }
>
> image = list_entry(resource->entry, struct vme_slave_resource, list);
>
> if (!bridge->slave_set) {
> - printk(KERN_ERR "Function not supported\n");
> + dev_err(bridge->parent, "Function not supported\n");
> return -ENOSYS;
> }
>
> if (!(((image->address_attr & aspace) == aspace) &&
> ((image->cycle_attr & cycle) == cycle))) {
> - printk(KERN_ERR "Invalid attributes\n");
> + dev_err(bridge->parent, "Invalid attributes\n");
> return -EINVAL;
> }
>
> @@ -411,14 +417,14 @@ int vme_slave_get(struct vme_resource *resource, int *enabled,
> struct vme_slave_resource *image;
>
> if (resource->type != VME_SLAVE) {
> - printk(KERN_ERR "Not a slave resource\n");
> + dev_err(bridge->parent, "Not a slave resource\n");
> return -EINVAL;
> }
>
> image = list_entry(resource->entry, struct vme_slave_resource, list);
>
> if (!bridge->slave_get) {
> - printk(KERN_ERR "vme_slave_get not supported\n");
> + dev_err(bridge->parent, "vme_slave_get not supported\n");
> return -EINVAL;
> }
>
> @@ -435,24 +441,25 @@ EXPORT_SYMBOL(vme_slave_get);
> */
> void vme_slave_free(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_slave_resource *slave_image;
>
> if (resource->type != VME_SLAVE) {
> - printk(KERN_ERR "Not a slave resource\n");
> + dev_err(bridge->parent, "Not a slave resource\n");
> return;
> }
>
> slave_image = list_entry(resource->entry, struct vme_slave_resource,
> list);
> if (!slave_image) {
> - printk(KERN_ERR "Can't find slave resource\n");
> + dev_err(bridge->parent, "Can't find slave resource\n");
> return;
> }
>
> /* Unlock image */
> mutex_lock(&slave_image->mtx);
> if (slave_image->locked == 0)
> - printk(KERN_ERR "Image is already free\n");
> + dev_err(bridge->parent, "Image is already free\n");
>
> slave_image->locked = 0;
> mutex_unlock(&slave_image->mtx);
> @@ -484,14 +491,15 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> goto err_bus;
> }
>
> /* Loop through master resources */
> list_for_each_entry(master_image, &bridge->master_resources, list) {
> if (!master_image) {
> - printk(KERN_WARNING "Registered NULL master resource\n");
> + dev_warn(bridge->parent,
> + "Registered NULL master resource\n");
> continue;
> }
>
> @@ -511,7 +519,7 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
>
> /* Check to see if we found a resource */
> if (!allocated_image) {
> - printk(KERN_ERR "Can't find a suitable resource\n");
> + dev_err(&vdev->dev, "Can't find a suitable resource\n");
> goto err_image;
> }
>
> @@ -561,21 +569,21 @@ int vme_master_set(struct vme_resource *resource, int enabled,
> int retval;
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> image = list_entry(resource->entry, struct vme_master_resource, list);
>
> if (!bridge->master_set) {
> - printk(KERN_WARNING "vme_master_set not supported\n");
> + dev_warn(bridge->parent, "vme_master_set not supported\n");
> return -EINVAL;
> }
>
> if (!(((image->address_attr & aspace) == aspace) &&
> ((image->cycle_attr & cycle) == cycle) &&
> ((image->width_attr & dwidth) == dwidth))) {
> - printk(KERN_WARNING "Invalid attributes\n");
> + dev_warn(bridge->parent, "Invalid attributes\n");
> return -EINVAL;
> }
>
> @@ -611,14 +619,14 @@ int vme_master_get(struct vme_resource *resource, int *enabled,
> struct vme_master_resource *image;
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> image = list_entry(resource->entry, struct vme_master_resource, list);
>
> if (!bridge->master_get) {
> - printk(KERN_WARNING "%s not supported\n", __func__);
> + dev_warn(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -650,12 +658,13 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
> size_t length;
>
> if (!bridge->master_read) {
> - printk(KERN_WARNING "Reading from resource not supported\n");
> + dev_warn(bridge->parent,
> + "Reading from resource not supported\n");
> return -EINVAL;
> }
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> @@ -664,7 +673,7 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
> length = vme_get_size(resource);
>
> if (offset > length) {
> - printk(KERN_WARNING "Invalid Offset\n");
> + dev_warn(bridge->parent, "Invalid Offset\n");
> return -EFAULT;
> }
>
> @@ -698,12 +707,12 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
> size_t length;
>
> if (!bridge->master_write) {
> - printk(KERN_WARNING "Writing to resource not supported\n");
> + dev_warn(bridge->parent, "Writing to resource not supported\n");
> return -EINVAL;
> }
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> @@ -712,7 +721,7 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
> length = vme_get_size(resource);
>
> if (offset > length) {
> - printk(KERN_WARNING "Invalid Offset\n");
> + dev_warn(bridge->parent, "Invalid Offset\n");
> return -EFAULT;
> }
>
> @@ -749,12 +758,12 @@ unsigned int vme_master_rmw(struct vme_resource *resource, unsigned int mask,
> struct vme_master_resource *image;
>
> if (!bridge->master_rmw) {
> - printk(KERN_WARNING "Writing to resource not supported\n");
> + dev_warn(bridge->parent, "Writing to resource not supported\n");
> return -EINVAL;
> }
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> @@ -777,12 +786,13 @@ EXPORT_SYMBOL(vme_master_rmw);
> */
> int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_master_resource *image;
> phys_addr_t phys_addr;
> unsigned long vma_size;
>
> if (resource->type != VME_MASTER) {
> - pr_err("Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return -EINVAL;
> }
>
> @@ -791,7 +801,7 @@ int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
> vma_size = vma->vm_end - vma->vm_start;
>
> if (phys_addr + vma_size > image->bus_resource.end + 1) {
> - pr_err("Map size cannot exceed the window size\n");
> + dev_err(bridge->parent, "Map size cannot exceed the window size\n");
> return -EFAULT;
> }
>
> @@ -809,24 +819,25 @@ EXPORT_SYMBOL(vme_master_mmap);
> */
> void vme_master_free(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_master_resource *master_image;
>
> if (resource->type != VME_MASTER) {
> - printk(KERN_ERR "Not a master resource\n");
> + dev_err(bridge->parent, "Not a master resource\n");
> return;
> }
>
> master_image = list_entry(resource->entry, struct vme_master_resource,
> list);
> if (!master_image) {
> - printk(KERN_ERR "Can't find master resource\n");
> + dev_err(bridge->parent, "Can't find master resource\n");
> return;
> }
>
> /* Unlock image */
> spin_lock(&master_image->lock);
> if (master_image->locked == 0)
> - printk(KERN_ERR "Image is already free\n");
> + dev_err(bridge->parent, "Image is already free\n");
>
> master_image->locked = 0;
> spin_unlock(&master_image->lock);
> @@ -854,18 +865,19 @@ struct vme_resource *vme_dma_request(struct vme_dev *vdev, u32 route)
> struct vme_resource *resource = NULL;
>
> /* XXX Not checking resource attributes */
> - printk(KERN_ERR "No VME resource Attribute tests done\n");
> + dev_err(&vdev->dev, "No VME resource Attribute tests done\n");
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> goto err_bus;
> }
>
> /* Loop through DMA resources */
> list_for_each_entry(dma_ctrlr, &bridge->dma_resources, list) {
> if (!dma_ctrlr) {
> - printk(KERN_ERR "Registered NULL DMA resource\n");
> + dev_err(bridge->parent,
> + "Registered NULL DMA resource\n");
> continue;
> }
>
> @@ -917,10 +929,11 @@ EXPORT_SYMBOL(vme_dma_request);
> */
> struct vme_dma_list *vme_new_dma_list(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_dma_list *dma_list;
>
> if (resource->type != VME_DMA) {
> - printk(KERN_ERR "Not a DMA resource\n");
> + dev_err(bridge->parent, "Not a DMA resource\n");
> return NULL;
> }
>
> @@ -1098,12 +1111,13 @@ int vme_dma_list_add(struct vme_dma_list *list, struct vme_dma_attr *src,
> int retval;
>
> if (!bridge->dma_list_add) {
> - printk(KERN_WARNING "Link List DMA generation not supported\n");
> + dev_warn(bridge->parent,
> + "Link List DMA generation not supported\n");
> return -EINVAL;
> }
>
> if (!mutex_trylock(&list->mtx)) {
> - printk(KERN_ERR "Link List already submitted\n");
> + dev_err(bridge->parent, "Link List already submitted\n");
> return -EINVAL;
> }
>
> @@ -1131,7 +1145,8 @@ int vme_dma_list_exec(struct vme_dma_list *list)
> int retval;
>
> if (!bridge->dma_list_exec) {
> - printk(KERN_ERR "Link List DMA execution not supported\n");
> + dev_err(bridge->parent,
> + "Link List DMA execution not supported\n");
> return -EINVAL;
> }
>
> @@ -1160,12 +1175,13 @@ int vme_dma_list_free(struct vme_dma_list *list)
> int retval;
>
> if (!bridge->dma_list_empty) {
> - printk(KERN_WARNING "Emptying of Link Lists not supported\n");
> + dev_warn(bridge->parent,
> + "Emptying of Link Lists not supported\n");
> return -EINVAL;
> }
>
> if (!mutex_trylock(&list->mtx)) {
> - printk(KERN_ERR "Link List in use\n");
> + dev_err(bridge->parent, "Link List in use\n");
> return -EBUSY;
> }
>
> @@ -1175,7 +1191,7 @@ int vme_dma_list_free(struct vme_dma_list *list)
> */
> retval = bridge->dma_list_empty(list);
> if (retval) {
> - printk(KERN_ERR "Unable to empty link-list entries\n");
> + dev_err(bridge->parent, "Unable to empty link-list entries\n");
> mutex_unlock(&list->mtx);
> return retval;
> }
> @@ -1197,22 +1213,24 @@ EXPORT_SYMBOL(vme_dma_list_free);
> */
> int vme_dma_free(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_dma_resource *ctrlr;
>
> if (resource->type != VME_DMA) {
> - printk(KERN_ERR "Not a DMA resource\n");
> + dev_err(bridge->parent, "Not a DMA resource\n");
> return -EINVAL;
> }
>
> ctrlr = list_entry(resource->entry, struct vme_dma_resource, list);
>
> if (!mutex_trylock(&ctrlr->mtx)) {
> - printk(KERN_ERR "Resource busy, can't free\n");
> + dev_err(bridge->parent, "Resource busy, can't free\n");
> return -EBUSY;
> }
>
> if (!(list_empty(&ctrlr->pending) && list_empty(&ctrlr->running))) {
> - printk(KERN_WARNING "Resource still processing transfers\n");
> + dev_warn(bridge->parent,
> + "Resource still processing transfers\n");
> mutex_unlock(&ctrlr->mtx);
> return -EBUSY;
> }
> @@ -1290,8 +1308,9 @@ void vme_irq_handler(struct vme_bridge *bridge, int level, int statid)
> if (call)
> call(level, statid, priv_data);
> else
> - printk(KERN_WARNING "Spurious VME interrupt, level:%x, vector:%x\n",
> - level, statid);
> + dev_warn(bridge->parent,
> + "Spurious VME interrupt, level:%x, vector:%x\n", level,
> + statid);
> }
> EXPORT_SYMBOL(vme_irq_handler);
>
> @@ -1319,17 +1338,18 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> return -EINVAL;
> }
>
> if ((level < 1) || (level > 7)) {
> - printk(KERN_ERR "Invalid interrupt level\n");
> + dev_err(bridge->parent, "Invalid interrupt level\n");
> return -EINVAL;
> }
>
> if (!bridge->irq_set) {
> - printk(KERN_ERR "Configuring interrupts not supported\n");
> + dev_err(bridge->parent,
> + "Configuring interrupts not supported\n");
> return -EINVAL;
> }
>
> @@ -1337,7 +1357,7 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
>
> if (bridge->irq[level - 1].callback[statid].func) {
> mutex_unlock(&bridge->irq_mtx);
> - printk(KERN_WARNING "VME Interrupt already taken\n");
> + dev_warn(bridge->parent, "VME Interrupt already taken\n");
> return -EBUSY;
> }
>
> @@ -1368,17 +1388,18 @@ void vme_irq_free(struct vme_dev *vdev, int level, int statid)
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> return;
> }
>
> if ((level < 1) || (level > 7)) {
> - printk(KERN_ERR "Invalid interrupt level\n");
> + dev_err(bridge->parent, "Invalid interrupt level\n");
> return;
> }
>
> if (!bridge->irq_set) {
> - printk(KERN_ERR "Configuring interrupts not supported\n");
> + dev_err(bridge->parent,
> + "Configuring interrupts not supported\n");
> return;
> }
>
> @@ -1415,17 +1436,18 @@ int vme_irq_generate(struct vme_dev *vdev, int level, int statid)
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> return -EINVAL;
> }
>
> if ((level < 1) || (level > 7)) {
> - printk(KERN_WARNING "Invalid interrupt level\n");
> + dev_warn(bridge->parent, "Invalid interrupt level\n");
> return -EINVAL;
> }
>
> if (!bridge->irq_generate) {
> - printk(KERN_WARNING "Interrupt generation not supported\n");
> + dev_warn(bridge->parent,
> + "Interrupt generation not supported\n");
> return -EINVAL;
> }
>
> @@ -1452,14 +1474,15 @@ struct vme_resource *vme_lm_request(struct vme_dev *vdev)
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> goto err_bus;
> }
>
> /* Loop through LM resources */
> list_for_each_entry(lm, &bridge->lm_resources, list) {
> if (!lm) {
> - printk(KERN_ERR "Registered NULL Location Monitor resource\n");
> + dev_err(bridge->parent,
> + "Registered NULL Location Monitor resource\n");
> continue;
> }
>
> @@ -1511,10 +1534,11 @@ EXPORT_SYMBOL(vme_lm_request);
> */
> int vme_lm_count(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return -EINVAL;
> }
>
> @@ -1545,14 +1569,14 @@ int vme_lm_set(struct vme_resource *resource, unsigned long long lm_base,
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return -EINVAL;
> }
>
> lm = list_entry(resource->entry, struct vme_lm_resource, list);
>
> if (!bridge->lm_set) {
> - printk(KERN_ERR "vme_lm_set not supported\n");
> + dev_err(bridge->parent, "vme_lm_set not supported\n");
> return -EINVAL;
> }
>
> @@ -1581,14 +1605,14 @@ int vme_lm_get(struct vme_resource *resource, unsigned long long *lm_base,
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return -EINVAL;
> }
>
> lm = list_entry(resource->entry, struct vme_lm_resource, list);
>
> if (!bridge->lm_get) {
> - printk(KERN_ERR "vme_lm_get not supported\n");
> + dev_err(bridge->parent, "vme_lm_get not supported\n");
> return -EINVAL;
> }
>
> @@ -1618,14 +1642,14 @@ int vme_lm_attach(struct vme_resource *resource, int monitor,
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return -EINVAL;
> }
>
> lm = list_entry(resource->entry, struct vme_lm_resource, list);
>
> if (!bridge->lm_attach) {
> - printk(KERN_ERR "vme_lm_attach not supported\n");
> + dev_err(bridge->parent, "vme_lm_attach not supported\n");
> return -EINVAL;
> }
>
> @@ -1651,14 +1675,14 @@ int vme_lm_detach(struct vme_resource *resource, int monitor)
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return -EINVAL;
> }
>
> lm = list_entry(resource->entry, struct vme_lm_resource, list);
>
> if (!bridge->lm_detach) {
> - printk(KERN_ERR "vme_lm_detach not supported\n");
> + dev_err(bridge->parent, "vme_lm_detach not supported\n");
> return -EINVAL;
> }
>
> @@ -1680,10 +1704,11 @@ EXPORT_SYMBOL(vme_lm_detach);
> */
> void vme_lm_free(struct vme_resource *resource)
> {
> + struct vme_bridge *bridge = find_bridge(resource);
> struct vme_lm_resource *lm;
>
> if (resource->type != VME_LM) {
> - printk(KERN_ERR "Not a Location Monitor resource\n");
> + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> return;
> }
>
> @@ -1720,12 +1745,12 @@ int vme_slot_num(struct vme_dev *vdev)
>
> bridge = vdev->bridge;
> if (!bridge) {
> - printk(KERN_ERR "Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> return -EINVAL;
> }
>
> if (!bridge->slot_get) {
> - printk(KERN_WARNING "vme_slot_num not supported\n");
> + dev_warn(bridge->parent, "vme_slot_num not supported\n");
> return -EINVAL;
> }
>
> @@ -1748,7 +1773,7 @@ int vme_bus_num(struct vme_dev *vdev)
>
> bridge = vdev->bridge;
> if (!bridge) {
> - pr_err("Can't find VME bus\n");
> + dev_err(&vdev->dev, "Can't find VME bus\n");
> return -EINVAL;
> }
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*()
2023-10-19 10:45 ` Karolina Stolarek
@ 2023-10-19 15:31 ` Greg Kroah-Hartman
2023-10-20 2:06 ` Soumya Negi
1 sibling, 0 replies; 11+ messages in thread
From: Greg Kroah-Hartman @ 2023-10-19 15:31 UTC (permalink / raw)
To: Karolina Stolarek
Cc: Soumya Negi, Martyn Welch, Manohar Vanga, Andi Shyti,
Julia Lawall, outreachy, linux-kernel, linux-staging
On Thu, Oct 19, 2023 at 12:45:30PM +0200, Karolina Stolarek wrote:
> On 19.10.2023 09:20, Soumya Negi wrote:
> > vme.c uses printk() to log messages. To improve and standardize message
> > formatting, use logging mechanisms pr_err()/pr_warn() and
> > dev_err()/dev_warn() instead. Retain the printk log levels of the
> > messages during replacement.
> >
> > Issue found by checkpatch.pl
> >
> > Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
> > ---
> > Changes in v2:
> > * Change the pr_*() calls used to replace printk's in v1 to dev_*()
> > wherever possible, as it adds more context to the message.
> > (as per feedback from julia.lawall@inria.fr &
> > gregkh@linuxfoundation.org)
>
> You don't have to include the change log here, as it's already in the cover
> letter. This space is usually used when a patch is not sent as a part of the
> series.
Either or both is fine, this is not a real issue at all, thanks
greg k-h
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name
2023-10-19 7:20 ` [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name Soumya Negi
@ 2023-10-19 15:41 ` Andi Shyti
2023-10-19 19:14 ` Soumya Negi
0 siblings, 1 reply; 11+ messages in thread
From: Andi Shyti @ 2023-10-19 15:41 UTC (permalink / raw)
To: Soumya Negi
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall, outreachy, linux-kernel, linux-staging
Hi Soumya,
On Thu, Oct 19, 2023 at 12:20:10AM -0700, Soumya Negi wrote:
> Replace function names in message strings with __func__ to fix
> all checkpatch warnings like:
>
> WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
> this function's name, in a string
>
> Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
you forgot my ack here:
Acked-by: Andi Shyti <andi.shyti@linux.intel.com>
Andi
> ---
> Changes in v2:
> * None
>
> drivers/staging/vme_user/vme.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
> index 640b2dda3ac6..e533cce8e54e 100644
> --- a/drivers/staging/vme_user/vme.c
> +++ b/drivers/staging/vme_user/vme.c
> @@ -424,7 +424,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) {
> - dev_err(bridge->parent, "vme_slave_get not supported\n");
> + dev_err(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -576,7 +576,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) {
> - dev_warn(bridge->parent, "vme_master_set not supported\n");
> + dev_warn(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -1576,7 +1576,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) {
> - dev_err(bridge->parent, "vme_lm_set not supported\n");
> + dev_err(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -1612,7 +1612,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) {
> - dev_err(bridge->parent, "vme_lm_get not supported\n");
> + dev_err(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -1649,7 +1649,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) {
> - dev_err(bridge->parent, "vme_lm_attach not supported\n");
> + dev_err(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -1682,7 +1682,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) {
> - dev_err(bridge->parent, "vme_lm_detach not supported\n");
> + dev_err(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> @@ -1750,7 +1750,7 @@ int vme_slot_num(struct vme_dev *vdev)
> }
>
> if (!bridge->slot_get) {
> - dev_warn(bridge->parent, "vme_slot_num not supported\n");
> + dev_warn(bridge->parent, "%s not supported\n", __func__);
> return -EINVAL;
> }
>
> --
> 2.42.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*()
2023-10-19 7:20 ` [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*() Soumya Negi
2023-10-19 10:45 ` Karolina Stolarek
@ 2023-10-19 15:42 ` Andi Shyti
1 sibling, 0 replies; 11+ messages in thread
From: Andi Shyti @ 2023-10-19 15:42 UTC (permalink / raw)
To: Soumya Negi
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall, outreachy, linux-kernel, linux-staging
Hi,
On Thu, Oct 19, 2023 at 12:20:09AM -0700, Soumya Negi wrote:
> vme.c uses printk() to log messages. To improve and standardize message
> formatting, use logging mechanisms pr_err()/pr_warn() and
> dev_err()/dev_warn() instead. Retain the printk log levels of the
> messages during replacement.
>
> Issue found by checkpatch.pl
>
> Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
Acked-by: Andi Shyti <andi.shyti@linux.intel.com>
Andi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name
2023-10-19 15:41 ` Andi Shyti
@ 2023-10-19 19:14 ` Soumya Negi
2023-10-19 21:32 ` Andi Shyti
0 siblings, 1 reply; 11+ messages in thread
From: Soumya Negi @ 2023-10-19 19:14 UTC (permalink / raw)
To: Andi Shyti
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Julia Lawall,
outreachy, linux-kernel, linux-staging
On Thu, Oct 19, 2023 at 05:41:12PM +0200, Andi Shyti wrote:
> Hi Soumya,
>
> On Thu, Oct 19, 2023 at 12:20:10AM -0700, Soumya Negi wrote:
> > Replace function names in message strings with __func__ to fix
> > all checkpatch warnings like:
> >
> > WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
> > this function's name, in a string
> >
> > Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
>
> you forgot my ack here:
>
> Acked-by: Andi Shyti <andi.shyti@linux.intel.com>
>
> Andi
Sorry I forgot the tag Andi. I'll add the tags in v3(Greg has suggested
more changes). There will be some new patches, so I'll leave the tags out in
those as you may want to review them first.
Regards,
Soumya
>
> > ---
> > Changes in v2:
> > * None
> >
> > drivers/staging/vme_user/vme.c | 14 +++++++-------
> > 1 file changed, 7 insertions(+), 7 deletions(-)
> >
> > diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
> > index 640b2dda3ac6..e533cce8e54e 100644
> > --- a/drivers/staging/vme_user/vme.c
> > +++ b/drivers/staging/vme_user/vme.c
> > @@ -424,7 +424,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) {
> > - dev_err(bridge->parent, "vme_slave_get not supported\n");
> > + dev_err(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -576,7 +576,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) {
> > - dev_warn(bridge->parent, "vme_master_set not supported\n");
> > + dev_warn(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -1576,7 +1576,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) {
> > - dev_err(bridge->parent, "vme_lm_set not supported\n");
> > + dev_err(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -1612,7 +1612,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) {
> > - dev_err(bridge->parent, "vme_lm_get not supported\n");
> > + dev_err(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -1649,7 +1649,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) {
> > - dev_err(bridge->parent, "vme_lm_attach not supported\n");
> > + dev_err(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -1682,7 +1682,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) {
> > - dev_err(bridge->parent, "vme_lm_detach not supported\n");
> > + dev_err(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > @@ -1750,7 +1750,7 @@ int vme_slot_num(struct vme_dev *vdev)
> > }
> >
> > if (!bridge->slot_get) {
> > - dev_warn(bridge->parent, "vme_slot_num not supported\n");
> > + dev_warn(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> >
> > --
> > 2.42.0
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name
2023-10-19 19:14 ` Soumya Negi
@ 2023-10-19 21:32 ` Andi Shyti
2023-10-19 22:13 ` Soumya Negi
0 siblings, 1 reply; 11+ messages in thread
From: Andi Shyti @ 2023-10-19 21:32 UTC (permalink / raw)
To: Soumya Negi
Cc: Andi Shyti, Martyn Welch, Manohar Vanga, Greg Kroah-Hartman,
Julia Lawall, outreachy, linux-kernel, linux-staging
Hi Soumya,
> > On Thu, Oct 19, 2023 at 12:20:10AM -0700, Soumya Negi wrote:
> > > Replace function names in message strings with __func__ to fix
> > > all checkpatch warnings like:
> > >
> > > WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
> > > this function's name, in a string
> > >
> > > Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
> >
> > you forgot my ack here:
> >
> > Acked-by: Andi Shyti <andi.shyti@linux.intel.com>
> >
> > Andi
>
> Sorry I forgot the tag Andi. I'll add the tags in v3(Greg has suggested
> more changes). There will be some new patches, so I'll leave the tags out in
> those as you may want to review them first.
no problem.
I don't see reviews from Greg in this v2 series. If you are
referring to the changelog, then you don't need to resend.
Andi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name
2023-10-19 21:32 ` Andi Shyti
@ 2023-10-19 22:13 ` Soumya Negi
0 siblings, 0 replies; 11+ messages in thread
From: Soumya Negi @ 2023-10-19 22:13 UTC (permalink / raw)
To: Andi Shyti
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Julia Lawall,
outreachy, linux-kernel, linux-staging
On Thu, Oct 19, 2023 at 11:32:04PM +0200, Andi Shyti wrote:
> Hi Soumya,
>
> > > On Thu, Oct 19, 2023 at 12:20:10AM -0700, Soumya Negi wrote:
> > > > Replace function names in message strings with __func__ to fix
> > > > all checkpatch warnings like:
> > > >
> > > > WARNING: Prefer using '"%s...", __func__' to using 'vme_lm_get',
> > > > this function's name, in a string
> > > >
> > > > Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
> > >
> > > you forgot my ack here:
> > >
> > > Acked-by: Andi Shyti <andi.shyti@linux.intel.com>
> > >
> > > Andi
> >
> > Sorry I forgot the tag Andi. I'll add the tags in v3(Greg has suggested
> > more changes). There will be some new patches, so I'll leave the tags out in
> > those as you may want to review them first.
>
> no problem.
>
> I don't see reviews from Greg in this v2 series. If you are
> referring to the changelog, then you don't need to resend.
>
> Andi
Hi Andi,
I had sent in v2 after suggestions from Julia and Greg. Got some more
feedback from Greg in the v1 email thread though. Will have to send a v3 due
to that.
Regards,
Soumya
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*()
2023-10-19 10:45 ` Karolina Stolarek
2023-10-19 15:31 ` Greg Kroah-Hartman
@ 2023-10-20 2:06 ` Soumya Negi
1 sibling, 0 replies; 11+ messages in thread
From: Soumya Negi @ 2023-10-20 2:06 UTC (permalink / raw)
To: Karolina Stolarek
Cc: Martyn Welch, Manohar Vanga, Greg Kroah-Hartman, Andi Shyti,
Julia Lawall, outreachy, linux-kernel, linux-staging
On Thu, Oct 19, 2023 at 12:45:30PM +0200, Karolina Stolarek wrote:
> On 19.10.2023 09:20, Soumya Negi wrote:
> > vme.c uses printk() to log messages. To improve and standardize message
> > formatting, use logging mechanisms pr_err()/pr_warn() and
> > dev_err()/dev_warn() instead. Retain the printk log levels of the
> > messages during replacement.
> >
> > Issue found by checkpatch.pl
> >
> > Signed-off-by: Soumya Negi <soumya.negi97@gmail.com>
> > ---
> > Changes in v2:
> > * Change the pr_*() calls used to replace printk's in v1 to dev_*()
> > wherever possible, as it adds more context to the message.
> > (as per feedback from julia.lawall@inria.fr &
> > gregkh@linuxfoundation.org)
>
> You don't have to include the change log here, as it's already in the cover
> letter. This space is usually used when a patch is not sent as a part of the
> series.
>
> All the best,
> Karolina
Hi Karolina,
Thanks for letting me know. In v3, I have kept the changelog changes
just in the cover letter. Definitely made things easier for me.
Regards,
Soumya
>
> >
> > drivers/staging/vme_user/vme.c | 193 +++++++++++++++++++--------------
> > 1 file changed, 109 insertions(+), 84 deletions(-)
> >
> > diff --git a/drivers/staging/vme_user/vme.c b/drivers/staging/vme_user/vme.c
> > index 6519a7c994a0..640b2dda3ac6 100644
> > --- a/drivers/staging/vme_user/vme.c
> > +++ b/drivers/staging/vme_user/vme.c
> > @@ -9,6 +9,8 @@
> > * Copyright 2004 Motorola Inc.
> > */
> > +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> > +
> > #include <linux/init.h>
> > #include <linux/export.h>
> > #include <linux/mm.h>
> > @@ -62,7 +64,7 @@ static struct vme_bridge *find_bridge(struct vme_resource *resource)
> > return list_entry(resource->entry, struct vme_lm_resource,
> > list)->parent;
> > default:
> > - printk(KERN_ERR "Unknown resource type\n");
> > + pr_err("Unknown resource type\n");
> > return NULL;
> > }
> > }
> > @@ -84,24 +86,25 @@ void *vme_alloc_consistent(struct vme_resource *resource, size_t size,
> > struct vme_bridge *bridge;
> > if (!resource) {
> > - printk(KERN_ERR "No resource\n");
> > + pr_err("No resource\n");
> > return NULL;
> > }
> > bridge = find_bridge(resource);
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find bridge\n");
> > + pr_err("Can't find bridge\n");
> > return NULL;
> > }
> > if (!bridge->parent) {
> > - printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
> > + pr_err("Dev entry NULL for bridge %s\n", bridge->name);
> > return NULL;
> > }
> > if (!bridge->alloc_consistent) {
> > - printk(KERN_ERR "alloc_consistent not supported by bridge %s\n",
> > - bridge->name);
> > + dev_err(bridge->parent,
> > + "alloc_consistent not supported by bridge %s\n",
> > + bridge->name);
> > return NULL;
> > }
> > @@ -124,24 +127,25 @@ void vme_free_consistent(struct vme_resource *resource, size_t size,
> > struct vme_bridge *bridge;
> > if (!resource) {
> > - printk(KERN_ERR "No resource\n");
> > + pr_err("No resource\n");
> > return;
> > }
> > bridge = find_bridge(resource);
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find bridge\n");
> > + pr_err("Can't find bridge\n");
> > return;
> > }
> > if (!bridge->parent) {
> > - printk(KERN_ERR "Dev entry NULL for bridge %s\n", bridge->name);
> > + pr_err("Dev entry NULL for bridge %s\n", bridge->name);
> > return;
> > }
> > if (!bridge->free_consistent) {
> > - printk(KERN_ERR "free_consistent not supported by bridge %s\n",
> > - bridge->name);
> > + dev_err(bridge->parent,
> > + "free_consistent not supported by bridge %s\n",
> > + bridge->name);
> > return;
> > }
> > @@ -161,6 +165,7 @@ EXPORT_SYMBOL(vme_free_consistent);
> > */
> > size_t vme_get_size(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > int enabled, retval;
> > unsigned long long base, size;
> > dma_addr_t buf_base;
> > @@ -184,7 +189,7 @@ size_t vme_get_size(struct vme_resource *resource)
> > case VME_DMA:
> > return 0;
> > default:
> > - printk(KERN_ERR "Unknown resource type\n");
> > + dev_err(bridge->parent, "Unknown resource type\n");
> > return 0;
> > }
> > }
> > @@ -225,7 +230,7 @@ int vme_check_window(u32 aspace, unsigned long long vme_base,
> > /* User Defined */
> > break;
> > default:
> > - printk(KERN_ERR "Invalid address space\n");
> > + pr_err("Invalid address space\n");
> > retval = -EINVAL;
> > break;
> > }
> > @@ -288,14 +293,15 @@ struct vme_resource *vme_slave_request(struct vme_dev *vdev, u32 address,
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > goto err_bus;
> > }
> > /* Loop through slave resources */
> > list_for_each_entry(slave_image, &bridge->slave_resources, list) {
> > if (!slave_image) {
> > - printk(KERN_ERR "Registered NULL Slave resource\n");
> > + dev_err(bridge->parent,
> > + "Registered NULL Slave resource\n");
> > continue;
> > }
> > @@ -362,20 +368,20 @@ int vme_slave_set(struct vme_resource *resource, int enabled,
> > int retval;
> > if (resource->type != VME_SLAVE) {
> > - printk(KERN_ERR "Not a slave resource\n");
> > + dev_err(bridge->parent, "Not a slave resource\n");
> > return -EINVAL;
> > }
> > image = list_entry(resource->entry, struct vme_slave_resource, list);
> > if (!bridge->slave_set) {
> > - printk(KERN_ERR "Function not supported\n");
> > + dev_err(bridge->parent, "Function not supported\n");
> > return -ENOSYS;
> > }
> > if (!(((image->address_attr & aspace) == aspace) &&
> > ((image->cycle_attr & cycle) == cycle))) {
> > - printk(KERN_ERR "Invalid attributes\n");
> > + dev_err(bridge->parent, "Invalid attributes\n");
> > return -EINVAL;
> > }
> > @@ -411,14 +417,14 @@ int vme_slave_get(struct vme_resource *resource, int *enabled,
> > struct vme_slave_resource *image;
> > if (resource->type != VME_SLAVE) {
> > - printk(KERN_ERR "Not a slave resource\n");
> > + dev_err(bridge->parent, "Not a slave resource\n");
> > return -EINVAL;
> > }
> > image = list_entry(resource->entry, struct vme_slave_resource, list);
> > if (!bridge->slave_get) {
> > - printk(KERN_ERR "vme_slave_get not supported\n");
> > + dev_err(bridge->parent, "vme_slave_get not supported\n");
> > return -EINVAL;
> > }
> > @@ -435,24 +441,25 @@ EXPORT_SYMBOL(vme_slave_get);
> > */
> > void vme_slave_free(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_slave_resource *slave_image;
> > if (resource->type != VME_SLAVE) {
> > - printk(KERN_ERR "Not a slave resource\n");
> > + dev_err(bridge->parent, "Not a slave resource\n");
> > return;
> > }
> > slave_image = list_entry(resource->entry, struct vme_slave_resource,
> > list);
> > if (!slave_image) {
> > - printk(KERN_ERR "Can't find slave resource\n");
> > + dev_err(bridge->parent, "Can't find slave resource\n");
> > return;
> > }
> > /* Unlock image */
> > mutex_lock(&slave_image->mtx);
> > if (slave_image->locked == 0)
> > - printk(KERN_ERR "Image is already free\n");
> > + dev_err(bridge->parent, "Image is already free\n");
> > slave_image->locked = 0;
> > mutex_unlock(&slave_image->mtx);
> > @@ -484,14 +491,15 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > goto err_bus;
> > }
> > /* Loop through master resources */
> > list_for_each_entry(master_image, &bridge->master_resources, list) {
> > if (!master_image) {
> > - printk(KERN_WARNING "Registered NULL master resource\n");
> > + dev_warn(bridge->parent,
> > + "Registered NULL master resource\n");
> > continue;
> > }
> > @@ -511,7 +519,7 @@ struct vme_resource *vme_master_request(struct vme_dev *vdev, u32 address,
> > /* Check to see if we found a resource */
> > if (!allocated_image) {
> > - printk(KERN_ERR "Can't find a suitable resource\n");
> > + dev_err(&vdev->dev, "Can't find a suitable resource\n");
> > goto err_image;
> > }
> > @@ -561,21 +569,21 @@ int vme_master_set(struct vme_resource *resource, int enabled,
> > int retval;
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > image = list_entry(resource->entry, struct vme_master_resource, list);
> > if (!bridge->master_set) {
> > - printk(KERN_WARNING "vme_master_set not supported\n");
> > + dev_warn(bridge->parent, "vme_master_set not supported\n");
> > return -EINVAL;
> > }
> > if (!(((image->address_attr & aspace) == aspace) &&
> > ((image->cycle_attr & cycle) == cycle) &&
> > ((image->width_attr & dwidth) == dwidth))) {
> > - printk(KERN_WARNING "Invalid attributes\n");
> > + dev_warn(bridge->parent, "Invalid attributes\n");
> > return -EINVAL;
> > }
> > @@ -611,14 +619,14 @@ int vme_master_get(struct vme_resource *resource, int *enabled,
> > struct vme_master_resource *image;
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > image = list_entry(resource->entry, struct vme_master_resource, list);
> > if (!bridge->master_get) {
> > - printk(KERN_WARNING "%s not supported\n", __func__);
> > + dev_warn(bridge->parent, "%s not supported\n", __func__);
> > return -EINVAL;
> > }
> > @@ -650,12 +658,13 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
> > size_t length;
> > if (!bridge->master_read) {
> > - printk(KERN_WARNING "Reading from resource not supported\n");
> > + dev_warn(bridge->parent,
> > + "Reading from resource not supported\n");
> > return -EINVAL;
> > }
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > @@ -664,7 +673,7 @@ ssize_t vme_master_read(struct vme_resource *resource, void *buf, size_t count,
> > length = vme_get_size(resource);
> > if (offset > length) {
> > - printk(KERN_WARNING "Invalid Offset\n");
> > + dev_warn(bridge->parent, "Invalid Offset\n");
> > return -EFAULT;
> > }
> > @@ -698,12 +707,12 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
> > size_t length;
> > if (!bridge->master_write) {
> > - printk(KERN_WARNING "Writing to resource not supported\n");
> > + dev_warn(bridge->parent, "Writing to resource not supported\n");
> > return -EINVAL;
> > }
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > @@ -712,7 +721,7 @@ ssize_t vme_master_write(struct vme_resource *resource, void *buf,
> > length = vme_get_size(resource);
> > if (offset > length) {
> > - printk(KERN_WARNING "Invalid Offset\n");
> > + dev_warn(bridge->parent, "Invalid Offset\n");
> > return -EFAULT;
> > }
> > @@ -749,12 +758,12 @@ unsigned int vme_master_rmw(struct vme_resource *resource, unsigned int mask,
> > struct vme_master_resource *image;
> > if (!bridge->master_rmw) {
> > - printk(KERN_WARNING "Writing to resource not supported\n");
> > + dev_warn(bridge->parent, "Writing to resource not supported\n");
> > return -EINVAL;
> > }
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > @@ -777,12 +786,13 @@ EXPORT_SYMBOL(vme_master_rmw);
> > */
> > int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_master_resource *image;
> > phys_addr_t phys_addr;
> > unsigned long vma_size;
> > if (resource->type != VME_MASTER) {
> > - pr_err("Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return -EINVAL;
> > }
> > @@ -791,7 +801,7 @@ int vme_master_mmap(struct vme_resource *resource, struct vm_area_struct *vma)
> > vma_size = vma->vm_end - vma->vm_start;
> > if (phys_addr + vma_size > image->bus_resource.end + 1) {
> > - pr_err("Map size cannot exceed the window size\n");
> > + dev_err(bridge->parent, "Map size cannot exceed the window size\n");
> > return -EFAULT;
> > }
> > @@ -809,24 +819,25 @@ EXPORT_SYMBOL(vme_master_mmap);
> > */
> > void vme_master_free(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_master_resource *master_image;
> > if (resource->type != VME_MASTER) {
> > - printk(KERN_ERR "Not a master resource\n");
> > + dev_err(bridge->parent, "Not a master resource\n");
> > return;
> > }
> > master_image = list_entry(resource->entry, struct vme_master_resource,
> > list);
> > if (!master_image) {
> > - printk(KERN_ERR "Can't find master resource\n");
> > + dev_err(bridge->parent, "Can't find master resource\n");
> > return;
> > }
> > /* Unlock image */
> > spin_lock(&master_image->lock);
> > if (master_image->locked == 0)
> > - printk(KERN_ERR "Image is already free\n");
> > + dev_err(bridge->parent, "Image is already free\n");
> > master_image->locked = 0;
> > spin_unlock(&master_image->lock);
> > @@ -854,18 +865,19 @@ struct vme_resource *vme_dma_request(struct vme_dev *vdev, u32 route)
> > struct vme_resource *resource = NULL;
> > /* XXX Not checking resource attributes */
> > - printk(KERN_ERR "No VME resource Attribute tests done\n");
> > + dev_err(&vdev->dev, "No VME resource Attribute tests done\n");
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > goto err_bus;
> > }
> > /* Loop through DMA resources */
> > list_for_each_entry(dma_ctrlr, &bridge->dma_resources, list) {
> > if (!dma_ctrlr) {
> > - printk(KERN_ERR "Registered NULL DMA resource\n");
> > + dev_err(bridge->parent,
> > + "Registered NULL DMA resource\n");
> > continue;
> > }
> > @@ -917,10 +929,11 @@ EXPORT_SYMBOL(vme_dma_request);
> > */
> > struct vme_dma_list *vme_new_dma_list(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_dma_list *dma_list;
> > if (resource->type != VME_DMA) {
> > - printk(KERN_ERR "Not a DMA resource\n");
> > + dev_err(bridge->parent, "Not a DMA resource\n");
> > return NULL;
> > }
> > @@ -1098,12 +1111,13 @@ int vme_dma_list_add(struct vme_dma_list *list, struct vme_dma_attr *src,
> > int retval;
> > if (!bridge->dma_list_add) {
> > - printk(KERN_WARNING "Link List DMA generation not supported\n");
> > + dev_warn(bridge->parent,
> > + "Link List DMA generation not supported\n");
> > return -EINVAL;
> > }
> > if (!mutex_trylock(&list->mtx)) {
> > - printk(KERN_ERR "Link List already submitted\n");
> > + dev_err(bridge->parent, "Link List already submitted\n");
> > return -EINVAL;
> > }
> > @@ -1131,7 +1145,8 @@ int vme_dma_list_exec(struct vme_dma_list *list)
> > int retval;
> > if (!bridge->dma_list_exec) {
> > - printk(KERN_ERR "Link List DMA execution not supported\n");
> > + dev_err(bridge->parent,
> > + "Link List DMA execution not supported\n");
> > return -EINVAL;
> > }
> > @@ -1160,12 +1175,13 @@ int vme_dma_list_free(struct vme_dma_list *list)
> > int retval;
> > if (!bridge->dma_list_empty) {
> > - printk(KERN_WARNING "Emptying of Link Lists not supported\n");
> > + dev_warn(bridge->parent,
> > + "Emptying of Link Lists not supported\n");
> > return -EINVAL;
> > }
> > if (!mutex_trylock(&list->mtx)) {
> > - printk(KERN_ERR "Link List in use\n");
> > + dev_err(bridge->parent, "Link List in use\n");
> > return -EBUSY;
> > }
> > @@ -1175,7 +1191,7 @@ int vme_dma_list_free(struct vme_dma_list *list)
> > */
> > retval = bridge->dma_list_empty(list);
> > if (retval) {
> > - printk(KERN_ERR "Unable to empty link-list entries\n");
> > + dev_err(bridge->parent, "Unable to empty link-list entries\n");
> > mutex_unlock(&list->mtx);
> > return retval;
> > }
> > @@ -1197,22 +1213,24 @@ EXPORT_SYMBOL(vme_dma_list_free);
> > */
> > int vme_dma_free(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_dma_resource *ctrlr;
> > if (resource->type != VME_DMA) {
> > - printk(KERN_ERR "Not a DMA resource\n");
> > + dev_err(bridge->parent, "Not a DMA resource\n");
> > return -EINVAL;
> > }
> > ctrlr = list_entry(resource->entry, struct vme_dma_resource, list);
> > if (!mutex_trylock(&ctrlr->mtx)) {
> > - printk(KERN_ERR "Resource busy, can't free\n");
> > + dev_err(bridge->parent, "Resource busy, can't free\n");
> > return -EBUSY;
> > }
> > if (!(list_empty(&ctrlr->pending) && list_empty(&ctrlr->running))) {
> > - printk(KERN_WARNING "Resource still processing transfers\n");
> > + dev_warn(bridge->parent,
> > + "Resource still processing transfers\n");
> > mutex_unlock(&ctrlr->mtx);
> > return -EBUSY;
> > }
> > @@ -1290,8 +1308,9 @@ void vme_irq_handler(struct vme_bridge *bridge, int level, int statid)
> > if (call)
> > call(level, statid, priv_data);
> > else
> > - printk(KERN_WARNING "Spurious VME interrupt, level:%x, vector:%x\n",
> > - level, statid);
> > + dev_warn(bridge->parent,
> > + "Spurious VME interrupt, level:%x, vector:%x\n", level,
> > + statid);
> > }
> > EXPORT_SYMBOL(vme_irq_handler);
> > @@ -1319,17 +1338,18 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > return -EINVAL;
> > }
> > if ((level < 1) || (level > 7)) {
> > - printk(KERN_ERR "Invalid interrupt level\n");
> > + dev_err(bridge->parent, "Invalid interrupt level\n");
> > return -EINVAL;
> > }
> > if (!bridge->irq_set) {
> > - printk(KERN_ERR "Configuring interrupts not supported\n");
> > + dev_err(bridge->parent,
> > + "Configuring interrupts not supported\n");
> > return -EINVAL;
> > }
> > @@ -1337,7 +1357,7 @@ int vme_irq_request(struct vme_dev *vdev, int level, int statid,
> > if (bridge->irq[level - 1].callback[statid].func) {
> > mutex_unlock(&bridge->irq_mtx);
> > - printk(KERN_WARNING "VME Interrupt already taken\n");
> > + dev_warn(bridge->parent, "VME Interrupt already taken\n");
> > return -EBUSY;
> > }
> > @@ -1368,17 +1388,18 @@ void vme_irq_free(struct vme_dev *vdev, int level, int statid)
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > return;
> > }
> > if ((level < 1) || (level > 7)) {
> > - printk(KERN_ERR "Invalid interrupt level\n");
> > + dev_err(bridge->parent, "Invalid interrupt level\n");
> > return;
> > }
> > if (!bridge->irq_set) {
> > - printk(KERN_ERR "Configuring interrupts not supported\n");
> > + dev_err(bridge->parent,
> > + "Configuring interrupts not supported\n");
> > return;
> > }
> > @@ -1415,17 +1436,18 @@ int vme_irq_generate(struct vme_dev *vdev, int level, int statid)
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > return -EINVAL;
> > }
> > if ((level < 1) || (level > 7)) {
> > - printk(KERN_WARNING "Invalid interrupt level\n");
> > + dev_warn(bridge->parent, "Invalid interrupt level\n");
> > return -EINVAL;
> > }
> > if (!bridge->irq_generate) {
> > - printk(KERN_WARNING "Interrupt generation not supported\n");
> > + dev_warn(bridge->parent,
> > + "Interrupt generation not supported\n");
> > return -EINVAL;
> > }
> > @@ -1452,14 +1474,15 @@ struct vme_resource *vme_lm_request(struct vme_dev *vdev)
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > goto err_bus;
> > }
> > /* Loop through LM resources */
> > list_for_each_entry(lm, &bridge->lm_resources, list) {
> > if (!lm) {
> > - printk(KERN_ERR "Registered NULL Location Monitor resource\n");
> > + dev_err(bridge->parent,
> > + "Registered NULL Location Monitor resource\n");
> > continue;
> > }
> > @@ -1511,10 +1534,11 @@ EXPORT_SYMBOL(vme_lm_request);
> > */
> > int vme_lm_count(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return -EINVAL;
> > }
> > @@ -1545,14 +1569,14 @@ int vme_lm_set(struct vme_resource *resource, unsigned long long lm_base,
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return -EINVAL;
> > }
> > lm = list_entry(resource->entry, struct vme_lm_resource, list);
> > if (!bridge->lm_set) {
> > - printk(KERN_ERR "vme_lm_set not supported\n");
> > + dev_err(bridge->parent, "vme_lm_set not supported\n");
> > return -EINVAL;
> > }
> > @@ -1581,14 +1605,14 @@ int vme_lm_get(struct vme_resource *resource, unsigned long long *lm_base,
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return -EINVAL;
> > }
> > lm = list_entry(resource->entry, struct vme_lm_resource, list);
> > if (!bridge->lm_get) {
> > - printk(KERN_ERR "vme_lm_get not supported\n");
> > + dev_err(bridge->parent, "vme_lm_get not supported\n");
> > return -EINVAL;
> > }
> > @@ -1618,14 +1642,14 @@ int vme_lm_attach(struct vme_resource *resource, int monitor,
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return -EINVAL;
> > }
> > lm = list_entry(resource->entry, struct vme_lm_resource, list);
> > if (!bridge->lm_attach) {
> > - printk(KERN_ERR "vme_lm_attach not supported\n");
> > + dev_err(bridge->parent, "vme_lm_attach not supported\n");
> > return -EINVAL;
> > }
> > @@ -1651,14 +1675,14 @@ int vme_lm_detach(struct vme_resource *resource, int monitor)
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return -EINVAL;
> > }
> > lm = list_entry(resource->entry, struct vme_lm_resource, list);
> > if (!bridge->lm_detach) {
> > - printk(KERN_ERR "vme_lm_detach not supported\n");
> > + dev_err(bridge->parent, "vme_lm_detach not supported\n");
> > return -EINVAL;
> > }
> > @@ -1680,10 +1704,11 @@ EXPORT_SYMBOL(vme_lm_detach);
> > */
> > void vme_lm_free(struct vme_resource *resource)
> > {
> > + struct vme_bridge *bridge = find_bridge(resource);
> > struct vme_lm_resource *lm;
> > if (resource->type != VME_LM) {
> > - printk(KERN_ERR "Not a Location Monitor resource\n");
> > + dev_err(bridge->parent, "Not a Location Monitor resource\n");
> > return;
> > }
> > @@ -1720,12 +1745,12 @@ int vme_slot_num(struct vme_dev *vdev)
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - printk(KERN_ERR "Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > return -EINVAL;
> > }
> > if (!bridge->slot_get) {
> > - printk(KERN_WARNING "vme_slot_num not supported\n");
> > + dev_warn(bridge->parent, "vme_slot_num not supported\n");
> > return -EINVAL;
> > }
> > @@ -1748,7 +1773,7 @@ int vme_bus_num(struct vme_dev *vdev)
> > bridge = vdev->bridge;
> > if (!bridge) {
> > - pr_err("Can't find VME bus\n");
> > + dev_err(&vdev->dev, "Can't find VME bus\n");
> > return -EINVAL;
> > }
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2023-10-20 2:06 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-19 7:20 [PATCH v2 0/2] staging: vme_user: Replace printk's & clean up log messages Soumya Negi
2023-10-19 7:20 ` [PATCH v2 1/2] staging: vme_user: Replace printk() with pr_*(),dev_*() Soumya Negi
2023-10-19 10:45 ` Karolina Stolarek
2023-10-19 15:31 ` Greg Kroah-Hartman
2023-10-20 2:06 ` Soumya Negi
2023-10-19 15:42 ` Andi Shyti
2023-10-19 7:20 ` [PATCH v2 2/2] staging: vme_user: Use __func__ instead of function name Soumya Negi
2023-10-19 15:41 ` Andi Shyti
2023-10-19 19:14 ` Soumya Negi
2023-10-19 21:32 ` Andi Shyti
2023-10-19 22:13 ` Soumya Negi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).