Messages in this thread | | | From | Doug Anderson <> | Date | Tue, 7 Dec 2021 13:59:38 -0800 | Subject | Re: [PATCH v3] drm/bridge: sn65dsi86: defer if there is no dsi host |
| |
Hi,
On Tue, Dec 7, 2021 at 1:52 PM Rob Clark <robdclark@gmail.com> wrote: > > From: Rob Clark <robdclark@chromium.org> > > 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@chromium.org> > Reviewed-by: Doug Anderson <dianders@chromium.org> > --- > 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@chromium.org>
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
| |