| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <d1fe3178-ac31-4db5-ac58-89866f308f46@quicinc.com>
Date: Thu, 21 Dec 2023 15:54:44 +0800
From: Tengfei Fan <quic_tengfan@...cinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
CC: <andersson@...nel.org>, <konrad.dybcio@...aro.org>, <robh+dt@...nel.org>,
<krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
<linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, Fenglin Wu <quic_fenglinw@...cinc.com>,
<kernel@...cinc.com>
Subject: Re: [PATCH v3 5/6] arm64: dts: qcom: add base AIM300 dtsi
On 12/20/2023 3:00 PM, Dmitry Baryshkov wrote:
> On Wed, 20 Dec 2023 at 03:23, Tengfei Fan <quic_tengfan@...cinc.com> wrote:
>>
>>
>>
>> On 12/19/2023 6:25 PM, Dmitry Baryshkov wrote:
>>> On Tue, 19 Dec 2023 at 02:52, Tengfei Fan <quic_tengfan@...cinc.com> wrote:
>>>>
>>>> AIM300 is a module with QCS8550 SoC, UFS and PMIC chip.
>>>> AIM300 Series is a highly optimized family of modules designed to support
>>>> AIoT applications. The Module is mounted onto Qualcomm AIoT carrier board
>>>> to support verification, evaluation and development.
>>>>
>>>> Co-developed-by: Fenglin Wu <quic_fenglinw@...cinc.com>
>>>> Signed-off-by: Fenglin Wu <quic_fenglinw@...cinc.com>
>>>> Signed-off-by: Tengfei Fan <quic_tengfan@...cinc.com>
>>>> ---
>>>> arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi | 371 +++++++++++++++++++
>>>> 1 file changed, 371 insertions(+)
>>>> create mode 100644 arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
>>>>
>>>> diff --git a/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi b/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
>>>> new file mode 100644
>>>> index 000000000000..04ca4597f952
>>>> --- /dev/null
>>>> +++ b/arch/arm64/boot/dts/qcom/qcs8550-aim300.dtsi
>>>> @@ -0,0 +1,371 @@
>>>> +// SPDX-License-Identifier: BSD-3-Clause
>>>> +/*
>>>> + * Copyright (c) 2023, Qualcomm Innovation Center, Inc. All rights reserved.
>>>> + */
>>>> +
>>>> +#include <dt-bindings/regulator/qcom,rpmh-regulator.h>
>>>> +#include "qcs8550.dtsi"
>>>> +#include "pm8010.dtsi"
>>>> +#include "pm8550.dtsi"
>>>> +#include "pm8550b.dtsi"
>>>> +#define PMK8550VE_SID 5
>>>> +#include "pm8550ve.dtsi"
>>>> +#include "pm8550vs.dtsi"
>>>> +#include "pmk8550.dtsi"
>>>> +#include "pmr735d_a.dtsi"
>>>> +#include "pmr735d_b.dtsi"
>>>> +
>>>> +/ {
>>>> + vph_pwr: vph-pwr-regulator {
>>>> + compatible = "regulator-fixed";
>>>> + regulator-name = "vph_pwr";
>>>> + regulator-min-microvolt = <3700000>;
>>>> + regulator-max-microvolt = <3700000>;
>>>> +
>>>> + regulator-always-on;
>>>> + regulator-boot-on;
>>>> + };
>>>> +};
>>>
>>> Is VPH_PWR regulator present on the SoM or on the carrier board?VPH_PWR regulator present on the SoM.
>
> If not for gmail, I could have missed your reply.
>
> Maybe it is a good idea to add an explicit comment here.
________________________________________
|AIM300 SoM |
| _____ |
| | | |
| |--->| UFS | |
| | |_____| |
| | |
| _________________ | _________ |
3.8v | | | | | | |
-------->| Regulatorss |----->| QCS8550 | |
| |_________________| | |_________| |
| | |
| | _____ |
| | | | |
| |--->| ... | |
| |_____| |
| |
|________________________________________|
VPH_PWR is obtained by vonverting 12v voltage into 3.8 voltage with a
3.8v buck. VPH_PWR is power supply for regulators in AIM300 SOM.
VPH_PWR regulator is defined in AIM300 SOM dtsi file.
>
> BTW: what regulators are present on the carrier board rather than on the SoM?
The regulators in pm8550, pm8550ve and pm8550vs are present on the SoM.
The regulators in pm8010 and pmr73d are present on the carrier board.
In the next version patch series, pm8010 and pmr73d will be moved from
SOM dtsi to carrier board dts file.
>
>>>
>>>> +
>>>> +&apps_rsc {
>>>> + regulators-0 {
>>>> + compatible = "qcom,pm8550-rpmh-regulators";
>>>> + qcom,pmic-id = "b";
>>>> +
>>>> + vdd-bob1-supply = <&vph_pwr>;
>>>> + vdd-bob2-supply = <&vph_pwr>;
>>>> + vdd-l1-l4-l10-supply = <&vreg_s6g_1p86>;
>>>> + vdd-l2-l13-l14-supply = <&vreg_bob1>;
>>>> + vdd-l3-supply = <&vreg_s4g_1p25>;
>>>> + vdd-l5-l16-supply = <&vreg_bob1>;
>>>> + vdd-l6-l7-supply = <&vreg_bob1>;
>>>> + vdd-l8-l9-supply = <&vreg_bob1>;
>>>> + vdd-l11-supply = <&vreg_s4g_1p25>;
>>>> + vdd-l12-supply = <&vreg_s6g_1p86>;
>>>> + vdd-l15-supply = <&vreg_s6g_1p86>;
>>>> + vdd-l17-supply = <&vreg_bob2>;
>>>> +
>>>> + vreg_bob1: bob1 {
>>>> + regulator-name = "vreg_bob1";
>>>> + regulator-min-microvolt = <3296000>;
>>>> + regulator-max-microvolt = <3960000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_bob2: bob2 {
>>>> + regulator-name = "vreg_bob2";
>>>> + regulator-min-microvolt = <2720000>;
>>>> + regulator-max-microvolt = <3960000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l1b_1p8: ldo1 {
>>>> + regulator-name = "vreg_l1b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <1800000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l2b_3p0: ldo2 {
>>>> + regulator-name = "vreg_l2b_3p0";
>>>> + regulator-min-microvolt = <3008000>;
>>>> + regulator-max-microvolt = <3008000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l5b_3p1: ldo5 {
>>>> + regulator-name = "vreg_l5b_3p1";
>>>> + regulator-min-microvolt = <3104000>;
>>>> + regulator-max-microvolt = <3104000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l6b_1p8: ldo6 {
>>>> + regulator-name = "vreg_l6b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <3008000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l7b_1p8: ldo7 {
>>>> + regulator-name = "vreg_l7b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <3008000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l8b_1p8: ldo8 {
>>>> + regulator-name = "vreg_l8b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <3008000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l9b_2p9: ldo9 {
>>>> + regulator-name = "vreg_l9b_2p9";
>>>> + regulator-min-microvolt = <2960000>;
>>>> + regulator-max-microvolt = <3008000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l11b_1p2: ldo11 {
>>>> + regulator-name = "vreg_l11b_1p2";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1504000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l12b_1p8: ldo12 {
>>>> + regulator-name = "vreg_l12b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <1800000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l13b_3p0: ldo13 {
>>>> + regulator-name = "vreg_l13b_3p0";
>>>> + regulator-min-microvolt = <3000000>;
>>>> + regulator-max-microvolt = <3000000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l14b_3p2: ldo14 {
>>>> + regulator-name = "vreg_l14b_3p2";
>>>> + regulator-min-microvolt = <3200000>;
>>>> + regulator-max-microvolt = <3200000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l15b_1p8: ldo15 {
>>>> + regulator-name = "vreg_l15b_1p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <1800000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l16b_2p8: ldo16 {
>>>> + regulator-name = "vreg_l16b_2p8";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <3300000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l17b_2p5: ldo17 {
>>>> + regulator-name = "vreg_l17b_2p5";
>>>> + regulator-min-microvolt = <2504000>;
>>>> + regulator-max-microvolt = <2504000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +
>>>> + regulators-1 {
>>>> + compatible = "qcom,pm8550vs-rpmh-regulators";
>>>> + qcom,pmic-id = "c";
>>>> +
>>>> + vdd-l1-supply = <&vreg_s4g_1p25>;
>>>> + vdd-l2-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l3-supply = <&vreg_s4e_0p95>;
>>>> +
>>>> + vreg_l3c_0p9: ldo3 {
>>>> + regulator-name = "vreg_l3c_0p9";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <912000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +
>>>> + regulators-2 {
>>>> + compatible = "qcom,pm8550vs-rpmh-regulators";
>>>> + qcom,pmic-id = "d";
>>>> +
>>>> + vdd-l1-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l2-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l3-supply = <&vreg_s4e_0p95>;
>>>> +
>>>> + vreg_l1d_0p88: ldo1 {
>>>> + regulator-name = "vreg_l1d_0p88";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <920000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +
>>>> + regulators-3 {
>>>> + compatible = "qcom,pm8550vs-rpmh-regulators";
>>>> + qcom,pmic-id = "e";
>>>> +
>>>> + vdd-l1-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l2-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l3-supply = <&vreg_s4g_1p25>;
>>>> + vdd-s4-supply = <&vph_pwr>;
>>>> + vdd-s5-supply = <&vph_pwr>;
>>>> +
>>>> + vreg_s4e_0p95: smps4 {
>>>> + regulator-name = "vreg_s4e_0p95";
>>>> + regulator-min-microvolt = <904000>;
>>>> + regulator-max-microvolt = <984000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s5e_1p08: smps5 {
>>>> + regulator-name = "vreg_s5e_1p08";
>>>> + regulator-min-microvolt = <1010000>;
>>>> + regulator-max-microvolt = <1120000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l1e_0p88: ldo1 {
>>>> + regulator-name = "vreg_l1e_0p88";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <912000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l2e_0p9: ldo2 {
>>>> + regulator-name = "vreg_l2e_0p9";
>>>> + regulator-min-microvolt = <870000>;
>>>> + regulator-max-microvolt = <970000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l3e_1p2: ldo3 {
>>>> + regulator-name = "vreg_l3e_1p2";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1200000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +
>>>> + regulators-4 {
>>>> + compatible = "qcom,pm8550ve-rpmh-regulators";
>>>> + qcom,pmic-id = "f";
>>>> +
>>>> + vdd-l1-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l2-supply = <&vreg_s4e_0p95>;
>>>> + vdd-l3-supply = <&vreg_s4e_0p95>;
>>>> + vdd-s4-supply = <&vph_pwr>;
>>>> +
>>>> + vreg_s4f_0p5: smps4 {
>>>> + regulator-name = "vreg_s4f_0p5";
>>>> + regulator-min-microvolt = <500000>;
>>>> + regulator-max-microvolt = <600000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l1f_0p9: ldo1 {
>>>> + regulator-name = "vreg_l1f_0p9";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <912000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l2f_0p88: ldo2 {
>>>> + regulator-name = "vreg_l2f_0p88";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <912000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l3f_0p88: ldo3 {
>>>> + regulator-name = "vreg_l3f_0p88";
>>>> + regulator-min-microvolt = <880000>;
>>>> + regulator-max-microvolt = <912000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +
>>>> + regulators-5 {
>>>> + compatible = "qcom,pm8550vs-rpmh-regulators";
>>>> + qcom,pmic-id = "g";
>>>> + vdd-l1-supply = <&vreg_s4g_1p25>;
>>>> + vdd-l2-supply = <&vreg_s4g_1p25>;
>>>> + vdd-l3-supply = <&vreg_s4g_1p25>;
>>>> + vdd-s1-supply = <&vph_pwr>;
>>>> + vdd-s2-supply = <&vph_pwr>;
>>>> + vdd-s3-supply = <&vph_pwr>;
>>>> + vdd-s4-supply = <&vph_pwr>;
>>>> + vdd-s5-supply = <&vph_pwr>;
>>>> + vdd-s6-supply = <&vph_pwr>;
>>>> +
>>>> + vreg_s1g_1p25: smps1 {
>>>> + regulator-name = "vreg_s1g_1p25";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1300000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s2g_0p85: smps2 {
>>>> + regulator-name = "vreg_s2g_0p85";
>>>> + regulator-min-microvolt = <800000>;
>>>> + regulator-max-microvolt = <1000000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s3g_0p8: smps3 {
>>>> + regulator-name = "vreg_s3g_0p8";
>>>> + regulator-min-microvolt = <300000>;
>>>> + regulator-max-microvolt = <1004000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s4g_1p25: smps4 {
>>>> + regulator-name = "vreg_s4g_1p25";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1300000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s5g_0p85: smps5 {
>>>> + regulator-name = "vreg_s5g_0p85";
>>>> + regulator-min-microvolt = <500000>;
>>>> + regulator-max-microvolt = <1004000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_s6g_1p86: smps6 {
>>>> + regulator-name = "vreg_s6g_1p86";
>>>> + regulator-min-microvolt = <1800000>;
>>>> + regulator-max-microvolt = <2000000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l1g_1p2: ldo1 {
>>>> + regulator-name = "vreg_l1g_1p2";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1200000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l2g_1p2: ldo2 {
>>>> + regulator-name = "vreg_l2g_1p2";
>>>> + regulator-min-microvolt = <1100000>;
>>>> + regulator-max-microvolt = <1200000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> +
>>>> + vreg_l3g_1p2: ldo3 {
>>>> + regulator-name = "vreg_l3g_1p2";
>>>> + regulator-min-microvolt = <1200000>;
>>>> + regulator-max-microvolt = <1200000>;
>>>> + regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
>>>> + };
>>>> + };
>>>> +};
>>>> +
>>>> +&ufs_mem_hc {
>>>> + reset-gpios = <&tlmm 210 GPIO_ACTIVE_LOW>;
>>>> + vcc-supply = <&vreg_l17b_2p5>;
>>>> + vcc-max-microamp = <1300000>;
>>>> + vccq-supply = <&vreg_l1g_1p2>;
>>>> + vccq-max-microamp = <1200000>;
>>>> + vdd-hba-supply = <&vreg_l3g_1p2>;
>>>> +
>>>> + status = "okay";
>>>> +};
>>>> +
>>>> +&ufs_mem_phy {
>>>> + vdda-phy-supply = <&vreg_l1d_0p88>;
>>>> + vdda-pll-supply = <&vreg_l3e_1p2>;
>>>> +
>>>> + status = "okay";
>>>> +};
>>>> --
>>>> 2.17.1
>>>>
>>>>
>>>
>>>
>>
>> --
>> Thx and BRs,
>> Tengfei Fan
>
>
>
--
Thx and BRs,
Tengfei Fan
Powered by blists - more mailing lists