Hello Rob,

On 8/3/2019 1:38 AM, Rob Herring wrote:

On Fri, Aug 2, 2019 at 1:47 PM Stefan-gabriel Mirea
[off-list ref] wrote:

quoted

+      - description: S32V234 Customer Evaluation Board

Most of the entries in this file are for all the boards for an SoC.

quoted

+        items:
+          - enum:
+              - fsl,s32v234-evb

If that's not going to be the case here, you can use 'const' here.

We also intend to submit patches for the SBC-S32V234[1] board in the
future and I believe that its 'compatible' should share this entry.
Would it therefore be preferable to update the description at this
moment and add a comment, like:

      - description: S32V234 based Boards
        items:
          - enum:
              - fsl,s32v234-evb           # S32V234-EVB2 Customer Evaluation Board
          - const: fsl,s32v234

or just replace the single-value 'enum' with a 'const' for now?

Regards,
Stefan

[1] https://www.nxp.com/design/development-boards/automotive-development-platforms/s32v-mpus-platforms/s32v-vision-and-sensor-fusion-evaluation-board:SBC-S32V234

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel