* [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300
@ 2024-09-11 6:56 Jingyi Wang
2024-09-16 14:53 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Jingyi Wang @ 2024-09-11 6:56 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, linux-phy, devicetree, linux-kernel, quic_jingyw,
Xin Liu
From: Xin Liu <quic_liuxin@quicinc.com>
Document the QMP UFS PHY compatible for Qualcomm QCS8300 to support
physical layer functionality for UFS found on the SoC. Use fallback to
indicate the compatibility of the QMP UFS PHY on the QCS8300 with that
on the SA8775P.
Signed-off-by: Xin Liu <quic_liuxin@quicinc.com>
Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
---
Changes in v2:
- decoupled from the original series.
- Use fallback to indicate compatibility with SA8775P.
- typo fixup
- Link to v1: https://lore.kernel.org/r/20240904-qcs8300_initial_dtsi-v1-0-d0ea9afdc007@quicinc.com
---
.../bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml | 46 ++++++++++++----------
1 file changed, 26 insertions(+), 20 deletions(-)
diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
index f9cfbd0b2de6..626a2039e177 100644
--- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
@@ -15,26 +15,31 @@ description:
properties:
compatible:
- enum:
- - qcom,msm8996-qmp-ufs-phy
- - qcom,msm8998-qmp-ufs-phy
- - qcom,sa8775p-qmp-ufs-phy
- - qcom,sc7180-qmp-ufs-phy
- - qcom,sc7280-qmp-ufs-phy
- - qcom,sc8180x-qmp-ufs-phy
- - qcom,sc8280xp-qmp-ufs-phy
- - qcom,sdm845-qmp-ufs-phy
- - qcom,sm6115-qmp-ufs-phy
- - qcom,sm6125-qmp-ufs-phy
- - qcom,sm6350-qmp-ufs-phy
- - qcom,sm7150-qmp-ufs-phy
- - qcom,sm8150-qmp-ufs-phy
- - qcom,sm8250-qmp-ufs-phy
- - qcom,sm8350-qmp-ufs-phy
- - qcom,sm8450-qmp-ufs-phy
- - qcom,sm8475-qmp-ufs-phy
- - qcom,sm8550-qmp-ufs-phy
- - qcom,sm8650-qmp-ufs-phy
+ oneOf:
+ - items:
+ - enum:
+ - qcom,qcs8300-qmp-ufs-phy
+ - const: qcom,sa8775p-qmp-ufs-phy
+ - enum:
+ - qcom,msm8996-qmp-ufs-phy
+ - qcom,msm8998-qmp-ufs-phy
+ - qcom,sa8775p-qmp-ufs-phy
+ - qcom,sc7180-qmp-ufs-phy
+ - qcom,sc7280-qmp-ufs-phy
+ - qcom,sc8180x-qmp-ufs-phy
+ - qcom,sc8280xp-qmp-ufs-phy
+ - qcom,sdm845-qmp-ufs-phy
+ - qcom,sm6115-qmp-ufs-phy
+ - qcom,sm6125-qmp-ufs-phy
+ - qcom,sm6350-qmp-ufs-phy
+ - qcom,sm7150-qmp-ufs-phy
+ - qcom,sm8150-qmp-ufs-phy
+ - qcom,sm8250-qmp-ufs-phy
+ - qcom,sm8350-qmp-ufs-phy
+ - qcom,sm8450-qmp-ufs-phy
+ - qcom,sm8475-qmp-ufs-phy
+ - qcom,sm8550-qmp-ufs-phy
+ - qcom,sm8650-qmp-ufs-phy
reg:
maxItems: 1
@@ -85,6 +90,7 @@ allOf:
contains:
enum:
- qcom,msm8998-qmp-ufs-phy
+ - qcom,qcs8300-qmp-ufs-phy
- qcom,sa8775p-qmp-ufs-phy
- qcom,sc7180-qmp-ufs-phy
- qcom,sc7280-qmp-ufs-phy
---
base-commit: 100cc857359b5d731407d1038f7e76cd0e871d94
change-id: 20240911-qcs8300_ufs_phy_binding-84640e3e7bb7
Best regards,
--
Jingyi Wang <quic_jingyw@quicinc.com>
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300
2024-09-11 6:56 [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300 Jingyi Wang
@ 2024-09-16 14:53 ` Krzysztof Kozlowski
2024-09-25 3:16 ` Jingyi Wang
0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-09-16 14:53 UTC (permalink / raw)
To: Jingyi Wang
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-phy,
devicetree, linux-kernel, Xin Liu
On Wed, Sep 11, 2024 at 02:56:16PM +0800, Jingyi Wang wrote:
> From: Xin Liu <quic_liuxin@quicinc.com>
>
> Document the QMP UFS PHY compatible for Qualcomm QCS8300 to support
> physical layer functionality for UFS found on the SoC. Use fallback to
> indicate the compatibility of the QMP UFS PHY on the QCS8300 with that
> on the SA8775P.
>
> Signed-off-by: Xin Liu <quic_liuxin@quicinc.com>
> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
> ---
> Changes in v2:
> - decoupled from the original series.
> - Use fallback to indicate compatibility with SA8775P.
> - typo fixup
> - Link to v1: https://lore.kernel.org/r/20240904-qcs8300_initial_dtsi-v1-0-d0ea9afdc007@quicinc.com
> ---
> .../bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml | 46 ++++++++++++----------
> 1 file changed, 26 insertions(+), 20 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> index f9cfbd0b2de6..626a2039e177 100644
> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
> @@ -15,26 +15,31 @@ description:
>
> properties:
> compatible:
> - enum:
> - - qcom,msm8996-qmp-ufs-phy
> - - qcom,msm8998-qmp-ufs-phy
> - - qcom,sa8775p-qmp-ufs-phy
> - - qcom,sc7180-qmp-ufs-phy
> - - qcom,sc7280-qmp-ufs-phy
> - - qcom,sc8180x-qmp-ufs-phy
> - - qcom,sc8280xp-qmp-ufs-phy
> - - qcom,sdm845-qmp-ufs-phy
> - - qcom,sm6115-qmp-ufs-phy
> - - qcom,sm6125-qmp-ufs-phy
> - - qcom,sm6350-qmp-ufs-phy
> - - qcom,sm7150-qmp-ufs-phy
> - - qcom,sm8150-qmp-ufs-phy
> - - qcom,sm8250-qmp-ufs-phy
> - - qcom,sm8350-qmp-ufs-phy
> - - qcom,sm8450-qmp-ufs-phy
> - - qcom,sm8475-qmp-ufs-phy
> - - qcom,sm8550-qmp-ufs-phy
> - - qcom,sm8650-qmp-ufs-phy
> + oneOf:
> + - items:
> + - enum:
> + - qcom,qcs8300-qmp-ufs-phy
> + - const: qcom,sa8775p-qmp-ufs-phy
> + - enum:
> + - qcom,msm8996-qmp-ufs-phy
> + - qcom,msm8998-qmp-ufs-phy
> + - qcom,sa8775p-qmp-ufs-phy
> + - qcom,sc7180-qmp-ufs-phy
> + - qcom,sc7280-qmp-ufs-phy
> + - qcom,sc8180x-qmp-ufs-phy
> + - qcom,sc8280xp-qmp-ufs-phy
> + - qcom,sdm845-qmp-ufs-phy
> + - qcom,sm6115-qmp-ufs-phy
> + - qcom,sm6125-qmp-ufs-phy
> + - qcom,sm6350-qmp-ufs-phy
> + - qcom,sm7150-qmp-ufs-phy
> + - qcom,sm8150-qmp-ufs-phy
> + - qcom,sm8250-qmp-ufs-phy
> + - qcom,sm8350-qmp-ufs-phy
> + - qcom,sm8450-qmp-ufs-phy
> + - qcom,sm8475-qmp-ufs-phy
> + - qcom,sm8550-qmp-ufs-phy
> + - qcom,sm8650-qmp-ufs-phy
>
> reg:
> maxItems: 1
> @@ -85,6 +90,7 @@ allOf:
> contains:
> enum:
> - qcom,msm8998-qmp-ufs-phy
> + - qcom,qcs8300-qmp-ufs-phy
Not needed.
BTW, please be sure you organize your patchsets per subsystem. IOW, that
you do not split same subsystem bindings patches into separate
patchsets.
Best regards,
Krzysztof
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300
2024-09-16 14:53 ` Krzysztof Kozlowski
@ 2024-09-25 3:16 ` Jingyi Wang
2024-09-25 7:14 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Jingyi Wang @ 2024-09-25 3:16 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-phy,
devicetree, linux-kernel, Xin Liu
On 9/16/2024 10:53 PM, Krzysztof Kozlowski wrote:
> On Wed, Sep 11, 2024 at 02:56:16PM +0800, Jingyi Wang wrote:
>> From: Xin Liu <quic_liuxin@quicinc.com>
>>
>> Document the QMP UFS PHY compatible for Qualcomm QCS8300 to support
>> physical layer functionality for UFS found on the SoC. Use fallback to
>> indicate the compatibility of the QMP UFS PHY on the QCS8300 with that
>> on the SA8775P.
>>
>> Signed-off-by: Xin Liu <quic_liuxin@quicinc.com>
>> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
>> ---
>> Changes in v2:
>> - decoupled from the original series.
>> - Use fallback to indicate compatibility with SA8775P.
>> - typo fixup
>> - Link to v1: https://lore.kernel.org/r/20240904-qcs8300_initial_dtsi-v1-0-d0ea9afdc007@quicinc.com
>> ---
>> .../bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml | 46 ++++++++++++----------
>> 1 file changed, 26 insertions(+), 20 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>> index f9cfbd0b2de6..626a2039e177 100644
>> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>> @@ -15,26 +15,31 @@ description:
>>
>> properties:
>> compatible:
>> - enum:
>> - - qcom,msm8996-qmp-ufs-phy
>> - - qcom,msm8998-qmp-ufs-phy
>> - - qcom,sa8775p-qmp-ufs-phy
>> - - qcom,sc7180-qmp-ufs-phy
>> - - qcom,sc7280-qmp-ufs-phy
>> - - qcom,sc8180x-qmp-ufs-phy
>> - - qcom,sc8280xp-qmp-ufs-phy
>> - - qcom,sdm845-qmp-ufs-phy
>> - - qcom,sm6115-qmp-ufs-phy
>> - - qcom,sm6125-qmp-ufs-phy
>> - - qcom,sm6350-qmp-ufs-phy
>> - - qcom,sm7150-qmp-ufs-phy
>> - - qcom,sm8150-qmp-ufs-phy
>> - - qcom,sm8250-qmp-ufs-phy
>> - - qcom,sm8350-qmp-ufs-phy
>> - - qcom,sm8450-qmp-ufs-phy
>> - - qcom,sm8475-qmp-ufs-phy
>> - - qcom,sm8550-qmp-ufs-phy
>> - - qcom,sm8650-qmp-ufs-phy
>> + oneOf:
>> + - items:
>> + - enum:
>> + - qcom,qcs8300-qmp-ufs-phy
>> + - const: qcom,sa8775p-qmp-ufs-phy
>> + - enum:
>> + - qcom,msm8996-qmp-ufs-phy
>> + - qcom,msm8998-qmp-ufs-phy
>> + - qcom,sa8775p-qmp-ufs-phy
>> + - qcom,sc7180-qmp-ufs-phy
>> + - qcom,sc7280-qmp-ufs-phy
>> + - qcom,sc8180x-qmp-ufs-phy
>> + - qcom,sc8280xp-qmp-ufs-phy
>> + - qcom,sdm845-qmp-ufs-phy
>> + - qcom,sm6115-qmp-ufs-phy
>> + - qcom,sm6125-qmp-ufs-phy
>> + - qcom,sm6350-qmp-ufs-phy
>> + - qcom,sm7150-qmp-ufs-phy
>> + - qcom,sm8150-qmp-ufs-phy
>> + - qcom,sm8250-qmp-ufs-phy
>> + - qcom,sm8350-qmp-ufs-phy
>> + - qcom,sm8450-qmp-ufs-phy
>> + - qcom,sm8475-qmp-ufs-phy
>> + - qcom,sm8550-qmp-ufs-phy
>> + - qcom,sm8650-qmp-ufs-phy
>>
>> reg:
>> maxItems: 1
>> @@ -85,6 +90,7 @@ allOf:
>> contains:
>> enum:
>> - qcom,msm8998-qmp-ufs-phy
>> + - qcom,qcs8300-qmp-ufs-phy
>
> Not needed.
>
Well noted.
> BTW, please be sure you organize your patchsets per subsystem. IOW, that
> you do not split same subsystem bindings patches into separate
> patchsets.
>
Do you mean combine the binding of qmp-ufs-phy with ufs controller? I noticed
that this patch belongs to "GENERIC PHY FRAMEWORK" subsystem and ufs controller
binding belongs to "UNIVERSAL FLASH STORAGE HOST CONTROLLER DRIVER". If necessary,
we can pay attention to combine them next time
> Best regards,
> Krzysztof
>
Thanks,
Jingyi
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300
2024-09-25 3:16 ` Jingyi Wang
@ 2024-09-25 7:14 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-09-25 7:14 UTC (permalink / raw)
To: Jingyi Wang
Cc: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, linux-arm-msm, linux-phy,
devicetree, linux-kernel, Xin Liu
On 25/09/2024 05:16, Jingyi Wang wrote:
>
>
> On 9/16/2024 10:53 PM, Krzysztof Kozlowski wrote:
>> On Wed, Sep 11, 2024 at 02:56:16PM +0800, Jingyi Wang wrote:
>>> From: Xin Liu <quic_liuxin@quicinc.com>
>>>
>>> Document the QMP UFS PHY compatible for Qualcomm QCS8300 to support
>>> physical layer functionality for UFS found on the SoC. Use fallback to
>>> indicate the compatibility of the QMP UFS PHY on the QCS8300 with that
>>> on the SA8775P.
>>>
>>> Signed-off-by: Xin Liu <quic_liuxin@quicinc.com>
>>> Signed-off-by: Jingyi Wang <quic_jingyw@quicinc.com>
>>> ---
>>> Changes in v2:
>>> - decoupled from the original series.
>>> - Use fallback to indicate compatibility with SA8775P.
>>> - typo fixup
>>> - Link to v1: https://lore.kernel.org/r/20240904-qcs8300_initial_dtsi-v1-0-d0ea9afdc007@quicinc.com
>>> ---
>>> .../bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml | 46 ++++++++++++----------
>>> 1 file changed, 26 insertions(+), 20 deletions(-)
>>>
>>> diff --git a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>>> index f9cfbd0b2de6..626a2039e177 100644
>>> --- a/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>>> +++ b/Documentation/devicetree/bindings/phy/qcom,sc8280xp-qmp-ufs-phy.yaml
>>> @@ -15,26 +15,31 @@ description:
>>>
>>> properties:
>>> compatible:
>>> - enum:
>>> - - qcom,msm8996-qmp-ufs-phy
>>> - - qcom,msm8998-qmp-ufs-phy
>>> - - qcom,sa8775p-qmp-ufs-phy
>>> - - qcom,sc7180-qmp-ufs-phy
>>> - - qcom,sc7280-qmp-ufs-phy
>>> - - qcom,sc8180x-qmp-ufs-phy
>>> - - qcom,sc8280xp-qmp-ufs-phy
>>> - - qcom,sdm845-qmp-ufs-phy
>>> - - qcom,sm6115-qmp-ufs-phy
>>> - - qcom,sm6125-qmp-ufs-phy
>>> - - qcom,sm6350-qmp-ufs-phy
>>> - - qcom,sm7150-qmp-ufs-phy
>>> - - qcom,sm8150-qmp-ufs-phy
>>> - - qcom,sm8250-qmp-ufs-phy
>>> - - qcom,sm8350-qmp-ufs-phy
>>> - - qcom,sm8450-qmp-ufs-phy
>>> - - qcom,sm8475-qmp-ufs-phy
>>> - - qcom,sm8550-qmp-ufs-phy
>>> - - qcom,sm8650-qmp-ufs-phy
>>> + oneOf:
>>> + - items:
>>> + - enum:
>>> + - qcom,qcs8300-qmp-ufs-phy
>>> + - const: qcom,sa8775p-qmp-ufs-phy
>>> + - enum:
>>> + - qcom,msm8996-qmp-ufs-phy
>>> + - qcom,msm8998-qmp-ufs-phy
>>> + - qcom,sa8775p-qmp-ufs-phy
>>> + - qcom,sc7180-qmp-ufs-phy
>>> + - qcom,sc7280-qmp-ufs-phy
>>> + - qcom,sc8180x-qmp-ufs-phy
>>> + - qcom,sc8280xp-qmp-ufs-phy
>>> + - qcom,sdm845-qmp-ufs-phy
>>> + - qcom,sm6115-qmp-ufs-phy
>>> + - qcom,sm6125-qmp-ufs-phy
>>> + - qcom,sm6350-qmp-ufs-phy
>>> + - qcom,sm7150-qmp-ufs-phy
>>> + - qcom,sm8150-qmp-ufs-phy
>>> + - qcom,sm8250-qmp-ufs-phy
>>> + - qcom,sm8350-qmp-ufs-phy
>>> + - qcom,sm8450-qmp-ufs-phy
>>> + - qcom,sm8475-qmp-ufs-phy
>>> + - qcom,sm8550-qmp-ufs-phy
>>> + - qcom,sm8650-qmp-ufs-phy
>>>
>>> reg:
>>> maxItems: 1
>>> @@ -85,6 +90,7 @@ allOf:
>>> contains:
>>> enum:
>>> - qcom,msm8998-qmp-ufs-phy
>>> + - qcom,qcs8300-qmp-ufs-phy
>>
>> Not needed.
>>
> Well noted.
>> BTW, please be sure you organize your patchsets per subsystem. IOW, that
>> you do not split same subsystem bindings patches into separate
>> patchsets.
>>
> Do you mean combine the binding of qmp-ufs-phy with ufs controller? I noticed
> that this patch belongs to "GENERIC PHY FRAMEWORK" subsystem and ufs controller
> binding belongs to "UNIVERSAL FLASH STORAGE HOST CONTROLLER DRIVER". If necessary,
> we can pay attention to combine them next time
No, I mean group patches targeting the same subsystem.
Best regards,
Krzysztof
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-09-25 7:14 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-11 6:56 [PATCH v2] dt-bindings: phy: Add QMP UFS PHY comptible for QCS8300 Jingyi Wang
2024-09-16 14:53 ` Krzysztof Kozlowski
2024-09-25 3:16 ` Jingyi Wang
2024-09-25 7:14 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).