[2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

Message ID 20230320135710.1989-3-johan+linaro@kernel.org
State New
Headers
Series arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property |

Commit Message

Johan Hovold March 20, 2023, 1:57 p.m. UTC
  The SPMI PMIC register region width is fixed and should not be encoded
in the devicetree.

Fixes: 42f45cc655d0 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
---
 arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Krzysztof Kozlowski March 20, 2023, 2:11 p.m. UTC | #1
On 20/03/2023 14:57, Johan Hovold wrote:
> The SPMI PMIC register region width is fixed and should not be encoded
> in the devicetree.

It also violates size-cells=0.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof
  
Johan Hovold March 20, 2023, 2:37 p.m. UTC | #2
On Mon, Mar 20, 2023 at 02:57:09PM +0100, Johan Hovold wrote:
> The SPMI PMIC register region width is fixed and should not be encoded
> in the devicetree.
> 
> Fixes: 42f45cc655d0 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")

Bah, that should have been:

Fixes: d6dbbda37ab5 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")

Can you fix that up when applying, Bjorn?

Johan
  
Johan Hovold March 20, 2023, 2:38 p.m. UTC | #3
On Mon, Mar 20, 2023 at 03:11:39PM +0100, Krzysztof Kozlowski wrote:
> On 20/03/2023 14:57, Johan Hovold wrote:
> > The SPMI PMIC register region width is fixed and should not be encoded
> > in the devicetree.
> 
> It also violates size-cells=0.

That's the same thing.

Johan
  

Patch

diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
index 1411edd42b25..c35e7f6bd657 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
@@ -107,7 +107,7 @@  pmk8280_rtc: rtc@6100 {
 
 		pmk8280_sdam_6: nvram@8500 {
 			compatible = "qcom,spmi-sdam";
-			reg = <0x8500 0x100>;
+			reg = <0x8500>;
 			#address-cells = <1>;
 			#size-cells = <1>;
 			ranges = <0 0x8500 0x100>;