linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Stephan Gerhold <stephan@gerhold.net>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/4] ARM: dts: ux500: Break out DB8500 DTSI
Date: Tue, 26 Nov 2019 19:03:26 +0100	[thread overview]
Message-ID: <20191126180326.GB12811@gerhold.net> (raw)
In-Reply-To: <20191126124738.77690-1-linus.walleij@linaro.org>

On Tue, Nov 26, 2019 at 01:47:35PM +0100, Linus Walleij wrote:
> The DB8500 exists in an enhanced variant named DB8520
> for some machines. To clearly distinguish between the
> different machines, create an explicit db8500.dtsi
> and move the operating points (only known difference so
> far) to that file, so we can add an explicit db8520.dtsi
> after this.
> 
> Cc: Stephan Gerhold <stephan@gerhold.net>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Reviewed-by: Stephan Gerhold <stephan@gerhold.net>

> ---
>  arch/arm/boot/dts/ste-db8500.dtsi           | 15 +++++++++++++++
>  arch/arm/boot/dts/ste-dbx5x0.dtsi           |  5 -----
>  arch/arm/boot/dts/ste-href.dtsi             |  1 -
>  arch/arm/boot/dts/ste-hrefprev60-stuib.dts  |  3 +--
>  arch/arm/boot/dts/ste-hrefprev60-tvk.dts    |  1 +
>  arch/arm/boot/dts/ste-hrefprev60.dtsi       |  1 -
>  arch/arm/boot/dts/ste-hrefv60plus-stuib.dts |  3 +--
>  arch/arm/boot/dts/ste-hrefv60plus-tvk.dts   |  1 +
>  arch/arm/boot/dts/ste-hrefv60plus.dtsi      |  1 -
>  arch/arm/boot/dts/ste-snowball.dts          |  2 +-
>  10 files changed, 20 insertions(+), 13 deletions(-)
>  create mode 100644 arch/arm/boot/dts/ste-db8500.dtsi
> 
> diff --git a/arch/arm/boot/dts/ste-db8500.dtsi b/arch/arm/boot/dts/ste-db8500.dtsi
> new file mode 100644
> index 000000000000..d309fad32229
> --- /dev/null
> +++ b/arch/arm/boot/dts/ste-db8500.dtsi
> @@ -0,0 +1,15 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +
> +#include "ste-dbx5x0.dtsi"
> +
> +/ {
> +	cpus {
> +		cpu@300 {
> +			/* cpufreq controls */
> +			operating-points = <998400 0
> +					    800000 0
> +					    400000 0
> +					    200000 0>;
> +		};
> +	};
> +};
> diff --git a/arch/arm/boot/dts/ste-dbx5x0.dtsi b/arch/arm/boot/dts/ste-dbx5x0.dtsi
> index d4a55369452d..4e02b1b6c0bf 100644
> --- a/arch/arm/boot/dts/ste-dbx5x0.dtsi
> +++ b/arch/arm/boot/dts/ste-dbx5x0.dtsi
> @@ -51,11 +51,6 @@
>  			device_type = "cpu";
>  			compatible = "arm,cortex-a9";
>  			reg = <0x300>;
> -			/* cpufreq controls */
> -			operating-points = <998400 0
> -					    800000 0
> -					    400000 0
> -					    200000 0>;
>  			clocks = <&prcmu_clk PRCMU_ARMSS>;
>  			clock-names = "cpu";
>  			clock-latency = <20000>;
> diff --git a/arch/arm/boot/dts/ste-href.dtsi b/arch/arm/boot/dts/ste-href.dtsi
> index 5eafd5d8a8cd..33e3b0b3c53d 100644
> --- a/arch/arm/boot/dts/ste-href.dtsi
> +++ b/arch/arm/boot/dts/ste-href.dtsi
> @@ -4,7 +4,6 @@
>   */
>  
>  #include <dt-bindings/interrupt-controller/irq.h>
> -#include "ste-dbx5x0.dtsi"
>  #include "ste-href-family-pinctrl.dtsi"
>  
>  / {
> diff --git a/arch/arm/boot/dts/ste-hrefprev60-stuib.dts b/arch/arm/boot/dts/ste-hrefprev60-stuib.dts
> index d2405133860a..8ce6b723abf2 100644
> --- a/arch/arm/boot/dts/ste-hrefprev60-stuib.dts
> +++ b/arch/arm/boot/dts/ste-hrefprev60-stuib.dts
> @@ -4,8 +4,7 @@
>   */
>  
>  /dts-v1/;
> -#include <dt-bindings/gpio/gpio.h>
> -#include <dt-bindings/interrupt-controller/irq.h>
> +#include "ste-db8500.dtsi"
>  #include "ste-hrefprev60.dtsi"
>  #include "ste-href-stuib.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-hrefprev60-tvk.dts b/arch/arm/boot/dts/ste-hrefprev60-tvk.dts
> index 54b0f8282b2c..8b88c91dca6c 100644
> --- a/arch/arm/boot/dts/ste-hrefprev60-tvk.dts
> +++ b/arch/arm/boot/dts/ste-hrefprev60-tvk.dts
> @@ -4,6 +4,7 @@
>   */
>  
>  /dts-v1/;
> +#include "ste-db8500.dtsi"
>  #include "ste-hrefprev60.dtsi"
>  #include "ste-href-tvk1281618.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-hrefprev60.dtsi b/arch/arm/boot/dts/ste-hrefprev60.dtsi
> index 937f942f0961..115495de8612 100644
> --- a/arch/arm/boot/dts/ste-hrefprev60.dtsi
> +++ b/arch/arm/boot/dts/ste-hrefprev60.dtsi
> @@ -5,7 +5,6 @@
>   * Device Tree for the HREF+ prior to the v60 variant.
>   */
>  
> -#include "ste-dbx5x0.dtsi"
>  #include "ste-href-ab8500.dtsi"
>  #include "ste-href.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-hrefv60plus-stuib.dts b/arch/arm/boot/dts/ste-hrefv60plus-stuib.dts
> index 36163c0b5267..1316886e6bcb 100644
> --- a/arch/arm/boot/dts/ste-hrefv60plus-stuib.dts
> +++ b/arch/arm/boot/dts/ste-hrefv60plus-stuib.dts
> @@ -6,8 +6,7 @@
>   */
>  
>  /dts-v1/;
> -#include <dt-bindings/gpio/gpio.h>
> -#include <dt-bindings/interrupt-controller/irq.h>
> +#include "ste-db8500.dtsi"
>  #include "ste-hrefv60plus.dtsi"
>  #include "ste-href-stuib.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-hrefv60plus-tvk.dts b/arch/arm/boot/dts/ste-hrefv60plus-tvk.dts
> index cf59e9bb9a74..b31e43d5ff89 100644
> --- a/arch/arm/boot/dts/ste-hrefv60plus-tvk.dts
> +++ b/arch/arm/boot/dts/ste-hrefv60plus-tvk.dts
> @@ -6,6 +6,7 @@
>   */
>  
>  /dts-v1/;
> +#include "ste-db8500.dtsi"
>  #include "ste-hrefv60plus.dtsi"
>  #include "ste-href-tvk1281618.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-hrefv60plus.dtsi b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
> index aece8eb2924f..05b4fbbba57f 100644
> --- a/arch/arm/boot/dts/ste-hrefv60plus.dtsi
> +++ b/arch/arm/boot/dts/ste-hrefv60plus.dtsi
> @@ -3,7 +3,6 @@
>   * Copyright 2012 ST-Ericsson AB
>   */
>  
> -#include "ste-dbx5x0.dtsi"
>  #include "ste-href-ab8500.dtsi"
>  #include "ste-href.dtsi"
>  
> diff --git a/arch/arm/boot/dts/ste-snowball.dts b/arch/arm/boot/dts/ste-snowball.dts
> index f8dec3976c91..be90e73c923e 100644
> --- a/arch/arm/boot/dts/ste-snowball.dts
> +++ b/arch/arm/boot/dts/ste-snowball.dts
> @@ -4,7 +4,7 @@
>   */
>  
>  /dts-v1/;
> -#include "ste-dbx5x0.dtsi"
> +#include "ste-db8500.dtsi"
>  #include "ste-href-ab8500.dtsi"
>  #include "ste-href-family-pinctrl.dtsi"
>  
> -- 
> 2.23.0
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

      parent reply	other threads:[~2019-11-26 18:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-26 12:47 [PATCH 1/4] ARM: dts: ux500: Break out DB8500 DTSI Linus Walleij
2019-11-26 12:47 ` [PATCH 2/4] ARM: dts: ux500: Split TVK DTSI files in two Linus Walleij
2019-11-26 12:47 ` [PATCH 3/4] ARM: dts: ux500: Add devicetree for HREF520 Linus Walleij
2019-11-26 12:47 ` [PATCH 4/4] ARM: dts: ux500: Fix up DSI controller nodes Linus Walleij
2019-11-26 18:14   ` Stephan Gerhold
2019-11-27 14:04     ` Linus Walleij
2019-11-26 18:03 ` Stephan Gerhold [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=20191126180326.GB12811@gerhold.net \
    --to=stephan@gerhold.net \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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).