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

[Chaoyi Chen <chaoyi.chen@rock-chips.com>]

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@rock-chips.com>
>>
>> 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@oss.nxp.com>
>> Signed-off-by: Chaoyi Chen <chaoyi.chen@rock-chips.com>
>> ---
>>  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@airkyi.com/

> thanks,
> 
> greg k-h
> 
> 

-- 
Best, 
Chaoyi