> -----Original Message----- > From: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > Sent: Tuesday, September 3, 2024 12:07 PM > To: Claudiu.Beznea <claudiu.beznea@xxxxxxxxx>; Ulf Hansson <ulf.hansson@xxxxxxxxxx> > Cc: vkoul@xxxxxxxxxx; kishon@xxxxxxxxxx; robh@xxxxxxxxxx; krzk+dt@xxxxxxxxxx; conor+dt@xxxxxxxxxx; > p.zabel@xxxxxxxxxxxxxx; geert+renesas@xxxxxxxxx; magnus.damm@xxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; > mturquette@xxxxxxxxxxxx; sboyd@xxxxxxxxxx; Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>; > linux-phy@xxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux- > renesas-soc@xxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux- > clk@xxxxxxxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx> > Subject: RE: [PATCH 00/16] Add initial USB support for the Renesas RZ/G3S SoC > > Hi Claudiu, > > > -----Original Message----- > > From: claudiu beznea <claudiu.beznea@xxxxxxxxx> > > Sent: Tuesday, September 3, 2024 12:00 PM > > Subject: Re: [PATCH 00/16] Add initial USB support for the Renesas > > RZ/G3S SoC > > > > > > > > On 03.09.2024 13:31, Biju Das wrote: > > >>> During boot clr USB PWR READY signal in TF-A. > > >>> STR case, suspend set USB PWR READY signal in TF-A. > > >>> STR case, resume clr USB PWR READY signal in TF-A. > > >> As I said previously, it can be done in different ways. My point > > >> was to let Linux set what it needs for all it's devices to work. I > > >> think the way to go forward is a > > maintainer decision. > > > > > > I agree, there can be n number of solution for a problem. > > > > > > Since you modelled system state signal (USB PWRRDY) as reset control > > > signal, it is reset/DT maintainer's decision to say the final word > > > whether this signal fits in reset > > system framework or not? > > > > I was thinking: > > 1/ Geert would be the best to say if he considers it OK to handle this > > in Linux > > I agree Geert is the right person for taking SYSTEM decisions, since the signal is used only during > state transitions (Table 41.6.4 AWO to ALL_ON and 41.6.3 ALL_ON to AWO) One more info, as per [1], this USB PWRRDY signal setting to be before Linux kernel boots. All clocks/reset happens after setting USB PWRRDY signal https://pasteboard.co/qbz021q7KPyi.png Cheers, Biju