mbox series

[00/44] ASoC: sync parameter naming : rate / sample_bits

Message ID 87zh1aolkt.wl-kuninori.morimoto.gx@renesas.com
Headers show
Series ASoC: sync parameter naming : rate / sample_bits | expand

Message

Kuninori Morimoto Jan. 15, 2021, 4:51 a.m. UTC
Hi Mark

snd_pcm_runtime / snd_soc_dai / snd_soc_dai_driver / snd_soc_dai_link
have related parameter which is similar but not same naming.

	struct snd_pcm_runtime {
		...
(A)		unsigned int rate;
		...
(B)		unsigned int sample_bits;
		...
	};

	struct snd_soc_dai {
		...
(A)		unsigned int rate;
(B)		unsigned int sample_bits;
		...
	};

	struct snd_soc_dai_driver {
		...
(A)		unsigned int symmetric_rates:1;
(B)		unsigned int symmetric_samplebits:1;
		...
	};

	struct snd_soc_dai_link {
		...
(A)		unsigned int symmetric_rates:1;
(B)		unsigned int symmetric_samplebits:1;
		...
	};

Because it is similar but not same naming rule,
code can be verbose / can't share macro.

This patch-set sync naming rule
to struct snd_pcm_runtime base.

	- xxx_rates;
	+ xxx_rate;

	- xxx_samplebits;
	+ xxx_sample_bits;

Kuninori Morimoto (44):
  ASoC: soc-pcm: revert soc_pcm_apply_symmetry()
  ASoC: sync parameter naming : rate / sample_bits
  ASoC: adi: sync parameter naming (rate/sample_bits)
  ASoC: atmel: sync parameter naming (rate/sample_bits)
  ASoC: au1x: sync parameter naming (rate/sample_bits)
  ASoC: bcm: sync parameter naming (rate/sample_bits)
  ASoC: cirrus: sync parameter naming (rate/sample_bits)
  ASoC: tegra: sync parameter naming (rate/sample_bits)
  ASoC: rockchip: sync parameter naming (rate/sample_bits)
  ASoC: samsung: sync parameter naming (rate/sample_bits)
  ASoC: sh: sync parameter naming (rate/sample_bits)
  ASoC: ti: sync parameter naming (rate/sample_bits)
  ASoC: pxa: sync parameter naming (rate/sample_bits)
  ASoC: mediatek: sync parameter naming (rate/sample_bits)
  ASoC: fsl: sync parameter naming (rate/sample_bits)
  ASoC: wm*: sync parameter naming (rate/sample_bits)
  ASoC: tlv*: sync parameter naming (rate/sample_bits)
  ASoC: rt*: sync parameter naming (rate/sample_bits)
  ASoC: nau*: sync parameter naming (rate/sample_bits)
  ASoC: tas*: sync parameter naming (rate/sample_bits)
  ASoC: da*: sync parameter naming (rate/sample_bits)
  ASoC: es*: sync parameter naming (rate/sample_bits)
  ASoC: max*: sync parameter naming (rate/sample_bits)
  ASoC: cs*: sync parameter naming (rate/sample_bits)
  ASoC: tscs*: sync parameter naming (rate/sample_bits)
  ASoC: ak*: sync parameter naming (rate/sample_bits)
  ASoC: adau*: sync parameter naming (rate/sample_bits)
  ASoC: jz4740: sync parameter naming (rate/sample_bits)
  ASoC: sunxi: sync parameter naming (rate/sample_bits)
  ASoC: zl38060: sync parameter naming (rate/sample_bits)
  ASoC: ssm2602: sync parameter naming (rate/sample_bits)
  ASoC: sgtl5000: sync parameter naming (rate/sample_bits)
  ASoC: mc13783: sync parameter naming (rate/sample_bits)
  ASoC: mt6660: sync parameter naming (rate/sample_bits)
  ASoC: lochnagar-sc: sync parameter naming (rate/sample_bits)
  ASoC: ab8500: sync parameter naming (rate/sample_bits)
  ASoC: alc5632: sync parameter naming (rate/sample_bits)
  ASoC: cx2072x: sync parameter naming (rate/sample_bits)
  ASoC: inno_rk3036: sync parameter naming (rate/sample_bits)
  ASoC: lm49453: sync parameter naming (rate/sample_bits)
  ASoC: ml26124: sync parameter naming (rate/sample_bits)
  ASoC: soc-dai.h: remove symmetric_rates/samplebits
  ASoC: soc-pcm: cleanup soc_pcm_apply_symmetry()
  ASoC: soc-pcm: cleanup soc_pcm_params_symmetry()

 include/sound/soc-dai.h                    |  4 +-
 include/sound/soc.h                        |  4 +-
 sound/soc/adi/axi-i2s.c                    |  2 +-
 sound/soc/atmel/atmel-i2s.c                |  2 +-
 sound/soc/atmel/mchp-i2s-mcc.c             |  4 +-
 sound/soc/au1x/i2sc.c                      |  2 +-
 sound/soc/bcm/bcm2835-i2s.c                |  4 +-
 sound/soc/bcm/bcm63xx-i2s-whistler.c       |  2 +-
 sound/soc/cirrus/ep93xx-i2s.c              |  2 +-
 sound/soc/codecs/ab8500-codec.c            |  4 +-
 sound/soc/codecs/adau1372.c                |  2 +-
 sound/soc/codecs/adau1373.c                |  6 +-
 sound/soc/codecs/adau1701.c                |  2 +-
 sound/soc/codecs/ak4554.c                  |  2 +-
 sound/soc/codecs/ak4613.c                  |  2 +-
 sound/soc/codecs/ak4641.c                  |  4 +-
 sound/soc/codecs/ak4642.c                  |  2 +-
 sound/soc/codecs/alc5632.c                 |  2 +-
 sound/soc/codecs/cs35l32.c                 |  2 +-
 sound/soc/codecs/cs35l33.c                 |  2 +-
 sound/soc/codecs/cs35l34.c                 |  2 +-
 sound/soc/codecs/cs35l35.c                 |  2 +-
 sound/soc/codecs/cs35l36.c                 |  2 +-
 sound/soc/codecs/cs4234.c                  |  2 +-
 sound/soc/codecs/cs4271.c                  |  2 +-
 sound/soc/codecs/cs42l73.c                 |  6 +-
 sound/soc/codecs/cs43130.c                 |  6 +-
 sound/soc/codecs/cs4341.c                  |  2 +-
 sound/soc/codecs/cs4349.c                  |  2 +-
 sound/soc/codecs/cs47l15.c                 | 12 ++--
 sound/soc/codecs/cs47l24.c                 | 12 ++--
 sound/soc/codecs/cs47l35.c                 | 12 ++--
 sound/soc/codecs/cs47l85.c                 | 16 ++---
 sound/soc/codecs/cs47l90.c                 | 16 ++---
 sound/soc/codecs/cs47l92.c                 | 12 ++--
 sound/soc/codecs/cs53l30.c                 |  2 +-
 sound/soc/codecs/cx2072x.c                 |  2 +-
 sound/soc/codecs/da7210.c                  |  2 +-
 sound/soc/codecs/da7213.c                  |  2 +-
 sound/soc/codecs/da7218.c                  |  4 +-
 sound/soc/codecs/da7219.c                  |  4 +-
 sound/soc/codecs/da9055.c                  |  2 +-
 sound/soc/codecs/es8316.c                  |  2 +-
 sound/soc/codecs/es8328.c                  |  2 +-
 sound/soc/codecs/inno_rk3036.c             |  2 +-
 sound/soc/codecs/jz4740.c                  |  2 +-
 sound/soc/codecs/lm49453.c                 |  2 +-
 sound/soc/codecs/lochnagar-sc.c            | 12 ++--
 sound/soc/codecs/max9860.c                 |  2 +-
 sound/soc/codecs/max9867.c                 |  2 +-
 sound/soc/codecs/mc13783.c                 |  2 +-
 sound/soc/codecs/ml26124.c                 |  2 +-
 sound/soc/codecs/mt6660.c                  |  4 +-
 sound/soc/codecs/nau8810.c                 |  2 +-
 sound/soc/codecs/nau8822.c                 |  2 +-
 sound/soc/codecs/rt274.c                   |  2 +-
 sound/soc/codecs/rt286.c                   |  4 +-
 sound/soc/codecs/rt298.c                   |  4 +-
 sound/soc/codecs/rt5670.c                  |  4 +-
 sound/soc/codecs/sgtl5000.c                |  2 +-
 sound/soc/codecs/ssm2602.c                 |  4 +-
 sound/soc/codecs/tas2764.c                 |  2 +-
 sound/soc/codecs/tas2770.c                 |  2 +-
 sound/soc/codecs/tlv320adcx140.c           |  2 +-
 sound/soc/codecs/tlv320aic31xx.c           |  4 +-
 sound/soc/codecs/tlv320aic32x4.c           |  2 +-
 sound/soc/codecs/tlv320aic3x.c             |  2 +-
 sound/soc/codecs/tscs42xx.c                |  4 +-
 sound/soc/codecs/tscs454.c                 | 12 ++--
 sound/soc/codecs/wm5102.c                  | 12 ++--
 sound/soc/codecs/wm5110.c                  | 12 ++--
 sound/soc/codecs/wm8510.c                  |  2 +-
 sound/soc/codecs/wm8731.c                  |  2 +-
 sound/soc/codecs/wm8770.c                  |  2 +-
 sound/soc/codecs/wm8804.c                  |  2 +-
 sound/soc/codecs/wm8903.c                  |  2 +-
 sound/soc/codecs/wm8904.c                  |  2 +-
 sound/soc/codecs/wm8940.c                  |  2 +-
 sound/soc/codecs/wm8960.c                  |  2 +-
 sound/soc/codecs/wm8962.c                  |  2 +-
 sound/soc/codecs/wm8974.c                  |  2 +-
 sound/soc/codecs/wm8978.c                  |  2 +-
 sound/soc/codecs/wm8983.c                  |  2 +-
 sound/soc/codecs/wm8985.c                  |  2 +-
 sound/soc/codecs/wm8988.c                  |  2 +-
 sound/soc/codecs/wm8993.c                  |  2 +-
 sound/soc/codecs/wm8994.c                  |  4 +-
 sound/soc/codecs/wm8997.c                  |  8 +--
 sound/soc/codecs/wm8998.c                  | 12 ++--
 sound/soc/codecs/wm9713.c                  |  2 +-
 sound/soc/codecs/zl38060.c                 |  4 +-
 sound/soc/fsl/fsl_esai.c                   |  4 +-
 sound/soc/fsl/fsl_sai.c                    |  8 +--
 sound/soc/fsl/fsl_ssi.c                    |  4 +-
 sound/soc/jz4740/jz4740-i2s.c              |  2 +-
 sound/soc/mediatek/mt2701/mt2701-afe-pcm.c | 10 +--
 sound/soc/mediatek/mt6797/mt6797-dai-pcm.c |  8 +--
 sound/soc/mediatek/mt8173/mt8173-afe-pcm.c |  2 +-
 sound/soc/mediatek/mt8183/mt8183-dai-pcm.c |  8 +--
 sound/soc/mediatek/mt8192/mt8192-dai-pcm.c |  8 +--
 sound/soc/pxa/pxa2xx-i2s.c                 |  2 +-
 sound/soc/rockchip/rockchip_i2s.c          |  4 +-
 sound/soc/rockchip/rockchip_pdm.c          |  2 +-
 sound/soc/samsung/i2s.c                    |  2 +-
 sound/soc/samsung/pcm.c                    |  2 +-
 sound/soc/sh/rcar/core.c                   |  4 +-
 sound/soc/soc-pcm.c                        | 80 +++++++---------------
 sound/soc/soc-topology.c                   |  8 +--
 sound/soc/sunxi/sun4i-i2s.c                |  2 +-
 sound/soc/sunxi/sun8i-codec.c              | 12 ++--
 sound/soc/tegra/tegra186_dspk.c            |  2 +-
 sound/soc/tegra/tegra20_i2s.c              |  2 +-
 sound/soc/tegra/tegra210_dmic.c            |  2 +-
 sound/soc/tegra/tegra210_i2s.c             |  2 +-
 sound/soc/tegra/tegra30_i2s.c              |  2 +-
 sound/soc/ti/davinci-mcasp.c               |  2 +-
 116 files changed, 253 insertions(+), 283 deletions(-)

Comments

Mark Brown Jan. 21, 2021, 7:39 p.m. UTC | #1
On 15 Jan 2021 13:51:56 +0900, Kuninori Morimoto wrote:
> snd_pcm_runtime / snd_soc_dai / snd_soc_dai_driver / snd_soc_dai_link
> have related parameter which is similar but not same naming.
> 
> 	struct snd_pcm_runtime {
> 		...
> (A)		unsigned int rate;
> 		...
> (B)		unsigned int sample_bits;
> 		...
> 	};
> 
> [...]

Applied to

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

Thanks!

[01/44] ASoC: soc-pcm: revert soc_pcm_apply_symmetry()
        commit: 80f454e283708c28f5220eda68511fc495e5ffaf
[02/44] ASoC: sync parameter naming : rate / sample_bits
        commit: f14654ddf2e982537ab476d392a87fcbf90374c3
[03/44] ASoC: adi: sync parameter naming (rate/sample_bits)
        commit: 55d0056ba6a1bc8be2ae804cadf106ff2ae712ca
[04/44] ASoC: atmel: sync parameter naming (rate/sample_bits)
        commit: ba471f8d1655753bd1fad226a1c47ffc4568c02e
[05/44] ASoC: au1x: sync parameter naming (rate/sample_bits)
        commit: 3470c2398ce9f6e80a5434c1c8649b036544a1d4
[06/44] ASoC: bcm: sync parameter naming (rate/sample_bits)
        commit: a8e940222bdf60b9957515e80064b1230a3fc774
[07/44] ASoC: cirrus: sync parameter naming (rate/sample_bits)
        commit: f1e6b04baba690e3bd08669c73e8d78032c2b905
[08/44] ASoC: tegra: sync parameter naming (rate/sample_bits)
        commit: c6d152a8de560f50a37e9580b7550dc2001eaccf
[09/44] ASoC: rockchip: sync parameter naming (rate/sample_bits)
        commit: fadaed3023138197000d37c2fafe15b2208ec9bc
[10/44] ASoC: samsung: sync parameter naming (rate/sample_bits)
        commit: d11ff0bf6ac1a85fcf51f4ca52934b0fb6ac47a6
[11/44] ASoC: sh: sync parameter naming (rate/sample_bits)
        commit: f1cd5f3bbcef13980734ea18797f192c344d850e
[12/44] ASoC: ti: sync parameter naming (rate/sample_bits)
        commit: c756316f0027a8659562d7df76fb24a9ff7c2068
[13/44] ASoC: pxa: sync parameter naming (rate/sample_bits)
        commit: c658b2185818ac39ca5b71001f30ab14a517219a
[14/44] ASoC: mediatek: sync parameter naming (rate/sample_bits)
        commit: 4b7ead032ecd1b1af5ce0dfbc2479cfdee7063b3
[15/44] ASoC: fsl: sync parameter naming (rate/sample_bits)
        commit: cb2f6927faf44d0cd3e55d3f9a9211f2aad26850
[16/44] ASoC: wm*: sync parameter naming (rate/sample_bits)
        commit: 0769575294e4c965cc2a4f4817fa696e20c2c05f
[17/44] ASoC: tlv*: sync parameter naming (rate/sample_bits)
        commit: a9aef184f7b47c355e4d78aa08792ff74fd5c4f5
[18/44] ASoC: rt*: sync parameter naming (rate/sample_bits)
        commit: bc03b3910ef6e228ef8b192e08030a1e177ad273
[19/44] ASoC: nau*: sync parameter naming (rate/sample_bits)
        commit: 42c9b5bec1c5218b33eda3b519381b38b89d5ccd
[20/44] ASoC: tas*: sync parameter naming (rate/sample_bits)
        commit: fa056c07367e443271837c8500de8c7e6d9b2722
[21/44] ASoC: da*: sync parameter naming (rate/sample_bits)
        commit: 1c6d1c4f6874345a31d12a9f3843876c49d637fa
[22/44] ASoC: es*: sync parameter naming (rate/sample_bits)
        commit: 4060064e8baf047a64b6286f5fd87cb0a34d8b89
[23/44] ASoC: max*: sync parameter naming (rate/sample_bits)
        commit: cb40d1b4581c0f6e2a828ebfcb7cc9a04dca1e11
[24/44] ASoC: cs*: sync parameter naming (rate/sample_bits)
        commit: 260b668cf356dbab5f73e651cfc96a90ed220b98
[25/44] ASoC: tscs*: sync parameter naming (rate/sample_bits)
        commit: 9b4488efdeccbe1d4dceb13aee9e58371f6ba926
[26/44] ASoC: ak*: sync parameter naming (rate/sample_bits)
        commit: 870b76ea7d0c0e5d6e5f305d8eca9ba45b89241a
[27/44] ASoC: adau*: sync parameter naming (rate/sample_bits)
        commit: f88f45861bbd5da303a997f92800fa0d7958c929
[28/44] ASoC: jz4740: sync parameter naming (rate/sample_bits)
        commit: 0547dece8dcbb80983b3c37ad20ceca76a1f06a5
[29/44] ASoC: sunxi: sync parameter naming (rate/sample_bits)
        commit: 813857086191aace3a39c056c8acb6946ff75f03
[30/44] ASoC: zl38060: sync parameter naming (rate/sample_bits)
        commit: b33c088a9b0b15df987dae05177ef9b0816dc4cf
[31/44] ASoC: ssm2602: sync parameter naming (rate/sample_bits)
        commit: 4735c8cadd54ef74726718ad93a27d16b23089b0
[32/44] ASoC: sgtl5000: sync parameter naming (rate/sample_bits)
        commit: bea17630c4ad398b6cd9b454c1649a962a7e0258
[33/44] ASoC: mc13783: sync parameter naming (rate/sample_bits)
        commit: 23df7f69ff134d108ddffcde622b7ffc904ac789
[34/44] ASoC: mt6660: sync parameter naming (rate/sample_bits)
        commit: 852d4daa9689188f25cd1aa92d4daf3a88d73101
[35/44] ASoC: lochnagar-sc: sync parameter naming (rate/sample_bits)
        commit: 65ba325625e74f44fe7c0052a85bcdfac2b012ec
[36/44] ASoC: ab8500: sync parameter naming (rate/sample_bits)
        commit: 6058bcb4974418bdec55738383755b83e57e555e
[37/44] ASoC: alc5632: sync parameter naming (rate/sample_bits)
        commit: 278eacf3dc717f27463d56d605c2ea919e308e49
[38/44] ASoC: cx2072x: sync parameter naming (rate/sample_bits)
        commit: 15d27c2b129fc8234ed709bdcbd17fa3fff0808e
[39/44] ASoC: inno_rk3036: sync parameter naming (rate/sample_bits)
        commit: d5d396c8b01e1fea696c68d255c2d7206de6499f
[40/44] ASoC: lm49453: sync parameter naming (rate/sample_bits)
        commit: 3b940ecd2ebfdd9a2b4a7a97aef1f25d1844d3ba
[41/44] ASoC: ml26124: sync parameter naming (rate/sample_bits)
        commit: 6944db0ab8c4b7c19b7ff9fbbe98322efcd9595a
[42/44] ASoC: soc-dai.h: remove symmetric_rates/samplebits
        commit: fa31a2c787aeaf61d02b2a57bd9765ca5e67d949
[43/44] ASoC: soc-pcm: cleanup soc_pcm_apply_symmetry()
        commit: fac110cbcddec115c7d4b79d7491aebebbe44469
[44/44] ASoC: soc-pcm: cleanup soc_pcm_params_symmetry()
        commit: 3a906721112263f1698d7752f6f0133c78fc12c2

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