Hi Rob, On Thu, Jul 15, 2021 at 4:32 PM Rob Herring <robh@xxxxxxxxxx> wrote: > On Thu, Jul 15, 2021 at 1:12 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > On Wed, Jul 14, 2021 at 10:36 PM Rob Herring <robh@xxxxxxxxxx> wrote: > > > On Fri, Jun 25, 2021 at 02:58:46PM +0200, Geert Uytterhoeven wrote: > > > > The Holtek HT16K33 LED controller is not only used for driving > > > > dot-matrix displays, but also for driving segment displays. > > > > > > > > Document compatible values for the Adafruit 7-segment[1] and > > > > 14-segment[2] FeatherWing expansion boards with red displays. According > > > > to the schematics, all other Adafruit 7-segment and 14-segment display > > > > backpack and FeatherWing expansion boards (including bare boards and > > > > boards fitted with displays) are compatible with these two boards. > > > > Add a "color" property to support the different color variants. > > > > > > > > [1] https://www.adafruit.com/product/3108 > > > > [2] https://www.adafruit.com/product/3130 > > > > > > > > Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > > > > > --- a/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml > > > > +++ b/Documentation/devicetree/bindings/auxdisplay/holtek,ht16k33.yaml > > > > @@ -14,14 +14,23 @@ allOf: > > > > > > > > properties: > > > > compatible: > > > > - const: holtek,ht16k33 > > > > + oneOf: > > > > + - items: > > > > + - const: adafruit,3108 # 0.56" 4-Digit 7-Segment FeatherWing Display (Red) > > > > + - const: holtek,ht16k33 > > > > + > > > > + - items: > > > > + - const: adafruit,3130 # 0.54" Quad Alphanumeric FeatherWing Display (Red) > > > > + - const: holtek,ht16k33 > > > > > > These 2 entries can be combined. > > > > Right. This split dates back from when I considered adding all > > possible compatible values. It can indeed be simplified to: > > > > - enum: > > - adafruit,3108 # 0.56" 4-Digit 7-Segment FeatherWing > > Display (Red) > > - adafruit,3130 # 0.54" Quad Alphanumeric FeatherWing > > Display (Red) > > - const: holtek,ht16k33 > > > > > Or make the comment a 'description'. > > > > What do you mean? > > Adding this: > > description: '0.54" Quad Alphanumeric FeatherWing Display (Red)' Sorry, I still don't get it. Note that there are two different adafruit,NNNN compatible values. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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
