[PATCH v5 phy-next 09/27] scsi: ufs: exynos: stop poking into struct phy guts

[Alim Akhtar]

Hi Vladimir

> -----Original Message-----
> From: Vladimir Oltean <vladimir.oltean at nxp.com>
> Sent: Monday, March 23, 2026 5:29 PM
> To: Martin K. Petersen <martin.petersen at oracle.com>
> Cc: linux-phy at lists.infradead.org; Vinod Koul <vkoul at kernel.org>; Neil
> Armstrong <neil.armstrong at linaro.org>; dri-devel at lists.freedesktop.org;
> freedreno at lists.freedesktop.org; linux-arm-kernel at lists.infradead.org;
> linux-arm-msm at vger.kernel.org; linux-can at vger.kernel.org; linux-
> gpio at vger.kernel.org; linux-ide at vger.kernel.org; linux-
> kernel at vger.kernel.org; linux-media at vger.kernel.org; linux-
> pci at vger.kernel.org; linux-renesas-soc at vger.kernel.org; linux-
> riscv at lists.infradead.org; linux-rockchip at lists.infradead.org;
linux-samsung-
> soc at vger.kernel.org; linux-scsi at vger.kernel.org;
linux-sunxi at lists.linux.dev;
> linux-tegra at vger.kernel.org; linux-usb at vger.kernel.org;
> netdev at vger.kernel.org; spacemit at lists.linux.dev;
> UNGLinuxDriver at microchip.com; Bart Van Assche <bvanassche at acm.org>;
> Alim Akhtar <alim.akhtar at samsung.com>; Peter Griffin
> <peter.griffin at linaro.org>; James E.J. Bottomley
> <James.Bottomley at HansenPartnership.com>; Krzysztof Kozlowski
> <krzk at kernel.org>; Chanho Park <chanho61.park at samsung.com>
> Subject: Re: [PATCH v5 phy-next 09/27] scsi: ufs: exynos: stop poking into
> struct phy guts
> 
> On Thu, Mar 19, 2026 at 10:15:17PM -0400, Martin K. Petersen wrote:
> > Vladimir,
> >
> > > The Exynos host controller driver is clearly a PHY consumer (gets
> > > the
> > > ufs->phy using devm_phy_get()), but pokes into the guts of struct
> > > ufs->phy
> > > to get the generic_phy->power_count.
> >
> > Ah, newer version. Would still like an ack from Samsung.
> >
> > And I hit the wrong key, I did not actually apply this...
> 
> I will have to resend v6 because of an armv7 build error I've caused for
ufs-
> qcom.c (which doesn't #include <linux/interrupt.h>, and relies on a
transitive
> inclusion from <linux/phy/phy.h>). It would be nice to get the ack from
> Samsung, but I'll send the next version in the upcoming hours regardless.
Will review and possibly test on one of the board later tonight