RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver

[나소원/SOWON NA]

Hi Vinod,

> -----Original Message-----
> From: Sowon Na <sowon.na@xxxxxxxxxxx>
> Sent: Thursday, December 26, 2024 12:12 PM
> To: robh@xxxxxxxxxx; krzk@xxxxxxxxxx; conor+dt@xxxxxxxxxx;
> vkoul@xxxxxxxxxx; alim.akhtar@xxxxxxxxxxx; kishon@xxxxxxxxxx
> Cc: krzk+dt@xxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx;
> sowon.na@xxxxxxxxxxx
> Subject: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
> 
> This patchset introduces ExynosAuto v920 SoC ufs phy driver as Generic PHY
> driver framework.
> 
> Changes from v3:
> - Use lower case for all addresses
> - Add empty line between macro and function
> 
> Changes from v2:
> - simplify function name from samsung_exynosautov920_ufs_phy_wait_cdr_lock
>   to exynosautov920_ufs_phy_wait_cdr_lock
> - return immediately after getting the CDR lock
> - add comment for wait CDR lock
> 
> Changes from v1:
> - use exynosautov920 instead of exynosauto to specify
> - remove obvious comment
> - change soc name as ExynosAutov920 to keep consistent
> - use macros instead of magic numbers
> - specify function name
> - add error handling for CDR lock failure
> 
> 
> Sowon Na (3):
>   dt-bindings: phy: Add ExynosAutov920 UFS PHY bindings
>   phy: samsung-ufs: support ExynosAutov920 ufs phy driver
>   arm64: dts: exynosautov920: add ufs phy for ExynosAutov920 SoC
> 
>  .../bindings/phy/samsung,ufs-phy.yaml         |   1 +
>  .../arm64/boot/dts/exynos/exynosautov920.dtsi |  11 ++
>  drivers/phy/samsung/Makefile                  |   1 +
>  drivers/phy/samsung/phy-exynosautov920-ufs.c  | 168 ++++++++++++++++++
>  drivers/phy/samsung/phy-samsung-ufs.c         |   9 +-
>  drivers/phy/samsung/phy-samsung-ufs.h         |   4 +
>  6 files changed, 191 insertions(+), 3 deletions(-)  create mode 100644
> drivers/phy/samsung/phy-exynosautov920-ufs.c
> 
> --
> 2.45.2
> 

I can't see these patches in -next yet, 
do let me know if anything is missing to be addressed from myside.

Best regards,
Sowon Na.