[PATCH] drm/bridge: Add stubs for devm_drm_of_get_bridge when OF is disabled

Randy Dunlap rdunlap at infradead.org
Tue Sep 28 18:20:40 UTC 2021


On 9/28/21 11:13 AM, Maxime Ripard wrote:
> If CONFIG_OF is disabled, devm_drm_of_get_bridge won't be compiled in
> and drivers using that function will fail to build.
> 
> Add an inline stub so that we can still build-test those cases.
> 
> Reported-by: Randy Dunlap <rdunlap at infradead.org>
> Signed-off-by: Maxime Ripard <maxime at cerno.tech>

Acked-by: Randy Dunlap <rdunlap at infradead.org> # build-tested

Thanks.


> ---
>   include/drm/drm_bridge.h | 13 ++++++++++++-
>   1 file changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
> index 9cdbd209388e..1648ce265cba 100644
> --- a/include/drm/drm_bridge.h
> +++ b/include/drm/drm_bridge.h
> @@ -911,9 +911,20 @@ struct drm_bridge *devm_drm_panel_bridge_add(struct device *dev,
>   struct drm_bridge *devm_drm_panel_bridge_add_typed(struct device *dev,
>   						   struct drm_panel *panel,
>   						   u32 connector_type);
> +struct drm_connector *drm_panel_bridge_connector(struct drm_bridge *bridge);
> +#endif
> +
> +#if defined(CONFIG_OF) && defined(CONFIG_DRM_PANEL_BRIDGE)
>   struct drm_bridge *devm_drm_of_get_bridge(struct device *dev, struct device_node *node,
>   					  u32 port, u32 endpoint);
> -struct drm_connector *drm_panel_bridge_connector(struct drm_bridge *bridge);
> +#else
> +static inline struct drm_bridge *devm_drm_of_get_bridge(struct device *dev,
> +							struct device_node *node,
> +							u32 port,
> +							u32 endpoint)
> +{
> +	return ERR_PTR(-ENODEV);
> +}
>   #endif
>   
>   #endif
> 


-- 
~Randy


More information about the dri-devel mailing list