Re: [PATCH 1/5] dt-bindings: memory: fsl: convert ifc binding to yaml schema

On Mon, Oct 4, 2021 at 4:31 AM Krzysztof Kozlowski
[off-list ref] wrote:

On 01/10/2021 18:17, Li Yang wrote:

quoted

On Fri, Oct 1, 2021 at 5:01 AM Krzysztof Kozlowski
[off-list ref] wrote:

quoted

(...)

quoted

quoted

quoted

+
+  interrupts:
+    minItems: 1
+    maxItems: 2
+    description: |
+      IFC may have one or two interrupts.  If two interrupt specifiers are
+      present, the first is the "common" interrupt (CM_EVTER_STAT), and the
+      second is the NAND interrupt (NAND_EVTER_STAT).  If there is only one,
+      that interrupt reports both types of event.
+
+  little-endian:
+    $ref: '/schemas/types.yaml#/definitions/flag'

type: boolean

It will not have a true or false value, but only present or not.  Is
the boolean type taking care of this too?

boolean is for a property which does not accept values and true/false
depends on its presence.
See:
Documentation/devicetree/bindings/phy/lantiq,vrx200-pcie-phy.yaml
Documentation/devicetree/bindings/thermal/qoriq-thermal.yaml

They are equivalent, so either can be used.

Really what is needed here is a common schema for the endianness
properties defining the type once. Then any binding using a property
can just do 'little-endian: true'.

Rob