diff options
| author | Dan Carpenter <[email protected]> | 2020-09-18 17:33:38 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2020-09-22 09:49:34 +0200 |
| commit | dbbc8fdf2ad57629436bcdb421c2d4ad9aa1974f (patch) | |
| tree | 6a23a7991a5c18891eb907df654a1d65ccd1fefb | |
| parent | e2bb57d233ce973a8e61cbb6ab79a91779a4a6c4 (diff) | |
staging: spmi: hisi-spmi-controller: Use devm_ version of ioremap().
The ioremap() was never unmapped in the probe error handling or in the
remove function. The fix is to use the devm_ioremap() function so it
gets cleaned up automatically.
Fixes: 70f59c90c819 ("staging: spmi: add Hikey 970 SPMI controller driver")
Signed-off-by: Dan Carpenter <[email protected]>
Link: https://lore.kernel.org/r/20200918143338.GE909725@mwanda
Signed-off-by: Greg Kroah-Hartman <[email protected]>
| -rw-r--r-- | drivers/staging/hikey9xx/hisi-spmi-controller.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/hikey9xx/hisi-spmi-controller.c b/drivers/staging/hikey9xx/hisi-spmi-controller.c index 5ffe79f45d9a..f831c43f4783 100644 --- a/drivers/staging/hikey9xx/hisi-spmi-controller.c +++ b/drivers/staging/hikey9xx/hisi-spmi-controller.c @@ -281,7 +281,8 @@ static int spmi_controller_probe(struct platform_device *pdev) return -EINVAL; } - spmi_controller->base = ioremap(iores->start, resource_size(iores)); + spmi_controller->base = devm_ioremap(&pdev->dev, iores->start, + resource_size(iores)); if (!spmi_controller->base) { dev_err(&pdev->dev, "can not remap base addr!\n"); return -EADDRNOTAVAIL; |