* [PATCH 1/3] dt-bindings: crypto: qcom,prng: Document kaanapali RNG
2025-09-24 23:38 [PATCH 0/3] Add Crypto support for Kaanapali SoC Jingyi Wang
@ 2025-09-24 23:38 ` Jingyi Wang
2025-10-12 3:08 ` Krzysztof Kozlowski
2025-09-24 23:38 ` [PATCH 2/3] dt-bindings: crypto: qcom-qce: Document the kaanapli crypto engine Jingyi Wang
2025-09-24 23:38 ` [PATCH 3/3] crypto: qce: add support for QCE major version 6 Jingyi Wang
2 siblings, 1 reply; 7+ messages in thread
From: Jingyi Wang @ 2025-09-24 23:38 UTC (permalink / raw)
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Vinod Koul, Thara Gopinath, Bjorn Andersson,
Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel,
Jingyi Wang, aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang,
Gaurav Kashyap
From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
Document kaanapali compatible for the True Random Number Generator.
Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
---
Documentation/devicetree/bindings/crypto/qcom,prng.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
index ed7e16bd11d3..597441d94cf1 100644
--- a/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
+++ b/Documentation/devicetree/bindings/crypto/qcom,prng.yaml
@@ -20,6 +20,7 @@ properties:
- qcom,ipq5332-trng
- qcom,ipq5424-trng
- qcom,ipq9574-trng
+ - qcom,kaanapali-trng
- qcom,qcs615-trng
- qcom,qcs8300-trng
- qcom,sa8255p-trng
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 1/3] dt-bindings: crypto: qcom,prng: Document kaanapali RNG
2025-09-24 23:38 ` [PATCH 1/3] dt-bindings: crypto: qcom,prng: Document kaanapali RNG Jingyi Wang
@ 2025-10-12 3:08 ` Krzysztof Kozlowski
0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-10-12 3:08 UTC (permalink / raw)
To: Jingyi Wang, Herbert Xu, David S. Miller, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Vinod Koul, Thara Gopinath,
Bjorn Andersson, Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel, aiqun.yu,
tingwei.zhang, trilok.soni, yijie.yang, Gaurav Kashyap
On 25/09/2025 01:38, Jingyi Wang wrote:
> From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
>
> Document kaanapali compatible for the True Random Number Generator.
>
> Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
> ---
> Documentation/devicetree/bindings/crypto/qcom,prng.yaml | 1 +
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/3] dt-bindings: crypto: qcom-qce: Document the kaanapli crypto engine
2025-09-24 23:38 [PATCH 0/3] Add Crypto support for Kaanapali SoC Jingyi Wang
2025-09-24 23:38 ` [PATCH 1/3] dt-bindings: crypto: qcom,prng: Document kaanapali RNG Jingyi Wang
@ 2025-09-24 23:38 ` Jingyi Wang
2025-10-12 3:08 ` Krzysztof Kozlowski
2025-09-24 23:38 ` [PATCH 3/3] crypto: qce: add support for QCE major version 6 Jingyi Wang
2 siblings, 1 reply; 7+ messages in thread
From: Jingyi Wang @ 2025-09-24 23:38 UTC (permalink / raw)
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Vinod Koul, Thara Gopinath, Bjorn Andersson,
Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel,
Jingyi Wang, aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang,
Gaurav Kashyap
From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
Document the crypto engine on the kaanapali platform.
Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
---
Documentation/devicetree/bindings/crypto/qcom-qce.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/crypto/qcom-qce.yaml b/Documentation/devicetree/bindings/crypto/qcom-qce.yaml
index e009cb712fb8..79d5be2548bc 100644
--- a/Documentation/devicetree/bindings/crypto/qcom-qce.yaml
+++ b/Documentation/devicetree/bindings/crypto/qcom-qce.yaml
@@ -45,6 +45,7 @@ properties:
- items:
- enum:
+ - qcom,kaanapali-qce
- qcom,qcs615-qce
- qcom,qcs8300-qce
- qcom,sa8775p-qce
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 2/3] dt-bindings: crypto: qcom-qce: Document the kaanapli crypto engine
2025-09-24 23:38 ` [PATCH 2/3] dt-bindings: crypto: qcom-qce: Document the kaanapli crypto engine Jingyi Wang
@ 2025-10-12 3:08 ` Krzysztof Kozlowski
0 siblings, 0 replies; 7+ messages in thread
From: Krzysztof Kozlowski @ 2025-10-12 3:08 UTC (permalink / raw)
To: Jingyi Wang, Herbert Xu, David S. Miller, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Vinod Koul, Thara Gopinath,
Bjorn Andersson, Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel, aiqun.yu,
tingwei.zhang, trilok.soni, yijie.yang, Gaurav Kashyap
On 25/09/2025 01:38, Jingyi Wang wrote:
> From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
>
> Document the crypto engine on the kaanapali platform.
>
> Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] crypto: qce: add support for QCE major version 6
2025-09-24 23:38 [PATCH 0/3] Add Crypto support for Kaanapali SoC Jingyi Wang
2025-09-24 23:38 ` [PATCH 1/3] dt-bindings: crypto: qcom,prng: Document kaanapali RNG Jingyi Wang
2025-09-24 23:38 ` [PATCH 2/3] dt-bindings: crypto: qcom-qce: Document the kaanapli crypto engine Jingyi Wang
@ 2025-09-24 23:38 ` Jingyi Wang
2025-09-25 11:48 ` Konrad Dybcio
2 siblings, 1 reply; 7+ messages in thread
From: Jingyi Wang @ 2025-09-24 23:38 UTC (permalink / raw)
To: Herbert Xu, David S. Miller, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Vinod Koul, Thara Gopinath, Bjorn Andersson,
Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel,
Jingyi Wang, aiqun.yu, tingwei.zhang, trilok.soni, yijie.yang,
Gaurav Kashyap
From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
The Qualcomm General Purpose Crypto Engine (GPCE) has
upgraded to version 6 on newer socs, this makes the
version check logic obsolete and needs to be updated
to account for v6.0.
As part of this, unnecessary version macros has also
been removed.
Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
---
drivers/crypto/qce/core.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/crypto/qce/core.c b/drivers/crypto/qce/core.c
index e95e84486d9a..b966f3365b7d 100644
--- a/drivers/crypto/qce/core.c
+++ b/drivers/crypto/qce/core.c
@@ -21,7 +21,6 @@
#include "sha.h"
#include "aead.h"
-#define QCE_MAJOR_VERSION5 0x05
#define QCE_QUEUE_LENGTH 1
#define QCE_DEFAULT_MEM_BANDWIDTH 393600
@@ -161,7 +160,7 @@ static int qce_check_version(struct qce_device *qce)
* the driver does not support v5 with minor 0 because it has special
* alignment requirements.
*/
- if (major != QCE_MAJOR_VERSION5 || minor == 0)
+ if (major == 5 && minor == 0)
return -ENODEV;
qce->burst_size = QCE_BAM_BURST_SIZE;
--
2.25.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] crypto: qce: add support for QCE major version 6
2025-09-24 23:38 ` [PATCH 3/3] crypto: qce: add support for QCE major version 6 Jingyi Wang
@ 2025-09-25 11:48 ` Konrad Dybcio
0 siblings, 0 replies; 7+ messages in thread
From: Konrad Dybcio @ 2025-09-25 11:48 UTC (permalink / raw)
To: Jingyi Wang, Herbert Xu, David S. Miller, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Vinod Koul, Thara Gopinath,
Bjorn Andersson, Konrad Dybcio
Cc: linux-arm-msm, linux-crypto, devicetree, linux-kernel, aiqun.yu,
tingwei.zhang, trilok.soni, yijie.yang, Gaurav Kashyap
On 9/25/25 1:38 AM, Jingyi Wang wrote:
> From: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
>
> The Qualcomm General Purpose Crypto Engine (GPCE) has
> upgraded to version 6 on newer socs, this makes the
> version check logic obsolete and needs to be updated
> to account for v6.0.
>
> As part of this, unnecessary version macros has also
> been removed.
>
> Signed-off-by: Gaurav Kashyap <gaurav.kashyap@oss.qualcomm.com>
> Signed-off-by: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
> ---
This commit message does not reflect what the code does (or at
least it does so very indirectly)
You can instead title the commit message "fix version check" and
mention in the commit text that this is necessary e.g. for v6
Konrad
^ permalink raw reply [flat|nested] 7+ messages in thread