Re: [PATCH 0/6] sbus: Convert to platform remove callback returning void

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

 



On 2024-03-04 23:30, Uwe Kleine-König wrote:
> Hello,
> 
> this series converts all platform drivers below drivers/sbus/platform to
> struct platform_driver::remove_new(). See commit 5c5a7680e67b
> ("platform: Provide a remove callback that returns no value") for an
> extended explanation and the eventual goal.
> 
> The first commit is a build fix. The remaining patches implement that
> actual conversion. All conversations are trivial, because their
> .remove() callbacks returned zero unconditionally.
> 
> Apart from patch #2 building on top of #1, there are no
> interdependencies between these patches, so they could be picked up
> individually. But I'd hope that they get picked up all together by
> Andreas (or Dave?).
> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (6):
>   sbus: Add prototype for bbc_envctrl_init and bbc_envctrl_cleanup to
>     header
>   sbus: bbc_i2c: Convert to platform remove callback returning void
>   sbus: display7seg: Convert to platform remove callback returning void
>   sbus: envctrl: Convert to platform remove callback returning void
>   sbus: flash: Convert to platform remove callback returning void
>   sbus: uctrl: Convert to platform remove callback returning void
> 
>  drivers/sbus/char/bbc_i2c.c     | 9 ++-------
>  drivers/sbus/char/bbc_i2c.h     | 3 +++
>  drivers/sbus/char/display7seg.c | 6 ++----
>  drivers/sbus/char/envctrl.c     | 6 ++----
>  drivers/sbus/char/flash.c       | 6 ++----
>  drivers/sbus/char/uctrl.c       | 5 ++---
>  6 files changed, 13 insertions(+), 22 deletions(-)
> 
> base-commit: 67908bf6954b7635d33760ff6dfc189fc26ccc89

For the whole series,

Reviewed-by: Andreas Larsson <andreas@xxxxxxxxxxx>

Picking this up to my for-next.

Thanks,
Andreas




[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux