[RFT PATCH] clk: qcom: dispcc-glymur: Fix (possibly) dumping regmap

Krzysztof Kozlowski posted 1 patch 1 month, 1 week ago
drivers/clk/qcom/dispcc-glymur.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[RFT PATCH] clk: qcom: dispcc-glymur: Fix (possibly) dumping regmap
Posted by Krzysztof Kozlowski 1 month, 1 week ago
Reading few registers at the end of the block (e.g. 0x10000) might
result in synchronous external abort, so limit the regmap to the last
readable register which allows dumping the regs for debugging.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

---

Register map is the same as for x1e80100 and sm8750, so I am guessing
same problem.
---
 drivers/clk/qcom/dispcc-glymur.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/qcom/dispcc-glymur.c b/drivers/clk/qcom/dispcc-glymur.c
index c4bb328d432f..412b92e227d6 100644
--- a/drivers/clk/qcom/dispcc-glymur.c
+++ b/drivers/clk/qcom/dispcc-glymur.c
@@ -1930,7 +1930,7 @@ static const struct regmap_config disp_cc_glymur_regmap_config = {
 	.reg_bits = 32,
 	.reg_stride = 4,
 	.val_bits = 32,
-	.max_register = 0x11014,
+	.max_register = 0xf004, /* 0x10000 and maybe others are for TZ */
 	.fast_io = true,
 };
 
-- 
2.51.0
Re: [RFT PATCH] clk: qcom: dispcc-glymur: Fix (possibly) dumping regmap
Posted by Konrad Dybcio 1 month ago
On 5/5/26 5:37 PM, Krzysztof Kozlowski wrote:
> Reading few registers at the end of the block (e.g. 0x10000) might
> result in synchronous external abort, so limit the regmap to the last
> readable register which allows dumping the regs for debugging.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
> 
> ---
> 
> Register map is the same as for x1e80100 and sm8750, so I am guessing
> same problem.
> ---

This is not necessary on Glymur, possibly in relation to lack of Gunyah

Konrad