diff mbox series

[v2] scsi: dt-bindings: ufs: Add sm8250, sm8350 compatible strings

Message ID 20210204165234.61939-1-vkoul@kernel.org
State Not Applicable, archived
Headers show
Series [v2] scsi: dt-bindings: ufs: Add sm8250, sm8350 compatible strings | expand

Checks

Context Check Description
robh/checkpatch success

Commit Message

Vinod Koul Feb. 4, 2021, 4:52 p.m. UTC
Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.

Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
---
 - add Bjorn's ack
 - split from phy series

 Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt | 2 ++
 1 file changed, 2 insertions(+)

Comments

Rob Herring (Arm) Feb. 9, 2021, 9:39 p.m. UTC | #1
On Thu, 04 Feb 2021 22:22:34 +0530, Vinod Koul wrote:
> Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
> Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
> documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.
> 
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---
>  - add Bjorn's ack
>  - split from phy series
> 
>  Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 

Reviewed-by: Rob Herring <robh@kernel.org>
Vinod Koul March 12, 2021, 5:09 a.m. UTC | #2
On 04-02-21, 22:22, Vinod Koul wrote:
> Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
> Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
> documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.

Gentle reminder for this patch, Rob has acked this

Thanks

> 
> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> Signed-off-by: Vinod Koul <vkoul@kernel.org>
> ---
>  - add Bjorn's ack
>  - split from phy series
> 
>  Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> index 415ccdd7442d..d8fd4df81743 100644
> --- a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> +++ b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
> @@ -14,6 +14,8 @@ Required properties:
>  			    "qcom,msm8998-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
>  			    "qcom,sdm845-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
>  			    "qcom,sm8150-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
> +			    "qcom,sm8250-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
> +			    "qcom,sm8350-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
>  - interrupts        : <interrupt mapping for UFS host controller IRQ>
>  - reg               : <registers mapping>
>  
> -- 
> 2.26.2
Martin K. Petersen March 17, 2021, 1:21 a.m. UTC | #3
Vinod,

>> Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
>> Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
>> documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.
>
> Gentle reminder for this patch, Rob has acked this

Was not really expecting a DT patch to be routed through SCSI. But no
worries. I've queued it up.
Vinod Koul March 17, 2021, 5:50 a.m. UTC | #4
Hello Martin,

On 16-03-21, 21:21, Martin K. Petersen wrote:
> 
> Vinod,
> 
> >> Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
> >> Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
> >> documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.
> >
> > Gentle reminder for this patch, Rob has acked this
> 
> Was not really expecting a DT patch to be routed through SCSI. But no
> worries. I've queued it up.

Thank you. You can blame MAINTAINERS for pointing it to you :)
Typically DT binding go thru subsystem and dts thru soc trees
Martin K. Petersen March 19, 2021, 3:46 a.m. UTC | #5
On Thu, 4 Feb 2021 22:22:34 +0530, Vinod Koul wrote:

> Document "qcom,sm8250-ufshc" and "qcom,sm8350-ufshc" compatible string.
> Use of "qcom,sm8250-ufshc" is already present upstream, so add misiing
> documentation. "qcom,sm8350-ufshc" is for UFS HC found in SM8350 SoC.

Applied to 5.13/scsi-queue, thanks!

[1/1] scsi: dt-bindings: ufs: Add sm8250, sm8350 compatible strings
      https://git.kernel.org/mkp/scsi/c/4517e77eb83d
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
index 415ccdd7442d..d8fd4df81743 100644
--- a/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
+++ b/Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt
@@ -14,6 +14,8 @@  Required properties:
 			    "qcom,msm8998-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
 			    "qcom,sdm845-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
 			    "qcom,sm8150-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
+			    "qcom,sm8250-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
+			    "qcom,sm8350-ufshc", "qcom,ufshc", "jedec,ufs-2.0"
 - interrupts        : <interrupt mapping for UFS host controller IRQ>
 - reg               : <registers mapping>