All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.