Re: [PATCH 03/13] dt-bindings: soc/microchip: update sys ctrlr compat string

On 09/11/2021 08:33, Geert Uytterhoeven wrote:

EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe

Hi Conor,

On Mon, Nov 8, 2021 at 4:06 PM [off-list ref] wrote:

quoted

From: Conor Dooley <conor.dooley@microchip.com>

Update 'compatible' strings for system controller drivers to the
approved Microchip name.

Signed-off-by: Conor Dooley <conor.dooley@microchip.com>

Thanks for your patch!

quoted

--- a/Documentation/devicetree/bindings/mailbox/microchip,polarfire-soc-mailbox.yaml
+++ b/Documentation/devicetree/bindings/mailbox/microchip,polarfire-soc-mailbox.yaml

@@ -11,7 +11,9 @@ maintainers:

  properties:
    compatible:
-    const: microchip,polarfire-soc-mailbox
+    enum:
+      - microchip,polarfire-soc-mailbox
+      - microchip,mpfs-mailbox

Is there any point in keeping the old compatible value?
Are there any real users? Most of the MPFS upstream DT is still in flux.

yeah, true. and now is as good a time as any to just outright change it.

Gr{oetje,eeting}s,

                         Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                 -- Linus Torvalds