From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: andersson@kernel.org, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, bp@alien8.de,
tony.luck@intel.com
Cc: konrad.dybcio@linaro.org, linux-arm-msm@vger.kernel.org,
linux-kernel@vger.kernel.org, james.morse@arm.com,
mchehab@kernel.org, rric@kernel.org, linux-edac@vger.kernel.org,
quic_ppareek@quicinc.com, luca.weiss@fairphone.com,
ahalaney@redhat.com, steev@kali.org,
Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Subject: [PATCH v8 00/14] Qcom: LLCC/EDAC: Fix base address used for LLCC banks
Date: Tue, 14 Mar 2023 13:34:29 +0530 [thread overview]
Message-ID: <20230314080443.64635-1-manivannan.sadhasivam@linaro.org> (raw)
The Qualcomm LLCC/EDAC drivers were using a fixed register stride for
accessing the (Control and Status Regsiters) CSRs of each LLCC bank.
This offset only works for some SoCs like SDM845 for which driver support
was initially added.
But the later SoCs use different register stride that vary between the
banks with holes in-between. So it is not possible to use a single register
stride for accessing the CSRs of each bank. By doing so could result in a
crash with the current drivers. So far this crash is not reported since
EDAC_QCOM driver is not enabled in ARM64 defconfig and no one tested the
driver extensively by triggering the EDAC IRQ (that's where each bank
CSRs are accessed).
For fixing this issue, let's obtain the base address of each LLCC bank from
devicetree and get rid of the fixed stride.
This series has been tested on SM8250, SM8450, SM6350, SC8280XP, SA8540P,
and SDM845.
Merging strategy
----------------
All patches should be merged to qcom tree due to LLCC dependency.
Thanks,
Mani
Changes in v8:
* Added the ECC polling support patch that was missed in v7
Changes in v7:
* Rebased on top of v6.3-rc1
* Dropped the patches applied for v6.3
* Dropped Sai from the binding maintainers list since he left Qcom
Changes in v6:
* Incorporated comments from Borislav for the EDAC patches and collected
review tags.
Changes in v5:
* Reduced the size of llcc0 to 0x45000 on SDM845 due to overlapping with BWMON
* Added a patch to disable creation of EDAC platform device on SDM845
* Rebase on top of v6.2-rc1
* Moved the EDAC specific patches to the start so that they can be applied
independently of LLCC patches
Changes in v4:
* Added a patch that fixes the use-after-free bug in qcom_edac driver
Changes in v3:
* Brought back reg-names property for compatibility (Krzysztof)
* Removed Fixes tag and stable list as backporting the drivers/binding/dts
patches alone would break (Krzysztof)
* Fixed the uninitialized variable issue (Kbot)
* Added a patch to make use of driver supplied polling interval (Luca)
* Added a patch for module autoloading (Andrew)
* Didn't collect Review tags from Sai as the dts patches were changed.
Changes in v2:
* Removed reg-names property and used index of reg property to parse LLCC
bank base address (Bjorn)
* Collected Ack from Sai for binding
* Added a new patch for polling mode (Luca)
* Renamed subject of patches targeting SC7180 and SM6350
Manivannan Sadhasivam (14):
dt-bindings: arm: msm: Update the maintainers for LLCC
dt-bindings: arm: msm: Fix register regions used for LLCC banks
arm64: dts: qcom: sdm845: Fix the base addresses of LLCC banks
arm64: dts: qcom: sc7180: Fix the base addresses of LLCC banks
arm64: dts: qcom: sc7280: Fix the base addresses of LLCC banks
arm64: dts: qcom: sc8280xp: Fix the base addresses of LLCC banks
arm64: dts: qcom: sm8150: Fix the base addresses of LLCC banks
arm64: dts: qcom: sm8250: Fix the base addresses of LLCC banks
arm64: dts: qcom: sm8350: Fix the base addresses of LLCC banks
arm64: dts: qcom: sm8450: Fix the base addresses of LLCC banks
arm64: dts: qcom: sm6350: Fix the base addresses of LLCC banks
qcom: llcc/edac: Fix the base address used for accessing LLCC banks
qcom: llcc/edac: Support polling mode for ECC handling
soc: qcom: llcc: Do not create EDAC platform device on SDM845
.../bindings/arm/msm/qcom,llcc.yaml | 128 ++++++++++++++++--
arch/arm64/boot/dts/qcom/sc7180.dtsi | 2 +-
arch/arm64/boot/dts/qcom/sc7280.dtsi | 5 +-
arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 10 +-
arch/arm64/boot/dts/qcom/sdm845.dtsi | 7 +-
arch/arm64/boot/dts/qcom/sm6350.dtsi | 2 +-
arch/arm64/boot/dts/qcom/sm8150.dtsi | 7 +-
arch/arm64/boot/dts/qcom/sm8250.dtsi | 7 +-
arch/arm64/boot/dts/qcom/sm8350.dtsi | 7 +-
arch/arm64/boot/dts/qcom/sm8450.dtsi | 7 +-
drivers/edac/qcom_edac.c | 64 +++++----
drivers/soc/qcom/llcc-qcom.c | 87 +++++++-----
include/linux/soc/qcom/llcc-qcom.h | 6 +-
13 files changed, 242 insertions(+), 97 deletions(-)
--
2.25.1
next reply other threads:[~2023-03-14 8:05 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-14 8:04 Manivannan Sadhasivam [this message]
2023-03-14 8:04 ` [PATCH v8 01/14] dt-bindings: arm: msm: Update the maintainers for LLCC Manivannan Sadhasivam
2023-03-19 12:46 ` Krzysztof Kozlowski
2023-03-14 8:04 ` [PATCH v8 02/14] dt-bindings: arm: msm: Fix register regions used for LLCC banks Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 03/14] arm64: dts: qcom: sdm845: Fix the base addresses of " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 04/14] arm64: dts: qcom: sc7180: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 05/14] arm64: dts: qcom: sc7280: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 06/14] arm64: dts: qcom: sc8280xp: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 07/14] arm64: dts: qcom: sm8150: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 08/14] arm64: dts: qcom: sm8250: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 09/14] arm64: dts: qcom: sm8350: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 10/14] arm64: dts: qcom: sm8450: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 11/14] arm64: dts: qcom: sm6350: " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 12/14] qcom: llcc/edac: Fix the base address used for accessing " Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 13/14] qcom: llcc/edac: Support polling mode for ECC handling Manivannan Sadhasivam
2023-03-14 8:04 ` [PATCH v8 14/14] soc: qcom: llcc: Do not create EDAC platform device on SDM845 Manivannan Sadhasivam
2023-03-15 23:35 ` (subset) [PATCH v8 00/14] Qcom: LLCC/EDAC: Fix base address used for LLCC banks Bjorn Andersson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230314080443.64635-1-manivannan.sadhasivam@linaro.org \
--to=manivannan.sadhasivam@linaro.org \
--cc=ahalaney@redhat.com \
--cc=andersson@kernel.org \
--cc=bp@alien8.de \
--cc=james.morse@arm.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luca.weiss@fairphone.com \
--cc=mchehab@kernel.org \
--cc=quic_ppareek@quicinc.com \
--cc=robh+dt@kernel.org \
--cc=rric@kernel.org \
--cc=steev@kali.org \
--cc=tony.luck@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).