projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
f03f5c7
)
pinctrl: zynqmp: Drop pinctrl_unregister for devm_ registered device
author
Yang Yingliang
<yangyingliang@huawei.com>
Thu, 29 Jul 2021 07:19:05 +0000
(15:19 +0800)
committer
Linus Walleij
<linus.walleij@linaro.org>
Wed, 11 Aug 2021 13:09:22 +0000
(15:09 +0200)
It's not necessary to unregister pin controller device registered
with devm_pinctrl_register() and using pinctrl_unregister() leads
to a double free.
Fixes: fa99e7013827 ("pinctrl: zynqmp: some code cleanups")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Michal Simek <michal.simek@xilinx.com>
Link:
https://lore.kernel.org/r/20210729071905.3235953-1-yangyingliang@huawei.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-zynqmp.c
patch
|
blob
|
history
diff --git
a/drivers/pinctrl/pinctrl-zynqmp.c
b/drivers/pinctrl/pinctrl-zynqmp.c
index bbde676b7313ab215546e7dfa07c431dec6cd33b..e14012209992f997d086e15fc2462b08bbd6e938 100644
(file)
--- a/
drivers/pinctrl/pinctrl-zynqmp.c
+++ b/
drivers/pinctrl/pinctrl-zynqmp.c
@@
-866,15
+866,6
@@
static int zynqmp_pinctrl_probe(struct platform_device *pdev)
return ret;
}
-static int zynqmp_pinctrl_remove(struct platform_device *pdev)
-{
- struct zynqmp_pinctrl *pctrl = platform_get_drvdata(pdev);
-
- pinctrl_unregister(pctrl->pctrl);
-
- return 0;
-}
-
static const struct of_device_id zynqmp_pinctrl_of_match[] = {
{ .compatible = "xlnx,zynqmp-pinctrl" },
{ }
@@
-887,7
+878,6
@@
static struct platform_driver zynqmp_pinctrl_driver = {
.of_match_table = zynqmp_pinctrl_of_match,
},
.probe = zynqmp_pinctrl_probe,
- .remove = zynqmp_pinctrl_remove,
};
module_platform_driver(zynqmp_pinctrl_driver);