Message ID | 20240422152413.1.Ib96985e197f3db620a127a84aa20f3f3017aaf57@changeid (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | Bluetooth: Populate hci_set_hw_info for Intel and Realtek | 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/CheckAllWarning | success | CheckAllWarning PASS |
tedd_an/CheckSparse | success | CheckSparse PASS |
tedd_an/CheckSmatch | fail | CheckSparse: FAIL: Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: net/bluetooth/hci_core.o] Error 139 make[4]: *** Deleting file 'net/bluetooth/hci_core.o' make[3]: *** [scripts/Makefile.build:485: net/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:485: net] Error 2 make[2]: *** Waiting for unfinished jobs.... Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: drivers/bluetooth/bcm203x.o] Error 139 make[4]: *** Deleting file 'drivers/bluetooth/bcm203x.o' make[4]: *** Waiting for unfinished jobs.... Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: drivers/bluetooth/bpa10x.o] Error 139 make[4]: *** Deleting file 'drivers/bluetooth/bpa10x.o' make[3]: *** [scripts/Makefile.build:485: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:485: drivers] Error 2 make[1]: *** [/github/workspace/src/src/Makefile:1919: .] Error 2 make: *** [Makefile:240: __sub-make] Error 2 |
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 | fail | TestRunner_mgmt-tester: Total: 492, Passed: 489 (99.4%), Failed: 1, Not Run: 2 |
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 |
Dear Archie, Thank you for your patch. Am 22.04.24 um 09:24 schrieb Archie Pusaka: > From: Archie Pusaka <apusaka@chromium.org> > > The hardware information surfaced via debugfs might be usable by the > userspace to set some configuration knobs. This patch sets the hw_info > for Intel and Realtek chipsets. Could you please add an example paste to the commit message? > Signed-off-by: Archie Pusaka <apusaka@chromium.org> > Reviewed-by: Abhishek Pandit-Subedi <abhishekpandit@google.com> > > --- > > drivers/bluetooth/btintel.c | 9 +++++++++ > drivers/bluetooth/btrtl.c | 7 +++++++ > 2 files changed, 16 insertions(+) > > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > index a19ebe47bd951..dc48352166a52 100644 > --- a/drivers/bluetooth/btintel.c > +++ b/drivers/bluetooth/btintel.c > @@ -2956,6 +2956,11 @@ static int btintel_setup_combined(struct hci_dev *hdev) > err = -EINVAL; > } > > + hci_set_hw_info(hdev, > + "INTEL platform=%u variant=%u revision=%u", > + ver.hw_platform, ver.hw_variant, > + ver.hw_revision); > + > goto exit_error; > } > > @@ -3060,6 +3065,10 @@ static int btintel_setup_combined(struct hci_dev *hdev) > break; > } > > + hci_set_hw_info(hdev, "INTEL platform=%u variant=%u", > + INTEL_HW_PLATFORM(ver_tlv.cnvi_bt), > + INTEL_HW_VARIANT(ver_tlv.cnvi_bt)); > + Why does it need to be added at two places? > exit_error: > kfree_skb(skb); > > diff --git a/drivers/bluetooth/btrtl.c b/drivers/bluetooth/btrtl.c > index cc50de69e8dc9..4f1e37b4f7802 100644 > --- a/drivers/bluetooth/btrtl.c > +++ b/drivers/bluetooth/btrtl.c > @@ -1339,6 +1339,13 @@ int btrtl_setup_realtek(struct hci_dev *hdev) > > btrtl_set_quirks(hdev, btrtl_dev); > > + hci_set_hw_info(hdev, > + "RTL lmp_subver=%u hci_rev=%u hci_ver=%u hci_bus=%u", > + btrtl_dev->ic_info->lmp_subver, > + btrtl_dev->ic_info->hci_rev, > + btrtl_dev->ic_info->hci_ver, > + btrtl_dev->ic_info->hci_bus); > + > btrtl_free(btrtl_dev); > return ret; > } Kind regards, Paul
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=846521 ---Test result--- Test Summary: CheckPatch PASS 1.14 seconds GitLint PASS 0.26 seconds SubjectPrefix PASS 0.09 seconds BuildKernel PASS 30.67 seconds CheckAllWarning PASS 33.38 seconds CheckSparse PASS 38.88 seconds CheckSmatch FAIL 36.77 seconds BuildKernel32 PASS 29.10 seconds TestRunnerSetup PASS 524.07 seconds TestRunner_l2cap-tester PASS 18.38 seconds TestRunner_iso-tester PASS 30.65 seconds TestRunner_bnep-tester PASS 4.68 seconds TestRunner_mgmt-tester FAIL 111.18 seconds TestRunner_rfcomm-tester PASS 7.23 seconds TestRunner_sco-tester PASS 14.97 seconds TestRunner_ioctl-tester PASS 7.51 seconds TestRunner_mesh-tester PASS 5.68 seconds TestRunner_smp-tester PASS 6.74 seconds TestRunner_userchan-tester PASS 4.81 seconds IncrementalBuild PASS 27.98 seconds Details ############################## Test: CheckSmatch - FAIL Desc: Run smatch tool with source Output: Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: net/bluetooth/hci_core.o] Error 139 make[4]: *** Deleting file 'net/bluetooth/hci_core.o' make[3]: *** [scripts/Makefile.build:485: net/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:485: net] Error 2 make[2]: *** Waiting for unfinished jobs.... Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: drivers/bluetooth/bcm203x.o] Error 139 make[4]: *** Deleting file 'drivers/bluetooth/bcm203x.o' make[4]: *** Waiting for unfinished jobs.... Segmentation fault (core dumped) make[4]: *** [scripts/Makefile.build:244: drivers/bluetooth/bpa10x.o] Error 139 make[4]: *** Deleting file 'drivers/bluetooth/bpa10x.o' make[3]: *** [scripts/Makefile.build:485: drivers/bluetooth] Error 2 make[2]: *** [scripts/Makefile.build:485: drivers] Error 2 make[1]: *** [/github/workspace/src/src/Makefile:1919: .] Error 2 make: *** [Makefile:240: __sub-make] Error 2 ############################## Test: TestRunner_mgmt-tester - FAIL Desc: Run mgmt-tester with test-runner Output: Total: 492, Passed: 489 (99.4%), Failed: 1, Not Run: 2 Failed Test Cases LL Privacy - Start Discovery 2 (Disable RL) Failed 0.170 seconds --- Regards, Linux Bluetooth
Hi Paul, On Mon, 22 Apr 2024 at 15:32, Paul Menzel <pmenzel@molgen.mpg.de> wrote: > > Dear Archie, > > > Thank you for your patch. > > Am 22.04.24 um 09:24 schrieb Archie Pusaka: > > From: Archie Pusaka <apusaka@chromium.org> > > > > The hardware information surfaced via debugfs might be usable by the > > userspace to set some configuration knobs. This patch sets the hw_info > > for Intel and Realtek chipsets. > > Could you please add an example paste to the commit message? > Sure, I will add those. > > Signed-off-by: Archie Pusaka <apusaka@chromium.org> > > Reviewed-by: Abhishek Pandit-Subedi <abhishekpandit@google.com> > > > > --- > > > > drivers/bluetooth/btintel.c | 9 +++++++++ > > drivers/bluetooth/btrtl.c | 7 +++++++ > > 2 files changed, 16 insertions(+) > > > > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > > index a19ebe47bd951..dc48352166a52 100644 > > --- a/drivers/bluetooth/btintel.c > > +++ b/drivers/bluetooth/btintel.c > > @@ -2956,6 +2956,11 @@ static int btintel_setup_combined(struct hci_dev *hdev) > > err = -EINVAL; > > } > > > > + hci_set_hw_info(hdev, > > + "INTEL platform=%u variant=%u revision=%u", > > + ver.hw_platform, ver.hw_variant, > > + ver.hw_revision); > > + > > goto exit_error; > > } > > > > @@ -3060,6 +3065,10 @@ static int btintel_setup_combined(struct hci_dev *hdev) > > break; > > } > > > > + hci_set_hw_info(hdev, "INTEL platform=%u variant=%u", > > + INTEL_HW_PLATFORM(ver_tlv.cnvi_bt), > > + INTEL_HW_VARIANT(ver_tlv.cnvi_bt)); > > + > > Why does it need to be added at two places? > Intel put the HW information into two structs, the "intel_version ver" and "intel_version_tlv ver_tlv". I don't know the history of the two structs, but that requires us to have the hw info set in two places. At most only one of them is executed though. > > exit_error: > > kfree_skb(skb); > > > > diff --git a/drivers/bluetooth/btrtl.c b/drivers/bluetooth/btrtl.c > > index cc50de69e8dc9..4f1e37b4f7802 100644 > > --- a/drivers/bluetooth/btrtl.c > > +++ b/drivers/bluetooth/btrtl.c > > @@ -1339,6 +1339,13 @@ int btrtl_setup_realtek(struct hci_dev *hdev) > > > > btrtl_set_quirks(hdev, btrtl_dev); > > > > + hci_set_hw_info(hdev, > > + "RTL lmp_subver=%u hci_rev=%u hci_ver=%u hci_bus=%u", > > + btrtl_dev->ic_info->lmp_subver, > > + btrtl_dev->ic_info->hci_rev, > > + btrtl_dev->ic_info->hci_ver, > > + btrtl_dev->ic_info->hci_bus); > > + > > btrtl_free(btrtl_dev); > > return ret; > > } > > > Kind regards, > > Paul Thanks! Archie
diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c index a19ebe47bd951..dc48352166a52 100644 --- a/drivers/bluetooth/btintel.c +++ b/drivers/bluetooth/btintel.c @@ -2956,6 +2956,11 @@ static int btintel_setup_combined(struct hci_dev *hdev) err = -EINVAL; } + hci_set_hw_info(hdev, + "INTEL platform=%u variant=%u revision=%u", + ver.hw_platform, ver.hw_variant, + ver.hw_revision); + goto exit_error; } @@ -3060,6 +3065,10 @@ static int btintel_setup_combined(struct hci_dev *hdev) break; } + hci_set_hw_info(hdev, "INTEL platform=%u variant=%u", + INTEL_HW_PLATFORM(ver_tlv.cnvi_bt), + INTEL_HW_VARIANT(ver_tlv.cnvi_bt)); + exit_error: kfree_skb(skb); diff --git a/drivers/bluetooth/btrtl.c b/drivers/bluetooth/btrtl.c index cc50de69e8dc9..4f1e37b4f7802 100644 --- a/drivers/bluetooth/btrtl.c +++ b/drivers/bluetooth/btrtl.c @@ -1339,6 +1339,13 @@ int btrtl_setup_realtek(struct hci_dev *hdev) btrtl_set_quirks(hdev, btrtl_dev); + hci_set_hw_info(hdev, + "RTL lmp_subver=%u hci_rev=%u hci_ver=%u hci_bus=%u", + btrtl_dev->ic_info->lmp_subver, + btrtl_dev->ic_info->hci_rev, + btrtl_dev->ic_info->hci_ver, + btrtl_dev->ic_info->hci_bus); + btrtl_free(btrtl_dev); return ret; }