Re: [PATCH v3 01/20] dt-bindings: arm: cpus: Add Apple A7-A11 CPU cores

[Nick Chan <towinchenmi@xxxxxxxxx>]

On 16/9/2024 22:34, Krzysztof Kozlowski wrote:
> On Sun, Sep 15, 2024 at 03:58:46PM +0800, Nick Chan wrote:
>> Add the following CPU cores:
>>
>> - apple,cyclone: A7 cores
>> - apple,typhoon: A8 cores
>> - apple,twister: A9 cores
>> - apple,hurricane-zephyr: A10 logical cores
>> - apple,monsoon: A11 performance cores
>> - apple,mistral: A11 efficiency cores
>>
>> In the Apple A10, there are physical performance-efficiency cores that
>> forms logical cores to software depending on the current p-state, and
>> only one type of core may be active at one time.
>>
>> This follows the existing newest-first order.
>>
>> Signed-off-by: Nick Chan <towinchenmi@xxxxxxxxx>
>> ---
>>  Documentation/devicetree/bindings/arm/cpus.yaml | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/cpus.yaml b/Documentation/devicetree/bindings/arm/cpus.yaml
>> index f308ff6c3532..3959e022079f 100644
>> --- a/Documentation/devicetree/bindings/arm/cpus.yaml
>> +++ b/Documentation/devicetree/bindings/arm/cpus.yaml
>> @@ -89,6 +89,12 @@ properties:
>>        - apple,blizzard
>>        - apple,icestorm
>>        - apple,firestorm
>> +      - apple,mistral
>> +      - apple,monsoon
>> +      - apple,hurricane-zephyr
>> +      - apple,twister
>> +      - apple,typhoon
>> +      - apple,cyclone
> 
> Please keep alphabetical order. And no, just because earlier Hector
> added stuff in reversed order, is not a reason to keep doing the same.
Ack. All bindings added in this series except

Documentation/devicetree/bindings/arm/apple.yaml

will be changed to alphabetical order in v2.

> 
> Best regards,
> Krzysztof
> 

Nick Chan