2022-03-17 05:43:56

by J. Neuschäfer

[permalink] [raw]
Subject: [PATCH v2] docs: media: uvcvideo: Use linux-media mailing list

As discussed with other developers, the linux-uvc-devel mailing list is
not very useful anymore, and it's better to send people to the general
linux-media mailing list.

Replace/remove the old mailing list address in uvcvideo.rst and
MAINTAINERS.

Signed-off-by: Jonathan Neuschäfer <[email protected]>
---

v2:
- Changed mailing list to [email protected] instead

v1:
- https://lore.kernel.org/lkml/[email protected]/
---
Documentation/userspace-api/media/drivers/uvcvideo.rst | 2 +-
MAINTAINERS | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/Documentation/userspace-api/media/drivers/uvcvideo.rst b/Documentation/userspace-api/media/drivers/uvcvideo.rst
index e5fd8fad333c9..a290f9fadae9d 100644
--- a/Documentation/userspace-api/media/drivers/uvcvideo.rst
+++ b/Documentation/userspace-api/media/drivers/uvcvideo.rst
@@ -7,7 +7,7 @@ This file documents some driver-specific aspects of the UVC driver, such as
driver-specific ioctls and implementation notes.

Questions and remarks can be sent to the Linux UVC development mailing list at
[email protected].
[email protected].


Extension Unit (XU) support
diff --git a/MAINTAINERS b/MAINTAINERS
index 05fd080b82f3a..8a0aeb1b98786 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -20154,7 +20154,6 @@ F: drivers/usb/host/uhci*

USB VIDEO CLASS
M: Laurent Pinchart <[email protected]>
-L: [email protected] (subscribers-only)
L: [email protected]
S: Maintained
W: http://www.ideasonboard.org/uvc/
--
2.35.1


2022-03-17 06:19:04

by Laurent Pinchart

[permalink] [raw]
Subject: Re: [PATCH v2] docs: media: uvcvideo: Use linux-media mailing list

Hi Jonathan,

Thank you for the patch.

On Tue, Mar 15, 2022 at 11:10:06PM +0100, Jonathan Neuschäfer wrote:
> As discussed with other developers, the linux-uvc-devel mailing list is
> not very useful anymore, and it's better to send people to the general
> linux-media mailing list.
>
> Replace/remove the old mailing list address in uvcvideo.rst and
> MAINTAINERS.
>
> Signed-off-by: Jonathan Neuschäfer <[email protected]>
> ---
>
> v2:
> - Changed mailing list to [email protected] instead
>
> v1:
> - https://lore.kernel.org/lkml/[email protected]/
> ---
> Documentation/userspace-api/media/drivers/uvcvideo.rst | 2 +-
> MAINTAINERS | 1 -

This looks good, but I think we should also add the following:

diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
index 5f394d4efc21..6c6a82af07ce 100644
--- a/drivers/media/usb/uvc/uvc_driver.c
+++ b/drivers/media/usb/uvc/uvc_driver.c
@@ -2443,7 +2443,7 @@ static int uvc_probe(struct usb_interface *intf,
"Forcing device quirks to 0x%x by module parameter for testing purpose.\n",
dev->quirks);
dev_info(&dev->udev->dev,
- "Please report required quirks to the linux-uvc-devel mailing list.\n");
+ "Please report required quirks to the linux-media mailing list.\n");
}

if (dev->info->uvc_version) {

If you're fine with that change, there's no need to resubmit, I can
handle this locally.

Reviewed-by: Laurent Pinchart <[email protected]>

> 2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/Documentation/userspace-api/media/drivers/uvcvideo.rst b/Documentation/userspace-api/media/drivers/uvcvideo.rst
> index e5fd8fad333c9..a290f9fadae9d 100644
> --- a/Documentation/userspace-api/media/drivers/uvcvideo.rst
> +++ b/Documentation/userspace-api/media/drivers/uvcvideo.rst
> @@ -7,7 +7,7 @@ This file documents some driver-specific aspects of the UVC driver, such as
> driver-specific ioctls and implementation notes.
>
> Questions and remarks can be sent to the Linux UVC development mailing list at
> [email protected].
> [email protected].
>
>
> Extension Unit (XU) support
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 05fd080b82f3a..8a0aeb1b98786 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -20154,7 +20154,6 @@ F: drivers/usb/host/uhci*
>
> USB VIDEO CLASS
> M: Laurent Pinchart <[email protected]>
> -L: [email protected] (subscribers-only)
> L: [email protected]
> S: Maintained
> W: http://www.ideasonboard.org/uvc/

--
Regards,

Laurent Pinchart

2022-03-17 06:22:50

by J. Neuschäfer

[permalink] [raw]
Subject: Re: [PATCH v2] docs: media: uvcvideo: Use linux-media mailing list

On Wed, Mar 16, 2022 at 12:18:55AM +0200, Laurent Pinchart wrote:
> Hi Jonathan,
>
> Thank you for the patch.
>
> On Tue, Mar 15, 2022 at 11:10:06PM +0100, Jonathan Neuschäfer wrote:
> > As discussed with other developers, the linux-uvc-devel mailing list is
> > not very useful anymore, and it's better to send people to the general
> > linux-media mailing list.
> >
> > Replace/remove the old mailing list address in uvcvideo.rst and
> > MAINTAINERS.
> >
> > Signed-off-by: Jonathan Neuschäfer <[email protected]>
> > ---
> >
> > v2:
> > - Changed mailing list to [email protected] instead
> >
> > v1:
> > - https://lore.kernel.org/lkml/[email protected]/
> > ---
> > Documentation/userspace-api/media/drivers/uvcvideo.rst | 2 +-
> > MAINTAINERS | 1 -
>
> This looks good, but I think we should also add the following:
>
> diff --git a/drivers/media/usb/uvc/uvc_driver.c b/drivers/media/usb/uvc/uvc_driver.c
> index 5f394d4efc21..6c6a82af07ce 100644
> --- a/drivers/media/usb/uvc/uvc_driver.c
> +++ b/drivers/media/usb/uvc/uvc_driver.c
> @@ -2443,7 +2443,7 @@ static int uvc_probe(struct usb_interface *intf,
> "Forcing device quirks to 0x%x by module parameter for testing purpose.\n",
> dev->quirks);
> dev_info(&dev->udev->dev,
> - "Please report required quirks to the linux-uvc-devel mailing list.\n");
> + "Please report required quirks to the linux-media mailing list.\n");
> }
>
> if (dev->info->uvc_version) {
>
> If you're fine with that change, there's no need to resubmit, I can
> handle this locally.
>
> Reviewed-by: Laurent Pinchart <[email protected]>

Ah, just before I saw your reply, I noticed this instance too, and sent
a separate patch for it.

I'll leave it up to you to handle it as you wish, I'm fine with it
either way.


Thanks,
Jonathan


Attachments:
(No filename) (1.92 kB)
signature.asc (849.00 B)
Download all attachments