linux-kernel - Re: [PATCH] pr_info("I changed uvcvideo driver in the Linux Kernel\n");

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <49d1dece-cb90-23f3-977e-98905b1a0abc@collabora.com>
Date:   Mon, 9 Jan 2023 17:13:57 +0100
From:   Andrzej Pietrasiewicz <andrzej.p@...labora.com>
To:     XiaxiShen <shenxiaxi26@...il.com>,
        laurent.pinchart@...asonboard.com, mchehab@...nel.org
Cc:     linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] pr_info("I changed uvcvideo driver in the Linux
 Kernel\n");

Hello XiaxiShen,

 From Documentation/process/researcher-guidelines.rst:

"The Linux kernel community expects that everyone interacting with the
project is participating in good faith to make Linux better."

Please explain how your patch will make Linux better. "I added a pr_info()
which emits information unrelated to work of the driver" is not enough of an
explanation.

Failing to do so and to adhere to the expectation quoted above
you risk any patches to the kernel submitted by you being disregarded.

Andrzej

W dniu 30.12.2022 o 04:46, XiaxiShen pisze:
> Signed-off-by: XiaxiShen <shenxiaxi26@...il.com>
> ---
>   drivers/media/usb/uvc/uvc_driver.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> index e4bcb5011360..60778a5a2b4e 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -2082,6 +2082,8 @@ static int uvc_probe(struct usb_interface *intf,
>   		(const struct uvc_device_info *)id->driver_info;
>   	int function;
>   	int ret;
> +	pr_info("I changed uvcvideo driver in the Linux Kernel\n");
> +
>   
>   	/* Allocate memory for the device and initialize it. */
>   	dev = kzalloc(sizeof(*dev), GFP_KERNEL);