| Message ID | 20221209-dt-binding-ufs-v4-0-14ced60f3d1b@fairphone.com |
|---|---|
| Headers | show |
| Series | Fix some issues in QCOM UFS bindings | expand |
On Mon Jun 26, 2023 at 10:27 AM CEST, Krzysztof Kozlowski wrote: > On 26/06/2023 10:15, Luca Weiss wrote: > > SM8450 actually supports ICE (Inline Crypto Engine) so adjust the > > bindings and the example to match. > > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Reviewed-by: Eric Biggers <ebiggers@google.com> > > Reviewed-by: Iskren Chernev <me@iskren.info> > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> > > SM8450 should be rather converted to qcom,ice. In v5 sm8450 is now using ICE, both in dtsi and binding example. But I guess you could also argue reg-names should be purged from dtsi and binding completely and to convert all existing dtsi to use the standalone ice node, right? But I'd also like for this series to finally land at some point, we can do this later, okay? Regards Luca > > Best regards, > Krzysztof
This series has been pending for quite some time, please somebody pick this up. Also has plenty of reviews by now (3-4 for each patch which should be plenty). Validation will fail without patch 1/5 from [0] so this depends on that patch at least for the check. [0] https://lore.kernel.org/linux-arm-msm/20230623113009.2512206-2-abel.vesa@linaro.org/ Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> --- Changes in v4: - Pick up tags - Rebase on linux-next (again) - Link to v3: https://lore.kernel.org/r/20221209-dt-binding-ufs-v3-0-499dff23a03c@fairphone.com Changes in v3: - Drop applied patch - Pick up sm6115 patch from v5 https://lore.kernel.org/all/20221030094258.486428-2-iskren.chernev@gmail.com/ - Rebase on linux-next - Link to v2: https://lore.kernel.org/r/20221209-dt-binding-ufs-v2-0-dc7a04699579@fairphone.com 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/20221209-dt-binding-ufs-v1-0-8d502f0e18d5@fairphone.com --- Iskren Chernev (1): dt-bindings: ufs: qcom: Add sm6115 binding Luca Weiss (2): dt-bindings: ufs: qcom: Add reg-names property for ICE dt-bindings: ufs: qcom: Fix sm8450 bindings .../devicetree/bindings/ufs/qcom,ufs.yaml | 61 ++++++++++++++++++++-- 1 file changed, 56 insertions(+), 5 deletions(-) --- base-commit: 3178cc4b7c9de986d63383cb59ee6570347002db change-id: 20221209-dt-binding-ufs-2d7f64797ff2 Best regards,