[PATCH v2 8/8] drm: bridge: anx7625: Switch to devm_drm_of_get_bridge

Linus Walleij linus.walleij at linaro.org
Tue Mar 1 23:20:06 UTC 2022


On Tue, Mar 1, 2022 at 3:13 PM Jagan Teki <jagan at amarulasolutions.com> wrote:

> +       bridge = devm_drm_of_get_bridge(dev, dev->of_node, 0, 0);
> +       if (IS_ERR(bridge)) {
> +               dev_err(dev, "error to get bridge\n");
> +               return PTR_ERR(bridge);
>         }
>
>         d->bridge_out = bridge;

Also notice how this bridge gets used in other places:

struct drm_connector *connector = drm_panel_bridge_connector(mcde->bridge);

Since you deleted:

-       } else if (bridge) {
-               /* TODO: AV8100 HDMI encoder goes here for example */
-               dev_info(dev, "connected to non-panel bridge (unsupported)\n");
-               return -ENODEV;

This will now have "interesting" effects. I think.

I don't know if there is a way to solve this though?

Yours,
Linus Walleij


More information about the dri-devel mailing list