| Message ID | 20221122034246.24408-1-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [-next] Bluetooth: Fix Kconfig warning for BT_HIDP | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/CheckPatch | success | CheckPatch PASS |
| tedd_an/GitLint | success | Gitlint PASS |
| tedd_an/SubjectPrefix | success | Gitlint PASS |
| tedd_an/BuildKernel | success | BuildKernel PASS |
| tedd_an/BuildKernel32 | success | BuildKernel32 PASS |
| tedd_an/TestRunnerSetup | success | TestRunnerSetup PASS |
| tedd_an/TestRunner_l2cap-tester | success | TestRunner PASS |
| tedd_an/TestRunner_iso-tester | success | TestRunner PASS |
| tedd_an/TestRunner_bnep-tester | success | TestRunner PASS |
| tedd_an/TestRunner_mgmt-tester | success | TestRunner PASS |
| tedd_an/TestRunner_rfcomm-tester | success | TestRunner PASS |
| tedd_an/TestRunner_sco-tester | success | TestRunner PASS |
| tedd_an/TestRunner_ioctl-tester | success | TestRunner PASS |
| tedd_an/TestRunner_mesh-tester | success | TestRunner PASS |
| tedd_an/TestRunner_smp-tester | success | TestRunner PASS |
| tedd_an/TestRunner_userchan-tester | success | TestRunner PASS |
| tedd_an/IncrementalBuild | success | Incremental Build PASS |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=697915 ---Test result--- Test Summary: CheckPatch PASS 0.49 seconds GitLint PASS 0.23 seconds SubjectPrefix PASS 0.06 seconds BuildKernel PASS 36.21 seconds BuildKernel32 PASS 32.77 seconds TestRunnerSetup PASS 466.59 seconds TestRunner_l2cap-tester PASS 16.68 seconds TestRunner_iso-tester PASS 17.23 seconds TestRunner_bnep-tester PASS 5.85 seconds TestRunner_mgmt-tester PASS 112.07 seconds TestRunner_rfcomm-tester PASS 10.06 seconds TestRunner_sco-tester PASS 9.36 seconds TestRunner_ioctl-tester PASS 11.30 seconds TestRunner_mesh-tester PASS 8.08 seconds TestRunner_smp-tester PASS 9.84 seconds TestRunner_userchan-tester PASS 6.65 seconds IncrementalBuild PASS 35.96 seconds --- Regards, Linux Bluetooth
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 … 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@huawei.com> > --- > 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
Hi, On Tue, Nov 22, 2022 at 9:37 AM Paul Menzel <pmenzel@molgen.mpg.de> 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 … > > > 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@huawei.com> > > --- FWIW, a fix is already in -next: https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git/commit/?h=for-6.2/hid-bpf&id=6cc90ccd4f6cfed98e2a3a378debc69f28d57473 But thanks for the patch nonetheless! Cheers, Benjamin > > > 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 >
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@huawei.com> >> --- >> 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 > > .
On 2022/11/22 17:06, Benjamin Tissoires wrote: > Hi, > > On Tue, Nov 22, 2022 at 9:37 AM Paul Menzel <pmenzel@molgen.mpg.de> 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 … >> >> >> 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@huawei.com> >>> --- > > > FWIW, a fix is already in -next: > https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git/commit/?h=for-6.2/hid-bpf&id=6cc90ccd4f6cfed98e2a3a378debc69f28d57473 OK, thanks for your info. > > But thanks for the patch nonetheless! > > Cheers, > Benjamin > >> >>> 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 >> > > . >
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
commit 25621bcc8976 add HID_SUPPORT, and HID depends on it now. 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@huawei.com> --- net/bluetooth/hidp/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)