diff options
| author | Arvind Yadav <arvind.yadav.cs@gmail.com> | 2017-05-29 13:12:12 +0530 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2017-06-11 15:08:20 +0100 |
| commit | 4e36a8ad4d69fbb52f0f07d9ff340cd6a7a2a503 (patch) | |
| tree | 804ef7e80b603c6a37ed7fa3a31ad81f1d089c4c /drivers/iio | |
| parent | 440d303c7775a5dff546dbbc9a39e8c6cbdd6d18 (diff) | |
| download | lwn-4e36a8ad4d69fbb52f0f07d9ff340cd6a7a2a503.tar.gz lwn-4e36a8ad4d69fbb52f0f07d9ff340cd6a7a2a503.zip | |
iio: Aspeed ADC - Handle return value of clk_prepare_enable
clk_prepare_enable() can fail here and we must check its return value.
Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Reviewed-by: Rick Altherr <raltherr@google.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio')
| -rw-r--r-- | drivers/iio/adc/aspeed_adc.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/iio/adc/aspeed_adc.c b/drivers/iio/adc/aspeed_adc.c index 62670cbfa2bb..e0ea411a0b2d 100644 --- a/drivers/iio/adc/aspeed_adc.c +++ b/drivers/iio/adc/aspeed_adc.c @@ -212,7 +212,10 @@ static int aspeed_adc_probe(struct platform_device *pdev) } /* Start all channels in normal mode. */ - clk_prepare_enable(data->clk_scaler->clk); + ret = clk_prepare_enable(data->clk_scaler->clk); + if (ret) + goto clk_enable_error; + adc_engine_control_reg_val = GENMASK(31, 16) | ASPEED_OPERATION_MODE_NORMAL | ASPEED_ENGINE_ENABLE; writel(adc_engine_control_reg_val, @@ -236,6 +239,7 @@ iio_register_error: writel(ASPEED_OPERATION_MODE_POWER_DOWN, data->base + ASPEED_REG_ENGINE_CONTROL); clk_disable_unprepare(data->clk_scaler->clk); +clk_enable_error: clk_hw_unregister_divider(data->clk_scaler); scaler_error: |