[PATCH v3 18/20] ASoC: uda1380: use devm_kmemdup_array()

Raag Jadav posted 20 patches 1 year ago
There is a newer version of this series
[PATCH v3 18/20] ASoC: uda1380: use devm_kmemdup_array()
Posted by Raag Jadav 1 year ago
Convert to use devm_kmemdup_array() which is more robust.

Signed-off-by: Raag Jadav <raag.jadav@intel.com>
---
 sound/soc/codecs/uda1380.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/sound/soc/codecs/uda1380.c b/sound/soc/codecs/uda1380.c
index 4f8fdd574585..6b474f9ee7c4 100644
--- a/sound/soc/codecs/uda1380.c
+++ b/sound/soc/codecs/uda1380.c
@@ -766,10 +766,8 @@ static int uda1380_i2c_probe(struct i2c_client *i2c)
 			return ret;
 	}
 
-	uda1380->reg_cache = devm_kmemdup(&i2c->dev,
-					uda1380_reg,
-					ARRAY_SIZE(uda1380_reg) * sizeof(u16),
-					GFP_KERNEL);
+	uda1380->reg_cache = devm_kmemdup_array(&i2c->dev, uda1380_reg, ARRAY_SIZE(uda1380_reg),
+						sizeof(u16), GFP_KERNEL);
 	if (!uda1380->reg_cache)
 		return -ENOMEM;
 
-- 
2.34.1
Re: [PATCH v3 18/20] ASoC: uda1380: use devm_kmemdup_array()
Posted by Andy Shevchenko 1 year ago
On Mon, Feb 03, 2025 at 01:39:00PM +0530, Raag Jadav wrote:
> Convert to use devm_kmemdup_array() which is more robust.

...

> -	uda1380->reg_cache = devm_kmemdup(&i2c->dev,
> -					uda1380_reg,
> -					ARRAY_SIZE(uda1380_reg) * sizeof(u16),
> -					GFP_KERNEL);
> +	uda1380->reg_cache = devm_kmemdup_array(&i2c->dev, uda1380_reg, ARRAY_SIZE(uda1380_reg),
> +						sizeof(u16), GFP_KERNEL);

sizeof(*uda1380->reg_cache) ?

>  	if (!uda1380->reg_cache)
>  		return -ENOMEM;

-- 
With Best Regards,
Andy Shevchenko