[cip-dev] [PATCH 4.19.y-cip v2 56/57] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string

Fabrizio Castro
2019-08-30T08:59:35Z

Hi,

>It's not that important, but why is there a space between 'arm,' and 'armv8'
>in subject?
>Original subject does not have space.

It is actually VERY, VERY, important, as the email I have sent out doesn't contain the space.
I have even run hexdump on the patch file on my disk, it confirms there is no space.
I am replying to my own email, and as you can see, there is no space.

>The following patch have similar problem.
>soc: renesas: rcar-sysc: Remove rcar_sysc_power_{down, up}() helpers

Same thing here, there is no space in the email I have sent out, which means it's
probably down to patchwork? Somehow a space gets automatically added, I wonder if
this is happening with the code too?

How do we find out if the problem is with patchwork?

Thanks,
Fab

> From: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Sent: 28 August 2019 14:33
> Subject: [cip-dev][PATCH 4.19.y-cip v2 56/57] arm64: dts: Remove inconsistent use of 'arm,armv8' compatible string
>
> From: Rob Herring <robh@kernel.org>
>
> commit 31af04cd60d3162a58213363fd740a2b0cf0a08e upstream.
>
> 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>
> Acked-by: Antoine Tenart <antoine.tenart@bootlin.com>
> Acked-by: Nishanth Menon <nm@ti.com>
> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
> Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Acked-by: Chanho Min <chanho.min@lge.com>
> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Acked-by: Gregory CLEMENT <gregory.clement@bootlin.com>
> Acked-by: Thierry Reding <treding@nvidia.com>
> Acked-by: Heiko Stuebner <heiko@sntech.de>
> Acked-by: Simon Horman <horms+renesas@verge.net.au>
> Acked-by: Tero Kristo <t-kristo@ti.com>
> Acked-by: Wei Xu <xuwei5@hisilicon.com>
> Acked-by: Liviu Dudau <liviu.dudau@arm.com>
> Acked-by: Matthias Brugger <matthias.bgg@gmail.com>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
> Acked-by: Scott Branden <scott.branden@broadcom.com>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
> Acked-by: Chunyan Zhang <zhang.lyra@gmail.com>
> Acked-by: Robert Richter <rrichter@cavium.com>
> Acked-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
> Acked-by: Dinh Nguyen <dinguyen@kernel.org>
> Signed-off-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> [fab: dropped changes not related to r8a774a1.dtsi]
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> ---
> arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
> index 1969649..a77de9e 100644
> --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi
> @@ -61,7 +61,7 @@
> #size-cells = <0>;
>
> a57_0: cpu at 0 {
> - compatible = "arm,cortex-a57", "arm,armv8";
> + compatible = "arm,cortex-a57";
> reg = <0x0>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA57_CPU0>;
> @@ -71,7 +71,7 @@
> };
>
> a57_1: cpu at 1 {
> - compatible = "arm,cortex-a57", "arm,armv8";
> + compatible = "arm,cortex-a57";
> reg = <0x1>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA57_CPU1>;
> @@ -81,7 +81,7 @@
> };
>
> a53_0: cpu at 100 {
> - compatible = "arm,cortex-a53", "arm,armv8";
> + compatible = "arm,cortex-a53";
> reg = <0x100>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA53_CPU0>;
> @@ -91,7 +91,7 @@
> };
>
> a53_1: cpu at 101 {
> - compatible = "arm,cortex-a53", "arm,armv8";
> + compatible = "arm,cortex-a53";
> reg = <0x101>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA53_CPU1>;
> @@ -101,7 +101,7 @@
> };
>
> a53_2: cpu at 102 {
> - compatible = "arm,cortex-a53", "arm,armv8";
> + compatible = "arm,cortex-a53";
> reg = <0x102>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA53_CPU2>;
> @@ -111,7 +111,7 @@
> };
>
> a53_3: cpu at 103 {
> - compatible = "arm,cortex-a53", "arm,armv8";
> + compatible = "arm,cortex-a53";
> reg = <0x103>;
> device_type = "cpu";
> power-domains = <&sysc R8A774A1_PD_CA53_CPU3>;
> --
> 2.7.4