[alsa-devel] [PATCH] UCM: bytcr/PlatformEnableSeq.conf update some comments

Takashi Iwai tiwai at suse.de
Thu Feb 7 08:22:50 CET 2019


On Sun, 03 Feb 2019 12:37:41 +0100,
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>

Applied now.  Thanks.


Takashi


More information about the Alsa-devel mailing list