Re: [PATCH] ARM: dts: Kill off skeleton{64}.dtsi
Vladimir Zapolskiy <[email protected]> Tue, 4 Dec 2018 00:27:36 +0200
| Newsgroups | io.groups.linux-oxnas,org.infradead.lists.linux-amlogic,org.infradead.lists.linux-mediatek,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree |
|---|---|
| Message-ID | <[email protected]> |
On 11/30/2018 07:06 PM, Rob Herring wrote:
> Remove the usage of skeleton.dtsi in the remaining dts files. It was
> deprecated since commit 9c0da3cc61f1 ("ARM: dts: explicitly mark
> skeleton.dtsi as deprecated"). This will make adding a unit-address to
> memory nodes easier.
>
> The main tricky part to removing skeleton.dtsi is we could end up with
> no /memory node at all when a bootloader depends on one being present. I
> hacked up dtc to check for this condition.
>
> Signed-off-by: Rob Herring <[email protected]>
[snip]
> diff --git a/arch/arm/boot/dts/lpc32xx.dtsi b/arch/arm/boot/dts/lpc32xx.dtsi
> index b7303a4e4236..44b468e4c37a 100644
> --- a/arch/arm/boot/dts/lpc32xx.dtsi
> +++ b/arch/arm/boot/dts/lpc32xx.dtsi
> @@ -11,12 +11,12 @@
> * http://www.gnu.org/copyleft/gpl.html
> */
>
> -#include "skeleton.dtsi"
> -
> #include <dt-bindings/clock/lpc32xx-clock.h>
> #include <dt-bindings/interrupt-controller/irq.h>
>
> / {
> + #address-cells = <1>;
> + #size-cells = <1>;
> compatible = "nxp,lpc3220";
> interrupt-parent = <&mic>;
>
For lpc32xx:
Acked-by: Vladimir Zapolskiy <[email protected]>
--
Best wishes,
Vladimir