Currently drm_bridge_connector_init() always returns the added connector or errors out. When adding bridge hotplug the bridge-connector can be successfully initialized without creating a connector, which can be added later when the pipeline will be complete.
For this the internal function drm_bridge_connector_add_connector() must be able to return a valid drm_bridge_connector even without any drm_connector. In preparation to support bridge hotplug, change its return value to be the same drm_bridge_connector pointer it gets as input, or a PTR_ERR. No functional changes, just changing an internal API. Note the return value could now become an int (0 or negative error) because returning the same value received as input does not carry any added value. However this would be change a lot of lines, so leave such change as a future cleanup. Signed-off-by: Luca Ceresoli <[email protected]> --- drivers/gpu/drm/display/drm_bridge_connector.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/display/drm_bridge_connector.c b/drivers/gpu/drm/display/drm_bridge_connector.c index 6813c6008985..eb1912611cd6 100644 --- a/drivers/gpu/drm/display/drm_bridge_connector.c +++ b/drivers/gpu/drm/display/drm_bridge_connector.c @@ -780,10 +780,10 @@ static void drm_bridge_connector_put_bridges(struct drm_device *dev, void *data) * @bridge_connector->drm and @bridge_connector->encoder fields are * assigned at least) * - * Returns a pointer to the new drm_connector on success, or a negative - * error pointer otherwise. + * Returns a pointer to @bridge_connector on success, or a negative error + * pointer otherwise. */ -static struct drm_connector * +static struct drm_bridge_connector * drm_bridge_connector_add_connector(struct drm_bridge_connector *bridge_connector) { struct drm_connector *connector; @@ -1049,7 +1049,7 @@ drm_bridge_connector_add_connector(struct drm_bridge_connector *bridge_connector if (ret) return ERR_PTR(ret); - return connector; + return bridge_connector; } /** @@ -1086,6 +1086,10 @@ struct drm_connector *drm_bridge_connector_init(struct drm_device *drm, bridge_connector->drm = drm; bridge_connector->encoder = encoder; - return drm_bridge_connector_add_connector(bridge_connector); + bridge_connector = drm_bridge_connector_add_connector(bridge_connector); + if (IS_ERR(bridge_connector)) + return ERR_CAST(bridge_connector); + + return &bridge_connector->base; } EXPORT_SYMBOL_GPL(drm_bridge_connector_init); -- 2.54.0
