On Mon, 04 Sep 2023 16:01:40 +0200,
Shenghao Ding wrote:
>
> Revert structure name back to cs35l41_dev_name and redefine
> tas2781_generic_fixup.
Again, please describe *why* you need to do this.
That's the most important part. And, if you write the reason for the
change, you can re-evaluate whether it really makes sense in that
regard.
thanks,
Takashi
>
> Signed-off-by: Shenghao Ding <shenghao-ding@xxxxxx>
>
> ---
> Changes in v2:
> - Add more description on why revert structure name back to
> cs35l41_dev_name
> - Redefine tas2781_generic_fixup, remove hid argument, and do not use
> structure scodec_dev_name any more in tas2781_generic_fixup.
> - remove cs35l41_dev_name from comp_match_tas2781_dev_name, which is
> useless. bus name is passed by tas2781_generic_fixup, hid is actually
> "TIAS2781", can be hardcode and unneeded argument passed from
> tas2781_generic_fixup.
> Index is a redundant member, and never used in tas2781 hda driver.
> - revert from scodec_dev_name back to cs35l41_dev_name, tas2781 pass only
> bus name instead of cs35l41_dev_name, so it seemed unnecessary for
> tas2781.
> ---
> sound/pci/hda/patch_realtek.c | 27 ++++++++++-----------------
> 1 file changed, 10 insertions(+), 17 deletions(-)
>
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index a07df6f929..c3e410152b 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -6745,7 +6745,7 @@ static void comp_generic_playback_hook(struct hda_pcm_stream *hinfo, struct hda_
> }
> }
>
> -struct scodec_dev_name {
> +struct cs35l41_dev_name {
> const char *bus;
> const char *hid;
> int index;
> @@ -6754,7 +6754,7 @@ struct scodec_dev_name {
> /* match the device name in a slightly relaxed manner */
> static int comp_match_cs35l41_dev_name(struct device *dev, void *data)
> {
> - struct scodec_dev_name *p = data;
> + struct cs35l41_dev_name *p = data;
> const char *d = dev_name(dev);
> int n = strlen(p->bus);
> char tmp[32];
> @@ -6773,19 +6773,19 @@ static int comp_match_cs35l41_dev_name(struct device *dev, void *data)
> static int comp_match_tas2781_dev_name(struct device *dev,
> void *data)
> {
> - struct scodec_dev_name *p = data;
> + const char *bus = data;
> const char *d = dev_name(dev);
> - int n = strlen(p->bus);
> + int n = strlen(bus);
> char tmp[32];
>
> /* check the bus name */
> - if (strncmp(d, p->bus, n))
> + if (strncmp(d, bus, n))
> return 0;
> /* skip the bus number */
> if (isdigit(d[n]))
> n++;
> /* the rest must be exact matching */
> - snprintf(tmp, sizeof(tmp), "-%s:00", p->hid);
> + snprintf(tmp, sizeof(tmp), "-%s:00", "TIAS2781");
>
> return !strcmp(d + n, tmp);
> }
> @@ -6795,7 +6795,7 @@ static void cs35l41_generic_fixup(struct hda_codec *cdc, int action, const char
> {
> struct device *dev = hda_codec_dev(cdc);
> struct alc_spec *spec = cdc->spec;
> - struct scodec_dev_name *rec;
> + struct cs35l41_dev_name *rec;
> int ret, i;
>
> switch (action) {
> @@ -6824,24 +6824,17 @@ static void cs35l41_generic_fixup(struct hda_codec *cdc, int action, const char
> }
>
> static void tas2781_generic_fixup(struct hda_codec *cdc, int action,
> - const char *bus, const char *hid)
> + const char *bus)
> {
> struct device *dev = hda_codec_dev(cdc);
> struct alc_spec *spec = cdc->spec;
> - struct scodec_dev_name *rec;
> int ret;
>
> switch (action) {
> case HDA_FIXUP_ACT_PRE_PROBE:
> - rec = devm_kmalloc(dev, sizeof(*rec), GFP_KERNEL);
> - if (!rec)
> - return;
> - rec->bus = bus;
> - rec->hid = hid;
> - rec->index = 0;
> spec->comps[0].codec = cdc;
> component_match_add(dev, &spec->match,
> - comp_match_tas2781_dev_name, rec);
> + comp_match_tas2781_dev_name, (void *)bus);
> ret = component_master_add_with_match(dev, &comp_master_ops,
> spec->match);
> if (ret)
> @@ -6888,7 +6881,7 @@ static void alc287_fixup_legion_16ithg6_speakers(struct hda_codec *cdc, const st
> static void tas2781_fixup_i2c(struct hda_codec *cdc,
> const struct hda_fixup *fix, int action)
> {
> - tas2781_generic_fixup(cdc, action, "i2c", "TIAS2781");
> + tas2781_generic_fixup(cdc, action, "i2c");
> }
>
> /* for alc295_fixup_hp_top_speakers */
> --
> 2.34.1
>
