lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <87h66cpvn6.wl-kuninori.morimoto.gx@renesas.com>
Date: Mon, 6 Jan 2025 01:40:29 +0000
From: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
To: Stephen Gordon <gordoste@...et.net.au>
Cc: perex@...ex.cz,
	tiwai@...e.com,
	linux-sound@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] asoc: simple_card: Show if link is unidirectional


Hi Stephen

> It is handy to know whether the link is playback-only/capture-only when
> debugging.
> 
> Signed-off-by: Stephen Gordon <gordoste@...et.net.au>
> ---
>  include/sound/simple_card_utils.h | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/include/sound/simple_card_utils.h b/include/sound/simple_card_utils.h
> index af79f680c72a..fc149fc18ad6 100644
> --- a/include/sound/simple_card_utils.h
> +++ b/include/sound/simple_card_utils.h
> @@ -267,6 +267,9 @@ static inline void simple_util_debug_info(struct simple_util_priv *priv)
>  			dev_dbg(dev, "link name = %s\n", link->name);
>  		if (link->dai_fmt)
>  			dev_dbg(dev, "link format = %04x\n", link->dai_fmt);
> +		if (link->playback_only || link->capture_only)
> +			dev_dbg(dev, "link is %s only",
> +				link->playback_only ? "PLAYBACK" : "CAPTURE");
>  		if (props->adata.convert_rate)
>  			dev_dbg(dev, "convert_rate = %d\n", props->adata.convert_rate);
>  		if (props->adata.convert_channels)

I like this instead. It will be less misunderstand.

	if (link->playback_only)
		dev_dbg(dev, "link has playback_only");
	if (link->capture_only)
		dev_dbg(dev, "link has capture_only");

Please add "To: Mark Brown <broonie@...nel.org>" on this patch.

In Subject,
	- asoc: ...
	+ ASoC: ...

Thank you for your help !!

Best regards
---
Kuninori Morimoto

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ