linux-amlogic.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Rob Herring <robh@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org,
	linux-rockchip@lists.infradead.org,
	linux-arm-msm@vger.kernel.org, Will Deacon <will.deacon@arm.com>,
	linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	linux-tegra@vger.kernel.org, arm@kernel.org,
	linux-mediatek@lists.infradead.org,
	Matthias Brugger <matthias.bgg@gmail.com>,
	linux-amlogic@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string
Date: Thu, 10 Jan 2019 09:42:57 +0100	[thread overview]
Message-ID: <2664471.Eeqcqp8LJD@diego> (raw)
In-Reply-To: <20190109202934.29304-1-robh@kernel.org>

Am Mittwoch, 9. Januar 2019, 21:29:34 CET schrieb Rob Herring:
> The 'arm,armv8' compatible string is only for software models. It adds
> little value otherwise and is inconsistently used as a fallback on some
> platforms. Remove it from those platforms.
> 
> This fixes warnings generated by the DT schema.
> 
> Reported-by: Michal Simek <michal.simek@xilinx.com>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---

For the Rockchip-parts
Acked-by: Heiko Stuebner <heiko@sntech.de>

> diff --git a/arch/arm64/boot/dts/rockchip/px30.dtsi
> b/arch/arm64/boot/dts/rockchip/px30.dtsi index 9aa8d5ef9e45..eb992d60e6ba
> 100644
> --- a/arch/arm64/boot/dts/rockchip/px30.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/px30.dtsi
> @@ -40,7 +40,7 @@
> 
>  		cpu0: cpu@0 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a35", "arm,armv8";
> +			compatible = "arm,cortex-a35";
>  			reg = <0x0 0x0>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLK>;
> @@ -52,7 +52,7 @@
> 
>  		cpu1: cpu@1 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a35", "arm,armv8";
> +			compatible = "arm,cortex-a35";
>  			reg = <0x0 0x1>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLK>;
> @@ -64,7 +64,7 @@
> 
>  		cpu2: cpu@2 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a35", "arm,armv8";
> +			compatible = "arm,cortex-a35";
>  			reg = <0x0 0x2>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLK>;
> @@ -76,7 +76,7 @@
> 
>  		cpu3: cpu@3 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a35", "arm,armv8";
> +			compatible = "arm,cortex-a35";
>  			reg = <0x0 0x3>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLK>;
> diff --git a/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> b/arch/arm64/boot/dts/rockchip/rk3328.dtsi index ecd7f19c3542..0f72bb33ce86
> 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3328.dtsi
> @@ -37,7 +37,7 @@
> 
>  		cpu0: cpu@0 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x0>;
>  			clocks = <&cru ARMCLK>;
>  			#cooling-cells = <2>;
> @@ -49,7 +49,7 @@
> 
>  		cpu1: cpu@1 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x1>;
>  			clocks = <&cru ARMCLK>;
>  			#cooling-cells = <2>;
> @@ -61,7 +61,7 @@
> 
>  		cpu2: cpu@2 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x2>;
>  			clocks = <&cru ARMCLK>;
>  			#cooling-cells = <2>;
> @@ -73,7 +73,7 @@
> 
>  		cpu3: cpu@3 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x3>;
>  			clocks = <&cru ARMCLK>;
>  			#cooling-cells = <2>;
> diff --git a/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> b/arch/arm64/boot/dts/rockchip/rk3368.dtsi index 7014d10b954c..06e7c31d7d07
> 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> @@ -73,7 +73,7 @@
> 
>  		cpu_l0: cpu@0 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x0>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -81,7 +81,7 @@
> 
>  		cpu_l1: cpu@1 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x1>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -89,7 +89,7 @@
> 
>  		cpu_l2: cpu@2 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x2>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -97,7 +97,7 @@
> 
>  		cpu_l3: cpu@3 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x3>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -105,7 +105,7 @@
> 
>  		cpu_b0: cpu@100 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x100>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -113,7 +113,7 @@
> 
>  		cpu_b1: cpu@101 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x101>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -121,7 +121,7 @@
> 
>  		cpu_b2: cpu@102 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x102>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> @@ -129,7 +129,7 @@
> 
>  		cpu_b3: cpu@103 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x103>;
>  			enable-method = "psci";
>  			#cooling-cells = <2>; /* min followed by max */
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> b/arch/arm64/boot/dts/rockchip/rk3399.dtsi index 6cc1c9fa4ea6..db9d948c0b03
> 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> @@ -68,7 +68,7 @@
> 
>  		cpu_l0: cpu@0 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x0>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKL>;
> @@ -79,7 +79,7 @@
> 
>  		cpu_l1: cpu@1 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x1>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKL>;
> @@ -90,7 +90,7 @@
> 
>  		cpu_l2: cpu@2 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x2>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKL>;
> @@ -101,7 +101,7 @@
> 
>  		cpu_l3: cpu@3 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a53", "arm,armv8";
> +			compatible = "arm,cortex-a53";
>  			reg = <0x0 0x3>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKL>;
> @@ -112,7 +112,7 @@
> 
>  		cpu_b0: cpu@100 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a72", "arm,armv8";
> +			compatible = "arm,cortex-a72";
>  			reg = <0x0 0x100>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKB>;
> @@ -123,7 +123,7 @@
> 
>  		cpu_b1: cpu@101 {
>  			device_type = "cpu";
> -			compatible = "arm,cortex-a72", "arm,armv8";
> +			compatible = "arm,cortex-a72";
>  			reg = <0x0 0x101>;
>  			enable-method = "psci";
>  			clocks = <&cru ARMCLKB>;




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

  parent reply	other threads:[~2019-01-10  8:43 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-09 20:29 [PATCH] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string Rob Herring
2019-01-10  2:27 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8' " Nishanth Menon
2019-01-10 12:02   ` Tero Kristo
2019-01-10 14:16   ` Rob Herring
2019-01-10  6:56 ` Maxime Ripard
2019-01-10  7:09 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm, armv8' " Chanho Min
2019-01-10  7:12 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8' " Manivannan Sadhasivam
2019-01-10  7:48 ` Krzysztof Kozlowski
2019-01-10  8:01 ` yamada.masahiro
2019-01-10  8:06 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm, armv8' " Gregory CLEMENT
2019-01-10  8:13 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8' " Thierry Reding
2019-01-10  8:42 ` Heiko Stübner [this message]
2019-01-10  9:47 ` Simon Horman
2019-01-10 14:45 ` Richter, Robert
2019-01-11 13:57   ` Rob Herring
2019-01-15  8:45     ` Richter, Robert
2019-01-11 14:50 ` Antoine Tenart
2019-01-11 15:32 ` Wei Xu
2019-01-11 16:50 ` Liviu Dudau
2019-01-14 11:27 ` Matthias Brugger
2019-01-14 11:34 ` Michal Simek
2019-01-14 17:33 ` Scott Branden
2019-01-14 18:07 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm, armv8' " Kevin Hilman
2019-01-15  2:00 ` [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8' " Chunyan Zhang
2019-01-15  8:52 ` Richter, Robert
2019-01-28  2:41 ` Jisheng Zhang
2019-01-29 23:17 ` Dinh Nguyen

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=2664471.Eeqcqp8LJD@diego \
    --to=heiko@sntech.de \
    --cc=arm@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-tegra@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=matthias.bgg@gmail.com \
    --cc=robh@kernel.org \
    --cc=will.deacon@arm.com \
    /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).