Hi, On Tue, Mar 18, 2025 at 10:56 AM Doug Anderson <diand...@chromium.org> wrote: > > Hi, > > On Tue, Mar 18, 2025 at 8:56 AM Wolfram Sang > <wsa+rene...@sang-engineering.com> wrote: > > > > Read out and check the ID registers, so we can bail out if I2C > > communication does not work or if the device is unknown. Tested on a > > Renesas GrayHawk board (R-Car V4M) by using a wrong I2C address and by > > not enabling RuntimePM for the device. > > > > Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com> > > --- > > > > Changes since v2: > > * switched to a new approach suggested by Doug (Thanks!). We add a > > dedicated read instead of using the first read. This prevents creating > > the aux devices. As a side-gain, we check now if the chip at the address > > is really the one we want to support. > > > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > Looks good to me. > > Reviewed-by: Douglas Anderson <diand...@chromium.org>
Pushed to drm-misc-next: [1/1] drm/bridge: ti-sn65dsi86: Check bridge connection failure commit: d69362f55fba92eb4cac10fe8da618de52b49bfc