[PATCH] Modified uvc_driver

[PATCH] Modified uvc_driver

[Jayesh Puri]

---
 Kconfig                            | 0
 drivers/media/usb/uvc/uvc_driver.c | 2 ++
 2 files changed, 2 insertions(+)
 mode change 100644 => 100755 Kconfig

diff --git a/Kconfig b/Kconfig
old mode 100644
new mode 100755
diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
index aa3e8d295e0f..cec117328ab2 100644
--- a/drivers/media/usb/uvc/uvc_driver.c
+++ b/drivers/media/usb/uvc/uvc_driver.c
@@ -2191,6 +2191,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);
-- 
2.53.0

Re: [PATCH] Modified uvc_driver

[Laurent Pinchart]

On Thu, Feb 19, 2026 at 06:48:11PM +0530, Jayesh Puri wrote:
> ---
>  Kconfig                            | 0
>  drivers/media/usb/uvc/uvc_driver.c | 2 ++
>  2 files changed, 2 insertions(+)
>  mode change 100644 => 100755 Kconfig
> 
> diff --git a/Kconfig b/Kconfig
> old mode 100644
> new mode 100755
> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> index aa3e8d295e0f..cec117328ab2 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -2191,6 +2191,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");

Congratulations, but I'm afraid we're out of medals.

Shuah, I thought LFD103 has been updated to avoid spamming the lists
with such patches. What happened ?

>  
>  	/* Allocate memory for the device and initialize it. */
>  	dev = kzalloc(sizeof(*dev), GFP_KERNEL);

-- 
Regards,

Laurent Pinchart

Re: [PATCH] Modified uvc_driver

[Shuah Khan]

On 2/19/26 06:32, Laurent Pinchart wrote:
> On Thu, Feb 19, 2026 at 06:48:11PM +0530, Jayesh Puri wrote:
>> ---
>>   Kconfig                            | 0
>>   drivers/media/usb/uvc/uvc_driver.c | 2 ++
>>   2 files changed, 2 insertions(+)
>>   mode change 100644 => 100755 Kconfig
>>
>> diff --git a/Kconfig b/Kconfig
>> old mode 100644
>> new mode 100755
>> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
>> index aa3e8d295e0f..cec117328ab2 100644
>> --- a/drivers/media/usb/uvc/uvc_driver.c
>> +++ b/drivers/media/usb/uvc/uvc_driver.c
>> @@ -2191,6 +2191,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");
> 
> Congratulations, but I'm afraid we're out of medals.
> 
> Shuah, I thought LFD103 has been updated to avoid spamming the lists
> with such patches. What happened ?

It is changed a while back - not sure what's going on.

thanks,
-- Shuah

Re: [PATCH] Modified uvc_driver

[Laurent Pinchart]

On Thu, Feb 19, 2026 at 01:24:23PM -0700, Shuah Khan wrote:
> On 2/19/26 06:32, Laurent Pinchart wrote:
> > On Thu, Feb 19, 2026 at 06:48:11PM +0530, Jayesh Puri wrote:
> >> ---
> >>   Kconfig                            | 0
> >>   drivers/media/usb/uvc/uvc_driver.c | 2 ++
> >>   2 files changed, 2 insertions(+)
> >>   mode change 100644 => 100755 Kconfig
> >>
> >> diff --git a/Kconfig b/Kconfig
> >> old mode 100644
> >> new mode 100755
> >> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> >> index aa3e8d295e0f..cec117328ab2 100644
> >> --- a/drivers/media/usb/uvc/uvc_driver.c
> >> +++ b/drivers/media/usb/uvc/uvc_driver.c
> >> @@ -2191,6 +2191,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");
> > 
> > Congratulations, but I'm afraid we're out of medals.
> > 
> > Shuah, I thought LFD103 has been updated to avoid spamming the lists
> > with such patches. What happened ?
> 
> It is changed a while back - not sure what's going on.

Could you please investigate ?

-- 
Regards,

Laurent Pinchart

Re: [PATCH] Modified uvc_driver

[Shuah Khan]

On 2/20/26 03:27, Laurent Pinchart wrote:
> On Thu, Feb 19, 2026 at 01:24:23PM -0700, Shuah Khan wrote:
>> On 2/19/26 06:32, Laurent Pinchart wrote:
>>> On Thu, Feb 19, 2026 at 06:48:11PM +0530, Jayesh Puri wrote:
>>>> ---
>>>>    Kconfig                            | 0
>>>>    drivers/media/usb/uvc/uvc_driver.c | 2 ++
>>>>    2 files changed, 2 insertions(+)
>>>>    mode change 100644 => 100755 Kconfig
>>>>
>>>> diff --git a/Kconfig b/Kconfig
>>>> old mode 100644
>>>> new mode 100755
>>>> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
>>>> index aa3e8d295e0f..cec117328ab2 100644
>>>> --- a/drivers/media/usb/uvc/uvc_driver.c
>>>> +++ b/drivers/media/usb/uvc/uvc_driver.c
>>>> @@ -2191,6 +2191,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");
>>>
>>> Congratulations, but I'm afraid we're out of medals.
>>>
>>> Shuah, I thought LFD103 has been updated to avoid spamming the lists
>>> with such patches. What happened ?
>>
>> It is changed a while back - not sure what's going on.
> 
> Could you please investigate ?
> 

I will look into it

thanks,
-- Shuah

Re: [PATCH] Modified uvc_driver

[Laurent Pinchart]

On Fri, Feb 20, 2026 at 08:49:47AM -0700, Shuah Khan wrote:
> On 2/20/26 03:27, Laurent Pinchart wrote:
> > On Thu, Feb 19, 2026 at 01:24:23PM -0700, Shuah Khan wrote:
> >> On 2/19/26 06:32, Laurent Pinchart wrote:
> >>> On Thu, Feb 19, 2026 at 06:48:11PM +0530, Jayesh Puri wrote:
> >>>> ---
> >>>>    Kconfig                            | 0
> >>>>    drivers/media/usb/uvc/uvc_driver.c | 2 ++
> >>>>    2 files changed, 2 insertions(+)
> >>>>    mode change 100644 => 100755 Kconfig
> >>>>
> >>>> diff --git a/Kconfig b/Kconfig
> >>>> old mode 100644
> >>>> new mode 100755
> >>>> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> >>>> index aa3e8d295e0f..cec117328ab2 100644
> >>>> --- a/drivers/media/usb/uvc/uvc_driver.c
> >>>> +++ b/drivers/media/usb/uvc/uvc_driver.c
> >>>> @@ -2191,6 +2191,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");
> >>>
> >>> Congratulations, but I'm afraid we're out of medals.
> >>>
> >>> Shuah, I thought LFD103 has been updated to avoid spamming the lists
> >>> with such patches. What happened ?
> >>
> >> It is changed a while back - not sure what's going on.
> > 
> > Could you please investigate ?
> 
> I will look into it

Thank you.

Jayesh, I assume this conversation took you by surprise, so I'll provide
a bit of context. LFD103 used to provide instructions on how to modify a
driver, create a patch and send it to a mailing list. Those instructions
were then followed by a line to tell the reader to not actually run the
commands. Unsurprisingly, that line was often overlooked, and patches
such as this one spammed the list.

I got a bit fed up and reported the issue to Shuah, who ensured LFD103
would be improved to avoid the problem. That was months ago, so I'm
surprised to see the issue occurring again. If you've followed the old
LFD103, I'm blaming the original wording of the course, not you, for the
problem.

-- 
Regards,

Laurent Pinchart