Re: [PATCH v1 1/1] mux: mux-core: Unregister mux_class in mux_exit()

From: Peter Rosin
Date: Sat Jul 08 2017 - 16:57:55 EST


On 2017-07-07 23:50, sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx wrote:
> From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx>
>
> Signed-off-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx>

Good catch!

However, I do not like the empty commit message, please add a sentence
about how this fixes a typo or something, and also please add this tag:

Fixes: a3b02a9c6591 ("mux: minimal mux subsystem")

Cheers,
peda

> ---
> drivers/mux/mux-core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mux/mux-core.c b/drivers/mux/mux-core.c
> index 90b8995..2fe96c4 100644
> --- a/drivers/mux/mux-core.c
> +++ b/drivers/mux/mux-core.c
> @@ -46,7 +46,7 @@ static int __init mux_init(void)
>
> static void __exit mux_exit(void)
> {
> - class_register(&mux_class);
> + class_unregister(&mux_class);
> ida_destroy(&mux_ida);
> }
>
>