ALSA: aoa: make remove callback of soundbus driver void returned
authorDawei Li <set_pte_at@outlook.com>
Fri, 6 Jan 2023 15:17:46 +0000 (23:17 +0800)
committerTakashi Iwai <tiwai@suse.de>
Wed, 25 Jan 2023 08:34:29 +0000 (09:34 +0100)
Since commit fc7a6209d571 ("bus: Make remove callback return void")
forces bus_type::remove be void-returned, it doesn't make much sense
for any bus based driver implementing remove callbalk to return
non-void to its caller.

As such, change the remove function for soundbus based drivers to
return void.

Signed-off-by: Dawei Li <set_pte_at@outlook.com>
Link: https://lore.kernel.org/r/TYCP286MB23234FED40A3AE6797DEBAB7CAFB9@TYCP286MB2323.JPNP286.PROD.OUTLOOK.COM
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/aoa/fabrics/layout.c
sound/aoa/soundbus/soundbus.h

index ec4ef18..850dc8c 100644 (file)
@@ -1094,7 +1094,7 @@ static int aoa_fabric_layout_probe(struct soundbus_dev *sdev)
        return -ENODEV;
 }
 
-static int aoa_fabric_layout_remove(struct soundbus_dev *sdev)
+static void aoa_fabric_layout_remove(struct soundbus_dev *sdev)
 {
        struct layout_dev *ldev = dev_get_drvdata(&sdev->ofdev.dev);
        int i;
@@ -1123,7 +1123,6 @@ static int aoa_fabric_layout_remove(struct soundbus_dev *sdev)
        kfree(ldev);
        sdev->pcmid = -1;
        sdev->pcmname = NULL;
-       return 0;
 }
 
 #ifdef CONFIG_PM_SLEEP
index 3a99c1f..db40f9d 100644 (file)
@@ -185,7 +185,7 @@ struct soundbus_driver {
        /* we don't implement any matching at all */
 
        int     (*probe)(struct soundbus_dev* dev);
-       int     (*remove)(struct soundbus_dev* dev);
+       void    (*remove)(struct soundbus_dev *dev);
 
        int     (*shutdown)(struct soundbus_dev* dev);