diff options
| author | Changhuang Liang <changhuang.liang@starfivetech.com> | 2024-09-25 04:24:42 -0700 |
|---|---|---|
| committer | Philipp Zabel <p.zabel@pengutronix.de> | 2024-09-30 14:24:37 +0200 |
| commit | 2cf59663660799ce16f4dfbed97cdceac7a7fa11 (patch) | |
| tree | 2c8acc6402aba098615dae0fcfb99a67edc9acf3 /drivers/reset | |
| parent | e7b71bf181d03b4835e2d424fa587f4db2438fb2 (diff) | |
| download | lwn-2cf59663660799ce16f4dfbed97cdceac7a7fa11.tar.gz lwn-2cf59663660799ce16f4dfbed97cdceac7a7fa11.zip | |
reset: starfive: jh71x0: Fix accessing the empty member on JH7110 SoC
data->asserted will be NULL on JH7110 SoC since commit 82327b127d41
("reset: starfive: Add StarFive JH7110 reset driver") was added. Add
the judgment condition to avoid errors when calling reset_control_status
on JH7110 SoC.
Fixes: 82327b127d41 ("reset: starfive: Add StarFive JH7110 reset driver")
Signed-off-by: Changhuang Liang <changhuang.liang@starfivetech.com>
Acked-by: Hal Feng <hal.feng@starfivetech.com>
Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
Link: https://lore.kernel.org/r/20240925112442.1732416-1-changhuang.liang@starfivetech.com
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Diffstat (limited to 'drivers/reset')
| -rw-r--r-- | drivers/reset/starfive/reset-starfive-jh71x0.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/reset/starfive/reset-starfive-jh71x0.c b/drivers/reset/starfive/reset-starfive-jh71x0.c index 55bbbd2de52c..29ce3486752f 100644 --- a/drivers/reset/starfive/reset-starfive-jh71x0.c +++ b/drivers/reset/starfive/reset-starfive-jh71x0.c @@ -94,6 +94,9 @@ static int jh71x0_reset_status(struct reset_controller_dev *rcdev, void __iomem *reg_status = data->status + offset * sizeof(u32); u32 value = readl(reg_status); + if (!data->asserted) + return !(value & mask); + return !((value ^ data->asserted[offset]) & mask); } |