On 22.11.2024 2:20 PM, Yuvaraj Ranganathan wrote: > Add an ICE node to qcs8300 SoC description and enable it by adding a > phandle to the UFS node. > > Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx> Getting *a review* does *not* mean I gave you a Reviewed-by tag. https://docs.kernel.org/process/submitting-patches.html#using-reported-by-tested-by-reviewed-by-suggested-by-and-fixes > Signed-off-by: Yuvaraj Ranganathan <quic_yrangana@xxxxxxxxxxx> > --- > arch/arm64/boot/dts/qcom/qcs8300.dtsi | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcs8300.dtsi b/arch/arm64/boot/dts/qcom/qcs8300.dtsi > index 2c35f96c3f28..ab91c3b7bba6 100644 > --- a/arch/arm64/boot/dts/qcom/qcs8300.dtsi > +++ b/arch/arm64/boot/dts/qcom/qcs8300.dtsi > @@ -685,6 +685,7 @@ &mc_virt SLAVE_EBI1 0>, > <0 0>, > <0 0>, > <0 0>; > + qcom,ice = <&ice>; > status = "disabled"; > }; > > @@ -710,6 +711,13 @@ ufs_mem_phy: phy@1d87000 { > status = "disabled"; > }; > > + ice: crypto@1d88000 { > + compatible = "qcom,qcs8300-inline-crypto-engine", > + "qcom,inline-crypto-engine"; > + reg = <0x0 0x01d88000 0x0 0x18000>; > + clocks = <&gcc GCC_UFS_PHY_ICE_CORE_CLK>; > + }; Although this looks good now, so I'll allow you to keep it. I won't add it a second time here to avoid confusing the maintainer tools. Konrad
