Re: [PATCH 0/6] bus: fsl-mc: Make remove function return void

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 10, 2023 at 11:41:22PM +0100, Uwe Kleine-König wrote:
> Hello,
> 
> many bus remove functions return an integer which is a historic
> misdesign that makes driver authors assume that there is some kind of
> error handling in the upper layers. This is wrong however and returning
> and error code only yields an error message.
> 
> This series improves the fsl-mc bus by changing the remove callback to
> return no value instead. As a preparation all drivers are changed to
> return zero before so that they don't trigger the error message.
> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (6):
>   bus: fsl-mc: Only warn once about errors on device unbind
>   bus: fsl-mc: dprc: Push down error message from fsl_mc_driver_remove()
>   bus: fsl-mc: fsl-mc-allocator: Drop if block with always wrong
>     condition
>   bus: fsl-mc: fsl-mc-allocator: Improve error reporting
>   soc: fsl: dpio: Suppress duplicated error reporting on device remove
>   bus: fsl-mc: Make remove function return void
> 

Reviewed-by: Ioana Ciornei <ioana.ciornei@xxxxxxx>
Tested-by: Ioana Ciornei <ioana.ciornei@xxxxxxx> # sanity checks





[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux