Linux Sound subsystem development
 help / color / mirror / Atom feed
* [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions
@ 2026-01-31  2:01 Ethan Nelson-Moore
  2026-01-31  6:47 ` Dmitry Baryshkov
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Ethan Nelson-Moore @ 2026-01-31  2:01 UTC (permalink / raw)
  To: linux-sound, linux-arm-msm
  Cc: Ethan Nelson-Moore, Srinivas Kandagatla, Liam Girdwood,
	Mark Brown, Jaroslav Kysela, Takashi Iwai

The wcd-mbhc-v2 driver has unnecessary empty module_init and
module_exit functions. Remove them. Note that if a module_init function
exists, a module_exit function must also exist; otherwise, the module
cannot be unloaded.

Signed-off-by: Ethan Nelson-Moore <enelsonmoore@gmail.com>
---
 sound/soc/codecs/wcd-mbhc-v2.c | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/sound/soc/codecs/wcd-mbhc-v2.c b/sound/soc/codecs/wcd-mbhc-v2.c
index 26ebcdadeb7d..0c842aaa7eec 100644
--- a/sound/soc/codecs/wcd-mbhc-v2.c
+++ b/sound/soc/codecs/wcd-mbhc-v2.c
@@ -1631,17 +1631,5 @@ void wcd_mbhc_deinit(struct wcd_mbhc *mbhc)
 }
 EXPORT_SYMBOL(wcd_mbhc_deinit);
 
-static int __init mbhc_init(void)
-{
-	return 0;
-}
-
-static void __exit mbhc_exit(void)
-{
-}
-
-module_init(mbhc_init);
-module_exit(mbhc_exit);
-
 MODULE_DESCRIPTION("wcd MBHC v2 module");
 MODULE_LICENSE("GPL");
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions
  2026-01-31  2:01 [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions Ethan Nelson-Moore
@ 2026-01-31  6:47 ` Dmitry Baryshkov
  2026-02-02 12:19 ` Mark Brown
  2026-02-02 23:33 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Dmitry Baryshkov @ 2026-01-31  6:47 UTC (permalink / raw)
  To: Ethan Nelson-Moore
  Cc: linux-sound, linux-arm-msm, Srinivas Kandagatla, Liam Girdwood,
	Mark Brown, Jaroslav Kysela, Takashi Iwai

On Fri, Jan 30, 2026 at 06:01:10PM -0800, Ethan Nelson-Moore wrote:
> The wcd-mbhc-v2 driver has unnecessary empty module_init and
> module_exit functions. Remove them. Note that if a module_init function
> exists, a module_exit function must also exist; otherwise, the module
> cannot be unloaded.
> 
> Signed-off-by: Ethan Nelson-Moore <enelsonmoore@gmail.com>
> ---
>  sound/soc/codecs/wcd-mbhc-v2.c | 12 ------------
>  1 file changed, 12 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>

> 

-- 
With best wishes
Dmitry

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions
  2026-01-31  2:01 [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions Ethan Nelson-Moore
  2026-01-31  6:47 ` Dmitry Baryshkov
@ 2026-02-02 12:19 ` Mark Brown
  2026-02-02 23:33 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2026-02-02 12:19 UTC (permalink / raw)
  To: Ethan Nelson-Moore
  Cc: linux-sound, linux-arm-msm, Srinivas Kandagatla, Liam Girdwood,
	Jaroslav Kysela, Takashi Iwai

[-- Attachment #1: Type: text/plain, Size: 630 bytes --]

On Fri, Jan 30, 2026 at 06:01:10PM -0800, Ethan Nelson-Moore wrote:
> The wcd-mbhc-v2 driver has unnecessary empty module_init and
> module_exit functions. Remove them. Note that if a module_init function
> exists, a module_exit function must also exist; otherwise, the module
> cannot be unloaded.

Please submit patches using subject lines reflecting the style for the
subsystem, this makes it easier for people to identify relevant patches.
Look at what existing commits in the area you're changing are doing and
make sure your subject lines visually resemble what they're doing.
There's no need to resubmit to fix this alone.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions
  2026-01-31  2:01 [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions Ethan Nelson-Moore
  2026-01-31  6:47 ` Dmitry Baryshkov
  2026-02-02 12:19 ` Mark Brown
@ 2026-02-02 23:33 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2026-02-02 23:33 UTC (permalink / raw)
  To: linux-sound, linux-arm-msm, Ethan Nelson-Moore
  Cc: Srinivas Kandagatla, Liam Girdwood, Jaroslav Kysela, Takashi Iwai

On Fri, 30 Jan 2026 18:01:10 -0800, Ethan Nelson-Moore wrote:
> The wcd-mbhc-v2 driver has unnecessary empty module_init and
> module_exit functions. Remove them. Note that if a module_init function
> exists, a module_exit function must also exist; otherwise, the module
> cannot be unloaded.
> 
> 

Applied to

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

Thanks!

[1/1] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions
      commit: 090c8844e5d454a898183ec4a7be8a75681ed262

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] 4+ messages in thread

end of thread, other threads:[~2026-02-02 23:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-31  2:01 [PATCH] sound: wcd-mbhc-v2: remove unnecessary module_init/exit functions Ethan Nelson-Moore
2026-01-31  6:47 ` Dmitry Baryshkov
2026-02-02 12:19 ` Mark Brown
2026-02-02 23:33 ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox