linux-kernel - Re: [PATCH 1/8] dt-bindings: power: supply: Add DT schema for Qualcomm SMBCHG

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 21 Nov 2022 09:26:59 +0100
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Yassine Oudjana <yassine.oudjana@...il.com>
Cc:     Sebastian Reichel <sre@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Alejandro Tafalla <atafalla@...on.com>,
        Konrad Dybcio <konrad.dybcio@...ainline.org>,
        linux-pm@...r.kernel.org, linux-arm-msm@...r.kernel.org,
        devicetree@...r.kernel.org, phone-devel@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Yassine Oudjana <y.oudjana@...tonmail.com>
Subject: Re: [PATCH 1/8] dt-bindings: power: supply: Add DT schema for
 Qualcomm SMBCHG

On 20/11/2022 16:46, Yassine Oudjana wrote:
>>> +  interrupts:
>>> +    items:
>>> +      - description: Charger error
>>> +      - description: Charger inhibited
>>> +      - description: Charger precharge safety timer timeout
>>> +      - description: Charger charge safety timer timeout
>>> +      - description: Charger pre to fast charging switch threshold reached
>>> +      - description: Charger recharge threshold reached
>>> +      - description: Charger taper threshold reached
>>> +      - description: Charger charge termination threshold reached
>>> +      - description: Battery hot
>>> +      - description: Battery warm
>>> +      - description: Battery cold
>>> +      - description: Battery cool
>>> +      - description: Battery overvoltage
>>> +      - description: Battery low
>>> +      - description: Battery missing
>>> +      - description: Battery thermistor missing # unconfirmed
>>> +      - description: USB input undervolt
>>> +      - description: USB input overvolt
>>> +      - description: USB input source detected
>>> +      - description: OTG regulator failure
>>> +      - description: OTG regulator overcurrent
>>> +      - description: Automatic input current limiting done
>>> +      - description: USB ID pin changed
>>> +      - description: DC input undervolt
>>> +      - description: DC input overvolt
>>> +      - description: Power OK
>>> +      - description: Temperature shutdown
>>> +      - description: Watchdog timeout
>>> +      - description: Flash failure
>>> +      - description: OTST2 # unknown
>>> +      - description: OTST3 # unknown
>>
>> It seems you listed register interrupts, not physical pins. This should
>> be interrupt lines.
> 
> I'm not sure what I'm supposed to do here. I couldn't find an interrupt-lines
> property used anywhere so that's not what you meant, right?

Are these physical interrupt lines this device has, register offsets or
virtual interrupts (e.g. passed via irq_chip)? Definitely not the first
and rather offsets for qpnpint_irq_domain_translate. Devicetree is not
for describing register layout of devices. IOW, register layout does not
change on different boards, because the device is exactly the same, so
there is no point to put it into DTS.

Best regards,
Krzysztof