Linux Sound subsystem development
 help / color / mirror / Atom feed
From: <qianghua.wang@senarytech.com>
To: "'Liao, Bard'" <bard.liao@intel.com>,
	"'Mark Brown'" <broonie@kernel.org>
Cc: <linux-sound@vger.kernel.org>,
	"'Liam Girdwood'" <lgirdwood@gmail.com>,
	"'Jaroslav Kysela'" <perex@perex.cz>,
	"'Takashi Iwai'" <tiwai@suse.com>,
	"'Rojewski, Cezary'" <cezary.rojewski@intel.com>,
	"'Peter Ujfalusi'" <peter.ujfalusi@linux.intel.com>,
	"'Bard Liao'" <yung-chuan.liao@linux.intel.com>,
	"'Kai Vehmanen'" <kai.vehmanen@linux.intel.com>,
	"'Pierre-Louis Bossart'" <pierre-louis.bossart@linux.dev>,
	"'bo liu'" <bo.liu@senarytech.com>
Subject: reply [PATCH v2] ASoC: sdw_utils: senary: handle a missing name_prefix
Date: Wed, 11 Nov 2026 14:36:53 +0800	[thread overview]
Message-ID: <004501dd71a7$0b2e3e40$218abac0$@senarytech.com> (raw)
In-Reply-To: <SJ2PR11MB8424FE5B64FEDEDA5F947D30FF722@SJ2PR11MB8424.namprd11.prod.outlook.com>

Hi Bard,
	Thanks for the review. I agree the right fix is to ensure
name_prefix is set via codec_conf / component probe, not to paper over NULL
in sdw_utils.
The v2 follow-up was based on debug logs and instrumentation from jfdnet, a
community contributor on GitHub issue #6012. We have not reproduced this on
SN6245 hardware ourselves yet (SN6244 only in our lab).
Please hold off on reviewing or merging
[PATCH v2] ASoC: sdw_utils: senary: handle a missing name_prefix until we
have the same hardware and can test and align the fix locally. 
I will follow up after that.
Thanks,
Qianghua Wang

From: Liao, Bard <bard.liao@intel.com> 
Sent: 2026年11月10日 22:45
To: Qianghua Wang <qianghua.wang@senarytech.com>; Mark Brown
<broonie@kernel.org>
cc: linux-sound@vger.kernel.org; Liam Girdwood <lgirdwood@gmail.com>;
Jaroslav Kysela <perex@perex.cz>; Takashi Iwai <tiwai@suse.com>; Rojewski,
Cezary <cezary.rojewski@intel.com>; Peter Ujfalusi
<peter.ujfalusi@linux.intel.com>; Bard Liao
<yung-chuan.liao@linux.intel.com>; Kai Vehmanen
<kai.vehmanen@linux.intel.com>; Pierre-Louis Bossart
<pierre-louis.bossart@linux.dev>; bo liu <bo.liu@senarytech.com>
Subject: RE: [PATCH v2] ASoC: sdw_utils: senary: handle a missing
name_prefix

> -----Original Message-----
> From: Qianghua Wang <qianghua.wang@senarytech.com>
> Sent: Tuesday, November 10, 2026 4:06 PM
> To: Mark Brown <broonie@kernel.org>
> Cc: linux-sound@vger.kernel.org; Liam Girdwood <lgirdwood@gmail.com>; 
> Jaroslav Kysela <perex@perex.cz>; Takashi Iwai <tiwai@suse.com>; 
> Rojewski, Cezary <cezary.rojewski@intel.com>; Peter Ujfalusi 
> <peter.ujfalusi@linux.intel.com>; Bard Liao 
> <yung-chuan.liao@linux.intel.com>;
> Kai Vehmanen <kai.vehmanen@linux.intel.com>; Pierre-Louis Bossart 
> <pierre- louis.bossart@linux.dev>; bo liu <bo.liu@senarytech.com>; 
> Qianghua Wang <qianghua.wang@senarytech.com>
> Subject: [PATCH v2] ASoC: sdw_utils: senary: handle a missing 
> name_prefix
> 
> asoc_sdw_senary_sdca_jack_rtd_init() passes component->name_prefix to 
> strstr(). When codec_conf does not apply the prefix, name_prefix is 
> NULL and card registration oopses.

In what case will codec_conf not apply the prefix?
Won't it be better to ensure that the name_prefix is always set?




      reply	other threads:[~2026-11-11  6:37 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-10  7:34 [PATCH] ASoC: sdw_utils: senary: handle a missing name_prefix Qianghua Wang
2026-11-10  7:44 ` sashiko-bot
2026-11-10  8:05 ` [PATCH v2] " Qianghua Wang
2026-11-10 14:45   ` Liao, Bard
2026-11-11  6:36     ` qianghua.wang [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='004501dd71a7$0b2e3e40$218abac0$@senarytech.com' \
    --to=qianghua.wang@senarytech.com \
    --cc=bard.liao@intel.com \
    --cc=bo.liu@senarytech.com \
    --cc=broonie@kernel.org \
    --cc=cezary.rojewski@intel.com \
    --cc=kai.vehmanen@linux.intel.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=peter.ujfalusi@linux.intel.com \
    --cc=pierre-louis.bossart@linux.dev \
    --cc=tiwai@suse.com \
    --cc=yung-chuan.liao@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox