Re: [PATCH v2 1/2] dt-bindings: soc: tegra: Document Nvidia Tegra modem pwrseq

[Bartosz Golaszewski]

On Tue, 26 May 2026 12:03:59 +0200, Svyatoslav Ryhel <clamor95@xxxxxxxxx> said:
> вт, 26 трав. 2026 р. о 12:51 Bartosz Golaszewski <brgl@xxxxxxxxxx> пише:
>>
>> On Sat, May 23, 2026 at 10:51 AM Svyatoslav Ryhel <clamor95@xxxxxxxxx> wrote:
>> >
>> > Document the Tegra modem pwseq used by various devices based on the Nvidia
>> > Tegra SoC, describing its usage. The power sequence provides interaction
>> > between the modem and SoC-specific interface configurations.
>> >
>> > Signed-off-by: Svyatoslav Ryhel <clamor95@xxxxxxxxx>
>> > ---
>>
>> ...
>>
>> > +
>> > +examples:
>> > +  - |
>> > +    #include <dt-bindings/gpio/gpio.h>
>> > +
>> > +    pwrseq-modem {
>> > +        compatible = "nvidia,tegra-modem-pwrseq";
>> > +
>> > +        enable-gpios = <&gpio 165 GPIO_ACTIVE_HIGH>;
>> > +        power-supply = <&vdd_3v3_vbat>;
>> > +
>> > +        nvidia,usb-bus = <&usb2>;
>> > +    };
>> > --
>> > 2.51.0
>> >
>>
>> I assume there's no such component as the tegra modem *pwrseq*?
>
> Hardware component, I assume no.
>
>> What is the relationship between this node and the "infineon,xmm6260"
>> compatible node?
>>
>
> The xmm6260 has a phandle reference to it. Since there is consumer
> matching logic in the pwrseq driver I am leaning towards removing this
> node entirely. This patches is still in non-final stage and I would
> like to hear any opinions on the way to improving it.
>

The node attached to the pwrseq provider device should represent a real
hardware component. Are the enable-gpios and power-supply lines connected
to the modem package? Is there an internal structure to it, like a PMIC to
which these lines connect that controls the actual modem IC?

Bart