| Message ID | 20240105-sm6350-qce-v1-0-416e5c7319ac@fairphone.com |
|---|---|
| Headers | show |
| Series | Add Crypto Engine support for SM6350 | expand |
On 05/01/2024 17:15, Luca Weiss wrote: > Add a compatible for the crypto block found on the SM6350 SoC. > > Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On 5.01.2024 17:15, Luca Weiss wrote: > Add the compatible and nodes for the QCE found on SM6350 SoC. > > Not completely sure how to fully test it but "kcapi-speed --all" shows > no issues. Let me know if I can/should test this more. I think I used `cryptsetup benchmark` with and without the ICE enabled a couple years back. IIRC the CPU should be faaar faster but also chug power while at it. Konrad
On Mon Jan 8, 2024 at 1:40 PM CET, Konrad Dybcio wrote: > On 5.01.2024 17:15, Luca Weiss wrote: > > Add the compatible and nodes for the QCE found on SM6350 SoC. > > > > Not completely sure how to fully test it but "kcapi-speed --all" shows > > no issues. Let me know if I can/should test this more. > > I think I used `cryptsetup benchmark` with and without the ICE enabled > a couple years back. IIRC the CPU should be faaar faster but also chug > power while at it. Are you sure you mean QCE here (which this patch is about) and not ICE? I'm not aware of them working together somehow but I wouldn't be surprised if there's something since I don't know much of this area at all. Regards Luca > > Konrad
On 1/9/24 12:27, Luca Weiss wrote: > On Mon Jan 8, 2024 at 1:40 PM CET, Konrad Dybcio wrote: >> On 5.01.2024 17:15, Luca Weiss wrote: >>> Add the compatible and nodes for the QCE found on SM6350 SoC. >>> >>> Not completely sure how to fully test it but "kcapi-speed --all" shows >>> no issues. Let me know if I can/should test this more. >> >> I think I used `cryptsetup benchmark` with and without the ICE enabled >> a couple years back. IIRC the CPU should be faaar faster but also chug >> power while at it. > > Are you sure you mean QCE here (which this patch is about) and not ICE? I.. think I do. It's been a while. > > I'm not aware of them working together somehow but I wouldn't be > surprised if there's something since I don't know much of this area at > all. No idea Konrad
Add the compatible and nodes for the QCE found on SM6350 SoC. Not completely sure how to fully test it but "kcapi-speed --all" shows no issues. Let me know if I can/should test this more. Signed-off-by: Luca Weiss <luca.weiss@fairphone.com> --- Luca Weiss (2): dt-bindings: qcom-qce: Add compatible for SM6350 arm64: dts: qcom: sm6350: Add Crypto Engine .../devicetree/bindings/crypto/qcom-qce.yaml | 1 + arch/arm64/boot/dts/qcom/sm6350.dtsi | 31 ++++++++++++++++++++++ 2 files changed, 32 insertions(+) --- base-commit: 610a9b8f49fbcf1100716370d3b5f6f884a2835a change-id: 20240105-sm6350-qce-c6233abbf54f Best regards,