[PATCH] nvmem: core: switch to use device_add_groups()

Greg Kroah-Hartman posted 1 patch 1 year, 10 months ago
drivers/nvmem/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] nvmem: core: switch to use device_add_groups()
Posted by Greg Kroah-Hartman 1 year, 10 months ago
devm_device_add_groups() is being removed from the kernel, so move the
nvmem driver to use device_add_groups() instead.  The logic is
identical, when the device is removed the driver core will properly
clean up and remove the groups, and the memory used by the attribute
groups will be freed because it was created with dev_* calls, so this is
functionally identical overall.

Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/nvmem/core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c
index 2c6b99402df8..e1ec3b7200d7 100644
--- a/drivers/nvmem/core.c
+++ b/drivers/nvmem/core.c
@@ -478,7 +478,7 @@ static int nvmem_populate_sysfs_cells(struct nvmem_device *nvmem)
 
 	nvmem_cells_group.bin_attrs = cells_attrs;
 
-	ret = devm_device_add_groups(&nvmem->dev, nvmem_cells_groups);
+	ret = device_add_groups(&nvmem->dev, nvmem_cells_groups);
 	if (ret)
 		goto unlock_mutex;
 
-- 
2.44.0
Re: [PATCH] nvmem: core: switch to use device_add_groups()
Posted by Srinivas Kandagatla 1 year, 10 months ago
On Wed, 27 Mar 2024 09:11:46 +0100, Greg Kroah-Hartman wrote:
> devm_device_add_groups() is being removed from the kernel, so move the
> nvmem driver to use device_add_groups() instead.  The logic is
> identical, when the device is removed the driver core will properly
> clean up and remove the groups, and the memory used by the attribute
> groups will be freed because it was created with dev_* calls, so this is
> functionally identical overall.
> 
> [...]

Applied, thanks!

[1/1] nvmem: core: switch to use device_add_groups()
      commit: ea8f9ec2bbb75adac49dbaaf267f8727ee26b91b

Best regards,
-- 
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>