Thread (89 messages) 89 messages, 11 authors, 2018-10-04

Re: [PATCH v2 05/29] nvmem: remove the name field from struct nvmem_device

From: Srinivas Kandagatla <hidden>
Date: 2018-08-10 08:33:46
Also in: linux-arm-kernel, linux-doc, linux-i2c, linux-omap, lkml


On 10/08/18 09:05, Bartosz Golaszewski wrote:
From: Bartosz Golaszewski <redacted>

This field is never set and is only used in a single error message.
Remove the field and use nvmem_dev_name() instead.

Signed-off-by: Bartosz Golaszewski <redacted>
---
Looks good to me!
quoted hunk ↗ jump to hunk
  drivers/nvmem/core.c | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c
index 31df2e6d6f72..ab3ced2d9a84 100644
--- a/drivers/nvmem/core.c
+++ b/drivers/nvmem/core.c
@@ -26,7 +26,6 @@
  #include <linux/slab.h>
  
  struct nvmem_device {
-	const char		*name;
  	struct module		*owner;
  	struct device		dev;
  	int			stride;
@@ -712,7 +711,7 @@ static struct nvmem_device *__nvmem_device_get(struct device_node *np,
  	if (!try_module_get(nvmem->owner)) {
  		dev_err(&nvmem->dev,
  			"could not increase module refcount for cell %s\n",
-			nvmem->name);
+			nvmem_dev_name(nvmem));
  
  		mutex_lock(&nvmem_mutex);
  		nvmem->users--;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help