[PATCH] dt-bindings: cpufreq: remove stale arm_big_little_dt entry

[PATCH] dt-bindings: cpufreq: remove stale arm_big_little_dt entry

[Sudeep Holla]

Most of the ARM platforms used v2 OPP bindings to support big-little
configurations. This arm_big_little_dt binding is incomplete and was
never used.

Commit f174e49e4906 ("cpufreq: remove unused arm_big_little_dt driver")
removes the driver supporting this binding. However the binding was left
unnoticed, so let's get rid of it now.

Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Viresh Kumar <viresh.kumar@linaro.org>
Cc: Rob Herring <robh+dt@kernel.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
---
 .../bindings/cpufreq/arm_big_little_dt.txt    | 65 -------------------
 1 file changed, 65 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt

Hi,

For some reason, I didn't expect the driver to have an entry in the
DT bindings and didn't look at it before. While I was searching references
to big-little that can be removed, I spotted this. Sorry for that.

Regards,
Sudeep

diff --git a/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt b/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt
deleted file mode 100644
index 2aa06ac0fac5..000000000000
--- a/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt
+++ /dev/null
@@ -1,65 +0,0 @@
-Generic ARM big LITTLE cpufreq driver's DT glue
------------------------------------------------
-
-This is DT specific glue layer for generic cpufreq driver for big LITTLE
-systems.
-
-Both required and optional properties listed below must be defined
-under node /cpus/cpu@x. Where x is the first cpu inside a cluster.
-
-FIXME: Cpus should boot in the order specified in DT and all cpus for a cluster
-must be present contiguously. Generic DT driver will check only node 'x' for
-cpu:x.
-
-Required properties:
-- operating-points: Refer to Documentation/devicetree/bindings/opp/opp.txt
-  for details
-
-Optional properties:
-- clock-latency: Specify the possible maximum transition latency for clock,
-  in unit of nanoseconds.
-
-Examples:
-
-cpus {
-	#address-cells = <1>;
-	#size-cells = <0>;
-
-	cpu@0 {
-		compatible = "arm,cortex-a15";
-		reg = <0>;
-		next-level-cache = <&L2>;
-		operating-points = <
-			/* kHz    uV */
-			792000  1100000
-			396000  950000
-			198000  850000
-		>;
-		clock-latency = <61036>; /* two CLK32 periods */
-	};
-
-	cpu@1 {
-		compatible = "arm,cortex-a15";
-		reg = <1>;
-		next-level-cache = <&L2>;
-	};
-
-	cpu@100 {
-		compatible = "arm,cortex-a7";
-		reg = <100>;
-		next-level-cache = <&L2>;
-		operating-points = <
-			/* kHz    uV */
-			792000  950000
-			396000  750000
-			198000  450000
-		>;
-		clock-latency = <61036>; /* two CLK32 periods */
-	};
-
-	cpu@101 {
-		compatible = "arm,cortex-a7";
-		reg = <101>;
-		next-level-cache = <&L2>;
-	};
-};
-- 
2.17.1

Re: [PATCH] dt-bindings: cpufreq: remove stale arm_big_little_dt entry

[Viresh Kumar]

On 07-11-18, 17:40, Sudeep Holla wrote:
> Most of the ARM platforms used v2 OPP bindings to support big-little
> configurations. This arm_big_little_dt binding is incomplete and was
> never used.
> 
> Commit f174e49e4906 ("cpufreq: remove unused arm_big_little_dt driver")
> removes the driver supporting this binding. However the binding was left
> unnoticed, so let's get rid of it now.
> 
> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> Cc: Viresh Kumar <viresh.kumar@linaro.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> ---
>  .../bindings/cpufreq/arm_big_little_dt.txt    | 65 -------------------
>  1 file changed, 65 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt
> 
> Hi,
> 
> For some reason, I didn't expect the driver to have an entry in the
> DT bindings and didn't look at it before. While I was searching references
> to big-little that can be removed, I spotted this. Sorry for that.
> 
> Regards,
> Sudeep
> 
> diff --git a/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt b/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt
> deleted file mode 100644
> index 2aa06ac0fac5..000000000000
> --- a/Documentation/devicetree/bindings/cpufreq/arm_big_little_dt.txt
> +++ /dev/null
> @@ -1,65 +0,0 @@
> -Generic ARM big LITTLE cpufreq driver's DT glue
> ------------------------------------------------
> -
> -This is DT specific glue layer for generic cpufreq driver for big LITTLE
> -systems.
> -
> -Both required and optional properties listed below must be defined
> -under node /cpus/cpu@x. Where x is the first cpu inside a cluster.
> -
> -FIXME: Cpus should boot in the order specified in DT and all cpus for a cluster
> -must be present contiguously. Generic DT driver will check only node 'x' for
> -cpu:x.
> -
> -Required properties:
> -- operating-points: Refer to Documentation/devicetree/bindings/opp/opp.txt
> -  for details
> -
> -Optional properties:
> -- clock-latency: Specify the possible maximum transition latency for clock,
> -  in unit of nanoseconds.
> -
> -Examples:
> -
> -cpus {
> -	#address-cells = <1>;
> -	#size-cells = <0>;
> -
> -	cpu@0 {
> -		compatible = "arm,cortex-a15";
> -		reg = <0>;
> -		next-level-cache = <&L2>;
> -		operating-points = <
> -			/* kHz    uV */
> -			792000  1100000
> -			396000  950000
> -			198000  850000
> -		>;
> -		clock-latency = <61036>; /* two CLK32 periods */
> -	};
> -
> -	cpu@1 {
> -		compatible = "arm,cortex-a15";
> -		reg = <1>;
> -		next-level-cache = <&L2>;
> -	};
> -
> -	cpu@100 {
> -		compatible = "arm,cortex-a7";
> -		reg = <100>;
> -		next-level-cache = <&L2>;
> -		operating-points = <
> -			/* kHz    uV */
> -			792000  950000
> -			396000  750000
> -			198000  450000
> -		>;
> -		clock-latency = <61036>; /* two CLK32 periods */
> -	};
> -
> -	cpu@101 {
> -		compatible = "arm,cortex-a7";
> -		reg = <101>;
> -		next-level-cache = <&L2>;
> -	};
> -};

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

-- 
viresh