mbox series

[0/2] ASoC: (SOF) topology: Regression fixes for next

Message ID 20230201112846.27707-1-peter.ujfalusi@linux.intel.com
Headers show
Series ASoC: (SOF) topology: Regression fixes for next | expand

Message

Peter Ujfalusi Feb. 1, 2023, 11:28 a.m. UTC 
Hi,

Today I came across two regressions in next with SOF:

The topology would not load with a failure of creating playback DAI
the first patch is fixing this which was caused by a missing 'else' in the patch

After fixing the topology loading, the module unloading caused kernel panic.
The second patch is correcting that which is I likely caused by copy-paste to
set wrong unload callback for the graph element.

With these patches applied SOF is working on next and modules can be unloaded

Regards,
Peter
---
Peter Ujfalusi (2):
  ASoC: SOF: topology: Add missed "else" in sof_connect_dai_widget
  ASoC: topology: Set correct unload callback for graph type

 sound/soc/soc-topology.c | 2 +-
 sound/soc/sof/topology.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Amadeusz Sławiński Feb. 1, 2023, 11:47 a.m. UTC | #1 
On 2/1/2023 12:28 PM, Peter Ujfalusi wrote:
> Using the control_unload for graph type of elem will lead surprises on
> module unload.
> 
> The correct callback to use is the dapm_route_unload.
> 
> Fixes: 31e9273912bf ("ASoC: topology: Use unload() op directly")
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>

LGTM, thanks!

Reviewed-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
Mark Brown Feb. 3, 2023, 2:55 p.m. UTC | #2 
On Wed, 01 Feb 2023 13:28:44 +0200, Peter Ujfalusi wrote:
> Today I came across two regressions in next with SOF:
> 
> The topology would not load with a failure of creating playback DAI
> the first patch is fixing this which was caused by a missing 'else' in the patch
> 
> After fixing the topology loading, the module unloading caused kernel panic.
> The second patch is correcting that which is I likely caused by copy-paste to
> set wrong unload callback for the graph element.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/2] ASoC: SOF: topology: Add missed "else" in sof_connect_dai_widget
      commit: afd7c141c750f3f043c755bd8d01a2ffee7e95b2
[2/2] ASoC: topology: Set correct unload callback for graph type
      commit: dd184c400e10295631e5742fc7318ba071c67007

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark