diff mbox series

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

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

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(+)

-- 
2.26.2

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


-- 
~Vinod
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

-- 
~Vinod
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

-- 
Martin K. Petersen	Oracle Linux Engineering
patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #6
Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

On Thu,  4 Feb 2021 22:22:34 +0530 you 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>

> 

> [...]


Here is the summary with links:
  - [v2] scsi: dt-bindings: ufs: Add sm8250, sm8350 compatible strings
    https://git.kernel.org/qcom/c/4517e77eb83d

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
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>