On 8/9/24 00:38, Rob Herring wrote:
On Thu, Aug 08, 2024 at 09:40:22PM +0300, Danila Tikhonov wrote:From: Eugene Lepshy <fekz115@xxxxxxxxx> The Snapdragon 778G (SM7325) / 778G+ (SM7325-AE) / 782G (SM7325-AF) is software-wise very similar to the Snapdragon 7c+ Gen 3 (SC7280). It uses the Kryo670. Signed-off-by: Eugene Lepshy <fekz115@xxxxxxxxx> Signed-off-by: Danila Tikhonov <danila@xxxxxxxxxxx> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> --- arch/arm64/boot/dts/qcom/sm7325.dtsi | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) create mode 100644 arch/arm64/boot/dts/qcom/sm7325.dtsi diff --git a/arch/arm64/boot/dts/qcom/sm7325.dtsi b/arch/arm64/boot/dts/qcom/sm7325.dtsi new file mode 100644 index 000000000000..5b4574484412 --- /dev/null +++ b/arch/arm64/boot/dts/qcom/sm7325.dtsi @@ -0,0 +1,17 @@ +// SPDX-License-Identifier: BSD-3-Clause +/* + * Copyright (c) 2024, Eugene Lepshy <fekz115@xxxxxxxxx> + * Copyright (c) 2024, Danila Tikhonov <danila@xxxxxxxxxxx> + */ + +#include "sc7280.dtsi" + +/* SM7325 uses Kryo 670 */ +&CPU0 { compatible = "qcom,kryo670"; }; +&CPU1 { compatible = "qcom,kryo670"; }; +&CPU2 { compatible = "qcom,kryo670"; }; +&CPU3 { compatible = "qcom,kryo670"; }; +&CPU4 { compatible = "qcom,kryo670"; }; +&CPU5 { compatible = "qcom,kryo670"; }; +&CPU6 { compatible = "qcom,kryo670"; }; +&CPU7 { compatible = "qcom,kryo670"; };No PMU? Because PMUs are also a per CPU model compatible string. I fixed most QCom platforms recently. Rob
Sorry for my long reply and thank you for your feedback. You are absolutely right. But at the moment I can't confirm that the sc7280 doesn't use cortex a55 and a78 either. I would fix that in the sc7280 DTS right away if I could confirm it. I am almost certain that the qcm6490 uses cortex a55 and a78 just like the sm7325 (and I hope Luca Weiss can confirm that too) If anyone with a sc7280 can read /proc/cpuinfo I would be very grateful. Best wishes, Danila
