[RFC] dt-bindings: arm: amlogic: add C3 bindings

Message ID 20230406081627.4083103-1-kelvin.zhang@amlogic.com
State New
Headers
Series [RFC] dt-bindings: arm: amlogic: add C3 bindings |

Commit Message

Kelvin Zhang April 6, 2023, 8:16 a.m. UTC
  Document the new C3 SoC/board device tree bindings.

C3 is an advanced edge AI processor designed for smart IP camera
applications, which does not belong to Meson series.

Therefore, modify the title field accordingly.

Signed-off-by: Kelvin Zhang <kelvin.zhang@amlogic.com>
---
 Documentation/devicetree/bindings/arm/amlogic.yaml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)


base-commit: e134c93f788fb93fd6a3ec3af9af850a2048c7e6
  

Comments

Neil Armstrong April 6, 2023, 8:35 a.m. UTC | #1
Hi Kelvin.

On 06/04/2023 10:16, Kelvin Zhang wrote:
> Document the new C3 SoC/board device tree bindings.
> 
> C3 is an advanced edge AI processor designed for smart IP camera
> applications, which does not belong to Meson series.
> 
> Therefore, modify the title field accordingly.

Please CC linux-amlogic@lists.infradead.org and linux-arm-kernel@lists.infradead.org aswell.

> 
> Signed-off-by: Kelvin Zhang <kelvin.zhang@amlogic.com>
> ---
>   Documentation/devicetree/bindings/arm/amlogic.yaml | 9 ++++++++-
>   1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
> index 274ee0890312..ade730f35519 100644
> --- a/Documentation/devicetree/bindings/arm/amlogic.yaml
> +++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
> @@ -4,7 +4,7 @@
>   $id: http://devicetree.org/schemas/arm/amlogic.yaml#
>   $schema: http://devicetree.org/meta-schemas/core.yaml#
>   
> -title: Amlogic MesonX
> +title: Amlogic SoC based Platforms
>   
>   maintainers:
>     - Kevin Hilman <khilman@baylibre.com>
> @@ -211,6 +211,13 @@ properties:
>                 - amlogic,aq222
>             - const: amlogic,s4
>   
> +      - description: Boards with the Amlogic C3 C302X/C308L SoC
> +        items:
> +          - enum:
> +              - amlogic,aw409
> +              - amlogic,aw419
> +          - const: amlogic,c3
> +
>   additionalProperties: true
>   
>   ...
> 
> base-commit: e134c93f788fb93fd6a3ec3af9af850a2048c7e6

Thanks,
Neil
  
Krzysztof Kozlowski April 6, 2023, 6:11 p.m. UTC | #2
On 06/04/2023 10:16, Kelvin Zhang wrote:
> Document the new C3 SoC/board device tree bindings.
> 
> C3 is an advanced edge AI processor designed for smart IP camera
> applications, which does not belong to Meson series.
> 
> Therefore, modify the title field accordingly.
> 
> Signed-off-by: Kelvin Zhang <kelvin.zhang@amlogic.com>
> ---
>  Documentation/devicetree/bindings/arm/amlogic.yaml | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
> index 274ee0890312..ade730f35519 100644
> --- a/Documentation/devicetree/bindings/arm/amlogic.yaml
> +++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
> @@ -4,7 +4,7 @@
>  $id: http://devicetree.org/schemas/arm/amlogic.yaml#
>  $schema: http://devicetree.org/meta-schemas/core.yaml#
>  
> -title: Amlogic MesonX
> +title: Amlogic SoC based Platforms
>  
>  maintainers:
>    - Kevin Hilman <khilman@baylibre.com>
> @@ -211,6 +211,13 @@ properties:
>                - amlogic,aq222
>            - const: amlogic,s4
>  
> +      - description: Boards with the Amlogic C3 C302X/C308L SoC
> +        items:
> +          - enum:
> +              - amlogic,aw409
> +              - amlogic,aw419
> +          - const: amlogic,c3

Try to keep some order, e.g. between a and s.

Best regards,
Krzysztof
  
Kelvin Zhang April 7, 2023, 3:26 a.m. UTC | #3
On 2023/4/7 02:11, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 06/04/2023 10:16, Kelvin Zhang wrote:
>> Document the new C3 SoC/board device tree bindings.
>>
>> C3 is an advanced edge AI processor designed for smart IP camera
>> applications, which does not belong to Meson series.
>>
>> Therefore, modify the title field accordingly.
>>
>> Signed-off-by: Kelvin Zhang <kelvin.zhang@amlogic.com>
>> ---
>>   Documentation/devicetree/bindings/arm/amlogic.yaml | 9 ++++++++-
>>   1 file changed, 8 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
>> index 274ee0890312..ade730f35519 100644
>> --- a/Documentation/devicetree/bindings/arm/amlogic.yaml
>> +++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
>> @@ -4,7 +4,7 @@
>>   $id: http://devicetree.org/schemas/arm/amlogic.yaml#
>>   $schema: http://devicetree.org/meta-schemas/core.yaml#
>>   
>> -title: Amlogic MesonX
>> +title: Amlogic SoC based Platforms
>>   
>>   maintainers:
>>     - Kevin Hilman <khilman@baylibre.com>
>> @@ -211,6 +211,13 @@ properties:
>>                 - amlogic,aq222
>>             - const: amlogic,s4
>>   
>> +      - description: Boards with the Amlogic C3 C302X/C308L SoC
>> +        items:
>> +          - enum:
>> +              - amlogic,aw409
>> +              - amlogic,aw419
>> +          - const: amlogic,c3
> Try to keep some order, e.g. between a and s.
Sure. Thanks!
>
> Best regards,
> Krzysztof
>
  

Patch

diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
index 274ee0890312..ade730f35519 100644
--- a/Documentation/devicetree/bindings/arm/amlogic.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
@@ -4,7 +4,7 @@ 
 $id: http://devicetree.org/schemas/arm/amlogic.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
-title: Amlogic MesonX
+title: Amlogic SoC based Platforms
 
 maintainers:
   - Kevin Hilman <khilman@baylibre.com>
@@ -211,6 +211,13 @@  properties:
               - amlogic,aq222
           - const: amlogic,s4
 
+      - description: Boards with the Amlogic C3 C302X/C308L SoC
+        items:
+          - enum:
+              - amlogic,aw409
+              - amlogic,aw419
+          - const: amlogic,c3
+
 additionalProperties: true
 
 ...