drm: don't link DP aux i2c adapter to the hardware device node
authorLucas Stach <l.stach@pengutronix.de>
Fri, 13 Jan 2017 17:36:30 +0000 (18:36 +0100)
committerAndrzej Hajda <a.hajda@samsung.com>
Mon, 20 Nov 2017 08:34:17 +0000 (09:34 +0100)
The i2c adapter on DP AUX is purely a software construct. Linking
it to the device node of the parent device is wrong, as it leads to
2 devices sharing the same device node, which is bad practice, as
well as the i2c trying to populate children of the i2c adapter by
looking at the child device nodes of the parent device.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170113173630.22138-1-l.stach@pengutronix.de
drivers/gpu/drm/drm_dp_helper.c

index b3d68964b407677148f11551dababb0d7cb71853..adf79be42c1e2a5b4fe3bdc42deae5c7d182185a 100644 (file)
@@ -1097,7 +1097,6 @@ int drm_dp_aux_register(struct drm_dp_aux *aux)
        aux->ddc.class = I2C_CLASS_DDC;
        aux->ddc.owner = THIS_MODULE;
        aux->ddc.dev.parent = aux->dev;
-       aux->ddc.dev.of_node = aux->dev->of_node;
 
        strlcpy(aux->ddc.name, aux->name ? aux->name : dev_name(aux->dev),
                sizeof(aux->ddc.name));