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

Sowon Na posted 3 patches 11 months, 3 weeks ago
.../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
[PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by Sowon Na 11 months, 3 weeks ago
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
RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by 나소원/SOWON NA 11 months, 1 week ago
Hi Vinod,

> -----Original Message-----
> From: Sowon Na <sowon.na@samsung.com>
> Sent: Thursday, December 26, 2024 12:12 PM
> To: robh@kernel.org; krzk@kernel.org; conor+dt@kernel.org;
> vkoul@kernel.org; alim.akhtar@samsung.com; kishon@kernel.org
> Cc: krzk+dt@kernel.org; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; linux-samsung-soc@vger.kernel.org;
> sowon.na@samsung.com
> 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.
RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by Alim Akhtar 10 months, 2 weeks ago
Hello Krzysztof / Vinod

> -----Original Message-----
> From: 나소원/SOWON NA <sowon.na@samsung.com>
> Sent: Monday, January 13, 2025 12:55 PM
> To: vkoul@kernel.org
> Cc: krzk+dt@kernel.org; linux-kernel@vger.kernel.org;
> devicetree@vger.kernel.org; linux-samsung-soc@vger.kernel.org;
> robh@kernel.org; krzk@kernel.org; conor+dt@kernel.org;
> alim.akhtar@samsung.com; kishon@kernel.org
> Subject: RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
> 
> Hi Vinod,
> 
> > -----Original Message-----
> > From: Sowon Na <sowon.na@samsung.com>
> 
[Snip]
> I can't see these patches in -next yet, do let me know if anything is missing to
> be addressed from myside.
> 
Which tree will this series go through? 

> Best regards,
> Sowon Na.


Re: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by Krzysztof Kozlowski 10 months, 2 weeks ago
On 05/02/2025 05:32, Alim Akhtar wrote:
> Hello Krzysztof / Vinod
> 
>> -----Original Message-----
>> From: 나소원/SOWON NA <sowon.na@samsung.com>
>> Sent: Monday, January 13, 2025 12:55 PM
>> To: vkoul@kernel.org
>> Cc: krzk+dt@kernel.org; linux-kernel@vger.kernel.org;
>> devicetree@vger.kernel.org; linux-samsung-soc@vger.kernel.org;
>> robh@kernel.org; krzk@kernel.org; conor+dt@kernel.org;
>> alim.akhtar@samsung.com; kishon@kernel.org
>> Subject: RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
>>
>> Hi Vinod,
>>
>>> -----Original Message-----
>>> From: Sowon Na <sowon.na@samsung.com>
>>
> [Snip]
>> I can't see these patches in -next yet, do let me know if anything is missing to
>> be addressed from myside.
>>
> Which tree will this series go through? 


phy goes through phy. I take the DTS once the bindings got accepted. I
cannot take it earlier for obvious reasons - warnings.

Best regards,
Krzysztof
Re: (subset) [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by Vinod Koul 10 months ago
On Thu, 26 Dec 2024 12:11:35 +0900, Sowon Na wrote:
> 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
> 
> [...]

Applied, thanks!

[1/3] dt-bindings: phy: Add ExynosAutov920 UFS PHY bindings
      commit: 0ee54dcfe76760a65d86437f66df7f93b8b81903
[2/3] phy: samsung-ufs: support ExynosAutov920 ufs phy driver
      commit: d2317767723b63d28e3b93da92760b7934935536

Best regards,
-- 
~Vinod