On Wed, May 5, 2021 at 12:03 PM Marek Vasut <ma...@denx.de> wrote: > Add driver for TI SN65DSI83 Single-link DSI to Single-link LVDS bridge > and TI SN65DSI84 Single-link DSI to Dual-link or 2x Single-link LVDS > bridge. TI SN65DSI85 is unsupported due to lack of hardware to test on, > but easy to add. > > The driver operates the chip via I2C bus. Currently the LVDS clock are > always derived from DSI clock lane, which is the usual mode of operation. > Support for clock from external oscillator is not implemented, but it is > easy to add if ever needed. Only RGB888 pixel format is implemented, the > LVDS666 is not supported, but could be added if needed. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Douglas Anderson <diand...@chromium.org> > Cc: Jagan Teki <ja...@amarulasolutions.com> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Cc: Linus Walleij <linus.wall...@linaro.org> > Cc: Loic Poulain <loic.poul...@linaro.org> > Cc: Philippe Schenker <philippe.schen...@toradex.com> > Cc: Sam Ravnborg <s...@ravnborg.org> > Cc: Stephen Boyd <swb...@chromium.org> > Cc: Valentin Raevsky <valen...@compulab.co.il> > To: dri-devel@lists.freedesktop.org
This looks very nice to me, despite the complex nature of the driver it is a bliss to read this code with all nice comments and explanations. I don't know the gory details as well as the bridge maintainers but from my helicopter perspective, hats off. Reviewed-by: Linus Walleij <linus.wall...@linaro.org> Yours, Linus Walleij _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel