Commit 62c0aff6 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Stephen Boyd
Browse files

clk: Remove never used devm_clk_*unregister() 



For the entire history of the devm_clk_*unregister() existence they were
used only once (*) in 2015. Remove them.

*) The commit 264e3b75 ("clk: s2mps11: Simplify s2mps11_clk_probe unwind
   paths") exactly supports the point of the change proposed here.

Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20220622171147.85603-1-andriy.shevchenko@linux.intel.com


Acked-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: default avatarStephen Boyd <sboyd@kernel.org>
parent 8b3d743f

drivers/clk/clk.c

+0 −48
Original line number Diff line number Diff line
@@ -4279,54 +4279,6 @@ int devm_clk_hw_register(struct device *dev, struct clk_hw *hw) 
}

EXPORT_SYMBOL_GPL(devm_clk_hw_register);



static int devm_clk_match(struct device *dev, void *res, void *data)

{

	struct clk *c = res;

	if (WARN_ON(!c))

		return 0;

	return c == data;

}



static int devm_clk_hw_match(struct device *dev, void *res, void *data)

{

	struct clk_hw *hw = res;



	if (WARN_ON(!hw))

		return 0;

	return hw == data;

}



/**

 * devm_clk_unregister - resource managed clk_unregister()

 * @dev: device that is unregistering the clock data

 * @clk: clock to unregister

 *

 * Deallocate a clock allocated with devm_clk_register(). Normally

 * this function will not need to be called and the resource management

 * code will ensure that the resource is freed.

 */

void devm_clk_unregister(struct device *dev, struct clk *clk)

{

	WARN_ON(devres_release(dev, devm_clk_unregister_cb, devm_clk_match, clk));

}

EXPORT_SYMBOL_GPL(devm_clk_unregister);



/**

 * devm_clk_hw_unregister - resource managed clk_hw_unregister()

 * @dev: device that is unregistering the hardware-specific clock data

 * @hw: link to hardware-specific clock data

 *

 * Unregister a clk_hw registered with devm_clk_hw_register(). Normally

 * this function will not need to be called and the resource management

 * code will ensure that the resource is freed.

 */

void devm_clk_hw_unregister(struct device *dev, struct clk_hw *hw)

{

	WARN_ON(devres_release(dev, devm_clk_hw_unregister_cb, devm_clk_hw_match,

				hw));

}

EXPORT_SYMBOL_GPL(devm_clk_hw_unregister);



static void devm_clk_release(struct device *dev, void *res)

{

	clk_put(*(struct clk **)res);

include/linux/clk-provider.h

+0 −2
Original line number Diff line number Diff line
@@ -1176,10 +1176,8 @@ int __must_check devm_clk_hw_register(struct device *dev, struct clk_hw *hw); 
int __must_check of_clk_hw_register(struct device_node *node, struct clk_hw *hw);



void clk_unregister(struct clk *clk);

void devm_clk_unregister(struct device *dev, struct clk *clk);



void clk_hw_unregister(struct clk_hw *hw);

void devm_clk_hw_unregister(struct device *dev, struct clk_hw *hw);



/* helper functions */

const char *__clk_get_name(const struct clk *clk);