[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ZwkG98S_wxK6xMEq@smile.fi.intel.com>
Date: Fri, 11 Oct 2024 14:07:35 +0300
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Baojun Xu <baojun.xu@...com>
Cc: tiwai@...e.de, robh+dt@...nel.org, lgirdwood@...il.com, perex@...ex.cz,
pierre-louis.bossart@...ux.intel.com, kevin-lu@...com,
shenghao-ding@...com, navada@...com, 13916275206@....com,
v-hampiholi@...com, v-po@...com, niranjan.hy@...com,
alsa-devel@...a-project.org, linux-kernel@...r.kernel.org,
liam.r.girdwood@...el.com, yung-chuan.liao@...ux.intel.com,
broonie@...nel.org, soyer@....hu
Subject: Re: [PATCH v1] ALSA: hda/tas2781: Add speaker id check for ASUS
projects
On Fri, Oct 11, 2024 at 04:53:03PM +0800, Baojun Xu wrote:
> Add speaker id check by gpio in ACPI for ASUS projects.
...
> + scnprintf(tas_priv->coef_binaryname, 64, "TAS2XXX%04X%01d.bin",
sizeof() ?
> + codec->core.subsystem_id & 0xffff, spk_id);
lower_16_bits() ?
...
> + scnprintf(tas_priv->coef_binaryname, 64, "TAS2XXX%04X.bin",
Ditto.
> + codec->core.subsystem_id & 0xffff);
Ditto.
...
> + tas_hda->priv->speaker_id = devm_gpiod_get(tas_hda->dev, "speakerid",
> + GPIOD_IN);
> + if (IS_ERR(tas_hda->priv->speaker_id))
> + dev_info(tas_hda->dev, "Failed to get Speaker id gpio.\n");
This is wrong. If it's okay to ignore, make it optional. Either way you have to
return an error code to the caller as you effectively ignore deferred probe,
for example.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists