[PATCH] bus: sunxi-rsb: make remove callback return void
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Thu Nov 26 05:35:01 EST 2020
On Thu, Nov 26, 2020 at 08:03:16AM +0000, Lee Jones wrote:
> On Thu, 26 Nov 2020, Chen-Yu Tsai wrote:
>
> > On Thu, Nov 26, 2020 at 3:41 PM Uwe Kleine-König
> > <u.kleine-koenig at pengutronix.de> wrote:
> > >
> > > The driver core ignores the return value of struct device_driver::remove
> > > because there is only little that can be done. To simplify the quest to
> > > make this function return void, let struct sunxi_rsb_driver::remove
> > > return void, too. All users already unconditionally return 0, this
> > > commit makes this obvious and ensures future users don't behave
> > > differently. To simplify even further, make axp20x_device_remove()
> > > return void instead of returning 0 unconditionally, too.
> > >
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> >
> > Looks good to me.
> >
> > Reviewed-by: Chen-Yu Tsai <wens at csie.org>
> >
> > Since this touches the mfd tree as well, we either need an Ack from Lee,
> > the mfd maintainer (CC-ed), or let Lee take it in his tree. For the latter,
>
> Right. Thank you Chen-Yu.
>
> Uwe, did you run get_maintainer.pl?
I thought I did, but obviously I somehow failed. (I have an idea what
went wrong, but I spare you the details.)
> Please re-submit this with the proper people on Cc.
sure, sorry for my confusion.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20201126/c2657c7b/attachment.sig>
More information about the linux-arm-kernel
mailing list