diff options
author | Liu Ying <[email protected]> | 2020-07-09 10:02:36 +0800 |
---|---|---|
committer | Sam Ravnborg <[email protected]> | 2020-07-10 20:21:45 +0200 |
commit | 0bf4f5b5d3972df7014df302b95b58b8de1a1e94 (patch) | |
tree | 9b35e10cf93dbf86b3d691bfdaea2552daafe0bb /drivers/fpga/xilinx-spi.c | |
parent | 2ae53e79f2dec41949d7b089c0b6d7edce292d10 (diff) |
drm/bridge: dw-hdmi: Always add the bridge in the global bridge list
It doesn't hurt to add the bridge in the global bridge list also for
platform specific dw-hdmi drivers which are based on the component
framework. This can be achieved by moving the drm_bridge_add() function
call from dw_hdmi_probe() to __dw_hdmi_probe(). A counterpart movement
for drm_bridge_remove() is also needed then. Moreover, since drm_bridge_add()
initializes &bridge->hpd_mutex, this may help those platform specific
dw-hdmi drivers(based on the component framework) avoid accessing the
uninitialized mutex in drm_bridge_hpd_notify() which is called in
dw_hdmi_irq(). Putting drm_bridge_add() in __dw_hdmi_probe() just before
it returns successfully should bring no logic change for platforms based
on the DRM bridge API, which is a good choice from safety point of view.
Also, __dw_hdmi_probe() is renamed to dw_hdmi_probe() since dw_hdmi_probe()
does nothing else but calling __dw_hdmi_probe(). Similar renaming applies
to the __dw_hdmi_remove()/dw_hdmi_remove() pair.
Fixes: ec971aaa6775 ("drm: bridge: dw-hdmi: Make connector creation optional")
Cc: Andrzej Hajda <[email protected]>
Cc: Neil Armstrong <[email protected]>
Cc: Laurent Pinchart <[email protected]>
Cc: Jonas Karlman <[email protected]>
Cc: Jernej Skrabec <[email protected]>
Cc: David Airlie <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Boris Brezillon <[email protected]>
Cc: Jerome Brunet <[email protected]>
Cc: Cheng-Yi Chiang <[email protected]>
Cc: Dariusz Marcinkiewicz <[email protected]>
Cc: Archit Taneja <[email protected]>
Cc: Jose Abreu <[email protected]>
Cc: Sam Ravnborg <[email protected]>
Cc: [email protected]
Cc: NXP Linux Team <[email protected]>
Signed-off-by: Liu Ying <[email protected]>
Signed-off-by: Sam Ravnborg <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'drivers/fpga/xilinx-spi.c')
0 files changed, 0 insertions, 0 deletions