# 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 
- List: linux-doc
- Page: https://ratatoskr.run/linux-doc/2018/08/5355029
- Thread: https://ratatoskr.run/linux-doc/2018/08/5354984/t.md

```text


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!

>   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--;
>
```