Hi, On Tue, Dec 7, 2021 at 1:52 PM Rob Clark <robdclark@xxxxxxxxx> wrote: > > From: Rob Clark <robdclark@xxxxxxxxxxxx> > > Otherwise we don't get another shot at it if the bridge probes before > the dsi host is registered. It seems like this is what *most* (but not > all) of the other bridges do. > > It looks like this was missed in the conversion to attach dsi host at > probe time. > > Fixes: c3b75d4734cb ("drm/bridge: sn65dsi86: Register and attach our DSI device at probe") > Signed-off-by: Rob Clark <robdclark@xxxxxxxxxxxx> > Reviewed-by: Doug Anderson <dianders@xxxxxxxxxxxx> > --- > v2: Drop DRM_ERROR() in favor of drm_err_probe() and shift around the > spot where we report the error > v3: Add \n and cull error msgs a bit further > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 18 +++++++----------- > 1 file changed, 7 insertions(+), 11 deletions(-) I can add a further: Tested-by: Doug Anderson <dianders@xxxxxxxxxxxx> Since it's pretty straightforward and I don't expect it to be controversial, I'll plan to land it tomorrow in drm-misc-next unless I hear otherwise. -Doug