diff mbox

ARM: dts: explicitly mark skeleton.dtsi as deprecated

Message ID 1472730037-4972-1-git-send-email-mark.rutland@arm.com
State Accepted
Commit 9c0da3cc61f1233c2782e2d3d91e3d0707dd4ba5
Headers show

Commit Message

Mark Rutland Sept. 1, 2016, 11:40 a.m. UTC
As noted in commit 3ebee5a2e141496b ("arm64: dts: kill skeleton.dtsi"),
there are a number of problems with skeleton.dtsi, and it would be
prefereable to remove it entirely. As there are a large number of
existing users, fixing these up will take a while.

This patch adds a note to arm's skeleton.dtsi noting that this is the
case, to make this more obvious and hopefully minimize new uptake of
skeleton.dtsi in the mean time.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>

Cc: Rob Herring <rob.herring@arm.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Olof Johansson <olof@lixom.net>
---
 arch/arm/boot/dts/skeleton.dtsi | 4 ++++
 1 file changed, 4 insertions(+)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Mark Rutland Sept. 1, 2016, 12:46 p.m. UTC | #1
On Thu, Sep 01, 2016 at 12:40:37PM +0100, Mark Rutland wrote:
> As noted in commit 3ebee5a2e141496b ("arm64: dts: kill skeleton.dtsi"),

> there are a number of problems with skeleton.dtsi, and it would be

> prefereable to remove it entirely. As there are a large number of

> existing users, fixing these up will take a while.

> 

> This patch adds a note to arm's skeleton.dtsi noting that this is the

> case, to make this more obvious and hopefully minimize new uptake of

> skeleton.dtsi in the mean time.

> 

> Signed-off-by: Mark Rutland <mark.rutland@arm.com>

> Cc: Rob Herring <rob.herring@arm.com>


This should have been linaro.org. not sure how I managed that.

Mark.

> Cc: Arnd Bergmann <arnd@arndb.de>

> Cc: Olof Johansson <olof@lixom.net>

> ---

>  arch/arm/boot/dts/skeleton.dtsi | 4 ++++

>  1 file changed, 4 insertions(+)

> 

> diff --git a/arch/arm/boot/dts/skeleton.dtsi b/arch/arm/boot/dts/skeleton.dtsi

> index b41d241..28b81d6 100644

> --- a/arch/arm/boot/dts/skeleton.dtsi

> +++ b/arch/arm/boot/dts/skeleton.dtsi

> @@ -1,4 +1,8 @@

>  /*

> + * This file is deprecated, and will be removed once existing users have been

> + * updated. New dts{,i} files should *not* include skeleton.dtsi, and should

> + * instead explicitly provide the below nodes only as required.

> + *

>   * Skeleton device tree; the bare minimum needed to boot; just include and

>   * add a compatible value.  The bootloader will typically populate the memory

>   * node.

> -- 

> 1.9.1

> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rob Herring (Arm) Sept. 1, 2016, 1:10 p.m. UTC | #2
On Thu, Sep 1, 2016 at 7:46 AM, Mark Rutland <mark.rutland@arm.com> wrote:
> On Thu, Sep 01, 2016 at 12:40:37PM +0100, Mark Rutland wrote:

>> As noted in commit 3ebee5a2e141496b ("arm64: dts: kill skeleton.dtsi"),

>> there are a number of problems with skeleton.dtsi, and it would be

>> prefereable to remove it entirely. As there are a large number of

>> existing users, fixing these up will take a while.

>>

>> This patch adds a note to arm's skeleton.dtsi noting that this is the

>> case, to make this more obvious and hopefully minimize new uptake of

>> skeleton.dtsi in the mean time.

>>

>> Signed-off-by: Mark Rutland <mark.rutland@arm.com>

>> Cc: Rob Herring <rob.herring@arm.com>

>

> This should have been linaro.org. not sure how I managed that.


Well, kernel.org really.

I was going to ask if we were getting close with the recent activity,
but I just looked and no there's still a ton.

Acked-by: Rob Herring <robh@kernel.org>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/skeleton.dtsi b/arch/arm/boot/dts/skeleton.dtsi
index b41d241..28b81d6 100644
--- a/arch/arm/boot/dts/skeleton.dtsi
+++ b/arch/arm/boot/dts/skeleton.dtsi
@@ -1,4 +1,8 @@ 
 /*
+ * This file is deprecated, and will be removed once existing users have been
+ * updated. New dts{,i} files should *not* include skeleton.dtsi, and should
+ * instead explicitly provide the below nodes only as required.
+ *
  * Skeleton device tree; the bare minimum needed to boot; just include and
  * add a compatible value.  The bootloader will typically populate the memory
  * node.