[0/2] ASoC: Intel: cht_bsw_nau8824: 2 fixes for usage with sof-audio-acpi

Message

Hans de Goede Dec. 6, 2020, 12:24 p.m. UTC

Hi All,

Here are 2 simple fixes which are necessary to make the
cht_bsw_nau8824 machine driver work together with the
sof-audio-acpi driver.

Note that atm the sof topology files are missing a .tplg
file for this setup. Simply copying over the standard
sof-byte-codec.tplg file does the trick, but then some
mixer setting changes are necessary to fix the right
speaker/headphones channel not working; and those mixer
settings break the right channel when used with the
sst-acpi driver.

I've been trying to fix this at the tplg level so that
we do not need to change the mixer settings, but no luck
sofar. I'll post a RFC with the topology changes which
I have and we can discuss this further there.

These 2 simple fixes are necessary to make the sof-audio-acpi
driver work regardless of the topology issue.

Regards,

Hans

Comments

Pierre-Louis Bossart Dec. 7, 2020, 3:14 p.m. UTC | #1

On 12/6/20 6:24 AM, Hans de Goede wrote:
> Hi All,
> 
> Here are 2 simple fixes which are necessary to make the
> cht_bsw_nau8824 machine driver work together with the
> sof-audio-acpi driver.
> 
> Note that atm the sof topology files are missing a .tplg
> file for this setup. Simply copying over the standard
> sof-byte-codec.tplg file does the trick, but then some
> mixer setting changes are necessary to fix the right
> speaker/headphones channel not working; and those mixer
> settings break the right channel when used with the
> sst-acpi driver.
> 
> I've been trying to fix this at the tplg level so that
> we do not need to change the mixer settings, but no luck
> sofar. I'll post a RFC with the topology changes which
> I have and we can discuss this further there.
> 
> These 2 simple fixes are necessary to make the sof-audio-acpi
> driver work regardless of the topology issue.

Thanks for your work Hans, much appreciated. Both changes are valid and 
were missed.

Acked-by: Pierre-Louis Bossart <[e-mail]>

Mark Brown Dec. 8, 2020, 5:09 p.m. UTC | #2

On Sun, 6 Dec 2020 13:24:34 +0100, Hans de Goede wrote:
> Here are 2 simple fixes which are necessary to make the
> cht_bsw_nau8824 machine driver work together with the
> sof-audio-acpi driver.
> 
> Note that atm the sof topology files are missing a .tplg
> file for this setup. Simply copying over the standard
> sof-byte-codec.tplg file does the trick, but then some
> mixer setting changes are necessary to fix the right
> speaker/headphones channel not working; and those mixer
> settings break the right channel when used with the
> sst-acpi driver.
> 
> [...]

Applied to

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

Thanks!

[1/2] ASoC: Intel: cht_bsw_nau8824: Drop compress-cpu-dai bits
      commit: e60ffc48fac4b6ba8f3ec500bd166909f3db03c3
[2/2] ASoC: Intel: cht_bsw_nau8824: Change SSP2-Codec DAI id to 0
      commit: 748e72e869718db8d735d773040bce95158c98c6

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