[v6,0/4] Fix some issues in QCOM UFS bindings

Message

Luca Weiss Aug. 14, 2023, 10:14 a.m. UTC

This series aims to solve the dtbs_check errors from the qcom ufs
bindings. It has changed in scope a bit since v1, so it may be a bit all
over the place.

Signed-off-by: Luca Weiss <[e-mail]>
---
Changes in v6:
- Rebase on linux-next
- Drop applied dts patch
- Pick up tags
- Link to v5: https://lore.kernel.org/r/[e-mail]

Changes in v5:
- Convert sm8450.dtsi to use qcom,ice property, so stop modifying schema
  for sm8450 and only add qcom,ice property.
- Move reg-names names to top-level with only minItems/maxItems in the
  'if'
- Link to v4: https://lore.kernel.org/r/[e-mail]

Changes in v4:
- Pick up tags
- Rebase on linux-next (again)
- Link to v3: https://lore.kernel.org/r/[e-mail]

Changes in v3:
- Drop applied patch
- Pick up sm6115 patch from v5 https://lore.kernel.org/all/[e-mail]/
- Rebase on linux-next
- Link to v2: https://lore.kernel.org/r/[e-mail]

Changes in v2:
- Add new patch adding reg-names to sm6115 & rebase series on top of sm6115
  addition
- Fix binding example after sm8450 move, split this patch from original patch
  since it became too big
- Move reg-names definition to top-level
- Link to v1: https://lore.kernel.org/r/[e-mail]

---
Iskren Chernev (1):
      dt-bindings: ufs: qcom: Add sm6115 binding

Luca Weiss (3):
      dt-bindings: ufs: qcom: Add reg-names property for ICE
      dt-bindings: ufs: qcom: Add ICE to sm8450 example
      dt-bindings: crypto: ice: Document sm8450 inline crypto engine

 .../bindings/crypto/qcom,inline-crypto-engine.yaml |  1 +
 .../devicetree/bindings/ufs/qcom,ufs.yaml          | 44 ++++++++++++++++++++++
 2 files changed, 45 insertions(+)
---
base-commit: 13055cce6d3df0a2704721d6e2fd9011e973f53d
change-id: 20221209-dt-binding-ufs-2d7f64797ff2

Best regards,

Comments

Rob Herring Aug. 24, 2023, 5:10 p.m. UTC | #1

On Mon, Aug 14, 2023 at 12:14:12PM +0200, Luca Weiss wrote:
> This series aims to solve the dtbs_check errors from the qcom ufs
> bindings. It has changed in scope a bit since v1, so it may be a bit all
> over the place.
> 
> Signed-off-by: Luca Weiss <[e-mail]>
> ---
> Changes in v6:
> - Rebase on linux-next
> - Drop applied dts patch
> - Pick up tags
> - Link to v5: https://lore.kernel.org/r/[e-mail]
> 
> Changes in v5:
> - Convert sm8450.dtsi to use qcom,ice property, so stop modifying schema
>   for sm8450 and only add qcom,ice property.
> - Move reg-names names to top-level with only minItems/maxItems in the
>   'if'
> - Link to v4: https://lore.kernel.org/r/[e-mail]
> 
> Changes in v4:
> - Pick up tags
> - Rebase on linux-next (again)
> - Link to v3: https://lore.kernel.org/r/[e-mail]
> 
> Changes in v3:
> - Drop applied patch
> - Pick up sm6115 patch from v5 https://lore.kernel.org/all/[e-mail]/
> - Rebase on linux-next
> - Link to v2: https://lore.kernel.org/r/[e-mail]
> 
> Changes in v2:
> - Add new patch adding reg-names to sm6115 & rebase series on top of sm6115
>   addition
> - Fix binding example after sm8450 move, split this patch from original patch
>   since it became too big
> - Move reg-names definition to top-level
> - Link to v1: https://lore.kernel.org/r/[e-mail]
> 
> ---
> Iskren Chernev (1):
>       dt-bindings: ufs: qcom: Add sm6115 binding
> 
> Luca Weiss (3):
>       dt-bindings: ufs: qcom: Add reg-names property for ICE
>       dt-bindings: ufs: qcom: Add ICE to sm8450 example
>       dt-bindings: crypto: ice: Document sm8450 inline crypto engine
> 
>  .../bindings/crypto/qcom,inline-crypto-engine.yaml |  1 +
>  .../devicetree/bindings/ufs/qcom,ufs.yaml          | 44 ++++++++++++++++++++++
>  2 files changed, 45 insertions(+)

I guess the subsystem maintainers aren't going to pick this up, so I've 
applied it.

Rob

Luca Weiss Aug. 25, 2023, 6:38 a.m. UTC | #2

On Thu Aug 24, 2023 at 7:10 PM CEST, Rob Herring wrote:
> On Mon, Aug 14, 2023 at 12:14:12PM +0200, Luca Weiss wrote:
> > This series aims to solve the dtbs_check errors from the qcom ufs
> > bindings. It has changed in scope a bit since v1, so it may be a bit all
> > over the place.
> > 
> > Signed-off-by: Luca Weiss <[e-mail]>
> > ---
> > Changes in v6:
> > - Rebase on linux-next
> > - Drop applied dts patch
> > - Pick up tags
> > - Link to v5: https://lore.kernel.org/r/[e-mail]
> > 
> > Changes in v5:
> > - Convert sm8450.dtsi to use qcom,ice property, so stop modifying schema
> >   for sm8450 and only add qcom,ice property.
> > - Move reg-names names to top-level with only minItems/maxItems in the
> >   'if'
> > - Link to v4: https://lore.kernel.org/r/[e-mail]
> > 
> > Changes in v4:
> > - Pick up tags
> > - Rebase on linux-next (again)
> > - Link to v3: https://lore.kernel.org/r/[e-mail]
> > 
> > Changes in v3:
> > - Drop applied patch
> > - Pick up sm6115 patch from v5 https://lore.kernel.org/all/[e-mail]/
> > - Rebase on linux-next
> > - Link to v2: https://lore.kernel.org/r/[e-mail]
> > 
> > Changes in v2:
> > - Add new patch adding reg-names to sm6115 & rebase series on top of sm6115
> >   addition
> > - Fix binding example after sm8450 move, split this patch from original patch
> >   since it became too big
> > - Move reg-names definition to top-level
> > - Link to v1: https://lore.kernel.org/r/[e-mail]
> > 
> > ---
> > Iskren Chernev (1):
> >       dt-bindings: ufs: qcom: Add sm6115 binding
> > 
> > Luca Weiss (3):
> >       dt-bindings: ufs: qcom: Add reg-names property for ICE
> >       dt-bindings: ufs: qcom: Add ICE to sm8450 example
> >       dt-bindings: crypto: ice: Document sm8450 inline crypto engine
> > 
> >  .../bindings/crypto/qcom,inline-crypto-engine.yaml |  1 +
> >  .../devicetree/bindings/ufs/qcom,ufs.yaml          | 44 ++++++++++++++++++++++
> >  2 files changed, 45 insertions(+)
>
> I guess the subsystem maintainers aren't going to pick this up, so I've 
> applied it.

Thanks Rob, appreciate it!

>
> Rob