[PATCH 0/2] New compatible for sa3ul for AM62x

[PATCH 0/2] New compatible for sa3ul for AM62x

[Jayesh Choudhary]

This series add the new compatible for sa3ul in the bindings and
driver file which will be used further in device tree nodes to
enable crypto support in TI SoC AM62x.

I will post the patch to enable crypto accelerator once this series
is merged.

Jayesh Choudhary (2):
  dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62
  crypto: sa2ul: Add the new compatible for AM62

 Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
 drivers/crypto/sa2ul.c                                 | 1 +
 2 files changed, 2 insertions(+)

-- 
2.17.1

[PATCH 1/2] dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62

[Jayesh Choudhary]

Add the AM62 version of sa3ul to the compatible list.

Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
 Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
index a410d2cedde6..02f47c2e7998 100644
--- a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
+++ b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
@@ -15,6 +15,7 @@ properties:
       - ti,j721e-sa2ul
       - ti,am654-sa2ul
       - ti,am64-sa2ul
+      - ti,am62-sa3ul
 
   reg:
     maxItems: 1
-- 
2.17.1

Re: [PATCH 1/2] dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62

[Krzysztof Kozlowski]

On 12/04/2022 09:30, Jayesh Choudhary wrote:
> Add the AM62 version of sa3ul to the compatible list.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>  Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
> index a410d2cedde6..02f47c2e7998 100644
> --- a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
> +++ b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
> @@ -15,6 +15,7 @@ properties:
>        - ti,j721e-sa2ul
>        - ti,am654-sa2ul
>        - ti,am64-sa2ul
> +      - ti,am62-sa3ul
>  

Just to be sure - dma-coherent is not required for this device (see
final "if:" in the bindings)?

Best regards,
Krzysztof

Re: [PATCH 1/2] dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62

[Jayesh Choudhary]

Hi,

On 12/04/22 5:33 pm, Krzysztof Kozlowski wrote:
> On 12/04/2022 09:30, Jayesh Choudhary wrote:
>> Add the AM62 version of sa3ul to the compatible list.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>>  Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
>> index a410d2cedde6..02f47c2e7998 100644
>> --- a/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
>> +++ b/Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml
>> @@ -15,6 +15,7 @@ properties:
>>        - ti,j721e-sa2ul
>>        - ti,am654-sa2ul
>>        - ti,am64-sa2ul
>> +      - ti,am62-sa3ul
>>  
> 
> Just to be sure - dma-coherent is not required for this device (see
> final "if:" in the bindings)?
> 

Yeah that's right.
dma-coherent is not required for this device.

Regards,
-Jayesh

Re: [PATCH 1/2] dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62

[Krzysztof Kozlowski]

On 12/04/2022 09:30, Jayesh Choudhary wrote:
> Add the AM62 version of sa3ul to the compatible list.
> 
> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
> ---
>  Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 


Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

[PATCH 2/2] crypto: sa2ul: Add the new compatible for AM62

[Jayesh Choudhary]

Add the new compatible for am62x in of_match_table.

Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
---
 drivers/crypto/sa2ul.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/crypto/sa2ul.c b/drivers/crypto/sa2ul.c
index 51b58e57153f..6957a125b447 100644
--- a/drivers/crypto/sa2ul.c
+++ b/drivers/crypto/sa2ul.c
@@ -2379,6 +2379,7 @@ static const struct of_device_id of_match[] = {
 	{ .compatible = "ti,j721e-sa2ul", .data = &am654_match_data, },
 	{ .compatible = "ti,am654-sa2ul", .data = &am654_match_data, },
 	{ .compatible = "ti,am64-sa2ul", .data = &am64_match_data, },
+	{ .compatible = "ti,am62-sa3ul", .data = &am64_match_data, },
 	{},
 };
 MODULE_DEVICE_TABLE(of, of_match);
-- 
2.17.1

Re: [PATCH 0/2] New compatible for sa3ul for AM62x

[Herbert Xu]

On Tue, Apr 12, 2022 at 01:00:14PM +0530, Jayesh Choudhary wrote:
> This series add the new compatible for sa3ul in the bindings and
> driver file which will be used further in device tree nodes to
> enable crypto support in TI SoC AM62x.
> 
> I will post the patch to enable crypto accelerator once this series
> is merged.
> 
> Jayesh Choudhary (2):
>   dt-bindings: crypto: ti,sa2ul: Add a new compatible for AM62
>   crypto: sa2ul: Add the new compatible for AM62
> 
>  Documentation/devicetree/bindings/crypto/ti,sa2ul.yaml | 1 +
>  drivers/crypto/sa2ul.c                                 | 1 +
>  2 files changed, 2 insertions(+)
> 
> -- 
> 2.17.1

All applied.  Thanks.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt