mbox series

[v2,00/11] Add Nothing Phone (1) support

Message ID 20240808184048.63030-1-danila@jiaxyga.com
Headers show
Series Add Nothing Phone (1) support | expand

Message

Danila Tikhonov Aug. 8, 2024, 6:40 p.m. UTC
This series of patches adds support for the Nothing Phone (1), identified
as nothing,spacewar. The Nothing Phone (1) is built on the Qualcomm
Snapdragon 778G+ (SM7325-AE, also known as yupik).

SM7325 is identical to SC7280 just as SM7125 is identical to SC7180, so
SM7325 devicetree imports SC7280 devicetree as a base.

All of these patches are essential for the integration of the Nothing
Phone (1) into the kernel. The inclusion of SoC IDs is particularly
important, as I encounter crash dumps if the device tree lacks msm and
board id information.

Changes in v2:
- Add Krzysztof's R-b tag (patches no. 1, 2, 10)
- Add Dmitry's R-b tag (patches no. 3, 4, 8)
- Document SM7325 as fallback to QCM6490 (patch no. 5)
- Drop patch no. 6 from v1
- Document PN553 NFC IC as fallback to nxp-nci-i2c (patch no. 6)
- Add Krzysztof's A-b tag (patches no. 7, 9)
- Switch nl.nothing.tech/nothing.tech in commit msg (patch no. 9)
- Add fallback compatibility for NFC (patch no. 10)
- Fix interrupt type for NFC (patch no. 10)
Note: Rob's A-b tag (patch no. 5) was not added because the patch was
fixed. Please look at it again.
- Link to v1:
https://lore.kernel.org/all/20240729201843.142918-1-danila@jiaxyga.com/

To: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzk+dt@kernel.org>
To: Conor Dooley <conor+dt@kernel.org>
To: Bjorn Andersson <andersson@kernel.org>
To: Konrad Dybcio <konradybcio@kernel.org>
To: "David S. Miller" <davem@davemloft.net>
To: Eric Dumazet <edumazet@google.com>
To: Jakub Kicinski <kuba@kernel.org>
To: Paolo Abeni <pabeni@redhat.com>
To: "Rafael J. Wysocki" <rafael@kernel.org>
To: Viresh Kumar <viresh.kumar@linaro.org>
To: Kees Cook <kees@kernel.org>
To: Tony Luck <tony.luck@intel.com>
To: "Guilherme G. Piccoli" <gpiccoli@igalia.com>
To: Ulf Hansson <ulf.hansson@linaro.org>
To: Andre Przywara <andre.przywara@arm.com>
To: Rajendra Nayak <quic_rjendra@quicinc.com>
To: David Wronek <davidwronek@gmail.com>
To: Neil Armstrong <neil.armstrong@linaro.org>
To: Heiko Stuebner <heiko.stuebner@cherry.de>
To: "Rafał Miłecki" <rafal@milecki.pl>
To: Chris Morgan <macromorgan@hotmail.com>
To: Linus Walleij <linus.walleij@linaro.org>
To: Lorenzo Pieralisi <lpieralisi@kernel.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Eugene Lepshy <fekz115@gmail.com>
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org
Cc: netdev@vger.kernel.org
Cc: linux-pm@vger.kernel.org
Cc: linux-hardening@vger.kernel.org
Cc: linux@mainlining.org
Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>

Danila Tikhonov (9):
  dt-bindings: arm: qcom,ids: Add IDs for SM7325 family
  soc: qcom: socinfo: Add Soc IDs for SM7325 family
  cpufreq: Add SM7325 to cpufreq-dt-platdev blocklist
  soc: qcom: pd_mapper: Add SM7325 compatible
  dt-bindings: soc: qcom: qcom,pmic-glink: Document SM7325 compatible
  dt-bindings: nfc: nxp,nci: Document PN553 compatible
  dt-bindings: arm: cpus: Add qcom kryo670 compatible
  dt-bindings: vendor-prefixes: Add Nothing Technology Limited
  dt-bindings: arm: qcom: Add SM7325 Nothing Phone 1

Eugene Lepshy (2):
  arm64: dts: qcom: Add SM7325 device tree
  arm64: dts: qcom: sm7325: Add device-tree for Nothing Phone 1

 .../devicetree/bindings/arm/cpus.yaml         |    1 +
 .../devicetree/bindings/arm/qcom.yaml         |    6 +
 .../devicetree/bindings/net/nfc/nxp,nci.yaml  |    1 +
 .../bindings/soc/qcom/qcom,pmic-glink.yaml    |    5 +
 .../devicetree/bindings/vendor-prefixes.yaml  |    2 +
 arch/arm64/boot/dts/qcom/Makefile             |    1 +
 .../boot/dts/qcom/sm7325-nothing-spacewar.dts | 1263 +++++++++++++++++
 arch/arm64/boot/dts/qcom/sm7325.dtsi          |   17 +
 drivers/cpufreq/cpufreq-dt-platdev.c          |    1 +
 drivers/soc/qcom/qcom_pd_mapper.c             |    1 +
 drivers/soc/qcom/socinfo.c                    |    2 +
 include/dt-bindings/arm/qcom,ids.h            |    2 +
 12 files changed, 1302 insertions(+)
 create mode 100644 arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dts
 create mode 100644 arch/arm64/boot/dts/qcom/sm7325.dtsi

Comments

Rob Herring (Arm) Aug. 8, 2024, 9:38 p.m. UTC | #1
On Thu, Aug 08, 2024 at 09:40:22PM +0300, Danila Tikhonov wrote:
> From: Eugene Lepshy <fekz115@gmail.com>
> 
> 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@gmail.com>
> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  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@gmail.com>
> + * Copyright (c) 2024, Danila Tikhonov <danila@jiaxyga.com>
> + */
> +
> +#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
Krzysztof Kozlowski Aug. 9, 2024, 5:39 a.m. UTC | #2
On 08/08/2024 20:40, Danila Tikhonov wrote:
> The PN553 is another NFC chip from NXP, document the compatible in the
> bindings.
> 
> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> ---
>  Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> index 6924aff0b2c5..364b36151180 100644
> --- a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> +++ b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> @@ -17,6 +17,7 @@ properties:
>            - enum:
>                - nxp,nq310
>                - nxp,pn547
> +              - nxp,pn553

Keep the list ordered.

Best regards,
Krzysztof
Danila Tikhonov Aug. 9, 2024, 5:38 p.m. UTC | #3
On 8/9/24 08:39, Krzysztof Kozlowski wrote:
> On 08/08/2024 20:40, Danila Tikhonov wrote:
>> The PN553 is another NFC chip from NXP, document the compatible in the
>> bindings.
>>
>> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
>> ---
>>   Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>> index 6924aff0b2c5..364b36151180 100644
>> --- a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>> +++ b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>> @@ -17,6 +17,7 @@ properties:
>>             - enum:
>>                 - nxp,nq310
>>                 - nxp,pn547
>> +              - nxp,pn553
> Keep the list ordered.
>
> Best regards,
> Krzysztof
>
Thanks for your comment.

Then I will fix it for nxp,pn547 too in this patch. Do you mind?

---
Best wishes,
Danila.
Rob Herring (Arm) Aug. 12, 2024, 3:09 p.m. UTC | #4
On Thu, 08 Aug 2024 21:40:14 +0300, Danila Tikhonov wrote:
> This series of patches adds support for the Nothing Phone (1), identified
> as nothing,spacewar. The Nothing Phone (1) is built on the Qualcomm
> Snapdragon 778G+ (SM7325-AE, also known as yupik).
> 
> SM7325 is identical to SC7280 just as SM7125 is identical to SC7180, so
> SM7325 devicetree imports SC7280 devicetree as a base.
> 
> All of these patches are essential for the integration of the Nothing
> Phone (1) into the kernel. The inclusion of SoC IDs is particularly
> important, as I encounter crash dumps if the device tree lacks msm and
> board id information.
> 
> Changes in v2:
> - Add Krzysztof's R-b tag (patches no. 1, 2, 10)
> - Add Dmitry's R-b tag (patches no. 3, 4, 8)
> - Document SM7325 as fallback to QCM6490 (patch no. 5)
> - Drop patch no. 6 from v1
> - Document PN553 NFC IC as fallback to nxp-nci-i2c (patch no. 6)
> - Add Krzysztof's A-b tag (patches no. 7, 9)
> - Switch nl.nothing.tech/nothing.tech in commit msg (patch no. 9)
> - Add fallback compatibility for NFC (patch no. 10)
> - Fix interrupt type for NFC (patch no. 10)
> Note: Rob's A-b tag (patch no. 5) was not added because the patch was
> fixed. Please look at it again.
> - Link to v1:
> https://lore.kernel.org/all/20240729201843.142918-1-danila@jiaxyga.com/
> 
> To: Rob Herring <robh@kernel.org>
> To: Krzysztof Kozlowski <krzk+dt@kernel.org>
> To: Conor Dooley <conor+dt@kernel.org>
> To: Bjorn Andersson <andersson@kernel.org>
> To: Konrad Dybcio <konradybcio@kernel.org>
> To: "David S. Miller" <davem@davemloft.net>
> To: Eric Dumazet <edumazet@google.com>
> To: Jakub Kicinski <kuba@kernel.org>
> To: Paolo Abeni <pabeni@redhat.com>
> To: "Rafael J. Wysocki" <rafael@kernel.org>
> To: Viresh Kumar <viresh.kumar@linaro.org>
> To: Kees Cook <kees@kernel.org>
> To: Tony Luck <tony.luck@intel.com>
> To: "Guilherme G. Piccoli" <gpiccoli@igalia.com>
> To: Ulf Hansson <ulf.hansson@linaro.org>
> To: Andre Przywara <andre.przywara@arm.com>
> To: Rajendra Nayak <quic_rjendra@quicinc.com>
> To: David Wronek <davidwronek@gmail.com>
> To: Neil Armstrong <neil.armstrong@linaro.org>
> To: Heiko Stuebner <heiko.stuebner@cherry.de>
> To: "Rafał Miłecki" <rafal@milecki.pl>
> To: Chris Morgan <macromorgan@hotmail.com>
> To: Linus Walleij <linus.walleij@linaro.org>
> To: Lorenzo Pieralisi <lpieralisi@kernel.org>
> To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> To: Eugene Lepshy <fekz115@gmail.com>
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: linux-arm-msm@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-hardening@vger.kernel.org
> Cc: linux@mainlining.org
> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> 
> Danila Tikhonov (9):
>   dt-bindings: arm: qcom,ids: Add IDs for SM7325 family
>   soc: qcom: socinfo: Add Soc IDs for SM7325 family
>   cpufreq: Add SM7325 to cpufreq-dt-platdev blocklist
>   soc: qcom: pd_mapper: Add SM7325 compatible
>   dt-bindings: soc: qcom: qcom,pmic-glink: Document SM7325 compatible
>   dt-bindings: nfc: nxp,nci: Document PN553 compatible
>   dt-bindings: arm: cpus: Add qcom kryo670 compatible
>   dt-bindings: vendor-prefixes: Add Nothing Technology Limited
>   dt-bindings: arm: qcom: Add SM7325 Nothing Phone 1
> 
> Eugene Lepshy (2):
>   arm64: dts: qcom: Add SM7325 device tree
>   arm64: dts: qcom: sm7325: Add device-tree for Nothing Phone 1
> 
>  .../devicetree/bindings/arm/cpus.yaml         |    1 +
>  .../devicetree/bindings/arm/qcom.yaml         |    6 +
>  .../devicetree/bindings/net/nfc/nxp,nci.yaml  |    1 +
>  .../bindings/soc/qcom/qcom,pmic-glink.yaml    |    5 +
>  .../devicetree/bindings/vendor-prefixes.yaml  |    2 +
>  arch/arm64/boot/dts/qcom/Makefile             |    1 +
>  .../boot/dts/qcom/sm7325-nothing-spacewar.dts | 1263 +++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm7325.dtsi          |   17 +
>  drivers/cpufreq/cpufreq-dt-platdev.c          |    1 +
>  drivers/soc/qcom/qcom_pd_mapper.c             |    1 +
>  drivers/soc/qcom/socinfo.c                    |    2 +
>  include/dt-bindings/arm/qcom,ids.h            |    2 +
>  12 files changed, 1302 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dts
>  create mode 100644 arch/arm64/boot/dts/qcom/sm7325.dtsi
> 
> --
> 2.45.2
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y qcom/sm7325-nothing-spacewar.dtb' for 20240808184048.63030-1-danila@jiaxyga.com:

arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: /: qcom,board-id: False schema does not allow [[65547, 0]]
	from schema $id: http://devicetree.org/schemas/arm/qcom.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: /: qcom,msm-id: False schema does not allow [[475, 65536]]
	from schema $id: http://devicetree.org/schemas/arm/qcom.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: pcie@1c08000: interrupts: [[0, 307, 4], [0, 308, 4], [0, 309, 4], [0, 312, 4], [0, 313, 4], [0, 314, 4], [0, 374, 4], [0, 375, 4]] is too long
	from schema $id: http://devicetree.org/schemas/pci/qcom,pcie-sc7280.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: pcie@1c08000: interrupt-names:0: 'msi' was expected
	from schema $id: http://devicetree.org/schemas/pci/qcom,pcie-sc7280.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: pcie@1c08000: interrupt-names: ['msi0', 'msi1', 'msi2', 'msi3', 'msi4', 'msi5', 'msi6', 'msi7'] is too long
	from schema $id: http://devicetree.org/schemas/pci/qcom,pcie-sc7280.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: usb@8cf8800: interrupt-names: ['pwr_event', 'hs_phy_irq', 'dp_hs_phy_irq', 'dm_hs_phy_irq'] is too short
	from schema $id: http://devicetree.org/schemas/usb/qcom,dwc3.yaml#
arch/arm64/boot/dts/qcom/sm7325-nothing-spacewar.dtb: video-codec@aa00000: iommus: [[68, 8576, 32]] is too short
	from schema $id: http://devicetree.org/schemas/media/qcom,sc7280-venus.yaml#
Rob Herring (Arm) Aug. 13, 2024, 7:33 p.m. UTC | #5
On Fri, Aug 09, 2024 at 07:39:53AM +0200, Krzysztof Kozlowski wrote:
> On 08/08/2024 20:40, Danila Tikhonov wrote:
> > The PN553 is another NFC chip from NXP, document the compatible in the
> > bindings.
> > 
> > Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
> > ---
> >  Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> > index 6924aff0b2c5..364b36151180 100644
> > --- a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> > +++ b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
> > @@ -17,6 +17,7 @@ properties:
> >            - enum:
> >                - nxp,nq310
> >                - nxp,pn547
> > +              - nxp,pn553
> 
> Keep the list ordered.

Looks ordered to me. n before p...

Rob
Krzysztof Kozlowski Aug. 14, 2024, 6 a.m. UTC | #6
On 13/08/2024 21:33, Rob Herring wrote:
> On Fri, Aug 09, 2024 at 07:39:53AM +0200, Krzysztof Kozlowski wrote:
>> On 08/08/2024 20:40, Danila Tikhonov wrote:
>>> The PN553 is another NFC chip from NXP, document the compatible in the
>>> bindings.
>>>
>>> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
>>> ---
>>>  Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>>> index 6924aff0b2c5..364b36151180 100644
>>> --- a/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>>> +++ b/Documentation/devicetree/bindings/net/nfc/nxp,nci.yaml
>>> @@ -17,6 +17,7 @@ properties:
>>>            - enum:
>>>                - nxp,nq310
>>>                - nxp,pn547
>>> +              - nxp,pn553
>>
>> Keep the list ordered.
> 
> Looks ordered to me. n before p...

Now it is... I think I misread 5->3.

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Bjorn Andersson Aug. 15, 2024, 9:56 p.m. UTC | #7
On Thu, 08 Aug 2024 21:40:14 +0300, Danila Tikhonov wrote:
> This series of patches adds support for the Nothing Phone (1), identified
> as nothing,spacewar. The Nothing Phone (1) is built on the Qualcomm
> Snapdragon 778G+ (SM7325-AE, also known as yupik).
> 
> SM7325 is identical to SC7280 just as SM7125 is identical to SC7180, so
> SM7325 devicetree imports SC7280 devicetree as a base.
> 
> [...]

Applied, thanks!

[01/11] dt-bindings: arm: qcom,ids: Add IDs for SM7325 family
        commit: c580e7bfc0cd140b8d3cf73183e08ca8b23326db
[02/11] soc: qcom: socinfo: Add Soc IDs for SM7325 family
        commit: 31150c9e87b4a8fe8e726a6f50ac0933f5075532
[04/11] soc: qcom: pd_mapper: Add SM7325 compatible
        commit: 79b26c110545530fa2945050a2ffbb3c4e270228
[05/11] dt-bindings: soc: qcom: qcom,pmic-glink: Document SM7325 compatible
        commit: e6b666de995e993bcda883ff045164f090e5506d

Best regards,
Danila Tikhonov Aug. 17, 2024, 6:09 p.m. UTC | #8
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@gmail.com>
>>
>> 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@gmail.com>
>> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>>   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@gmail.com>
>> + * Copyright (c) 2024, Danila Tikhonov <danila@jiaxyga.com>
>> + */
>> +
>> +#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
Danila Tikhonov Aug. 18, 2024, 7:33 p.m. UTC | #9
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@gmail.com>
>>
>> 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@gmail.com>
>> Signed-off-by: Danila Tikhonov <danila@jiaxyga.com>
>> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>>   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@gmail.com>
>> + * Copyright (c) 2024, Danila Tikhonov <danila@jiaxyga.com>
>> + */
>> +
>> +#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
A patch has been sent to fix this in SC7280:
https://lore.kernel.org/all/20240818192905.120477-1-danila@jiaxyga.com/

Best wishes,
Danila