Re: [PATCH 08/11] ASoC: simple-card: Use cleanup.h instead of devm_kfree()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Subject: Re: [PATCH 08/11] ASoC: simple-card: Use cleanup.h instead of devm_kfree()
From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
Date: Thu, 4 Jul 2024 02:53:54 +0000
Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>, Mark Brown <broonie@xxxxxxxxxx>, Jaroslav Kysela <perex@xxxxxxxx>, Takashi Iwai <tiwai@xxxxxxxx>, Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>, Banajit Goswami <bgoswami@xxxxxxxxxxx>, linux-sound@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, alsa-devel@xxxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx
In-reply-to: <20240703-asoc-cleanup-h-v1-8-71219dfd0aef@linaro.org>
References: <20240703-asoc-cleanup-h-v1-0-71219dfd0aef@linaro.org> <20240703-asoc-cleanup-h-v1-8-71219dfd0aef@linaro.org>
User-agent: Wanderlust/2.15.9 Emacs/29.3 Mule/6.0

Hi

> Allocate the memory with scoped/cleanup.h, instead of devm interface, to
> make the code more obvious that memory is not used outside this scope.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> ---

Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>

Thank you for your help !!

Best regards
---
Kuninori Morimoto

References:
- [PATCH 00/11] ASoC: Simplify code with cleanup.h
  - From: Krzysztof Kozlowski
- [PATCH 08/11] ASoC: simple-card: Use cleanup.h instead of devm_kfree()
  - From: Krzysztof Kozlowski

Prev by Date: Re: [PATCH 06/11] ASoC: audio-graph-card: Use cleanup.h instead of devm_kfree()
Next by Date: Re: [PATCH 07/11] ASoC: audio-graph-card2: Use cleanup.h instead of devm_kfree()
Previous by thread: [PATCH 08/11] ASoC: simple-card: Use cleanup.h instead of devm_kfree()
Next by thread: [PATCH 01/11] ASoC: codecs: audio-iio-aux: Simplify audio_iio_aux_add_dapms() with cleanup.h
Index(es):
- Date
- Thread

[Index of Archives] [ALSA User] [Linux Audio Users] [Pulse Audio] [Kernel Archive] [Asterisk PBX] [Photo Sharing] [Linux Sound] [Video 4 Linux] [Gimp] [Yosemite News]