All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Péter Ujfalusi" <peter.ujfalusi@linux.intel.com>
To: Mark Brown <broonie@kernel.org>
Cc: Cezary Rojewski <cezary.rojewski@intel.com>,
	tiwai@suse.com, perex@perex.cz, amade@asmblr.net,
	kuninori.morimoto.gx@renesas.com, linux-sound@vger.kernel.org,
	"Liao, Bard" <bard.liao@intel.com>,
	"Vehmanen, Kai" <kai.vehmanen@intel.com>,
	Charles Keepax <ckeepax@opensource.cirrus.com>,
	Liam Girdwood <liam.r.girdwood@linux.intel.com>,
	Richard Fitzgerald <rf@opensource.cirrus.com>,
	Simon Trimmer <simont@opensource.cirrus.com>
Subject: Re: [PATCH] ASoC: core: Move all users to deferrable card binding
Date: Tue, 26 May 2026 14:28:03 +0300	[thread overview]
Message-ID: <10d5daa6-5ec2-4f46-a204-4df9f83b100c@linux.intel.com> (raw)
In-Reply-To: <e6c5e5e0-69f1-4fdc-b7be-f6372e507b37@linux.intel.com>



On 26/05/2026 14:21, Péter Ujfalusi wrote:
> On 26/05/2026 08:45, Péter Ujfalusi wrote:
>> I also think that detaching from driver defer and use internal card
>> defer is a much better approach, but when majority of new PTL designs
>> will be broken with no patch in sight to prepare the SDCA class stack is
>> a regression.
>>
>> Charles, Bard, Richard, Simon, any thoughts on this?
> 
> I can force the sof_sdw driver to go back to driver defer queue if any of
> the expected (counted for) components yet to be registered, but I'm sure
> this has side-effects...

Yes, this defeats the purpose of the deferrable card concept.

> 
> diff --git a/sound/soc/sdw_utils/soc_sdw_utils.c b/sound/soc/sdw_utils/soc_sdw_utils.c
> index 9d0768f21ba4..e80c3904bc37 100644
> --- a/sound/soc/sdw_utils/soc_sdw_utils.c
> +++ b/sound/soc/sdw_utils/soc_sdw_utils.c
> @@ -2022,6 +2022,7 @@ int asoc_sdw_parse_sdw_endpoints(struct snd_soc_card *card,
>  					soc_aux->dlc.name = component->name;
>  				} else {
>  					soc_aux->dlc.name = codec_info->auxs[j].codec_name;
> +					return -EPROBE_DEFER;
>  				}
>  				soc_aux++;
>  			}
> 
>  
> 

-- 
Péter


  reply	other threads:[~2026-05-26 11:27 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-30 14:07 [PATCH] ASoC: core: Move all users to deferrable card binding Cezary Rojewski
2026-05-11  1:06 ` Mark Brown
2026-05-11  1:34 ` Kuninori Morimoto
2026-05-12  8:14   ` Rojewski, Cezary
2026-05-20  9:55 ` Alexander Stein
2026-05-20 10:33   ` Cezary Rojewski
2026-05-21  6:42     ` Alexander Stein
2026-05-21  8:13       ` Cezary Rojewski
2026-05-21 10:11         ` Alexander Stein
2026-05-21 14:41           ` Cezary Rojewski
2026-05-21 14:47             ` Alexander Stein
2026-05-22 11:21             ` Alexander Stein
2026-05-22 10:16 ` Péter Ujfalusi
2026-05-22 14:32   ` Cezary Rojewski
2026-05-22 14:58     ` Mark Brown
2026-05-22 15:08       ` Cezary Rojewski
2026-05-25 12:48       ` Péter Ujfalusi
2026-05-25 15:06         ` Mark Brown
2026-05-26  5:45           ` Péter Ujfalusi
2026-05-26 11:21             ` Péter Ujfalusi
2026-05-26 11:28               ` Péter Ujfalusi [this message]
2026-05-26 16:02             ` Mark Brown
2026-05-27  5:22               ` Péter Ujfalusi
2026-05-27 10:35                 ` Mark Brown
2026-05-25 10:44     ` Pierre-Louis Bossart
2026-05-25 21:10       ` Cezary Rojewski

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=10d5daa6-5ec2-4f46-a204-4df9f83b100c@linux.intel.com \
    --to=peter.ujfalusi@linux.intel.com \
    --cc=amade@asmblr.net \
    --cc=bard.liao@intel.com \
    --cc=broonie@kernel.org \
    --cc=cezary.rojewski@intel.com \
    --cc=ckeepax@opensource.cirrus.com \
    --cc=kai.vehmanen@intel.com \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=rf@opensource.cirrus.com \
    --cc=simont@opensource.cirrus.com \
    --cc=tiwai@suse.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.