From: Rob Herring <robh+dt@kernel.org> To: Krzysztof Kozlowski <k.kozlowski@samsung.com> Cc: Pawel Moll <pawel.moll@arm.com>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Kumar Gala <galak@codeaurora.org>, Kukjin Kim <kgene@kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org> Subject: Re: [PATCH 01/11] ARM: dts: exynos: Fix DTC unit name warnings in cros-adc-thermistors Date: Fri, 1 Apr 2016 10:31:34 -0500 [thread overview] Message-ID: <CAL_JsqJi0w_j2wcb8f60XKOL3MFxkHFZpoyFhT-CCno_chb9Rw@mail.gmail.com> (raw) In-Reply-To: <1459493874-2366-1-git-send-email-k.kozlowski@samsung.com> On Fri, Apr 1, 2016 at 1:57 AM, Krzysztof Kozlowski <k.kozlowski@samsung.com> wrote: > Fix following DTC warnings in cros-adc-thermistors: > Warning (unit_address_vs_reg): Node /adc@12D10000/ncp15wb473@3 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc@12D10000/ncp15wb473@4 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc@12D10000/ncp15wb473@5 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc@12D10000/ncp15wb473@6 has a unit name, but no reg property > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > --- > arch/arm/boot/dts/cros-adc-thermistors.dtsi | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/boot/dts/cros-adc-thermistors.dtsi b/arch/arm/boot/dts/cros-adc-thermistors.dtsi > index acd4fe1833f2..87e676917603 100644 > --- a/arch/arm/boot/dts/cros-adc-thermistors.dtsi > +++ b/arch/arm/boot/dts/cros-adc-thermistors.dtsi > @@ -13,28 +13,28 @@ > */ > > &adc { > - ncp15wb473@3 { > + ncp15wb473_3 { Don't use underscores. > compatible = "murata,ncp15wb473"; > pullup-uv = <1800000>; > pullup-ohm = <47000>; > pulldown-ohm = <0>; > io-channels = <&adc 3>; I've not looked at the full binding here, but this case may warrant adding a reg property as an ADC channel number is what i'd consider an addressable thing. This is also kind of strange having a phandle to the parent node. Rob
WARNING: multiple messages have this Message-ID (diff)
From: robh+dt@kernel.org (Rob Herring) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 01/11] ARM: dts: exynos: Fix DTC unit name warnings in cros-adc-thermistors Date: Fri, 1 Apr 2016 10:31:34 -0500 [thread overview] Message-ID: <CAL_JsqJi0w_j2wcb8f60XKOL3MFxkHFZpoyFhT-CCno_chb9Rw@mail.gmail.com> (raw) In-Reply-To: <1459493874-2366-1-git-send-email-k.kozlowski@samsung.com> On Fri, Apr 1, 2016 at 1:57 AM, Krzysztof Kozlowski <k.kozlowski@samsung.com> wrote: > Fix following DTC warnings in cros-adc-thermistors: > Warning (unit_address_vs_reg): Node /adc at 12D10000/ncp15wb473 at 3 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc at 12D10000/ncp15wb473 at 4 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc at 12D10000/ncp15wb473 at 5 has a unit name, but no reg property > Warning (unit_address_vs_reg): Node /adc at 12D10000/ncp15wb473 at 6 has a unit name, but no reg property > > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> > --- > arch/arm/boot/dts/cros-adc-thermistors.dtsi | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/boot/dts/cros-adc-thermistors.dtsi b/arch/arm/boot/dts/cros-adc-thermistors.dtsi > index acd4fe1833f2..87e676917603 100644 > --- a/arch/arm/boot/dts/cros-adc-thermistors.dtsi > +++ b/arch/arm/boot/dts/cros-adc-thermistors.dtsi > @@ -13,28 +13,28 @@ > */ > > &adc { > - ncp15wb473 at 3 { > + ncp15wb473_3 { Don't use underscores. > compatible = "murata,ncp15wb473"; > pullup-uv = <1800000>; > pullup-ohm = <47000>; > pulldown-ohm = <0>; > io-channels = <&adc 3>; I've not looked at the full binding here, but this case may warrant adding a reg property as an ADC channel number is what i'd consider an addressable thing. This is also kind of strange having a phandle to the parent node. Rob
next prev parent reply other threads:[~2016-04-01 15:32 UTC|newest] Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-01 6:57 [PATCH 01/11] ARM: dts: exynos: Fix DTC unit name warnings in cros-adc-thermistors Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 6:57 ` [PATCH 02/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos3250 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 14:27 ` Javier Martinez Canillas 2016-04-01 14:27 ` Javier Martinez Canillas 2016-04-01 14:27 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 03/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos4 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 14:30 ` Javier Martinez Canillas 2016-04-01 14:30 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 04/11] ARM: dts: exynos: Fix DTC unit name warnings in Trats2 board Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 8:17 ` Krzysztof Kozlowski 2016-04-01 8:17 ` Krzysztof Kozlowski 2016-04-01 14:35 ` Javier Martinez Canillas 2016-04-01 14:35 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 05/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos4x12 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 14:38 ` Javier Martinez Canillas 2016-04-01 14:38 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 06/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos5250 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:21 ` Javier Martinez Canillas 2016-04-01 17:21 ` Javier Martinez Canillas 2016-04-01 17:21 ` Javier Martinez Canillas [not found] ` <56FEAE29.6010405-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> 2016-04-03 2:56 ` Krzysztof Kozlowski 2016-04-03 2:56 ` Krzysztof Kozlowski 2016-04-03 2:56 ` Krzysztof Kozlowski 2016-04-03 4:08 ` Krzysztof Kozlowski 2016-04-03 4:08 ` Krzysztof Kozlowski 2016-04-01 6:57 ` [PATCH 07/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos542x Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:25 ` Javier Martinez Canillas 2016-04-01 17:25 ` Javier Martinez Canillas 2016-04-01 17:25 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 08/11] ARM: dts: exynos: Fix DTC unit name warnings in Peach Pit Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:27 ` Javier Martinez Canillas 2016-04-01 17:27 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 09/11] ARM: dts: exynos: Fix DTC unit name warnings in SMDK5420 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:30 ` Javier Martinez Canillas 2016-04-01 17:30 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 10/11] ARM: dts: exynos: Fix DTC unit name warnings in Exynos5440 Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:46 ` Javier Martinez Canillas 2016-04-01 17:46 ` Javier Martinez Canillas 2016-04-01 6:57 ` [PATCH 11/11] ARM: dts: s5p: Fix DTC unit name warnings in S5Pv210 boards Krzysztof Kozlowski 2016-04-01 6:57 ` Krzysztof Kozlowski 2016-04-01 17:50 ` Javier Martinez Canillas 2016-04-01 17:50 ` Javier Martinez Canillas 2016-04-01 17:50 ` Javier Martinez Canillas 2016-04-01 14:24 ` [PATCH 01/11] ARM: dts: exynos: Fix DTC unit name warnings in cros-adc-thermistors Javier Martinez Canillas 2016-04-01 14:24 ` Javier Martinez Canillas 2016-04-01 15:31 ` Rob Herring [this message] 2016-04-01 15:31 ` Rob Herring 2016-04-01 15:31 ` Rob Herring 2016-04-03 3:46 ` Krzysztof Kozlowski 2016-04-03 3:46 ` Krzysztof Kozlowski 2016-04-03 3:46 ` Krzysztof Kozlowski
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=CAL_JsqJi0w_j2wcb8f60XKOL3MFxkHFZpoyFhT-CCno_chb9Rw@mail.gmail.com \ --to=robh+dt@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=galak@codeaurora.org \ --cc=ijc+devicetree@hellion.org.uk \ --cc=k.kozlowski@samsung.com \ --cc=kgene@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=pawel.moll@arm.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.