Re: [PATCH 4/5 v3] dt-bindings: arm: Add Versatile Express and Juno YAML schema

[Rob Herring <robh@kernel.org>]

On Wed, 26 Feb 2020 12:07:14 +0100, Linus Walleij wrote:
> This implements the top-level schema for the ARM Versatile
> Express and Juno platforms.
> 
> Cc: Sudeep Holla <sudeep.holla@arm.com>
> Acked-by: Sudeep Holla <sudeep.holla@arm.com>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ChangeLog v2->v3:
> - Rename the smb@nnnnnnnn node to bus@nnnnnnnn, on a side
>   track in parallel proposed a patch to change this in all
>   device trees so we get coverage as well.
> - Drop an dash in front of maximum as minimum/maximum is a tuple
> - Move the simple-bus reference under the SMB node with an allOf
>   directive so we get the restrictions of simple-bus with the
>   additions of our compatible (etc)
> - Rewrite the compatible schema to use one enum and one const
>   item.
> - Duplicate the compatible schema for motherboard also directly
>   under the smb node with the option of being "simple-bus" only
>   and expand the text description so people understand what is
>   going on.
> - Indent the -if directive for required top-level properties
>   one step (syntactically incorrect but was incidentally working)
> ChangeLog v1->v2:
> - Add missing platforms: VE RTSM, FVP, foundation model
> - Properly define the arm,vexpress,site arm,vexpress,position
>   and arm,vexpress,dcc attributes. Maybe these are not the most
>   elegant bindings but they are used so we need to contain it
>   properly.
> - Add a patternProperty for the SMB (Static Memory Bus) which
>   was only described in text in the Vexpress bindings. It is a
>   "simple-bus" so just reference the existing bindings.
> - Define the layout of the "motherboard" node sometimes but
>   not always found below the SMB node, using two address-cells
>   with one for chipselect.
> - Make the arm,hbi property required on the ARMv7 variants.
> ---
>  .../bindings/arm/arm,vexpress-juno.yaml       | 223 ++++++++++++++++++
>  1 file changed, 223 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/arm,vexpress-juno.yaml
> 

Reviewed-by: Rob Herring <robh@kernel.org>