mbox series

[0/6] Fix MT8186/92/95 topology and idle state names

Message ID 20230126103526.417039-1-angelogioacchino.delregno@collabora.com (mailing list archive)
Headers show
Series Fix MT8186/92/95 topology and idle state names | expand

Message

AngeloGioacchino Del Regno Jan. 26, 2023, 10:35 a.m. UTC
MT8186, MT8192 and MT8195 use ARM DynamIQ, which combines big and
LITTLE cores in one single cluster! A two cluster topology is simply
wrong in this case.

While at it, after some research, I've also noticed that the names
assigned to the idle states are misleading, so I went on changing
them to reflect the actual function.

The names change commits, unlike the CPU map ones, didn't get any
Fixes tag, as naming changes aren't fixing anything that would be
meaningful for functionality (it's cosmetic, nothing else).

AngeloGioacchino Del Regno (6):
  arm64: dts: mt8195: Fix CPU map for single-cluster SoC
  arm64: dts: mt8192: Fix CPU map for single-cluster SoC
  arm64: dts: mt8186: Fix CPU map for single-cluster SoC
  arm64: dts: mt8195: Change idle states names to reflect actual
    function
  arm64: dts: mt8186: Change idle states names to reflect actual
    function
  arm64: dts: mt8192: Change idle states names to reflect actual
    function

 arch/arm64/boot/dts/mediatek/mt8186.dtsi | 30 ++++++++++----------
 arch/arm64/boot/dts/mediatek/mt8192.dtsi | 35 +++++++++++-------------
 arch/arm64/boot/dts/mediatek/mt8195.dtsi | 34 +++++++++++------------
 3 files changed, 46 insertions(+), 53 deletions(-)

Comments

Matthias Brugger Jan. 31, 2023, 11:42 a.m. UTC | #1
On 26/01/2023 11:35, AngeloGioacchino Del Regno wrote:
> MT8186, MT8192 and MT8195 use ARM DynamIQ, which combines big and
> LITTLE cores in one single cluster! A two cluster topology is simply
> wrong in this case.
> 
> While at it, after some research, I've also noticed that the names
> assigned to the idle states are misleading, so I went on changing
> them to reflect the actual function.
> 
> The names change commits, unlike the CPU map ones, didn't get any
> Fixes tag, as naming changes aren't fixing anything that would be
> meaningful for functionality (it's cosmetic, nothing else).
> 
> AngeloGioacchino Del Regno (6):
>    arm64: dts: mt8195: Fix CPU map for single-cluster SoC
>    arm64: dts: mt8192: Fix CPU map for single-cluster SoC
>    arm64: dts: mt8186: Fix CPU map for single-cluster SoC
>    arm64: dts: mt8195: Change idle states names to reflect actual
>      function
>    arm64: dts: mt8186: Change idle states names to reflect actual
>      function
>    arm64: dts: mt8192: Change idle states names to reflect actual
>      function
> 

Whole series applied, thanks!
Matthias

>   arch/arm64/boot/dts/mediatek/mt8186.dtsi | 30 ++++++++++----------
>   arch/arm64/boot/dts/mediatek/mt8192.dtsi | 35 +++++++++++-------------
>   arch/arm64/boot/dts/mediatek/mt8195.dtsi | 34 +++++++++++------------
>   3 files changed, 46 insertions(+), 53 deletions(-)
>