soc: qcom: llcc: Handle a second device without data corruption
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Tue, 26 Sep 2023 08:32:29 +0000 (10:32 +0200)
committerBjorn Andersson <andersson@kernel.org>
Wed, 27 Sep 2023 23:18:01 +0000 (16:18 -0700)
Usually there is only one llcc device. But if there were a second, even
a failed probe call would modify the global drv_data pointer. So check
if drv_data is valid before overwriting it.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fixes: a3134fb09e0b ("drivers: soc: Add LLCC driver")
Link: https://lore.kernel.org/r/20230926083229.2073890-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
drivers/soc/qcom/llcc-qcom.c

index feb21637ac20804311ec99854c5ab52ce7561bca..e3f262a50e4adf9a3642745a0aa94bf9308e65c6 100644 (file)
@@ -1120,6 +1120,9 @@ static int qcom_llcc_probe(struct platform_device *pdev)
        u32 version;
        struct regmap *regmap;
 
+       if (!IS_ERR(drv_data))
+               return -EBUSY;
+
        drv_data = devm_kzalloc(dev, sizeof(*drv_data), GFP_KERNEL);
        if (!drv_data) {
                ret = -ENOMEM;