mbox series

[0/5] arm64: dts: qcom: add QCE on SM8250 and SM8450 platforms

Message ID 20230323233735.2131020-1-vladimir.zapolskiy@linaro.org
Headers show
Series arm64: dts: qcom: add QCE on SM8250 and SM8450 platforms | expand

Message

Vladimir Zapolskiy March 23, 2023, 11:37 p.m. UTC 
The changeset updates Qualcomm Crypto Engine compatible values from
deprecated ones to new according to the recent changes in
Documentation/devicetree/bindings/crypto/qcom-qce.yaml, also two
more Qualcomm platforms SM8250 and SM8450 gained QCE and BAM DMA
device tree nodes.

Vladimir Zapolskiy (5):
  arm: dts: qcom: ipq4019: update a compatible for QCE IP on IPQ4019 SoC
  arm64: dts: qcom: msm8996: update QCE compatible according to a new scheme
  arm64: dts: qcom: sdm845: update QCE compatible according to a new scheme
  arm64: dts: qcom: sm8250: add description of Qualcomm Crypto Engine IP
  arm64: dts: qcom: sm8450: add description of Qualcomm Crypto Engine IP

 arch/arm/boot/dts/qcom-ipq4019.dtsi   |  2 +-
 arch/arm64/boot/dts/qcom/msm8996.dtsi |  2 +-
 arch/arm64/boot/dts/qcom/sdm845.dtsi  |  2 +-
 arch/arm64/boot/dts/qcom/sm8250.dtsi  | 24 ++++++++++++++++++++++++
 arch/arm64/boot/dts/qcom/sm8450.dtsi  | 22 ++++++++++++++++++++++
 5 files changed, 49 insertions(+), 3 deletions(-)

Comments

Krzysztof Kozlowski March 24, 2023, 9:51 a.m. UTC | #1 
On 24/03/2023 00:37, Vladimir Zapolskiy wrote:
> Change the old deprecated compatible name to a new one, which is specific
> to IPQ4019 SoC.
> 
> Signed-off-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
> ---
>  arch/arm/boot/dts/qcom-ipq4019.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom-ipq4019.dtsi
> index dfcfb3339c23..3968bf0f585b 100644
> --- a/arch/arm/boot/dts/qcom-ipq4019.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq4019.dtsi
> @@ -315,7 +315,7 @@ cryptobam: dma-controller@8e04000 {
>  		};
>  
>  		crypto: crypto@8e3a000 {
> -			compatible = "qcom,crypto-v5.1";
> +			compatible = "qcom,ipq4019-qce", "qcom,qce";

The bindings in current next do not allow such combination. Is there
some dependency here? Are you sure this passed dtbs_check?

Best regards,
Krzysztof