[alsa-devel] [PATCH] UCM: bytcr/PlatformEnableSeq.conf update some comments
Russell Parker
russell.parker7 at gmail.com
Thu Feb 7 03:42:46 CET 2019
On 02/03, Hans de Goede wrote:
> Commit f91cc3c7d6b7 ("Update chtrt5645 ucm variants to use
> bytcr/PlatformEnableSeq.conf component") updated the
> following 2 comments:
>
> # codec0_out settings (used if SSP2 is connected to aif1)
> # modem_out settings (used if SSP0 is connected to aif2)
>
> Specifically it added the " to aif1" resp. " to aif2" part of the comments.
>
> This is not correct, AIF1 / AIF2 are something which is present on
> Realtek codecs only, and either one can be used indepedent of
> SSP0 or SSP2 being used (the comments in the chtrt5645 UCM profile
> before this change were wrong / outdated).
>
> Besides there not being any relationship between SSP0 or SSP2 being
> used, bytcr/PlatformEnableSeq.conf is also used with other codecs,
> e.g. the ESS8316 codec where this is not applicable at all.
>
> Therefor this commit removes the " to aif?" part of the comments again
> to avoid confusing people reading this in the future.
>
> Cc: Russell Parker <russell.parker7 at gmail.com>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
Thanks for updating the comments; I saw that the notes were in the rt5645
sequences and incorrectly assumed that it applied to the platform too. Glad
you caught it!
More information about the Alsa-devel
mailing list