From: Krzysztof Kozlowski <krzk@kernel.org>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: Sylwester Nawrocki <snawrocki@kernel.org>,
Rob Herring <robh+dt@kernel.org>, Kukjin Kim <kgene@kernel.org>,
Alim Akhtar <alim.akhtar@samsung.com>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Chanwoo Choi <cw00.choi@samsung.com>,
Pankaj Dubey <pankaj.dubey@samsung.com>
Subject: Re: [PATCH v2] ARM: dts: exynos: Fix missing empty reg/ranges property regulators on Trats
Date: Thu, 2 Jul 2020 09:39:13 +0200 [thread overview]
Message-ID: <20200702073913.GA1187@kozik-lap> (raw)
In-Reply-To: <20200702061611.GC4175@kozik-lap>
On Thu, Jul 02, 2020 at 08:16:11AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Jun 30, 2020 at 08:27:01AM +0200, Marek Szyprowski wrote:
> > Hi Krzysztof,
> >
> > On 29.06.2020 22:59, Krzysztof Kozlowski wrote:
> > > Remove the regulators node entirely because its children do not have any
> > > unit addresses. This fixes DTC warning:
> > >
> > > Warning (simple_bus_reg): /regulators/regulator-0: missing or empty reg/ranges property
> > >
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> >
> > Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
> >
> > What about removing the regulators node from other boards:
> > exynos4412-origen.dts, exynos5420-smdk5420.dts and exynos5250-arndale.dts?
> >
> > On the other hand, maybe it would be really easier to add missing
> > address/size-cells properties to exynos4210-trats.dts/regulators node?
>
> Indeed let's keep it consistent so in such case better to add here
> proper address/size-cells.
Actually more of DTSes put fixed regulators directly in root node, not
under "regulators" node:
exynos3250-monk.dts
exynos4210-i9100.dts
exynos4210-origen.dts
exynos4210-universal_c210.dts
exynos4412-galaxy-s3.dtsi
exynos4412-midas.dtsi
exynos4412-n710x.dts
exynos4412-odroidx.dts
exynos5250-smdk5250.dts
exynos5250-snow-common.dtsi
exynos5420-peach-pit.dts
exynos5800-peach-pi.dts
If we want it to be consistent, it's easier to remove the regulator
nodes from exynos4412-origen.dts, exynos5420-smdk5420.dts and
exynos5250-arndale.dts.
Best regards,
Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
Sylwester Nawrocki <snawrocki@kernel.org>,
Pankaj Dubey <pankaj.dubey@samsung.com>,
linux-kernel@vger.kernel.org,
Chanwoo Choi <cw00.choi@samsung.com>,
Rob Herring <robh+dt@kernel.org>, Kukjin Kim <kgene@kernel.org>,
Alim Akhtar <alim.akhtar@samsung.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2] ARM: dts: exynos: Fix missing empty reg/ranges property regulators on Trats
Date: Thu, 2 Jul 2020 09:39:13 +0200 [thread overview]
Message-ID: <20200702073913.GA1187@kozik-lap> (raw)
In-Reply-To: <20200702061611.GC4175@kozik-lap>
On Thu, Jul 02, 2020 at 08:16:11AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Jun 30, 2020 at 08:27:01AM +0200, Marek Szyprowski wrote:
> > Hi Krzysztof,
> >
> > On 29.06.2020 22:59, Krzysztof Kozlowski wrote:
> > > Remove the regulators node entirely because its children do not have any
> > > unit addresses. This fixes DTC warning:
> > >
> > > Warning (simple_bus_reg): /regulators/regulator-0: missing or empty reg/ranges property
> > >
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> >
> > Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
> >
> > What about removing the regulators node from other boards:
> > exynos4412-origen.dts, exynos5420-smdk5420.dts and exynos5250-arndale.dts?
> >
> > On the other hand, maybe it would be really easier to add missing
> > address/size-cells properties to exynos4210-trats.dts/regulators node?
>
> Indeed let's keep it consistent so in such case better to add here
> proper address/size-cells.
Actually more of DTSes put fixed regulators directly in root node, not
under "regulators" node:
exynos3250-monk.dts
exynos4210-i9100.dts
exynos4210-origen.dts
exynos4210-universal_c210.dts
exynos4412-galaxy-s3.dtsi
exynos4412-midas.dtsi
exynos4412-n710x.dts
exynos4412-odroidx.dts
exynos5250-smdk5250.dts
exynos5250-snow-common.dtsi
exynos5420-peach-pit.dts
exynos5800-peach-pi.dts
If we want it to be consistent, it's easier to remove the regulator
nodes from exynos4412-origen.dts, exynos5420-smdk5420.dts and
exynos5250-arndale.dts.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-07-02 7:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20200629210025eucas1p219a52e75ecce9e813aa80f0126780189@eucas1p2.samsung.com>
2020-06-29 20:59 ` [PATCH v2] ARM: dts: exynos: Fix missing empty reg/ranges property regulators on Trats Krzysztof Kozlowski
2020-06-29 20:59 ` Krzysztof Kozlowski
2020-06-30 6:27 ` Marek Szyprowski
2020-06-30 6:27 ` Marek Szyprowski
2020-07-02 6:16 ` Krzysztof Kozlowski
2020-07-02 6:16 ` Krzysztof Kozlowski
2020-07-02 7:39 ` Krzysztof Kozlowski [this message]
2020-07-02 7:39 ` Krzysztof Kozlowski
2020-07-02 7:48 ` Marek Szyprowski
2020-07-02 7:48 ` Marek Szyprowski
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=20200702073913.GA1187@kozik-lap \
--to=krzk@kernel.org \
--cc=alim.akhtar@samsung.com \
--cc=b.zolnierkie@samsung.com \
--cc=cw00.choi@samsung.com \
--cc=devicetree@vger.kernel.org \
--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=m.szyprowski@samsung.com \
--cc=pankaj.dubey@samsung.com \
--cc=robh+dt@kernel.org \
--cc=snawrocki@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 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.