Thanks Laurent, On Sat, 3 Sept 2022 at 02:17, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > > Hi Chris, > > Thank you for the patch. > > On Fri, Sep 02, 2022 at 10:39:05AM -0500, Chris Morgan wrote: > > From: Chris Morgan <macromorgan@xxxxxxxxxxx> > > > > Update dt-binding documentation to add support for setting byteswap of > > chrontel ch7033. > > > > New property name of chrontel,byteswap added to set the byteswap order. > > This property is optional. > > > > Signed-off-by: Chris Morgan <macromorgan@xxxxxxxxxxx> > > Reviewed-by: Robert Foss <robert.foss@xxxxxxxxxx> > > --- > > .../bindings/display/bridge/chrontel,ch7033.yaml | 13 +++++++++++++ > > 1 file changed, 13 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/display/bridge/chrontel,ch7033.yaml b/Documentation/devicetree/bindings/display/bridge/chrontel,ch7033.yaml > > index bb6289c7d375..984b90893583 100644 > > --- a/Documentation/devicetree/bindings/display/bridge/chrontel,ch7033.yaml > > +++ b/Documentation/devicetree/bindings/display/bridge/chrontel,ch7033.yaml > > @@ -14,6 +14,19 @@ properties: > > compatible: > > const: chrontel,ch7033 > > > > + chrontel,byteswap: > > + $ref: /schemas/types.yaml#/definitions/uint8 > > + enum: > > + - 0 # BYTE_SWAP_RGB > > + - 1 # BYTE_SWAP_RBG > > + - 2 # BYTE_SWAP_GRB > > + - 3 # BYTE_SWAP_GBR > > + - 4 # BYTE_SWAP_BRG > > + - 5 # BYTE_SWAP_BGR > > + description: | > > + Set the byteswap value of the bridge. This is optional and if not > > + set value of BYTE_SWAP_BGR is used. > > I don't think this belongs to the device tree. The source of data > connected to the CH7033 input could use different formats. This > shouldn't be hardcoded, but queried at runtime, using the input and > output media bus formats infrastructure that the DRM bridge framework > includes. Chris, will you have a look at submitting a fix for this during the coming days? If not, we can revert this series and apply a fixed version later. > > > + > > reg: > > maxItems: 1 > > description: I2C address of the device > > -- > Regards, > > Laurent Pinchart
