[PATCH RESEND 0/3] alsa-lib/ASoC: use inclusive language for bclk/fsync/topology

Takashi Iwai tiwai at suse.de
Tue Nov 17 14:02:25 CET 2020


On Thu, 12 Nov 2020 17:29:37 +0100,
Pierre-Louis Bossart wrote:
> 
> This patchset is just a rebase of the code suggested in September [1],
> resent for convenience for comparison with kernel patches.
> 
> The SOF (Sound Open Firmware) tree contains a lot of references in
> topology files to 'codec_slave'/'codec_master' terms, which in turn
> come from alsa-lib and ALSA/ASoC topology support at the kernel
> level. These terms are no longer compatible with the guidelines
> adopted by the kernel community [2], standard organizations, and need
> to change in backwards-compatible ways.
> 
> The main/secondary terms typically suggested in guidelines don't mean
> anything for clocks, this patchset suggests instead the use of
> 'provider' and 'consumer' terms, with the 'codec' prefix kept to make
> it clear that the codec is the reference. The CM/CS suffixes are also
> replaced by CP/CC.
> 
> It can be argued that the change of suffix is invasive, but finding a
> replacement that keeps the M and S shortcuts has proven difficult in
> quite a few contexts.
> 
> The previous definitions are kept for backwards-compatibility so this
> change should not have any functional impact. It is suggested that new
> contributions only use the new terms but there is no requirement to
> transition immediately to the new definitions for existing code. Intel
> will however update all its past contributions related to bit
> clock/frame sync configurations immediately.
> 
> This patchset contains the alsa-lib changes only, the kernel changes
> will be shared in a different patchset. The SOF firmware changes are
> already applied, only topology-related changes will be modified after
> alsa-lib changes.
> 
> Feedback welcome
> ~Pierre
> 
> [1] https://mailman.alsa-project.org/pipermail/alsa-devel/2020-September/174387.html
> [2] https://lkml.org/lkml/2020/7/4/229
> 
> Changes since RFC:
> replaced 'follower' by 'consumer' as suggested by Jaroslav and Marc
> minor cleanups
> 
> Pierre-Louis Bossart (3):
>   topology: use inclusive language for bclk
>   topology: use inclusive language for fsync
>   topology: use inclusive language in documentation

Applied all three patches now.  Thanks.


Takashi


More information about the Alsa-devel mailing list