linux-kernel - Re: [PATCH 3/3] arm64: dts: qcom: pm6150: define USB-C related blocks

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
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b0b732b8-456a-4021-8277-cd51f01ead17@jiaxyga.com>
Date: Sun, 18 Feb 2024 11:05:21 +0300
From: Danila Tikhonov <danila@...xyga.com>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>, andersson@...nel.org,
 konrad.dybcio@...aro.org, lgirdwood@...il.com, broonie@...nel.org,
 robh@...nel.org, krzysztof.kozlowski+dt@...aro.org, conor+dt@...nel.org,
 gregkh@...uxfoundation.org, quic_wcheng@...cinc.com
Cc: linux-arm-msm@...r.kernel.org, linux-kernel@...r.kernel.org,
 devicetree@...r.kernel.org, linux-usb@...r.kernel.org, danila@...xyga.com
Subject: Re: [PATCH 3/3] arm64: dts: qcom: pm6150: define USB-C related blocks

I know that some interrupts have both for PM8150B, but for PM6150 all 
interrupts are rising.
Please look at the downstream kernel:
https://git.codelinaro.org/clo/la/kernel/msm-4.14/-/blob/187022f2721d584ac4ec92c0ac1af77da487521d/arch/arm64/boot/dts/qcom/pm6150.dtsi#L319
https://git.codelinaro.org/clo/la/kernel/msm-4.14/-/blob/187022f2721d584ac4ec92c0ac1af77da487521d/arch/arm64/boot/dts/qcom/pm8150b.dtsi#L292

---
Best wishes
Danila

On 2/18/24 02:19, Bryan O'Donoghue wrote:
> On 17/02/2024 16:32, Danila Tikhonov wrote:
>> Define VBUS regulator and the Type-C handling block as present on the
>> Quacomm PM6150 PMIC.
>>
>> Signed-off-by: Danila Tikhonov <danila@...xyga.com>
>
>> +        pm6150_typec: typec@...0 {
>> +            compatible = "qcom,pm6150-typec,
>> +                      qcom,pm8150b-typec";
>> +            reg = <0x1500>, <0x1700>;
>> +            interrupts = <0x0 0x15 0x00 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x01 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x02 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x03 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x04 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x05 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x06 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x15 0x07 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x00 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x01 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x02 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x03 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x04 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x05 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x06 IRQ_TYPE_EDGE_RISING>,
>> +                     <0x0 0x17 0x07 IRQ_TYPE_EDGE_RISING>;
>> +            interrupt-names = "or-rid-detect-change",
>> +                      "vpd-detect",
>> +                      "cc-state-change",
>> +                      "vconn-oc",
>> +                      "vbus-change",
>> +                      "attach-detach",
>> +                      "legacy-cable-detect",
>> +                      "try-snk-src-detect",
>> +                      "sig-tx",
>> +                      "sig-rx",
>> +                      "msg-tx",
>> +                      "msg-rx",
>> +                      "msg-tx-failed",
>> +                      "msg-tx-discarded",
>> +                      "msg-rx-discarded",
>> +                      "fr-swap";
>> +            status = "disabled";
>> +        };
>
> Should all of these be rising ? Looks incorrect to me.
>
> Please review: arch/arm64/boot/dts/qcom/pm8150b.dtsi
>
> pm8150b_typec: typec@...0 {
>         compatible = "qcom,pm8150b-typec";
>         status = "disabled";
>         reg = <0x1500>,
>               <0x1700>;
>
>     interrupts = <0x2 0x15 0x00 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x15 0x01 IRQ_TYPE_EDGE_BOTH>,
>              <0x2 0x15 0x02 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x15 0x03 IRQ_TYPE_EDGE_BOTH>,
>              <0x2 0x15 0x04 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x15 0x05 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x15 0x06 IRQ_TYPE_EDGE_BOTH>,
>              <0x2 0x15 0x07 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x00 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x01 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x02 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x03 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x04 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x05 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x06 IRQ_TYPE_EDGE_RISING>,
>              <0x2 0x17 0x07 IRQ_TYPE_EDGE_RISING>;
>
>         interrupt-names = "or-rid-detect-change",
>                   "vpd-detect",
>                   "cc-state-change",
>                   "vconn-oc",
>                   "vbus-change",
>                   "attach-detach",
>                   "legacy-cable-detect",
>                   "try-snk-src-detect",
>                   "sig-tx",
>                   "sig-rx",
>                   "msg-tx",
>                   "msg-rx",
>                   "msg-tx-failed",
>                   "msg-tx-discarded",
>                   "msg-rx-discarded",
>                   "fr-swap";
> }
>
> ---
> bod