Commit 72336966 authored by Uwe Kleine-König's avatar Uwe Kleine-König Committed by Jonathan Cameron
Browse files

iio: adc: xilinx-xadc: Benefit from devm_clk_get_enabled() to simplify 



Make use of devm_clk_get_enabled() to replace some code that effectively
open codes this new function.

Reviewed-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Link: https://lore.kernel.org/r/20220808204740.307667-9-u.kleine-koenig@pengutronix.de


Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 40c0c131

drivers/iio/adc/xilinx-xadc-core.c

+1 −17
Original line number Diff line number Diff line
@@ -1296,13 +1296,6 @@ static const char * const xadc_type_names[] = { 
	[XADC_TYPE_US] = "xilinx-system-monitor",

};



static void xadc_clk_disable_unprepare(void *data)

{

	struct clk *clk = data;



	clk_disable_unprepare(clk);

}



static void xadc_cancel_delayed_work(void *data)

{

	struct delayed_work *work = data;
@@ -1374,19 +1367,10 @@ static int xadc_probe(struct platform_device *pdev) 
		}

	}



	xadc->clk = devm_clk_get(dev, NULL);

	xadc->clk = devm_clk_get_enabled(dev, NULL);

	if (IS_ERR(xadc->clk))

		return PTR_ERR(xadc->clk);



	ret = clk_prepare_enable(xadc->clk);

	if (ret)

		return ret;



	ret = devm_add_action_or_reset(dev,

				       xadc_clk_disable_unprepare, xadc->clk);

	if (ret)

		return ret;



	/*

	 * Make sure not to exceed the maximum samplerate since otherwise the

	 * resulting interrupt storm will soft-lock the system.