Il 25/09/24 12:57, Fei Shao ha scritto:
The MT8188 SoC features two CPU clusters: one with 6 little Cortex-A55 cores, and the other with 2 big Cortex-A78 cores.
No, it doesn't. It features only one cluster, so...
Update the CPU topology to reflect the actual hardware configurations.
...the actual hardware configuration is already reflected by the currently declared CPU topology, so for this commit: NAK. This SoC uses the ARM DynamIQ technology and embeds both LITTLE and big cores in one single cluster. Check the MT8188 datasheet for more information :-) Cheers, Angelo
Signed-off-by: Fei Shao <fshao@xxxxxxxxxxxx> --- arch/arm64/boot/dts/mediatek/mt8188.dtsi | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi index cd27966d2e3c..51bf08b2ff9b 100644 --- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi @@ -195,12 +195,14 @@ core4 { core5 { cpu = <&cpu5>; }; + };- core6 {+ cluster1 { + core0 { cpu = <&cpu6>; };- core7 {+ core1 { cpu = <&cpu7>; }; };
