public inbox for linux-bluetooth@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Bluetooth btusb.c: Add pr_info() for logging.
@ 2025-06-22 16:57 Sumedh W. Jambekar
  2025-06-22 17:32 ` bluez.test.bot
  2025-06-23  9:42 ` [PATCH] " Bastien Nocera
  0 siblings, 2 replies; 3+ messages in thread
From: Sumedh W. Jambekar @ 2025-06-22 16:57 UTC (permalink / raw)
  To: marcel, luiz.dentz, linux-bluetooth, linux-kernel; +Cc: Sumedh W. Jambekar

In this commit, the pr_info() call is added in the
probe function for logging purposes. This is done
as part of the assignment in the LFD103 course.

Signed-off-by: Sumedh W. Jambekar <linuxsumedh@gmail.com>
---
 drivers/bluetooth/btusb.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
index 9ab661d2d1e6..5ecb9a61a33b 100644
--- a/drivers/bluetooth/btusb.c
+++ b/drivers/bluetooth/btusb.c
@@ -3858,6 +3858,7 @@ static int btusb_probe(struct usb_interface *intf,
 	unsigned ifnum_base;
 	int i, err, priv_size;
 
+    pr_info("SWJ changed the btusb driver in the Linux Kernel\n");
 	BT_DBG("intf %p id %p", intf, id);
 
 	if ((id->driver_info & BTUSB_IFNUM_2) &&
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* RE: Bluetooth btusb.c: Add pr_info() for logging.
  2025-06-22 16:57 [PATCH] Bluetooth btusb.c: Add pr_info() for logging Sumedh W. Jambekar
@ 2025-06-22 17:32 ` bluez.test.bot
  2025-06-23  9:42 ` [PATCH] " Bastien Nocera
  1 sibling, 0 replies; 3+ messages in thread
From: bluez.test.bot @ 2025-06-22 17:32 UTC (permalink / raw)
  To: linux-bluetooth, linuxsumedh

[-- Attachment #1: Type: text/plain, Size: 1849 bytes --]

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=974616

---Test result---

Test Summary:
CheckPatch                    PENDING   0.41 seconds
GitLint                       PENDING   0.31 seconds
SubjectPrefix                 FAIL      0.36 seconds
BuildKernel                   PASS      25.30 seconds
CheckAllWarning               PASS      27.98 seconds
CheckSparse                   PASS      31.31 seconds
BuildKernel32                 PASS      25.26 seconds
TestRunnerSetup               PASS      491.56 seconds
TestRunner_l2cap-tester       PASS      25.90 seconds
TestRunner_iso-tester         PASS      50.41 seconds
TestRunner_bnep-tester        PASS      6.02 seconds
TestRunner_mgmt-tester        PASS      134.71 seconds
TestRunner_rfcomm-tester      PASS      9.53 seconds
TestRunner_sco-tester         PASS      15.08 seconds
TestRunner_ioctl-tester       PASS      10.31 seconds
TestRunner_mesh-tester        PASS      7.58 seconds
TestRunner_smp-tester         PASS      8.95 seconds
TestRunner_userchan-tester    PASS      6.40 seconds
IncrementalBuild              PENDING   0.56 seconds

Details
##############################
Test: CheckPatch - PENDING
Desc: Run checkpatch.pl script
Output:

##############################
Test: GitLint - PENDING
Desc: Run gitlint
Output:

##############################
Test: SubjectPrefix - FAIL
Desc: Check subject contains "Bluetooth" prefix
Output:
"Bluetooth: " prefix is not specified in the subject
##############################
Test: IncrementalBuild - PENDING
Desc: Incremental build with the patches in the series
Output:



---
Regards,
Linux Bluetooth


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Bluetooth btusb.c: Add pr_info() for logging.
  2025-06-22 16:57 [PATCH] Bluetooth btusb.c: Add pr_info() for logging Sumedh W. Jambekar
  2025-06-22 17:32 ` bluez.test.bot
@ 2025-06-23  9:42 ` Bastien Nocera
  1 sibling, 0 replies; 3+ messages in thread
From: Bastien Nocera @ 2025-06-23  9:42 UTC (permalink / raw)
  To: Sumedh W. Jambekar, marcel, luiz.dentz, linux-bluetooth,
	linux-kernel

On Sun, 2025-06-22 at 22:27 +0530, Sumedh W. Jambekar wrote:
> In this commit, the pr_info() call is added in the
> probe function for logging purposes. This is done
> as part of the assignment in the LFD103 course.

Do not submit junk contributions to upstream as part of your course
work. This is a burden on upstream, and you failed to even run it
through checkpatch.pl which would have shown that you used the wrong
spacing.

This is a good way to get blocked upstream, don't send another patch
unless it is something that can be merged upstream.

> 
> Signed-off-by: Sumedh W. Jambekar <linuxsumedh@gmail.com>
> ---
>  drivers/bluetooth/btusb.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/bluetooth/btusb.c b/drivers/bluetooth/btusb.c
> index 9ab661d2d1e6..5ecb9a61a33b 100644
> --- a/drivers/bluetooth/btusb.c
> +++ b/drivers/bluetooth/btusb.c
> @@ -3858,6 +3858,7 @@ static int btusb_probe(struct usb_interface
> *intf,
>  	unsigned ifnum_base;
>  	int i, err, priv_size;
>  
> +    pr_info("SWJ changed the btusb driver in the Linux Kernel\n");
>  	BT_DBG("intf %p id %p", intf, id);
>  
>  	if ((id->driver_info & BTUSB_IFNUM_2) &&

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-06-23  9:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-22 16:57 [PATCH] Bluetooth btusb.c: Add pr_info() for logging Sumedh W. Jambekar
2025-06-22 17:32 ` bluez.test.bot
2025-06-23  9:42 ` [PATCH] " Bastien Nocera

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox