Re: [-next] ASoC: Intel: fix compiler warning of function ‘probe_codec’

From: Cezary Rojewski
Date: Tue Sep 13 2022 - 10:08:42 EST


On 2022-09-13 12:02 PM, Wang Yufen wrote:
Move the variable ‘skl’ define to
CONFIG_SND_SOC_INTEL_SKYLAKE_HDAUDIO_CODEC enabled, to fix the compiler
warning:
sound/soc/intel/skylake/skl.c: In function ‘probe_codec’:
sound/soc/intel/skylake/skl.c:729:18: warning: unused variable ‘skl’ [-Wunused-variable]
729 | struct skl_dev *skl = bus_to_skl(bus);
| ^~~

Signed-off-by: Wang Yufen <wangyufen@xxxxxxxxxx>

Fix is fine but it has been proposed already by Gaosheng [1].



[1]: https://lore.kernel.org/alsa-devel/20220822035133.2147381-1-cuigaosheng1@xxxxxxxxxx/

Regards,
Czarek

---
sound/soc/intel/skylake/skl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/soc/intel/skylake/skl.c b/sound/soc/intel/skylake/skl.c
index c7c1cad..52a041d 100644
--- a/sound/soc/intel/skylake/skl.c
+++ b/sound/soc/intel/skylake/skl.c
@@ -726,8 +726,8 @@ static int probe_codec(struct hdac_bus *bus, int addr)
unsigned int cmd = (addr << 28) | (AC_NODE_ROOT << 20) |
(AC_VERB_PARAMETERS << 8) | AC_PAR_VENDOR_ID;
unsigned int res = -1;
- struct skl_dev *skl = bus_to_skl(bus);
#if IS_ENABLED(CONFIG_SND_SOC_INTEL_SKYLAKE_HDAUDIO_CODEC)
+ struct skl_dev *skl = bus_to_skl(bus);
struct hdac_hda_priv *hda_codec;
#endif
struct hda_codec *codec;