From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
To: Rob Herring <robh+dt@kernel.org>
Cc: Kukjin Kim <kgene@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Kumar Gala <galak@codeaurora.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-samsung-soc@vger.kernel.org"
<linux-samsung-soc@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Javier Martinez Canillas <javier@osg.samsung.com>
Subject: Re: [PATCH 1/2] dt-bindings: Consolidate Exynos SoC bindings under Samsung directory
Date: Thu, 22 Oct 2015 09:26:40 +0900 [thread overview]
Message-ID: <56282D40.2050504@samsung.com> (raw)
In-Reply-To: <CAL_JsqLhbqJOhQrGwg3CZ2Th+wgf3oMR6-2oNNZ98yHNjxsx_A@mail.gmail.com>
On 22.10.2015 09:22, Rob Herring wrote:
> On Tue, Oct 20, 2015 at 8:30 PM, Krzysztof Kozlowski
> <k.kozlowski@samsung.com> wrote:
>> Exynos SoC Device Tree bindings are spread over arm/exynos/ and
>> arm/samsung/ directories. There is no need for that separation and it
>> actually confuses. Put everything under arm/samsung/.
>
> Thanks for the clean-up.
>
>>
>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>> Cc: Kukjin Kim <kgene@kernel.org>
>> ---
>> .../devicetree/bindings/arm/{exynos => samsung}/power_domain.txt | 0
>> Documentation/devicetree/bindings/arm/{ => samsung}/samsung-boards.txt | 0
>> Documentation/devicetree/bindings/arm/{exynos => samsung}/smp-sysram.txt | 0
>> 3 files changed, 0 insertions(+), 0 deletions(-)
>> rename Documentation/devicetree/bindings/arm/{exynos => samsung}/power_domain.txt (100%)
>
> I'd rather see this with other power domain bindings. So
> bindings/power/pd-samsung.txt I guess.
OK
>
>> rename Documentation/devicetree/bindings/arm/{ => samsung}/samsung-boards.txt (100%)
>> rename Documentation/devicetree/bindings/arm/{exynos => samsung}/smp-sysram.txt (100%)
>
> Probably need to move misc/sram.txt to bindings/sram/ and put both there.
Right, that make sense. I'll send v2 moving pd-samsung and smp-sysram to
these locations.
Best regards,
Krzysztof
prev parent reply other threads:[~2015-10-22 0:26 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-21 1:30 [PATCH 1/2] dt-bindings: Consolidate Exynos SoC bindings under Samsung directory Krzysztof Kozlowski
2015-10-21 1:30 ` [PATCH 2/2] dt-bindings: EXYNOS: Document compatibles from other vendors Krzysztof Kozlowski
2015-10-21 8:39 ` Javier Martinez Canillas
2015-10-21 8:38 ` [PATCH 1/2] dt-bindings: Consolidate Exynos SoC bindings under Samsung directory Javier Martinez Canillas
2015-10-22 0:22 ` Rob Herring
2015-10-22 0:26 ` Krzysztof Kozlowski [this message]
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=56282D40.2050504@samsung.com \
--to=k.kozlowski@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=javier@osg.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 \
--cc=robh+dt@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).