mbox series

[0/7] clk: Make clk_free return void

Message ID 20220115222504.617013-1-seanga2@gmail.com
Headers show
Series clk: Make clk_free return void | expand

Message

Sean Anderson Jan. 15, 2022, 10:24 p.m. UTC 
clk_free cleans up resources allocated by clk_request et. al. It returns an
error code, but it really shouldn't. Much like regular free(), there is
typically no way to handle an error, and errors from clk_free shouldn't prevent
progress in the rest of the program. Make clk_free (and rfree) return void.


Sean Anderson (7):
  clk: Make rfree return void
  dma: bcm6348: Don't check clk_free
  net: bcm63xx: Don't check clk_free
  phy: bcm63xx: Don't check clk_free
  spi: bcm63xx: Don't check clk_free
  spi: dw: Don't check clk_free
  clk: Make clk_free return void

 drivers/clk/clk-uclass.c       | 15 ++++++---------
 drivers/clk/clk_sandbox.c      |  6 +++---
 drivers/clk/clk_sandbox_test.c |  9 +++------
 drivers/dma/bcm6348-iudma.c    |  6 +-----
 drivers/net/bcm6348-eth.c      |  6 +-----
 drivers/net/bcm6368-eth.c      |  6 +-----
 drivers/phy/bcm6318-usbh-phy.c |  4 +---
 drivers/phy/bcm6348-usbh-phy.c |  4 +---
 drivers/phy/bcm6368-usbh-phy.c |  8 ++------
 drivers/spi/bcm63xx_hsspi.c    |  8 ++------
 drivers/spi/bcm63xx_spi.c      |  4 +---
 drivers/spi/designware_spi.c   |  2 +-
 include/clk-uclass.h           |  8 +++-----
 include/clk.h                  |  8 ++++----
 14 files changed, 30 insertions(+), 64 deletions(-)

Comments

Sean Anderson March 30, 2022, 7:21 p.m. UTC | #1 
On Sat, 15 Jan 2022 17:24:57 -0500, Sean Anderson wrote:
> clk_free cleans up resources allocated by clk_request et. al. It returns an
> error code, but it really shouldn't. Much like regular free(), there is
> typically no way to handle an error, and errors from clk_free shouldn't prevent
> progress in the rest of the program. Make clk_free (and rfree) return void.
> 
> 
> Sean Anderson (7):
>   clk: Make rfree return void
>   dma: bcm6348: Don't check clk_free
>   net: bcm63xx: Don't check clk_free
>   phy: bcm63xx: Don't check clk_free
>   spi: bcm63xx: Don't check clk_free
>   spi: dw: Don't check clk_free
>   clk: Make clk_free return void
> 
> [...]

Applied, thanks!

[1/7] clk: Make rfree return void
      commit: 276d446757e462c210768eb0bbd48450ae254f51
[2/7] dma: bcm6348: Don't check clk_free
      commit: 454af567edc0b02842c83aaf1a60bbcb766af0cd
[3/7] net: bcm63xx: Don't check clk_free
      commit: b2e0889abacfd453131359156fe279996727d95b
[4/7] phy: bcm63xx: Don't check clk_free
      commit: ad20358c7462159d5f9012facba9dec1e197aaca
[5/7] spi: bcm63xx: Don't check clk_free
      commit: dfdb227c3d018983f37cc97fe003e231a81a46ea
[6/7] spi: dw: Don't check clk_free
      commit: 3cbdd4cab951b8bd3f2e76066e6911f9780c4eb1
[7/7] clk: Make clk_free return void
      commit: ac15e789caecec19d29ee9c5869305d3c3ddfb42

Best regards,