lkml.org 
[lkml]   [2022]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v4 03/14] dt-bindings: i2c: add bindings for microchip mpfs i2c
    Hi Conor,

    On Mon, Jan 17, 2022 at 12:06 PM <conor.dooley@microchip.com> wrote:
    > From: Conor Dooley <conor.dooley@microchip.com>
    >
    > Add device tree bindings for the i2c controller on
    > the Microchip PolarFire SoC.
    >
    > Signed-off-by: Daire McNamara <daire.mcnamara@microchip.com>
    > Signed-off-by: Conor Dooley <conor.dooley@microchip.com>

    Thanks for your patch!

    > --- /dev/null
    > +++ b/Documentation/devicetree/bindings/i2c/microchip,mpfs-i2c.yaml
    > @@ -0,0 +1,55 @@
    > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
    > +%YAML 1.2
    > +---
    > +$id: http://devicetree.org/schemas/i2c/microchip,mpfs-i2c.yaml#
    > +$schema: http://devicetree.org/meta-schemas/core.yaml#
    > +
    > +title: Microchip MPFS I2C Controller Device Tree Bindings
    > +
    > +maintainers:
    > + - Daire McNamara <daire.mcnamara@microchip.com>
    > +
    > +allOf:
    > + - $ref: /schemas/i2c/i2c-controller.yaml#
    > +
    > +properties:
    > + compatible:
    > + enum:
    > + - microchip,mpfs-i2c # Microchip PolarFire SoC compatible SoCs
    > + - microchip,corei2c-rtl-v7 # Microchip Fabric based i2c IP core

    Wouldn't it be more logical to have:

    items:
    - const: microchip,mpfs-i2c # Microchip PolarFire SoC compatible SoCs
    - const: microchip,corei2c-rtl-v7 # Microchip Fabric based i2c IP core

    ?

    If the IP core is reused, it can become:

    items:
    - enum:
    - microchip,mpfs-i2c # Microchip PolarFire SoC compatible SoCs
    - microchip,<foo>-i2c # ...
    - const: microchip,corei2c-rtl-v7 # Microchip Fabric based i2c IP core

    That way the driver can just match on the second (fallback) value,
    and no further driver changes will be needed (until v8 or later).

    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

    \
     
     \ /
      Last update: 2022-01-20 09:37    [W:4.203 / U:0.068 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site