On Mon, Aug 01, 2022 at 03:19:00PM +0200, Marek Vasut wrote: > The ICN6211 is capable of swapping the output DPI RGB/BGR color channels, > document a DT property to select this swap in DT. This can be useful on > hardware where such swap happens. We should ensure this series[1] works for you instead. > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> > Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Cc: Rob Herring <robh@xxxxxxxxxx> > Cc: Robert Foss <robert.foss@xxxxxxxxxx> > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > Cc: devicetree@xxxxxxxxxxxxxxx > --- > .../devicetree/bindings/display/bridge/chipone,icn6211.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/chipone,icn6211.yaml b/Documentation/devicetree/bindings/display/bridge/chipone,icn6211.yaml > index 18563ebed1a96..e721dd76e6640 100644 > --- a/Documentation/devicetree/bindings/display/bridge/chipone,icn6211.yaml > +++ b/Documentation/devicetree/bindings/display/bridge/chipone,icn6211.yaml > @@ -33,6 +33,12 @@ properties: > Optional external clock connected to REF_CLK input. > The clock rate must be in 10..154 MHz range. > > + blue-and-red-swap: 'swap' assumes I know the default order and I don't. Better to make the property name describe the end result. ('invert' properties have the same issue) > + description: > + If defined, the DPI output blue and red channels are swapped, > + i.e. RGB becomes BGR. > + type: boolean > + > enable-gpios: > description: Bridge EN pin, chip is reset when EN is low. Rob [1] https://lore.kernel.org/r/20220628181838.2031-3-max.oss.09@xxxxxxxxx