Re: [PATCH v3 2/2] dt-bindings: Add Arm corstone500 platform

From: Krzysztof Kozlowski
Date: Thu Dec 22 2022 - 08:59:51 EST


On 22/12/2022 13:32, Emekcan Aras wrote:
> Add bindings to describe implementation of
> the ARM Corstone500 platform.
>
> Signed-off-by: Emekcan Aras <emekcan.aras@xxxxxxx>
> ---
> .../bindings/arm/arm,corstone500.yaml | 30 +++++++++++++++++++
> 1 file changed, 30 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/arm/arm,corstone500.yaml
>
> diff --git a/Documentation/devicetree/bindings/arm/arm,corstone500.yaml b/Documentation/devicetree/bindings/arm/arm,corstone500.yaml
> new file mode 100644
> index 000000000000..cfe41f7760fd
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/arm,corstone500.yaml

I don't think you solved my comments. Otherwise please point me how did
you solve this one:

"OK, so similar as Corstone 1000, but please tell me - any reason why
you keep them separate in the bindings? Next time new file for Corstone
1001, 1002, 2000, 2221 etc.?"


Best regards,
Krzysztof