Re: [PATCH v2] usb: typec: altmodes: select DRM_AUX_HPD_BRIDGE for TYPEC_DP_ALTMODE

[Greg Kroah-Hartman]

On Thu, Jul 09, 2026 at 11:00:43AM +0800, Chaoyi Chen wrote:
> Hello Greg,
> 
> On 7/8/2026 7:51 PM, Greg Kroah-Hartman wrote:
> > On Sun, Jul 05, 2026 at 02:58:32PM +0800, Chaoyi Chen wrote:
> >> From: Chaoyi Chen <chaoyi.chen@xxxxxxxxxxxxxx>
> >>
> >> When TYPEC_DP_ALTMODE is enabled, DRM_AUX_HPD_BRIDGE is most likely
> >> also needed for embedded platforms. Select it when available.
> >>
> >> Suggested-by: Xu Yang <xu.yang_2@xxxxxxxxxxx>
> >> Signed-off-by: Chaoyi Chen <chaoyi.chen@xxxxxxxxxxxxxx>
> >> ---
> >>  drivers/usb/typec/altmodes/Kconfig | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/usb/typec/altmodes/Kconfig b/drivers/usb/typec/altmodes/Kconfig
> >> index 7867fa7c405d..1f35b253d6b8 100644
> >> --- a/drivers/usb/typec/altmodes/Kconfig
> >> +++ b/drivers/usb/typec/altmodes/Kconfig
> >> @@ -5,6 +5,7 @@ menu "USB Type-C Alternate Mode drivers"
> >>  config TYPEC_DP_ALTMODE
> >>  	tristate "DisplayPort Alternate Mode driver"
> >>  	depends on DRM
> >> +	imply DRM_AUX_HPD_BRIDGE if DRM_BRIDGE && OF
> > 
> > I hate "imply", as this just makes things more complex.  Either this is
> > a dependency or not, right?  Why not fix that?
> >
> 
> Perhaps we can just use the v1 content directly? 
> Hopefully that's already simple enough.
> 
> https://lore.kernel.org/all/20260702022036.44-1-kernel@xxxxxxxxxx/

Maybe, send it as a v3 and see if the maintainers of those drivers
agree?

thanks,

greg k-h