Hi Bart, Thanks for your review feedback! On Fri, 28 Feb 2025 at 19:20, Bart Van Assche <bvanassche@xxxxxxx> wrote: > > On 2/26/25 2:04 PM, Peter Griffin wrote: > > +static void exynos_ufs_exit(struct ufs_hba *hba) > > +{ > > + struct exynos_ufs *ufs = ufshcd_get_variant(hba); > > + > > + phy_power_off(ufs->phy); > > + phy_exit(ufs->phy); > > +} > > + > > + > > For future patches, please follow the convention that is used elsewhere > in the kernel and separate functions with a single blank line instead of > two. That was an oversight on my part, will fix. Thanks, Peter