devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sasha Levin <sashal@kernel.org>
To: linux-kernel@vger.kernel.org, stable@vger.kernel.org
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Sasha Levin <sashal@kernel.org>,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org
Subject: [PATCH AUTOSEL 6.6 018/104] ARM: dts: samsung: s5pv210: fix camera unit addresses/ranges
Date: Tue, 16 Jan 2024 14:45:44 -0500	[thread overview]
Message-ID: <20240116194908.253437-18-sashal@kernel.org> (raw)
In-Reply-To: <20240116194908.253437-1-sashal@kernel.org>

From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

[ Upstream commit 07e6a553c2f1d385edfc9185081dee442a9dd38d ]

The camera node has both unit address and children within the same bus
mapping, thus needs proper ranges property to fix dtc W=1 warnings:

  Warning (unit_address_vs_reg): /soc/camera@fa600000: node has a unit name, but no reg or ranges property
  Warning (simple_bus_reg): /soc/camera@fa600000: missing or empty reg/ranges property

Subtract 0xfa600000 from all its children nodes.  No functional impact
expected.

Link: https://lore.kernel.org/r/20230722121719.150094-3-krzysztof.kozlowski@linaro.org
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 arch/arm/boot/dts/samsung/s5pv210.dtsi | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/arch/arm/boot/dts/samsung/s5pv210.dtsi b/arch/arm/boot/dts/samsung/s5pv210.dtsi
index f7de5b5f2f38..ed560c9a3aa1 100644
--- a/arch/arm/boot/dts/samsung/s5pv210.dtsi
+++ b/arch/arm/boot/dts/samsung/s5pv210.dtsi
@@ -549,17 +549,17 @@ i2c1: i2c@fab00000 {
 
 		camera: camera@fa600000 {
 			compatible = "samsung,fimc";
+			ranges = <0x0 0xfa600000 0xe01000>;
 			clocks = <&clocks SCLK_CAM0>, <&clocks SCLK_CAM1>;
 			clock-names = "sclk_cam0", "sclk_cam1";
 			#address-cells = <1>;
 			#size-cells = <1>;
 			#clock-cells = <1>;
 			clock-output-names = "cam_a_clkout", "cam_b_clkout";
-			ranges;
 
-			csis0: csis@fa600000 {
+			csis0: csis@0 {
 				compatible = "samsung,s5pv210-csis";
-				reg = <0xfa600000 0x4000>;
+				reg = <0x00000000 0x4000>;
 				interrupt-parent = <&vic2>;
 				interrupts = <29>;
 				clocks = <&clocks CLK_CSIS>,
@@ -572,9 +572,9 @@ csis0: csis@fa600000 {
 				#size-cells = <0>;
 			};
 
-			fimc0: fimc@fb200000 {
+			fimc0: fimc@c00000 {
 				compatible = "samsung,s5pv210-fimc";
-				reg = <0xfb200000 0x1000>;
+				reg = <0x00c00000 0x1000>;
 				interrupts = <5>;
 				interrupt-parent = <&vic2>;
 				clocks = <&clocks CLK_FIMC0>,
@@ -586,9 +586,9 @@ fimc0: fimc@fb200000 {
 				samsung,cam-if;
 			};
 
-			fimc1: fimc@fb300000 {
+			fimc1: fimc@d00000 {
 				compatible = "samsung,s5pv210-fimc";
-				reg = <0xfb300000 0x1000>;
+				reg = <0x00d00000 0x1000>;
 				interrupt-parent = <&vic2>;
 				interrupts = <6>;
 				clocks = <&clocks CLK_FIMC1>,
@@ -602,9 +602,9 @@ fimc1: fimc@fb300000 {
 				samsung,lcd-wb;
 			};
 
-			fimc2: fimc@fb400000 {
+			fimc2: fimc@e00000 {
 				compatible = "samsung,s5pv210-fimc";
-				reg = <0xfb400000 0x1000>;
+				reg = <0x00e00000 0x1000>;
 				interrupt-parent = <&vic2>;
 				interrupts = <7>;
 				clocks = <&clocks CLK_FIMC2>,
-- 
2.43.0


  parent reply	other threads:[~2024-01-16 19:49 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20240116194908.253437-1-sashal@kernel.org>
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 009/104] ARM: dts: qcom: strip prefix from PMIC files Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 010/104] ARM: dts: qcom: apq8064: fix PMIC node labels Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 011/104] ARM: dts: qcom: mdm9615: " Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 012/104] ARM: dts: qcom: msm8660: " Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 013/104] ARM: dts: qcom: msm8960: " Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 017/104] ARM: dts: samsung: exynos4: fix camera unit addresses/ranges Sasha Levin
2024-01-16 19:45 ` Sasha Levin [this message]
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 028/104] ARM: dts: imx7d: Fix coresight funnel ports Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 029/104] ARM: dts: imx7s: Fix lcdif compatible Sasha Levin
2024-01-16 19:45 ` [PATCH AUTOSEL 6.6 030/104] ARM: dts: imx7s: Fix nand-controller #size-cells Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 044/104] arm64: dts: qcom: sm8550: fix soundwire controllers node name Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 045/104] arm64: dts: qcom: sm8450: " Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 046/104] arm64: dts: qcom: sm8350: Fix remoteproc interrupt type Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 054/104] ARM: dts: rockchip: fix rk3036 hdmi ports node Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 055/104] ARM: dts: imx25/27-eukrea: Fix RTC node name Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 056/104] ARM: dts: imx: Use flash@0,0 pattern Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 057/104] ARM: dts: imx27: Fix sram node Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 058/104] ARM: dts: imx1: " Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 063/104] arm64: zynqmp: Move fixed clock to / for kv260 Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 064/104] arm64: zynqmp: Fix clock node name in kv260 cards Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 066/104] ARM: dts: imx25: Fix the iim compatible string Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 067/104] ARM: dts: imx25/27: Pass timing0 Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 068/104] ARM: dts: imx27-apf27dev: Fix LED name Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 069/104] ARM: dts: imx23-sansa: Use preferred i2c-gpios properties Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 070/104] ARM: dts: imx23/28: Fix the DMA controller node name Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 077/104] ARM: dts: marvell: Fix some common switch mistakes Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 078/104] ARM64: " Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 082/104] arm64: dts: qcom: msm8996: Fix 'in-ports' is a required property Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 083/104] arm64: dts: qcom: msm8998: Fix 'out-ports' " Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 084/104] arm64: dts: qcom: Fix coresight warnings in in-ports and out-ports Sasha Levin
2024-01-16 19:46 ` [PATCH AUTOSEL 6.6 086/104] arm64: dts: amlogic: fix format for s4 uart node Sasha Levin
2024-01-16 19:47 ` [PATCH AUTOSEL 6.6 099/104] arm64: dts: sprd: Add clock reference for pll2 on UMS512 Sasha Levin
2024-01-16 19:47 ` [PATCH AUTOSEL 6.6 100/104] arm64: dts: sprd: Change UMS512 idle-state nodename to match bindings Sasha Levin

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=20240116194908.253437-18-sashal@kernel.org \
    --to=sashal@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=stable@vger.kernel.org \
    /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).