Thread (1 message) 1 message, 1 author, 2021-07-14

Re: [PATCH v4 5/5] bus: Make remove callback return void

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2021-07-14 08:44:28
Also in: alsa-devel, dmaengine, kvm, linux-acpi, linux-arm-msm, linux-cxl, linux-fpga, linux-hyperv, linux-i2c, linux-i3c, linux-input, linux-m68k, linux-media, linux-mips, linux-mmc, linux-pci, linux-remoteproc, linux-s390, linux-scsi, linux-serial, linux-sh, linux-spi, linux-staging, linux-sunxi, linux-usb, linux-wireless, linuxppc-dev, lkml, netdev, nvdimm, platform-driver-x86, sparclinux, target-devel, xen-devel

On Tue, Jul 13, 2021 at 9:35 PM Uwe Kleine-König
[off-list ref] wrote:
The driver core ignores the return value of this callback because there
is only little it can do when a device disappears.

This is the final bit of a long lasting cleanup quest where several
buses were converted to also return void from their remove callback.
Additionally some resource leaks were fixed that were caused by drivers
returning an error code in the expectation that the driver won't go
away.

With struct bus_type::remove returning void it's prevented that newly
implemented buses return an ignored error code and so don't anticipate
wrong expectations for driver authors.
 drivers/zorro/zorro-driver.c              | 3 +--
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help