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

나소원/SOWON NA posted 3 patches 11 months, 2 weeks ago
Only 0 patches received!
There is a newer version of this series
RE: [PATCH v4 0/3] Support ExynosAutov920 ufs phy driver
Posted by 나소원/SOWON NA 11 months, 2 weeks 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, 3 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, 3 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