mbox series

[0/5] arm64: dts: qcom: Enable Crypto Engine for a few Qualcomm SoCs

Message ID 20230321190118.3327360-1-bhupesh.sharma@linaro.org
Headers
Series arm64: dts: qcom: Enable Crypto Engine for a few Qualcomm SoCs |

Message

Bhupesh Sharma March 21, 2023, 7:01 p.m. UTC 
  This patchset enables Crypto Engine support for Qualcomm SoCs like
SM6115, SM8150, SM8250 and SM8350.

While at it, also fix the compatible string for BAM DMA engine
used in sdm845.dtsi

Note that:
- SM8250 crypto engine patch utilizes the work already done by myself and
  Vladimir.
- SM8350 crypto engine patch utilizes the work already done by Robert.

This patchset depends on the dt-bindings patch which added support for
SM6115 and QCS2290 SoCs:
https://lore.kernel.org/linux-arm-msm/20230321184811.3325725-1-bhupesh.sharma@linaro.org/

Also this patchset is rebased on linux-next/master.

Bhupesh Sharma (5):
  arm64: dts: qcom: sdm845: Fix the BAM DMA engine compatible string
  arm64: dts: qcom: sm6115: Add Crypto Engine support
  arm64: dts: qcom: sm8150: Add Crypto Engine support
  arm64: dts: qcom: sm8250: Add Crypto Engine support
  arm64: dts: qcom: sm8350: Add Crypto Engine support

 arch/arm64/boot/dts/qcom/sdm845.dtsi |  2 +-
 arch/arm64/boot/dts/qcom/sm6115.dtsi | 26 ++++++++++++++++++++++++++
 arch/arm64/boot/dts/qcom/sm8150.dtsi | 26 ++++++++++++++++++++++++++
 arch/arm64/boot/dts/qcom/sm8250.dtsi | 26 ++++++++++++++++++++++++++
 arch/arm64/boot/dts/qcom/sm8350.dtsi | 26 ++++++++++++++++++++++++++
 5 files changed, 105 insertions(+), 1 deletion(-)

Comments

Konrad Dybcio March 21, 2023, 7:27 p.m. UTC | #1 
On 21.03.2023 20:01, Bhupesh Sharma wrote:
> This patchset enables Crypto Engine support for Qualcomm SoCs like
> SM6115, SM8150, SM8250 and SM8350.
> 
> While at it, also fix the compatible string for BAM DMA engine
> used in sdm845.dtsi
> 
> Note that:
> - SM8250 crypto engine patch utilizes the work already done by myself and
>   Vladimir.
> - SM8350 crypto engine patch utilizes the work already done by Robert.
> 
> This patchset depends on the dt-bindings patch which added support for
> SM6115 and QCS2290 SoCs:
> https://lore.kernel.org/linux-arm-msm/20230321184811.3325725-1-bhupesh.sharma@linaro.org/
You could have included that one here for easier tracking!

Konrad
> 
> Also this patchset is rebased on linux-next/master.
> 
> Bhupesh Sharma (5):
>   arm64: dts: qcom: sdm845: Fix the BAM DMA engine compatible string
>   arm64: dts: qcom: sm6115: Add Crypto Engine support
>   arm64: dts: qcom: sm8150: Add Crypto Engine support
>   arm64: dts: qcom: sm8250: Add Crypto Engine support
>   arm64: dts: qcom: sm8350: Add Crypto Engine support
> 
>  arch/arm64/boot/dts/qcom/sdm845.dtsi |  2 +-
>  arch/arm64/boot/dts/qcom/sm6115.dtsi | 26 ++++++++++++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8150.dtsi | 26 ++++++++++++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8250.dtsi | 26 ++++++++++++++++++++++++++
>  arch/arm64/boot/dts/qcom/sm8350.dtsi | 26 ++++++++++++++++++++++++++
>  5 files changed, 105 insertions(+), 1 deletion(-)
>
Bhupesh Sharma March 22, 2023, 8:33 a.m. UTC | #2 
On Wed, 22 Mar 2023 at 00:57, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>
>
>
> On 21.03.2023 20:01, Bhupesh Sharma wrote:
> > This patchset enables Crypto Engine support for Qualcomm SoCs like
> > SM6115, SM8150, SM8250 and SM8350.
> >
> > While at it, also fix the compatible string for BAM DMA engine
> > used in sdm845.dtsi
> >
> > Note that:
> > - SM8250 crypto engine patch utilizes the work already done by myself and
> >   Vladimir.
> > - SM8350 crypto engine patch utilizes the work already done by Robert.
> >
> > This patchset depends on the dt-bindings patch which added support for
> > SM6115 and QCS2290 SoCs:
> > https://lore.kernel.org/linux-arm-msm/20230321184811.3325725-1-bhupesh.sharma@linaro.org/
> You could have included that one here for easier tracking!

Err.. the dt-binding maintainers have been asking to split the
dt-binding patches from
the dts ones in the past. So, let's stick to that for now :)

Thanks.

> > Also this patchset is rebased on linux-next/master.
> >
> > Bhupesh Sharma (5):
> >   arm64: dts: qcom: sdm845: Fix the BAM DMA engine compatible string
> >   arm64: dts: qcom: sm6115: Add Crypto Engine support
> >   arm64: dts: qcom: sm8150: Add Crypto Engine support
> >   arm64: dts: qcom: sm8250: Add Crypto Engine support
> >   arm64: dts: qcom: sm8350: Add Crypto Engine support
> >
> >  arch/arm64/boot/dts/qcom/sdm845.dtsi |  2 +-
> >  arch/arm64/boot/dts/qcom/sm6115.dtsi | 26 ++++++++++++++++++++++++++
> >  arch/arm64/boot/dts/qcom/sm8150.dtsi | 26 ++++++++++++++++++++++++++
> >  arch/arm64/boot/dts/qcom/sm8250.dtsi | 26 ++++++++++++++++++++++++++
> >  arch/arm64/boot/dts/qcom/sm8350.dtsi | 26 ++++++++++++++++++++++++++
> >  5 files changed, 105 insertions(+), 1 deletion(-)
> >
Konrad Dybcio March 22, 2023, 8:34 a.m. UTC | #3 
On 22.03.2023 09:33, Bhupesh Sharma wrote:
> On Wed, 22 Mar 2023 at 00:57, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>
>>
>>
>> On 21.03.2023 20:01, Bhupesh Sharma wrote:
>>> This patchset enables Crypto Engine support for Qualcomm SoCs like
>>> SM6115, SM8150, SM8250 and SM8350.
>>>
>>> While at it, also fix the compatible string for BAM DMA engine
>>> used in sdm845.dtsi
>>>
>>> Note that:
>>> - SM8250 crypto engine patch utilizes the work already done by myself and
>>>   Vladimir.
>>> - SM8350 crypto engine patch utilizes the work already done by Robert.
>>>
>>> This patchset depends on the dt-bindings patch which added support for
>>> SM6115 and QCS2290 SoCs:
>>> https://lore.kernel.org/linux-arm-msm/20230321184811.3325725-1-bhupesh.sharma@linaro.org/
>> You could have included that one here for easier tracking!
> 
> Err.. the dt-binding maintainers have been asking to split the
> dt-binding patches from
> the dts ones in the past.
Quite the opposite!

So, let's stick to that for now :)


Konrad
> 
> Thanks.
> 
>>> Also this patchset is rebased on linux-next/master.
>>>
>>> Bhupesh Sharma (5):
>>>   arm64: dts: qcom: sdm845: Fix the BAM DMA engine compatible string
>>>   arm64: dts: qcom: sm6115: Add Crypto Engine support
>>>   arm64: dts: qcom: sm8150: Add Crypto Engine support
>>>   arm64: dts: qcom: sm8250: Add Crypto Engine support
>>>   arm64: dts: qcom: sm8350: Add Crypto Engine support
>>>
>>>  arch/arm64/boot/dts/qcom/sdm845.dtsi |  2 +-
>>>  arch/arm64/boot/dts/qcom/sm6115.dtsi | 26 ++++++++++++++++++++++++++
>>>  arch/arm64/boot/dts/qcom/sm8150.dtsi | 26 ++++++++++++++++++++++++++
>>>  arch/arm64/boot/dts/qcom/sm8250.dtsi | 26 ++++++++++++++++++++++++++
>>>  arch/arm64/boot/dts/qcom/sm8350.dtsi | 26 ++++++++++++++++++++++++++
>>>  5 files changed, 105 insertions(+), 1 deletion(-)
>>>
Krzysztof Kozlowski March 22, 2023, 8:35 a.m. UTC | #4 
On 22/03/2023 09:33, Bhupesh Sharma wrote:
> On Wed, 22 Mar 2023 at 00:57, Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>>
>>
>>
>> On 21.03.2023 20:01, Bhupesh Sharma wrote:
>>> This patchset enables Crypto Engine support for Qualcomm SoCs like
>>> SM6115, SM8150, SM8250 and SM8350.
>>>
>>> While at it, also fix the compatible string for BAM DMA engine
>>> used in sdm845.dtsi
>>>
>>> Note that:
>>> - SM8250 crypto engine patch utilizes the work already done by myself and
>>>   Vladimir.
>>> - SM8350 crypto engine patch utilizes the work already done by Robert.
>>>
>>> This patchset depends on the dt-bindings patch which added support for
>>> SM6115 and QCS2290 SoCs:
>>> https://lore.kernel.org/linux-arm-msm/20230321184811.3325725-1-bhupesh.sharma@linaro.org/
>> You could have included that one here for easier tracking!
> 
> Err.. the dt-binding maintainers have been asking to split the
> dt-binding patches from
> the dts ones in the past. So, let's stick to that for now :)

For submissions? No, we did not.

Best regards,
Krzysztof
Bjorn Andersson March 22, 2023, 2:45 p.m. UTC | #5 
On Wed, 22 Mar 2023 00:31:13 +0530, Bhupesh Sharma wrote:
> This patchset enables Crypto Engine support for Qualcomm SoCs like
> SM6115, SM8150, SM8250 and SM8350.
> 
> While at it, also fix the compatible string for BAM DMA engine
> used in sdm845.dtsi
> 
> Note that:
> - SM8250 crypto engine patch utilizes the work already done by myself and
>   Vladimir.
> - SM8350 crypto engine patch utilizes the work already done by Robert.
> 
> [...]

Applied, thanks!

[1/5] arm64: dts: qcom: sdm845: Fix the BAM DMA engine compatible string
      commit: b767d1b40cbca49452d712ca0f4f053dacfc5c45

Best regards,