Re: [PATCH -next] Bluetooth: Fix Kconfig warning for BT_HIDP

From: YueHaibing
Date: Tue Nov 22 2022 - 04:06:20 EST



On 2022/11/22 16:31, Paul Menzel wrote:
> Dear YueHaibing,
>
>
> Thank you for your patch.
>
>
> Am 22.11.22 um 04:42 schrieb YueHaibing:
>
> Maybe use the more specific summary below:
>
> Bluetooth: Add HID_SUPPORT dependency for BT_HIDP
>
>> commit 25621bcc8976 add HID_SUPPORT, and HID depends on it now.
>
> add*s*
>
> or
>
> Commit 25621bcc8976 ("HID: Kconfig: split HID support and hid-core compilation") introduces the new Kconfig symbol HID_SUPPORT …
>
>

Thanks for your review, v2 is on the way.

> Kind regards,
>
> Paul
>
>
>> Add HID_SUPPORT dependency for BT_HIDP to fix the warning:
>>
>> WARNING: unmet direct dependencies detected for HID
>>    Depends on [n]: HID_SUPPORT [=n]
>>    Selected by [m]:
>>    - BT_HIDP [=m] && NET [=y] && BT_BREDR [=y] && INPUT [=m]
>>
>> Fixes: 25621bcc8976 ("HID: Kconfig: split HID support and hid-core compilation")
>> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>
>> ---
>>   net/bluetooth/hidp/Kconfig | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/net/bluetooth/hidp/Kconfig b/net/bluetooth/hidp/Kconfig
>> index 14100f341f33..6746be07e222 100644
>> --- a/net/bluetooth/hidp/Kconfig
>> +++ b/net/bluetooth/hidp/Kconfig
>> @@ -1,7 +1,7 @@
>>   # SPDX-License-Identifier: GPL-2.0-only
>>   config BT_HIDP
>>       tristate "HIDP protocol support"
>> -    depends on BT_BREDR && INPUT
>> +    depends on BT_BREDR && INPUT && HID_SUPPORT
>>       select HID
>>       help
>>         HIDP (Human Interface Device Protocol) is a transport layer
>
> .