diff options
| author | Chen Ni <nichen@iscas.ac.cn> | 2024-05-10 16:31:56 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-08-03 08:53:25 +0200 |
| commit | 762384e9cc6562e1422f7451d9fedf681762ca1a (patch) | |
| tree | ffe2834196dc8d19b61ea9d8d37eb7f04adbd116 /drivers/soc/qcom | |
| parent | 4bf50c7dcd3938a912bc5d3a329672718f297a5d (diff) | |
| download | linux-762384e9cc6562e1422f7451d9fedf681762ca1a.tar.gz linux-762384e9cc6562e1422f7451d9fedf681762ca1a.tar.bz2 linux-762384e9cc6562e1422f7451d9fedf681762ca1a.zip | |
soc: qcom: pmic_glink: Handle the return value of pmic_glink_init
[ Upstream commit 0780c836673b25f5aad306630afcb1172d694cb4 ]
As platform_driver_register() and register_rpmsg_driver() can return
error numbers, it should be better to check the return value and deal
with the exception.
Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
Fixes: 58ef4ece1e41 ("soc: qcom: pmic_glink: Introduce base PMIC GLINK driver")
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Link: https://lore.kernel.org/r/20240510083156.1996783-1-nichen@iscas.ac.cn
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/soc/qcom')
| -rw-r--r-- | drivers/soc/qcom/pmic_glink.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/drivers/soc/qcom/pmic_glink.c b/drivers/soc/qcom/pmic_glink.c index 61a359938b6c..71d261ac8aa4 100644 --- a/drivers/soc/qcom/pmic_glink.c +++ b/drivers/soc/qcom/pmic_glink.c @@ -376,8 +376,17 @@ static struct platform_driver pmic_glink_driver = { static int pmic_glink_init(void) { - platform_driver_register(&pmic_glink_driver); - register_rpmsg_driver(&pmic_glink_rpmsg_driver); + int ret; + + ret = platform_driver_register(&pmic_glink_driver); + if (ret < 0) + return ret; + + ret = register_rpmsg_driver(&pmic_glink_rpmsg_driver); + if (ret < 0) { + platform_driver_unregister(&pmic_glink_driver); + return ret; + } return 0; }; |