mbox series

[SRU,B,0/2] uvcvideo: add mapping for HEVC payloads

Message ID 20200916100005.1879670-1-jesse.sung@canonical.com
Headers show
Series uvcvideo: add mapping for HEVC payloads | expand

Message

Wen-chien Jesse Sung Sept. 16, 2020, 10 a.m. UTC
BugLink: https://launchpad.net/bugs/1895803

These two patches add mapping for HEVC so that uvcvideo can support the format.

The first patch is already in mainline, while the second one is in the
maintainer tree waiting for the next merging window.
* https://patchwork.kernel.org/patch/11577395/
* https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b

The risk of regression is low since these patches only add a new format.


Dmitry Buzdyk (1):
  UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads

Smitha T Murthy (1):
  media: videodev2.h: Add v4l2 definition for HEVC

 drivers/media/usb/uvc/uvc_driver.c | 5 +++++
 drivers/media/usb/uvc/uvcvideo.h   | 3 +++
 include/uapi/linux/videodev2.h     | 1 +
 3 files changed, 9 insertions(+)

Comments

Stefan Bader Sept. 17, 2020, 7:16 a.m. UTC | #1
On 16.09.20 12:00, Wen-chien Jesse Sung wrote:
> BugLink: https://launchpad.net/bugs/1895803
> 
> These two patches add mapping for HEVC so that uvcvideo can support the format.
> 
> The first patch is already in mainline, while the second one is in the
> maintainer tree waiting for the next merging window.
> * https://patchwork.kernel.org/patch/11577395/
> * https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b
> 
> The risk of regression is low since these patches only add a new format.
> 
> 
> Dmitry Buzdyk (1):
>   UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads
> 
> Smitha T Murthy (1):
>   media: videodev2.h: Add v4l2 definition for HEVC
> 
>  drivers/media/usb/uvc/uvc_driver.c | 5 +++++
>  drivers/media/usb/uvc/uvcvideo.h   | 3 +++
>  include/uapi/linux/videodev2.h     | 1 +
>  3 files changed, 9 insertions(+)
> 
Acked-by: Stefan Bader <stefan.bader@canonical.com>
Colin Ian King Sept. 17, 2020, 8:40 a.m. UTC | #2
On 16/09/2020 11:00, Wen-chien Jesse Sung wrote:
> BugLink: https://launchpad.net/bugs/1895803
> 
> These two patches add mapping for HEVC so that uvcvideo can support the format.
> 
> The first patch is already in mainline, while the second one is in the
> maintainer tree waiting for the next merging window.
> * https://patchwork.kernel.org/patch/11577395/
> * https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b
> 
> The risk of regression is low since these patches only add a new format.
> 
> 
> Dmitry Buzdyk (1):
>   UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads
> 
> Smitha T Murthy (1):
>   media: videodev2.h: Add v4l2 definition for HEVC
> 
>  drivers/media/usb/uvc/uvc_driver.c | 5 +++++
>  drivers/media/usb/uvc/uvcvideo.h   | 3 +++
>  include/uapi/linux/videodev2.h     | 1 +
>  3 files changed, 9 insertions(+)
> 

As per SRU process, I believe we need this fix in Focal + Groovy too. Is
that happening?

The regression potential is low - I didn't see any evidence that this
has been tested. Has it been tested?
Wen-chien Jesse Sung Sept. 17, 2020, 2:22 p.m. UTC | #3
On Thu, Sep 17, 2020 at 4:40 PM Colin Ian King <colin.king@canonical.com> wrote:
>
> On 16/09/2020 11:00, Wen-chien Jesse Sung wrote:
> > BugLink: https://launchpad.net/bugs/1895803
> >
> > These two patches add mapping for HEVC so that uvcvideo can support the format.
> >
> > The first patch is already in mainline, while the second one is in the
> > maintainer tree waiting for the next merging window.
> > * https://patchwork.kernel.org/patch/11577395/
> > * https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b
> >
> > The risk of regression is low since these patches only add a new format.
> >
> >
> > Dmitry Buzdyk (1):
> >   UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads
> >
> > Smitha T Murthy (1):
> >   media: videodev2.h: Add v4l2 definition for HEVC
> >
> >  drivers/media/usb/uvc/uvc_driver.c | 5 +++++
> >  drivers/media/usb/uvc/uvcvideo.h   | 3 +++
> >  include/uapi/linux/videodev2.h     | 1 +
> >  3 files changed, 9 insertions(+)
> >
>
> As per SRU process, I believe we need this fix in Focal + Groovy too. Is
> that happening?

Yes, but for F/G I'd like to wait until the patch lands in the mainline.

>
> The regression potential is low - I didn't see any evidence that this
> has been tested. Has it been tested?

We have a test kernel with these two patches applied on top of Bionic
kernel and verified by QA, ODM, and other parties in the project.

Thanks,
Jesse
Marcelo Henrique Cerri Sept. 17, 2020, 2:36 p.m. UTC | #4
Acked-by: Marcelo Henrique Cerri <marcelo.cerri@canonical.com>

On Wed, Sep 16, 2020 at 06:00:03PM +0800, Wen-chien Jesse Sung wrote:
> BugLink: https://launchpad.net/bugs/1895803
> 
> These two patches add mapping for HEVC so that uvcvideo can support the format.
> 
> The first patch is already in mainline, while the second one is in the
> maintainer tree waiting for the next merging window.
> * https://patchwork.kernel.org/patch/11577395/
> * https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b
> 
> The risk of regression is low since these patches only add a new format.
> 
> 
> Dmitry Buzdyk (1):
>   UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads
> 
> Smitha T Murthy (1):
>   media: videodev2.h: Add v4l2 definition for HEVC
> 
>  drivers/media/usb/uvc/uvc_driver.c | 5 +++++
>  drivers/media/usb/uvc/uvcvideo.h   | 3 +++
>  include/uapi/linux/videodev2.h     | 1 +
>  3 files changed, 9 insertions(+)
> 
> -- 
> 2.25.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Kelsey Skunberg Sept. 18, 2020, 1:41 a.m. UTC | #5
Applied to Bionic/master-next. thank you! 

-Kelsey

On 2020-09-16 18:00:03 , Wen-chien Jesse Sung wrote:
> BugLink: https://launchpad.net/bugs/1895803
> 
> These two patches add mapping for HEVC so that uvcvideo can support the format.
> 
> The first patch is already in mainline, while the second one is in the
> maintainer tree waiting for the next merging window.
> * https://patchwork.kernel.org/patch/11577395/
> * https://git.linuxtv.org/media_tree.git/commit/?id=40b222b56ea81aa636a4aa7a84939786369e726b
> 
> The risk of regression is low since these patches only add a new format.
> 
> 
> Dmitry Buzdyk (1):
>   UBUNTU: SAUCE: media: uvcvideo: Add mapping for HEVC payloads
> 
> Smitha T Murthy (1):
>   media: videodev2.h: Add v4l2 definition for HEVC
> 
>  drivers/media/usb/uvc/uvc_driver.c | 5 +++++
>  drivers/media/usb/uvc/uvcvideo.h   | 3 +++
>  include/uapi/linux/videodev2.h     | 1 +
>  3 files changed, 9 insertions(+)
> 
> -- 
> 2.25.1
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team