diff mbox series

ASoC: madera: Use str_enabled_disabled() helper function

Message ID 20241229174043.781856-2-thorsten.blum@linux.dev (mailing list archive)
State Superseded
Headers show
Series ASoC: madera: Use str_enabled_disabled() helper function | expand

Commit Message

Thorsten Blum Dec. 29, 2024, 5:40 p.m. UTC 
Remove hard-coded strings by using the str_enabled_disabled() helper
function.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
 sound/soc/codecs/madera.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Charles Keepax Jan. 6, 2025, 4:02 p.m. UTC | #1 
On Sun, Dec 29, 2024 at 06:40:44PM +0100, Thorsten Blum wrote:
> Remove hard-coded strings by using the str_enabled_disabled() helper
> function.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
>  sound/soc/codecs/madera.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/sound/soc/codecs/madera.c b/sound/soc/codecs/madera.c
> index b24d6472ad5f..86abfbe56e1f 100644
> --- a/sound/soc/codecs/madera.c
> +++ b/sound/soc/codecs/madera.c
> @@ -3965,7 +3965,7 @@ static int madera_enable_fll(struct madera_fll *fll)
>  	}
>  
>  	madera_fll_dbg(fll, "Enabling FLL, initially %s\n",
> -		       already_enabled ? "enabled" : "disabled");
> +		       str_enabled_disabled(already_enabled));
>  
>  	if (fll->fout < MADERA_FLL_MIN_FOUT ||
>  	    fll->fout > MADERA_FLL_MAX_FOUT) {
> @@ -4252,7 +4252,7 @@ static int madera_enable_fll_ao(struct madera_fll *fll,
>  		pm_runtime_get_sync(madera->dev);
>  
>  	madera_fll_dbg(fll, "Enabling FLL_AO, initially %s\n",
> -		       already_enabled ? "enabled" : "disabled");
> +		       str_enabled_disabled(already_enabled));
>  
>  	/* FLL_AO_HOLD must be set before configuring any registers */
>  	regmap_update_bits(fll->madera->regmap,
> @@ -4576,7 +4576,7 @@ static int madera_fllhj_enable(struct madera_fll *fll)
>  		pm_runtime_get_sync(madera->dev);
>  
>  	madera_fll_dbg(fll, "Enabling FLL, initially %s\n",
> -		       already_enabled ? "enabled" : "disabled");
> +		       str_enabled_disabled(already_enabled));
>  
>  	/* FLLn_HOLD must be set before configuring any registers */
>  	regmap_update_bits(fll->madera->regmap,

This should also add an include for the string_choices header,
but otherwise looks fine to me.

Thanks,
Charles
diff mbox series

Patch

diff --git a/sound/soc/codecs/madera.c b/sound/soc/codecs/madera.c
index b24d6472ad5f..86abfbe56e1f 100644
--- a/sound/soc/codecs/madera.c
+++ b/sound/soc/codecs/madera.c
@@ -3965,7 +3965,7 @@  static int madera_enable_fll(struct madera_fll *fll)
 	}
 
 	madera_fll_dbg(fll, "Enabling FLL, initially %s\n",
-		       already_enabled ? "enabled" : "disabled");
+		       str_enabled_disabled(already_enabled));
 
 	if (fll->fout < MADERA_FLL_MIN_FOUT ||
 	    fll->fout > MADERA_FLL_MAX_FOUT) {
@@ -4252,7 +4252,7 @@  static int madera_enable_fll_ao(struct madera_fll *fll,
 		pm_runtime_get_sync(madera->dev);
 
 	madera_fll_dbg(fll, "Enabling FLL_AO, initially %s\n",
-		       already_enabled ? "enabled" : "disabled");
+		       str_enabled_disabled(already_enabled));
 
 	/* FLL_AO_HOLD must be set before configuring any registers */
 	regmap_update_bits(fll->madera->regmap,
@@ -4576,7 +4576,7 @@  static int madera_fllhj_enable(struct madera_fll *fll)
 		pm_runtime_get_sync(madera->dev);
 
 	madera_fll_dbg(fll, "Enabling FLL, initially %s\n",
-		       already_enabled ? "enabled" : "disabled");
+		       str_enabled_disabled(already_enabled));
 
 	/* FLLn_HOLD must be set before configuring any registers */
 	regmap_update_bits(fll->madera->regmap,