From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH V5 3/7] ARM: exynos: Use - instead of @ for DT OPP entries Date: Thu, 20 Apr 2017 18:09:30 +0200 Message-ID: <20170420160930.halyszjunt4q2634@kozik-lap> References: <8fd1dcd1bfa2011411dceab82bcd1587fe913cd1.1492685450.git.viresh.kumar@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <8fd1dcd1bfa2011411dceab82bcd1587fe913cd1.1492685450.git.viresh.kumar@linaro.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Viresh Kumar Cc: Mark Rutland , Rob Herring , linux-samsung-soc@vger.kernel.org, linaro-kernel@lists.linaro.org, arm@kernel.org, Javier Martinez Canillas , Masahiro Yamada , linux-pm@vger.kernel.org, Rafael Wysocki , linux-kernel@vger.kernel.org, Rob Herring , Chanwoo Choi , Kyungmin Park , Kukjin Kim , MyungJoo Ham , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Thu, Apr 20, 2017 at 04:25:07PM +0530, Viresh Kumar wrote: > Compiling the DT file with W=1, DTC warns like follows: > > Warning (unit_address_vs_reg): Node /opp_table0/opp@1000000000 has a > unit name, but no reg property > > Fix this by replacing '@' with '-' as the OPP nodes will never have a > "reg" property. > > Reported-by: Krzysztof Kozlowski > Reported-by: Masahiro Yamada > Suggested-by: Mark Rutland > Signed-off-by: Viresh Kumar > Reviewed-by: Chanwoo Choi > Reviewed-by: Krzysztof Kozlowski > Acked-by: Rob Herring > --- > .../devicetree/bindings/devfreq/exynos-bus.txt | 46 +++++++-------- > arch/arm/boot/dts/exynos3250.dtsi | 46 +++++++-------- > arch/arm/boot/dts/exynos4210.dtsi | 32 +++++------ > arch/arm/boot/dts/exynos4412-prime.dtsi | 4 +- > arch/arm/boot/dts/exynos4412.dtsi | 66 +++++++++++----------- > arch/arm/boot/dts/exynos5420.dtsi | 40 ++++++------- > arch/arm/boot/dts/exynos5800.dtsi | 56 +++++++++--------- > arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi | 48 ++++++++-------- > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 50 ++++++++-------- > 9 files changed, 194 insertions(+), 194 deletions(-) Thanks, split ARM64 from ARM and applied (for v4.12). arm-soc keeps DTS separated between ARM architectures. Technically this should require a resend but we had way too many resends for this simple patch. Best regards, Krzysztof