mbox series

[00/13] fpga: Convert to platform remove callback returning void

Message ID cover.1703006638.git.u.kleine-koenig@pengutronix.de (mailing list archive)
Headers show
Series fpga: Convert to platform remove callback returning void | expand

Message

Uwe Kleine-König Dec. 19, 2023, 5:31 p.m. UTC
Hello,

this series converts all drivers below drivers/fpga to use
.remove_new(). See commit 5c5a7680e67b ("platform: Provide a remove
callback that returns no value") for an extended explanation and the
eventual goal. The TL;DR; is to make it harder for driver authors to
leak resources without noticing.

This is merge window material. All patches are pairwise independent of
each other, so they could be applied individually. Getting them all in
together would be nicer though :-)

Best regards
Uwe

Uwe Kleine-König (13):
  fpga: altera-fpga2sdram: Convert to platform remove callback returning void
  fpga: altera-freeze-bridge: Convert to platform remove callback returning void
  fpga: altera-hps2fpga: Convert to platform remove callback returning void
  fpga: dfl-afu-main: Convert to platform remove callback returning void
  fpga: dfl-fme-br: Convert to platform remove callback returning void
  fpga: dfl-fme-main: Convert to platform remove callback returning void
  fpga: dfl-fme-region: Convert to platform remove callback returning void
  fpga: intel-m10-bmc-sec-update: Convert to platform remove callback returning void
  fpga: of-fpga-region: Convert to platform remove callback returning void
  fpga: socfpga-a10: Convert to platform remove callback returning void
  fpga: stratix10-soc: Convert to platform remove callback returning void
  fpga: xilinx-pr-decoupler: Convert to platform remove callback returning void
  fpga: zynq-fpga: Convert to platform remove callback returning void

 drivers/fpga/altera-fpga2sdram.c        | 6 ++----
 drivers/fpga/altera-freeze-bridge.c     | 6 ++----
 drivers/fpga/altera-hps2fpga.c          | 6 ++----
 drivers/fpga/dfl-afu-main.c             | 6 ++----
 drivers/fpga/dfl-fme-br.c               | 6 ++----
 drivers/fpga/dfl-fme-main.c             | 6 ++----
 drivers/fpga/dfl-fme-region.c           | 6 ++----
 drivers/fpga/intel-m10-bmc-sec-update.c | 6 ++----
 drivers/fpga/of-fpga-region.c           | 6 ++----
 drivers/fpga/socfpga-a10.c              | 6 ++----
 drivers/fpga/stratix10-soc.c            | 6 ++----
 drivers/fpga/xilinx-pr-decoupler.c      | 6 ++----
 drivers/fpga/zynq-fpga.c                | 6 ++----
 13 files changed, 26 insertions(+), 52 deletions(-)

base-commit: aa4db8324c4d0e67aa4670356df4e9fae14b4d37

Comments

Xu Yilun Dec. 25, 2023, 6:01 a.m. UTC | #1
On Tue, Dec 19, 2023 at 06:31:58PM +0100, Uwe Kleine-König wrote:
> Hello,
> 
> this series converts all drivers below drivers/fpga to use
> .remove_new(). See commit 5c5a7680e67b ("platform: Provide a remove
> callback that returns no value") for an extended explanation and the
> eventual goal. The TL;DR; is to make it harder for driver authors to
> leak resources without noticing.
> 
> This is merge window material. All patches are pairwise independent of
> each other, so they could be applied individually. Getting them all in
> together would be nicer though :-)
> 
> Best regards
> Uwe
> 
> Uwe Kleine-König (13):
>   fpga: altera-fpga2sdram: Convert to platform remove callback returning void
>   fpga: altera-freeze-bridge: Convert to platform remove callback returning void
>   fpga: altera-hps2fpga: Convert to platform remove callback returning void
>   fpga: dfl-afu-main: Convert to platform remove callback returning void
>   fpga: dfl-fme-br: Convert to platform remove callback returning void
>   fpga: dfl-fme-main: Convert to platform remove callback returning void
>   fpga: dfl-fme-region: Convert to platform remove callback returning void
>   fpga: intel-m10-bmc-sec-update: Convert to platform remove callback returning void
>   fpga: of-fpga-region: Convert to platform remove callback returning void
>   fpga: socfpga-a10: Convert to platform remove callback returning void
>   fpga: stratix10-soc: Convert to platform remove callback returning void
>   fpga: xilinx-pr-decoupler: Convert to platform remove callback returning void
>   fpga: zynq-fpga: Convert to platform remove callback returning void
> 
>  drivers/fpga/altera-fpga2sdram.c        | 6 ++----
>  drivers/fpga/altera-freeze-bridge.c     | 6 ++----
>  drivers/fpga/altera-hps2fpga.c          | 6 ++----
>  drivers/fpga/dfl-afu-main.c             | 6 ++----
>  drivers/fpga/dfl-fme-br.c               | 6 ++----
>  drivers/fpga/dfl-fme-main.c             | 6 ++----
>  drivers/fpga/dfl-fme-region.c           | 6 ++----
>  drivers/fpga/intel-m10-bmc-sec-update.c | 6 ++----
>  drivers/fpga/of-fpga-region.c           | 6 ++----
>  drivers/fpga/socfpga-a10.c              | 6 ++----
>  drivers/fpga/stratix10-soc.c            | 6 ++----
>  drivers/fpga/xilinx-pr-decoupler.c      | 6 ++----
>  drivers/fpga/zynq-fpga.c                | 6 ++----
>  13 files changed, 26 insertions(+), 52 deletions(-)

Acked-by: Xu Yilun <yilun.xu@intel.com>

And applied this series to for-next for 6.8-rc1.

> 
> base-commit: aa4db8324c4d0e67aa4670356df4e9fae14b4d37
> -- 
> 2.42.0
> 
>