mbox series

[00/11] ASoC: topology: Fixes and cleanups

Message ID 20230127231111.937721-1-amadeuszx.slawinski@linux.intel.com (mailing list archive)
Headers show
Series ASoC: topology: Fixes and cleanups | expand

Message

Amadeusz Sławiński Jan. 27, 2023, 11:11 p.m. UTC
Following is series of fixes and cleanups for core topology code. Few
patches fixing various problems all around and few fixing function
names.

v2:
 - fix commit messages
 - change order of last two patches
 - as there is no code changes, add Reviewed-by from Ranjani

Amadeusz Sławiński (11):
  ASoC: topology: Properly access value coming from topology file
  ASoC: topology: Remove unused SOC_TPLG_PASS_PINS constant
  ASoC: topology: Fix typo in functions name
  ASoC: topology: Fix function name
  ASoC: topology: Rename remove_ handlers
  ASoC: topology: Remove unnecessary forward declarations
  ASoC: topology: Pass correct pointer instead of casting
  ASoC: topology: Return an error on complete() failure
  ASoC: Topology: Remove unnecessary check for EOF
  ASoC: topology: Unify kcontrol removal code
  ASoC: topology: Use unload() op directly

 include/sound/soc-topology.h |   2 +-
 sound/soc/soc-topology.c     | 183 ++++++++++++++---------------------
 2 files changed, 74 insertions(+), 111 deletions(-)

Comments

Jaroslav Kysela Jan. 27, 2023, 4:27 p.m. UTC | #1
On 28. 01. 23 0:11, Amadeusz Sławiński wrote:
> Following is series of fixes and cleanups for core topology code. Few
> patches fixing various problems all around and few fixing function
> names.

Just an info - the sent date seems to be in future in your git patch sets.

					Jaroslav
Amadeusz Sławiński Jan. 30, 2023, 1:08 p.m. UTC | #2
On 1/27/2023 5:27 PM, Jaroslav Kysela wrote:
> On 28. 01. 23 0:11, Amadeusz Sławiński wrote:
>> Following is series of fixes and cleanups for core topology code. Few
>> patches fixing various problems all around and few fixing function
>> names.
> 
> Just an info - the sent date seems to be in future in your git patch sets.
> 

Eh..., I'm not sure how it got out of sync. Anyway fixed. Thanks!
Amadeusz Sławiński Jan. 30, 2023, 1:28 p.m. UTC | #3
On 1/28/2023 12:11 AM, Amadeusz Sławiński wrote:
> Following is series of fixes and cleanups for core topology code. Few
> patches fixing various problems all around and few fixing function
> names.
> 
> v2:
>   - fix commit messages
>   - change order of last two patches
>   - as there is no code changes, add Reviewed-by from Ranjani
> 

And I've just realized that I forgot -v2 switch when sending... I hope I 
don't need to resend ;)

> Amadeusz Sławiński (11):
>    ASoC: topology: Properly access value coming from topology file
>    ASoC: topology: Remove unused SOC_TPLG_PASS_PINS constant
>    ASoC: topology: Fix typo in functions name
>    ASoC: topology: Fix function name
>    ASoC: topology: Rename remove_ handlers
>    ASoC: topology: Remove unnecessary forward declarations
>    ASoC: topology: Pass correct pointer instead of casting
>    ASoC: topology: Return an error on complete() failure
>    ASoC: Topology: Remove unnecessary check for EOF
>    ASoC: topology: Unify kcontrol removal code
>    ASoC: topology: Use unload() op directly
> 
>   include/sound/soc-topology.h |   2 +-
>   sound/soc/soc-topology.c     | 183 ++++++++++++++---------------------
>   2 files changed, 74 insertions(+), 111 deletions(-)
>
Mark Brown Jan. 30, 2023, 9 p.m. UTC | #4
On Sat, 28 Jan 2023 00:11:00 +0100, Amadeusz Sławiński wrote:
> Following is series of fixes and cleanups for core topology code. Few
> patches fixing various problems all around and few fixing function
> names.
> 
> v2:
>  - fix commit messages
>  - change order of last two patches
>  - as there is no code changes, add Reviewed-by from Ranjani
> 
> [...]

Applied to

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

Thanks!

[01/11] ASoC: topology: Properly access value coming from topology file
        commit: c5d184c92df2b631fb81fe2ce6e96bfc5ba720e5
[02/11] ASoC: topology: Remove unused SOC_TPLG_PASS_PINS constant
        commit: 6257d224b894b676540998ae9563c211410c9436
[03/11] ASoC: topology: Fix typo in functions name
        commit: 8f9974d9d767d11ce17280bec0d0f2e95e91954d
[04/11] ASoC: topology: Fix function name
        commit: 23e591dc0f8ce0298857a1445993fa7549a1f2e0
[05/11] ASoC: topology: Rename remove_ handlers
        commit: 2abfd4bd7b0700df4996ae2b60a12f22a0ef633d
[06/11] ASoC: topology: Remove unnecessary forward declarations
        commit: 70a7cd09a6368e0c9d351185a8fbfb3bae5a74f3
[07/11] ASoC: topology: Pass correct pointer instead of casting
        commit: 9e2ee00039a8ff236ae4db2366f4d2325658bea6
[08/11] ASoC: topology: Return an error on complete() failure
        commit: b784617a407c4f7e079e1694c3161ab29eb4bab1
[09/11] ASoC: Topology: Remove unnecessary check for EOF
        commit: d9b07b790a5c47dd4fd66c9264a3b38a103fa09b
[10/11] ASoC: topology: Unify kcontrol removal code
        commit: fdfa3661f830c98fb0f6380c3876fae33bc83b1d
[11/11] ASoC: topology: Use unload() op directly
        commit: 31e9273912bf5e4c23a876b5dfe0760fbecde92c

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