Re: [PATCH v2 8/9] ASoC: simple-card: Add missing of_node_put() in case of error

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

To: Herve Codina <herve.codina@xxxxxxxxxxx>
Subject: Re: [PATCH v2 8/9] ASoC: simple-card: Add missing of_node_put() in case of error
From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
Date: Tue, 23 May 2023 23:24:34 +0000
Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>, Mark Brown <broonie@xxxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>, Jonathan Cameron <jic23@xxxxxxxxxx>, Lars-Peter Clausen <lars@xxxxxxxxxx>, Jaroslav Kysela <perex@xxxxxxxx>, Takashi Iwai <tiwai@xxxxxxxx>, alsa-devel@xxxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-iio@xxxxxxxxxxxxxxx, Christophe Leroy <christophe.leroy@xxxxxxxxxx>, Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx>
In-reply-to: <20230523151223.109551-9-herve.codina@bootlin.com>
References: <20230523151223.109551-1-herve.codina@bootlin.com> <20230523151223.109551-9-herve.codina@bootlin.com>
User-agent: Wanderlust/2.15.9 Emacs/27.1 Mule/6.0

Hi

> In the error path, a of_node_put() for platform is missing.
> Just add it.
> 
> Signed-off-by: Herve Codina <herve.codina@xxxxxxxxxxx>
> ---

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

Thank you for your help !!

Best regards
---
Kuninori Morimoto

References:
- [PATCH v2 0/9] Add support for IIO devices in ASoC
  - From: Herve Codina
- [PATCH v2 8/9] ASoC: simple-card: Add missing of_node_put() in case of error
  - From: Herve Codina

Prev by Date: Re: [PATCH] regmap: maple: Drop the RCU read lock while syncing registers
Next by Date: Re: [PATCH v2 9/9] ASoC: simple-card: Handle additional devices
Previous by thread: [PATCH v2 8/9] ASoC: simple-card: Add missing of_node_put() in case of error
Next by thread: Re: (subset) [PATCH v2 0/9] Add support for IIO devices in ASoC
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]