mbox series

[GIT,PULL,FOR,v6.3] uvcgadget changes

Message ID 20230124144929.GB19219@pengutronix.de (mailing list archive)
State Not Applicable
Headers
Series [GIT,PULL,FOR,v6.3] uvcgadget changes |

Pull-request

git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124

Message

Michael Grzeschik Jan. 24, 2023, 2:49 p.m. UTC 
  The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:

  Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)

are available in the Git repository at:

  git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124

for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:

  usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)

----------------------------------------------------------------
Marek Vasut (1):
      media: uvcvideo: Add GUID for BGRA/X 8:8:8:8

Michael Grzeschik (4):
      usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
      usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
      usb: uvc: make uvc_format_desc table const
      usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name

 drivers/media/common/Kconfig                  |   3 +
 drivers/media/common/Makefile                 |   1 +
 drivers/media/common/uvc.c                    | 183 ++++++++++++++++++++++
 drivers/media/usb/uvc/Kconfig                 |   1 +
 drivers/media/usb/uvc/uvc_ctrl.c              |   2 +-
 drivers/media/usb/uvc/uvc_driver.c            |   4 +-
 drivers/usb/gadget/Kconfig                    |   1 +
 drivers/usb/gadget/function/uvc_v4l2.c        |  16 +-
 include/{media/v4l2-uvc.h => linux/usb/uvc.h} | 209 +-------------------------
 9 files changed, 201 insertions(+), 219 deletions(-)
 create mode 100644 drivers/media/common/uvc.c
 rename include/{media/v4l2-uvc.h => linux/usb/uvc.h} (59%)

Comments

Greg KH Jan. 24, 2023, 3:35 p.m. UTC | #1 
On Tue, Jan 24, 2023 at 03:49:29PM +0100, Michael Grzeschik wrote:
> The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:
> 
>  Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)
> 
> are available in the Git repository at:
> 
>  git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124
> 
> for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:
> 
>  usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)
> 
> ----------------------------------------------------------------
> Marek Vasut (1):
>      media: uvcvideo: Add GUID for BGRA/X 8:8:8:8
> 
> Michael Grzeschik (4):
>      usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
>      usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
>      usb: uvc: make uvc_format_desc table const
>      usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name

Sorry, but I can't take patches like this as a pull request.  I want to
see the real patches and get people to review them properly that way.

thanks,

greg k-h
Michael Grzeschik Jan. 24, 2023, 4:17 p.m. UTC | #2 
On Tue, Jan 24, 2023 at 04:35:24PM +0100, Greg KH wrote:
>On Tue, Jan 24, 2023 at 03:49:29PM +0100, Michael Grzeschik wrote:
>> The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:
>>
>>  Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)
>>
>> are available in the Git repository at:
>>
>>  git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124
>>
>> for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:
>>
>>  usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)
>>
>> ----------------------------------------------------------------
>> Marek Vasut (1):
>>      media: uvcvideo: Add GUID for BGRA/X 8:8:8:8
>>
>> Michael Grzeschik (4):
>>      usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
>>      usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
>>      usb: uvc: make uvc_format_desc table const
>>      usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name
>
>Sorry, but I can't take patches like this as a pull request.  I want to
>see the real patches and get people to review them properly that way.

I just did like Laurent suggested. Since the series was already properly
reviewed I also thought this would be fine.

https://lore.kernel.org/linux-media/Y8VdpQGcqmoKESk8@pendragon.ideasonboard.com/

Anyway. I can send another round with the patch from Marek in the
series.

Thanks,
Michael
Michael Grzeschik Jan. 26, 2023, 10:33 p.m. UTC | #3 
Hi Laurent,

On Tue, Jan 24, 2023 at 05:17:09PM +0100, Michael Grzeschik wrote:
>On Tue, Jan 24, 2023 at 04:35:24PM +0100, Greg KH wrote:
>>On Tue, Jan 24, 2023 at 03:49:29PM +0100, Michael Grzeschik wrote:
>>>The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:
>>>
>>> Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)
>>>
>>>are available in the Git repository at:
>>>
>>> git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124
>>>
>>>for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:
>>>
>>> usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)
>>>
>>>----------------------------------------------------------------
>>>Marek Vasut (1):
>>>     media: uvcvideo: Add GUID for BGRA/X 8:8:8:8
>>>
>>>Michael Grzeschik (4):
>>>     usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
>>>     usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
>>>     usb: uvc: make uvc_format_desc table const
>>>     usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name
>>
>>Sorry, but I can't take patches like this as a pull request.  I want to
>>see the real patches and get people to review them properly that way.
>
>I just did like Laurent suggested. Since the series was already properly
>reviewed I also thought this would be fine.
>
>https://lore.kernel.org/linux-media/Y8VdpQGcqmoKESk8@pendragon.ideasonboard.com/
>
>Anyway. I can send another round with the patch from Marek in the
>series.

I am unsure how to proceed here.

I could send the series again and hope for Greg to take them. But
without Mauros merge of your changes underneath, Greg will not be
able to apply my series.


Thanks,
Michael
Laurent Pinchart Jan. 26, 2023, 10:48 p.m. UTC | #4 
Hello,

On Thu, Jan 26, 2023 at 11:33:30PM +0100, Michael Grzeschik wrote:
> On Tue, Jan 24, 2023 at 05:17:09PM +0100, Michael Grzeschik wrote:
> > On Tue, Jan 24, 2023 at 04:35:24PM +0100, Greg KH wrote:
> >> On Tue, Jan 24, 2023 at 03:49:29PM +0100, Michael Grzeschik wrote:
> >>> The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:
> >>>
> >>> Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)
> >>>
> >>> are available in the Git repository at:
> >>>
> >>> git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124
> >>>
> >>> for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:
> >>>
> >>> usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)
> >>>
> >>> ----------------------------------------------------------------
> >>> Marek Vasut (1):
> >>>     media: uvcvideo: Add GUID for BGRA/X 8:8:8:8
> >>>
> >>> Michael Grzeschik (4):
> >>>     usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
> >>>     usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
> >>>     usb: uvc: make uvc_format_desc table const
> >>>     usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name
> >>
> >> Sorry, but I can't take patches like this as a pull request.  I want to
> >> see the real patches and get people to review them properly that way.
> >
> > I just did like Laurent suggested. Since the series was already properly
> > reviewed I also thought this would be fine.
> >
> > https://lore.kernel.org/linux-media/Y8VdpQGcqmoKESk8@pendragon.ideasonboard.com/
> >
> > Anyway. I can send another round with the patch from Marek in the
> > series.
> 
> I am unsure how to proceed here.
> 
> I could send the series again and hope for Greg to take them. But
> without Mauros merge of your changes underneath, Greg will not be
> able to apply my series.

I think Greg would like to see a v3 of the series exactly as it exists
in your branch, based on top of media-uvc-next-20230115, with Marek's
patch included. Could you post that, and indicate in the cover letter
than it applies on top of

git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git media-uvc-next-20230115

which has been merged in the Linux media tree for v6.3 (as shown by
https://git.linuxtv.org/media_stage.git/commit/?id=82adfe34694c98f1115a7b75cff6af9c4a35dba9).

Greg, can you then merge that tag in your tree first, and then pick
Michael's patches up on top ?
Greg KH Jan. 27, 2023, 7:15 a.m. UTC | #5 
On Fri, Jan 27, 2023 at 12:48:28AM +0200, Laurent Pinchart wrote:
> Hello,
> 
> On Thu, Jan 26, 2023 at 11:33:30PM +0100, Michael Grzeschik wrote:
> > On Tue, Jan 24, 2023 at 05:17:09PM +0100, Michael Grzeschik wrote:
> > > On Tue, Jan 24, 2023 at 04:35:24PM +0100, Greg KH wrote:
> > >> On Tue, Jan 24, 2023 at 03:49:29PM +0100, Michael Grzeschik wrote:
> > >>> The following changes since commit 82adfe34694c98f1115a7b75cff6af9c4a35dba9:
> > >>>
> > >>> Merge tag 'media-uvc-next-20230115' of git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux into media_stage (2023-01-22 08:43:14 +0100)
> > >>>
> > >>> are available in the Git repository at:
> > >>>
> > >>> git://git.pengutronix.de/mgr/linux tags/uvc-gadget-next-20230124
> > >>>
> > >>> for you to fetch changes up to e88f760d0f7054f49aa68b75bf0530840f031d34:
> > >>>
> > >>> usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name (2023-01-24 15:26:07 +0100)
> > >>>
> > >>> ----------------------------------------------------------------
> > >>> Marek Vasut (1):
> > >>>     media: uvcvideo: Add GUID for BGRA/X 8:8:8:8
> > >>>
> > >>> Michael Grzeschik (4):
> > >>>     usb: uvc: move media/v4l2-uvc.h to usb/uvc.h
> > >>>     usb: uvc: move uvc_fmts and uvc_format_by_guid to own compile unit
> > >>>     usb: uvc: make uvc_format_desc table const
> > >>>     usb: uvc: use v4l2_fill_fmtdesc instead of open coded format name
> > >>
> > >> Sorry, but I can't take patches like this as a pull request.  I want to
> > >> see the real patches and get people to review them properly that way.
> > >
> > > I just did like Laurent suggested. Since the series was already properly
> > > reviewed I also thought this would be fine.
> > >
> > > https://lore.kernel.org/linux-media/Y8VdpQGcqmoKESk8@pendragon.ideasonboard.com/
> > >
> > > Anyway. I can send another round with the patch from Marek in the
> > > series.
> > 
> > I am unsure how to proceed here.
> > 
> > I could send the series again and hope for Greg to take them. But
> > without Mauros merge of your changes underneath, Greg will not be
> > able to apply my series.
> 
> I think Greg would like to see a v3 of the series exactly as it exists
> in your branch, based on top of media-uvc-next-20230115, with Marek's
> patch included. Could you post that, and indicate in the cover letter
> than it applies on top of
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/pinchartl/linux.git media-uvc-next-20230115
> 
> which has been merged in the Linux media tree for v6.3 (as shown by
> https://git.linuxtv.org/media_stage.git/commit/?id=82adfe34694c98f1115a7b75cff6af9c4a35dba9).
> 
> Greg, can you then merge that tag in your tree first, and then pick
> Michael's patches up on top ?

Yes, I will be glad to do that, thanks.

greg k-h