[B][D][E][SRU][PATCHv2 0/1] Enable VIMC module

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[B][D][E][SRU][PATCHv2 0/1] Enable VIMC module

Po-Hsu Lin (Sam)
BugLink: https://bugs.launchpad.net/bugs/1831482

v2: Add bug ID to annotation file, update regression potential

== SRU Justification ==
VIMC is a virtual media controller that can emulate a real media device.
This is useful for automated tests, and development of multimedia frameworks.

== Fix ==
Enable VIMC as a module in B/D/E kernel configs.

== Test ==
Test kernels could be found here:
https://people.canonical.com/~phlin/kernel/lp-1831482-cfg-vimc/

User has reported that the Eoan kernel works as expected.
I have also smoke tested the B/D kernel to make sure the VIMC modules
can be loaded properly.

== Regression Potential ==
Low, this patchset enables the VIMC module support, we might see some
bug reports against this module in the future. However the acutal impact
for a non-vimc user should be minimum.

There is a null pointer deference issue was found while inserting the
vimc module on some system (lp:1840028), it will be handled in that bug.


Po-Hsu Lin (1):
  UBUNTU: [Config] Enable VIMC module

 debian.master/config/annotations          | 2 ++
 debian.master/config/config.common.ubuntu | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

--
2.7.4

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[B][SRU][PATCHv2 1/1] UBUNTU: [Config] Enable VIMC module

Po-Hsu Lin (Sam)
BugLink: https://bugs.launchpad.net/bugs/1831482

VIMC is a virtual media controller that can emulate a real media device.
This is useful for automated tests, and development of multimedia frameworks.

Signed-off-by: Po-Hsu Lin <[hidden email]>
---
 debian.master/config/annotations          | 2 ++
 debian.master/config/config.common.ubuntu | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/debian.master/config/annotations b/debian.master/config/annotations
index e8f17ff..e9a9e15 100644
--- a/debian.master/config/annotations
+++ b/debian.master/config/annotations
@@ -4148,6 +4148,8 @@ CONFIG_VIDEO_VIVID                              policy<{'amd64': 'm', 'arm64': '
 CONFIG_VIDEO_VIVID_CEC                          policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'i386': 'y', 'ppc64el': 'y'}>
 CONFIG_VIDEO_VIVID_MAX_DEVS                     policy<{'amd64': '64', 'arm64': '64', 'armhf': '64', 'i386': '64', 'ppc64el': '64'}>
 CONFIG_VIDEO_VIM2M                              policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
+#
+CONFIG_VIDEO_VIMC                               note<LP#1831482>
 
 # Menu: Device Drivers >> Multimedia support >> Cameras/video grabbers support >> Memory-to-memory multimedia devices
 CONFIG_V4L_MEM2MEM_DRIVERS                      policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'i386': 'y', 'ppc64el': 'y'}>
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 67edc5e..cea83cd 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -10144,7 +10144,7 @@ CONFIG_VIDEO_V4L2_SUBDEV_API=y
 CONFIG_VIDEO_V4L2_TPG=m
 CONFIG_VIDEO_VIA_CAMERA=m
 CONFIG_VIDEO_VIM2M=m
-# CONFIG_VIDEO_VIMC is not set
+CONFIG_VIDEO_VIMC=m
 CONFIG_VIDEO_VIVID=m
 CONFIG_VIDEO_VIVID_CEC=y
 CONFIG_VIDEO_VIVID_MAX_DEVS=64
--
2.7.4

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[D][E][PATCHv2 1/1] UBUNTU: [Config] Enable VIMC module

Po-Hsu Lin (Sam)
In reply to this post by Po-Hsu Lin (Sam)
BugLink: https://bugs.launchpad.net/bugs/1831482

VIMC is a virtual media controller that can emulate a real media device.
This is useful for automated tests, and development of multimedia frameworks.

Signed-off-by: Po-Hsu Lin <[hidden email]>
---
 debian.master/config/annotations          | 2 ++
 debian.master/config/config.common.ubuntu | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/debian.master/config/annotations b/debian.master/config/annotations
index 8fcbca0..741bc2b 100644
--- a/debian.master/config/annotations
+++ b/debian.master/config/annotations
@@ -4290,6 +4290,8 @@ CONFIG_VIDEO_VIVID_CEC                          policy<{'amd64': 'y', 'arm64': '
 CONFIG_VIDEO_VIVID_MAX_DEVS                     policy<{'amd64': '64', 'arm64': '64', 'armhf': '64', 'i386': '64', 'ppc64el': '64'}>
 CONFIG_VIDEO_VIM2M                              policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
 CONFIG_VIDEO_VICODEC                            policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'i386': 'm', 'ppc64el': 'm'}>
+#
+CONFIG_VIDEO_VIMC                               note<LP#1831482>
 
 # Menu: Device Drivers >> Multimedia support >> Cameras/video grabbers support >> Memory-to-memory multimedia devices
 CONFIG_V4L_MEM2MEM_DRIVERS                      policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'i386': 'y', 'ppc64el': 'y'}>
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index bb855f1..a0ffc55 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -10635,7 +10635,7 @@ CONFIG_VIDEO_V4L2_TPG=m
 CONFIG_VIDEO_VIA_CAMERA=m
 CONFIG_VIDEO_VICODEC=m
 CONFIG_VIDEO_VIM2M=m
-# CONFIG_VIDEO_VIMC is not set
+CONFIG_VIDEO_VIMC=m
 CONFIG_VIDEO_VIVID=m
 CONFIG_VIDEO_VIVID_CEC=y
 CONFIG_VIDEO_VIVID_MAX_DEVS=64
--
2.7.4

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK / APPLIED[E/Unstable]: [B][D][E][SRU][PATCHv2 0/1] Enable VIMC module

Seth Forshee
In reply to this post by Po-Hsu Lin (Sam)
On Wed, Aug 14, 2019 at 02:58:17PM +0800, Po-Hsu Lin wrote:

> BugLink: https://bugs.launchpad.net/bugs/1831482
>
> v2: Add bug ID to annotation file, update regression potential
>
> == SRU Justification ==
> VIMC is a virtual media controller that can emulate a real media device.
> This is useful for automated tests, and development of multimedia frameworks.
>
> == Fix ==
> Enable VIMC as a module in B/D/E kernel configs.
>
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1831482-cfg-vimc/
>
> User has reported that the Eoan kernel works as expected.
> I have also smoke tested the B/D kernel to make sure the VIMC modules
> can be loaded properly.
>
> == Regression Potential ==
> Low, this patchset enables the VIMC module support, we might see some
> bug reports against this module in the future. However the acutal impact
> for a non-vimc user should be minimum.
>
> There is a null pointer deference issue was found while inserting the
> vimc module on some system (lp:1840028), it will be handled in that bug.

Acked-by: Seth Forshee <[hidden email]>

Applied to eoan/master-next and unstable/master, thanks!

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [B][D][SRU][PATCHv2 0/1] Enable VIMC module

Stefan Bader-2
In reply to this post by Po-Hsu Lin (Sam)
On 14.08.19 08:58, Po-Hsu Lin wrote:

> BugLink: https://bugs.launchpad.net/bugs/1831482
>
> v2: Add bug ID to annotation file, update regression potential
>
> == SRU Justification ==
> VIMC is a virtual media controller that can emulate a real media device.
> This is useful for automated tests, and development of multimedia frameworks.
>
> == Fix ==
> Enable VIMC as a module in B/D/E kernel configs.
>
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1831482-cfg-vimc/
>
> User has reported that the Eoan kernel works as expected.
> I have also smoke tested the B/D kernel to make sure the VIMC modules
> can be loaded properly.
>
> == Regression Potential ==
> Low, this patchset enables the VIMC module support, we might see some
> bug reports against this module in the future. However the acutal impact
> for a non-vimc user should be minimum.
>
> There is a null pointer deference issue was found while inserting the
> vimc module on some system (lp:1840028), it will be handled in that bug.
>
>
> Po-Hsu Lin (1):
>   UBUNTU: [Config] Enable VIMC module
>
>  debian.master/config/annotations          | 2 ++
>  debian.master/config/config.common.ubuntu | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
>
Acked-by: Stefan Bader <[hidden email]>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (849 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

ACK: [B][D][E][SRU][PATCHv2 0/1] Enable VIMC module

Connor Kuehl
In reply to this post by Po-Hsu Lin (Sam)
On 8/13/19 11:58 PM, Po-Hsu Lin wrote:

> BugLink: https://bugs.launchpad.net/bugs/1831482
>
> v2: Add bug ID to annotation file, update regression potential
>
> == SRU Justification ==
> VIMC is a virtual media controller that can emulate a real media device.
> This is useful for automated tests, and development of multimedia frameworks.
>
> == Fix ==
> Enable VIMC as a module in B/D/E kernel configs.
>
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1831482-cfg-vimc/
>
> User has reported that the Eoan kernel works as expected.
> I have also smoke tested the B/D kernel to make sure the VIMC modules
> can be loaded properly.
>
> == Regression Potential ==
> Low, this patchset enables the VIMC module support, we might see some
> bug reports against this module in the future. However the acutal impact
> for a non-vimc user should be minimum.
>
> There is a null pointer deference issue was found while inserting the
> vimc module on some system (lp:1840028), it will be handled in that bug.
>
>
> Po-Hsu Lin (1):
>    UBUNTU: [Config] Enable VIMC module
>
>   debian.master/config/annotations          | 2 ++
>   debian.master/config/config.common.ubuntu | 2 +-
>   2 files changed, 3 insertions(+), 1 deletion(-)
>

Acked-by: Connor Kuehl <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED[B/D]: [B][D][E][SRU][PATCHv2 0/1] Enable VIMC module

Kleber Souza
In reply to this post by Po-Hsu Lin (Sam)
On 8/14/19 8:58 AM, Po-Hsu Lin wrote:

> BugLink: https://bugs.launchpad.net/bugs/1831482
>
> v2: Add bug ID to annotation file, update regression potential
>
> == SRU Justification ==
> VIMC is a virtual media controller that can emulate a real media device.
> This is useful for automated tests, and development of multimedia frameworks.
>
> == Fix ==
> Enable VIMC as a module in B/D/E kernel configs.
>
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1831482-cfg-vimc/
>
> User has reported that the Eoan kernel works as expected.
> I have also smoke tested the B/D kernel to make sure the VIMC modules
> can be loaded properly.
>
> == Regression Potential ==
> Low, this patchset enables the VIMC module support, we might see some
> bug reports against this module in the future. However the acutal impact
> for a non-vimc user should be minimum.
>
> There is a null pointer deference issue was found while inserting the
> vimc module on some system (lp:1840028), it will be handled in that bug.
>
>
> Po-Hsu Lin (1):
>   UBUNTU: [Config] Enable VIMC module
>
>  debian.master/config/annotations          | 2 ++
>  debian.master/config/config.common.ubuntu | 2 +-
>  2 files changed, 3 insertions(+), 1 deletion(-)
>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team