Re: [PATCH v6 3/7] drm/display: Add soft depend for aux-hpd-typec-dp-bridge module

Sebastian Reichel <[email protected]>
Newsgroups org.freedesktop.lists.dri-devel,org.infradead.lists.linux-arm-kernel,org.infradead.lists.linux-phy,org.infradead.lists.linux-rockchip,org.kernel.vger.linux-kernel
Message-ID <anTLpHm_bIPSsdvz@venus>
Hi,

On Tue, Aug 04, 2026 at 03:07:26PM +0800, Chaoyi Chen wrote:
> From: Chaoyi Chen <[email protected]>
> 
> The aux-hpd-typec-dp-bridge module serves as a generic TypeC
> DisplayPort HPD bridge and is not required by any other
> module. Therefore, it will not be auto-loaded.
> 
> Given that the drm_display_helper module houses DisplayPort-related
> helper code, add a MODULE_SOFTDEP() within it to suggest loading the
> aux-hpd-typec-dp-bridge module beforehand.
> 
> Suggested-by: Sebastian Reichel <[email protected]>
> Signed-off-by: Chaoyi Chen <[email protected]>
> ---

Reviewed-by: Sebastian Reichel <[email protected]>
Tested-by: Sebastian Reichel <[email protected]>

Booting defconfig without manual HPD bridge registration works now :)

Greetings,

-- Sebastian

> 
> (no changes since v6)
> 
> ---
>  drivers/gpu/drm/display/drm_display_helper_mod.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/display/drm_display_helper_mod.c b/drivers/gpu/drm/display/drm_display_helper_mod.c
> index d8a6e6228773..f0152d6b0b2d 100644
> --- a/drivers/gpu/drm/display/drm_display_helper_mod.c
> +++ b/drivers/gpu/drm/display/drm_display_helper_mod.c
> @@ -18,5 +18,6 @@ static void __exit drm_display_helper_module_exit(void)
>  	drm_dp_aux_dev_exit();
>  }
>  
> +MODULE_SOFTDEP("pre: aux-hpd-typec-dp-bridge");
>  module_init(drm_display_helper_module_init);
>  module_exit(drm_display_helper_module_exit);
> -- 
> 2.53.0
>
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmp0y+gACgkQ2O7X88g7
+po1aRAAoVOEYS4tq1yTJqRuHjwNUobsS9PYSr5qD/DQPC1oCG4gOnrFa6dbHHJq
rUBAu1add5nl7fsbmBqBskTWJmavNvRP/MD1gFLf3O/2kaHGMTu8J/kGohL8lK1c
ZmCWvUErRM40JPnmDgi+HyLrAfewMZS+gcxolhfB7dvbR47MWiBXV52aZ8umglFW
arbLXrh6q8/ibncTclQ4NW6D0loqyrKhSdTLNtGisYCcwJzvLmGhZdCfGEIcdHz8
B7l8pHmmcL2M5udlUV1TFp9f6PR37gMhPFF6eu1EY0nr6+6RAR6RcSXKmz7fH+2K
FgWL3Cw9o2Q851SH3VDapSyi01F5NVHiDJZrUCuCl5VylS4xJ7ZohMXVqxhi/7BF
wnQEOY92uKvJ8SZErnu/mxuyttYFn7QAh26REhCDfmOBkvguo6fskjKSqrCKpJsZ
WCijZVU3WVoiZrhhp0NISmGU0qQu/+Gl6de6bYAZYh6ohdbhgu5oFSOlHQdK8cBI
2pYuo7inWqpmh02xPRs1HFrnLQCptZMkm1VIBrIiVSam95dQqhSvRLqi7xeXu6Hu
uYJzZ3c1gh8lRBqYaXBO/lWZ81/+uUVFXgFfHTvMNRrfFklmCOvus9i43qf/Z66k
1z3ifHGU1tZVyDyyEqOXWPdvk4KwHtPgwI57NWSEgep83nQ8fnw=
=6FUg
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.