Re: [PATCH] dt-bindings: phy: samsung,ufs-phy: add power-domains

[André Draszik]

Alim,

On Wed, 2025-10-08 at 09:07 +0100, André Draszik wrote:
> Hi Alim,
> 
> On Wed, 2025-10-08 at 11:56 +0530, Alim Akhtar wrote:
> > 
> > 
> > > -----Original Message-----
> > > From: André Draszik <andre.draszik@xxxxxxxxxx>
> > > Sent: Tuesday, October 7, 2025 9:32 PM
> > > To: Vinod Koul <vkoul@xxxxxxxxxx>; Kishon Vijay Abraham I
> > > <kishon@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski
> > > <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Alim Akhtar
> > > <alim.akhtar@xxxxxxxxxxx>
> > > Cc: Peter Griffin <peter.griffin@xxxxxxxxxx>; Tudor Ambarus
> > > <tudor.ambarus@xxxxxxxxxx>; Will McVicker <willmcvicker@xxxxxxxxxx>;
> > > kernel-team@xxxxxxxxxxx; linux-phy@xxxxxxxxxxxxxxxxxxx;
> > > devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; André Draszik
> > > <andre.draszik@xxxxxxxxxx>
> > > Subject: [PATCH] dt-bindings: phy: samsung,ufs-phy: add power-domains
> > > 
> > > The UFS phy can be part of a power domain, so we need to allow the
> > > relevant property 'power-domains'.
> > > 
> > Can you cross check if there is a separate power domain control for ufs-phy? 
> 
> I'm still having some other troubles with the hsi2 (UFS) power domain on gs101,
> but I can confirm the usb-phy is part of the hsi0 (USB) power domain. So for the
> ufs-phy I presume it's similar: it should be part of the hsi2 power domain. There
> certainly is no separate domain for the ufs-phy itself.

After double checking the HSI2 HDD, the ufs-phy is indeed part of PD_HSI2.


Cheers,
Andre'