All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ASoC: rk817: Add module alias for rk817-codec
@ 2021-11-21 15:05 ` Nicolas Frattaroli
  0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Frattaroli @ 2021-11-21 15:05 UTC (permalink / raw)
  To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Chris Morgan, Lee Jones, alsa-devel, linux-kernel
  Cc: Nicolas Frattaroli, linux-rockchip

Without a module alias, autoloading the driver does not occurr
when it is built as a module.

By adding a module alias, the driver now probes fine automatically
and therefore analog audio output works as it should.

Fixes: 0d6a04da9b25 ("ASoC: Add Rockchip rk817 audio CODEC support")
Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com>
---
 sound/soc/codecs/rk817_codec.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sound/soc/codecs/rk817_codec.c b/sound/soc/codecs/rk817_codec.c
index 943d7d933e81..03f24edfe4f6 100644
--- a/sound/soc/codecs/rk817_codec.c
+++ b/sound/soc/codecs/rk817_codec.c
@@ -539,3 +539,4 @@ module_platform_driver(rk817_codec_driver);
 MODULE_DESCRIPTION("ASoC RK817 codec driver");
 MODULE_AUTHOR("binyuan <kevan.lan@rock-chips.com>");
 MODULE_LICENSE("GPL v2");
+MODULE_ALIAS("platform:rk817-codec");
-- 
2.34.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] ASoC: rk817: Add module alias for rk817-codec
@ 2021-11-21 15:05 ` Nicolas Frattaroli
  0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Frattaroli @ 2021-11-21 15:05 UTC (permalink / raw)
  To: Liam Girdwood, Mark Brown, Jaroslav Kysela, Takashi Iwai,
	Chris Morgan, Lee Jones, alsa-devel, linux-kernel
  Cc: Nicolas Frattaroli, linux-rockchip

Without a module alias, autoloading the driver does not occurr
when it is built as a module.

By adding a module alias, the driver now probes fine automatically
and therefore analog audio output works as it should.

Fixes: 0d6a04da9b25 ("ASoC: Add Rockchip rk817 audio CODEC support")
Signed-off-by: Nicolas Frattaroli <frattaroli.nicolas@gmail.com>
---
 sound/soc/codecs/rk817_codec.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sound/soc/codecs/rk817_codec.c b/sound/soc/codecs/rk817_codec.c
index 943d7d933e81..03f24edfe4f6 100644
--- a/sound/soc/codecs/rk817_codec.c
+++ b/sound/soc/codecs/rk817_codec.c
@@ -539,3 +539,4 @@ module_platform_driver(rk817_codec_driver);
 MODULE_DESCRIPTION("ASoC RK817 codec driver");
 MODULE_AUTHOR("binyuan <kevan.lan@rock-chips.com>");
 MODULE_LICENSE("GPL v2");
+MODULE_ALIAS("platform:rk817-codec");
-- 
2.34.0


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: rk817: Add module alias for rk817-codec
  2021-11-21 15:05 ` Nicolas Frattaroli
@ 2021-11-22 23:59   ` Mark Brown
  -1 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2021-11-22 23:59 UTC (permalink / raw)
  To: linux-kernel, Chris Morgan, Jaroslav Kysela, Lee Jones,
	Liam Girdwood, Nicolas Frattaroli, alsa-devel, Takashi Iwai
  Cc: linux-rockchip

On Sun, 21 Nov 2021 16:05:20 +0100, Nicolas Frattaroli wrote:
> Without a module alias, autoloading the driver does not occurr
> when it is built as a module.
> 
> By adding a module alias, the driver now probes fine automatically
> and therefore analog audio output works as it should.
> 
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: rk817: Add module alias for rk817-codec
      commit: 428ee30a05cd1362c8aa86a4c909b0d1c6bc48a4

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

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: rk817: Add module alias for rk817-codec
@ 2021-11-22 23:59   ` Mark Brown
  0 siblings, 0 replies; 6+ messages in thread
From: Mark Brown @ 2021-11-22 23:59 UTC (permalink / raw)
  To: linux-kernel, Chris Morgan, Jaroslav Kysela, Lee Jones,
	Liam Girdwood, Nicolas Frattaroli, alsa-devel, Takashi Iwai
  Cc: linux-rockchip

On Sun, 21 Nov 2021 16:05:20 +0100, Nicolas Frattaroli wrote:
> Without a module alias, autoloading the driver does not occurr
> when it is built as a module.
> 
> By adding a module alias, the driver now probes fine automatically
> and therefore analog audio output works as it should.
> 
> 
> [...]

Applied to

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

Thanks!

[1/1] ASoC: rk817: Add module alias for rk817-codec
      commit: 428ee30a05cd1362c8aa86a4c909b0d1c6bc48a4

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

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: rk817: Add module alias for rk817-codec
  2021-11-22 23:59   ` Mark Brown
@ 2021-11-23  0:38     ` Christopher Morgan
  -1 siblings, 0 replies; 6+ messages in thread
From: Christopher Morgan @ 2021-11-23  0:38 UTC (permalink / raw)
  To: Mark Brown
  Cc: linux-kernel, Jaroslav Kysela, Lee Jones, Liam Girdwood,
	Nicolas Frattaroli, alsa-devel, Takashi Iwai, linux-rockchip

I’m sure it’s okay, but I am out of town and will test this one next Sunday when I get back. Thank you!

> On Nov 22, 2021, at 5:59 PM, Mark Brown <broonie@kernel.org> wrote:
> 
> On Sun, 21 Nov 2021 16:05:20 +0100, Nicolas Frattaroli wrote:
>> Without a module alias, autoloading the driver does not occurr
>> when it is built as a module.
>> 
>> By adding a module alias, the driver now probes fine automatically
>> and therefore analog audio output works as it should.
>> 
>> 
>> [...]
> 
> Applied to
> 
>   https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fbroonie%2Fsound.git&amp;data=04%7C01%7C%7C10fc56dea69848ff206b08d9ae142feb%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637732223989143500%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=h9Ff1gI%2B89ziJH%2B%2BNAdRfRqXNfVzM1RKStD7UN3xK7Y%3D&amp;reserved=0 for-linus
> 
> Thanks!
> 
> [1/1] ASoC: rk817: Add module alias for rk817-codec
>      commit: 428ee30a05cd1362c8aa86a4c909b0d1c6bc48a4
> 
> 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
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: rk817: Add module alias for rk817-codec
@ 2021-11-23  0:38     ` Christopher Morgan
  0 siblings, 0 replies; 6+ messages in thread
From: Christopher Morgan @ 2021-11-23  0:38 UTC (permalink / raw)
  To: Mark Brown
  Cc: alsa-devel, Takashi Iwai, Liam Girdwood, linux-kernel,
	Nicolas Frattaroli, linux-rockchip, Lee Jones

I’m sure it’s okay, but I am out of town and will test this one next Sunday when I get back. Thank you!

> On Nov 22, 2021, at 5:59 PM, Mark Brown <broonie@kernel.org> wrote:
> 
> On Sun, 21 Nov 2021 16:05:20 +0100, Nicolas Frattaroli wrote:
>> Without a module alias, autoloading the driver does not occurr
>> when it is built as a module.
>> 
>> By adding a module alias, the driver now probes fine automatically
>> and therefore analog audio output works as it should.
>> 
>> 
>> [...]
> 
> Applied to
> 
>   https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fbroonie%2Fsound.git&amp;data=04%7C01%7C%7C10fc56dea69848ff206b08d9ae142feb%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C637732223989143500%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=h9Ff1gI%2B89ziJH%2B%2BNAdRfRqXNfVzM1RKStD7UN3xK7Y%3D&amp;reserved=0 for-linus
> 
> Thanks!
> 
> [1/1] ASoC: rk817: Add module alias for rk817-codec
>      commit: 428ee30a05cd1362c8aa86a4c909b0d1c6bc48a4
> 
> 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

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-11-23  0:39 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-21 15:05 [PATCH] ASoC: rk817: Add module alias for rk817-codec Nicolas Frattaroli
2021-11-21 15:05 ` Nicolas Frattaroli
2021-11-22 23:59 ` Mark Brown
2021-11-22 23:59   ` Mark Brown
2021-11-23  0:38   ` Christopher Morgan
2021-11-23  0:38     ` Christopher Morgan

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.