* [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe
@ 2015-08-04 16:03 Jacob Stiffler
2015-08-05 9:55 ` Devshatwar, Nikhil
0 siblings, 1 reply; 4+ messages in thread
From: Jacob Stiffler @ 2015-08-04 16:03 UTC (permalink / raw)
To: meta-arago
* The module ti-vip and ti-vpe will use the device node /dev/video0
and /dev/video1
* Which module gets which device node depends on the order they are
inserted.
* The vpe gstreamer plugin has the assumption that ti-vpe is connected
to /dev/video0
* This conf file will ensure that ti-vpe is inserted first, and gets
node /dev/video0
Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
.../gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf | 1 +
.../gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb | 11 +++++++++--
2 files changed, 10 insertions(+), 2 deletions(-)
create mode 100644 meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf
diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf
new file mode 100644
index 0000000..339a0c1
--- /dev/null
+++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf
@@ -0,0 +1 @@
+install ti-vip /sbin/modprobe ti-vpe; /sbin/modprobe --ignore-install ti-vip
diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb
index e549b3b..73a7723 100644
--- a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb
+++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb
@@ -5,7 +5,14 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
require gstreamer1.0-plugins-ti.inc
-PR = "${INC_PR}.2"
+PR = "${INC_PR}.3"
SRCREV = "876d28b2de5fa02d0f5fa836cba7be9a14b819eb"
-SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git"
+SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git \
+ file://ti-video.conf \
+"
+
+do_install_append() {
+ install -d ${D}/etc/modprobe.d
+ install -m 644 ${WORKDIR}/ti-video.conf ${D}/etc/modprobe.d
+}
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe
2015-08-04 16:03 [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe Jacob Stiffler
@ 2015-08-05 9:55 ` Devshatwar, Nikhil
2015-08-10 17:00 ` Denys Dmytriyenko
0 siblings, 1 reply; 4+ messages in thread
From: Devshatwar, Nikhil @ 2015-08-05 9:55 UTC (permalink / raw)
To: Stiffler, Jacob, meta-arago@arago-project.org
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> Sent: Tuesday, August 04, 2015 9:33 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config
> for ti-vip and ti-vpe
>
> * The module ti-vip and ti-vpe will use the device node /dev/video0
> and /dev/video1
> * Which module gets which device node depends on the order they are
> inserted.
> * The vpe gstreamer plugin has the assumption that ti-vpe is connected
> to /dev/video0
> * This conf file will ensure that ti-vpe is inserted first, and gets
> node /dev/video0
>
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
> .../gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf | 1 +
> .../gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb | 11 +++++++++--
> 2 files changed, 10 insertions(+), 2 deletions(-) create mode 100644 meta-
> arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-
> video.conf
>
> diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> plugins-vpe/ti-video.conf b/meta-arago-extras/recipes-multimedia/gst-plugins-
> ti/gstreamer1.0-plugins-vpe/ti-video.conf
> new file mode 100644
> index 0000000..339a0c1
> --- /dev/null
> +++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-p
> +++ lugins-vpe/ti-video.conf
> @@ -0,0 +1 @@
> +install ti-vip /sbin/modprobe ti-vpe; /sbin/modprobe --ignore-install
> +ti-vip
This would work only if the VIP and VPE are compiled as modules
Also, it won't work with if the USB camera registers a video device.
The better way would be for plugin to not hard code the /dev/video0 device
If you want to hard code the device name, it better be /dev/v4l/by-path/<the vpe platform device name>
This won't change irrespective of other modules are there or not, and works even in case VIP,VPE is compiled as builtin
> diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> plugins-vpe_git.bb b/meta-arago-extras/recipes-multimedia/gst-plugins-
> ti/gstreamer1.0-plugins-vpe_git.bb
> index e549b3b..73a7723 100644
> --- a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> plugins-vpe_git.bb
> +++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-p
> +++ lugins-vpe_git.bb
> @@ -5,7 +5,14 @@ LIC_FILES_CHKSUM =
> "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
>
> require gstreamer1.0-plugins-ti.inc
>
> -PR = "${INC_PR}.2"
> +PR = "${INC_PR}.3"
> SRCREV = "876d28b2de5fa02d0f5fa836cba7be9a14b819eb"
>
> -SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git"
> +SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git \
> + file://ti-video.conf \
> +"
> +
> +do_install_append() {
> + install -d ${D}/etc/modprobe.d
> + install -m 644 ${WORKDIR}/ti-video.conf ${D}/etc/modprobe.d }
> --
> 1.9.1
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe
2015-08-05 9:55 ` Devshatwar, Nikhil
@ 2015-08-10 17:00 ` Denys Dmytriyenko
2015-08-12 18:19 ` Jacob Stiffler
0 siblings, 1 reply; 4+ messages in thread
From: Denys Dmytriyenko @ 2015-08-10 17:00 UTC (permalink / raw)
To: Devshatwar, Nikhil; +Cc: meta-arago@arago-project.org
On Wed, Aug 05, 2015 at 09:55:57AM +0000, Devshatwar, Nikhil wrote:
>
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> > Sent: Tuesday, August 04, 2015 9:33 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config
> > for ti-vip and ti-vpe
> >
> > * The module ti-vip and ti-vpe will use the device node /dev/video0
> > and /dev/video1
> > * Which module gets which device node depends on the order they are
> > inserted.
> > * The vpe gstreamer plugin has the assumption that ti-vpe is connected
> > to /dev/video0
> > * This conf file will ensure that ti-vpe is inserted first, and gets
> > node /dev/video0
> >
> > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > ---
> > .../gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf | 1 +
> > .../gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb | 11 +++++++++--
> > 2 files changed, 10 insertions(+), 2 deletions(-) create mode 100644 meta-
> > arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-
> > video.conf
> >
> > diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> > plugins-vpe/ti-video.conf b/meta-arago-extras/recipes-multimedia/gst-plugins-
> > ti/gstreamer1.0-plugins-vpe/ti-video.conf
> > new file mode 100644
> > index 0000000..339a0c1
> > --- /dev/null
> > +++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-p
> > +++ lugins-vpe/ti-video.conf
> > @@ -0,0 +1 @@
> > +install ti-vip /sbin/modprobe ti-vpe; /sbin/modprobe --ignore-install
> > +ti-vip
>
> This would work only if the VIP and VPE are compiled as modules
> Also, it won't work with if the USB camera registers a video device.
> The better way would be for plugin to not hard code the /dev/video0 device
> If you want to hard code the device name, it better be /dev/v4l/by-path/<the vpe platform device name>
> This won't change irrespective of other modules are there or not, and works
> even in case VIP,VPE is compiled as builtin
Jake,
Do you plan to make changes suggested by Nikhil?
--
Denys
> > diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> > plugins-vpe_git.bb b/meta-arago-extras/recipes-multimedia/gst-plugins-
> > ti/gstreamer1.0-plugins-vpe_git.bb
> > index e549b3b..73a7723 100644
> > --- a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
> > plugins-vpe_git.bb
> > +++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-p
> > +++ lugins-vpe_git.bb
> > @@ -5,7 +5,14 @@ LIC_FILES_CHKSUM =
> > "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
> >
> > require gstreamer1.0-plugins-ti.inc
> >
> > -PR = "${INC_PR}.2"
> > +PR = "${INC_PR}.3"
> > SRCREV = "876d28b2de5fa02d0f5fa836cba7be9a14b819eb"
> >
> > -SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git"
> > +SRC_URI = "git://git.ti.com/glsdk/gst-plugin-vpe.git;protocol=git \
> > + file://ti-video.conf \
> > +"
> > +
> > +do_install_append() {
> > + install -d ${D}/etc/modprobe.d
> > + install -m 644 ${WORKDIR}/ti-video.conf ${D}/etc/modprobe.d }
> > --
> > 1.9.1
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe
2015-08-10 17:00 ` Denys Dmytriyenko
@ 2015-08-12 18:19 ` Jacob Stiffler
0 siblings, 0 replies; 4+ messages in thread
From: Jacob Stiffler @ 2015-08-12 18:19 UTC (permalink / raw)
To: Denys Dmytriyenko, Devshatwar, Nikhil; +Cc: meta-arago@arago-project.org
On 8/10/2015 1:00 PM, Denys Dmytriyenko wrote:
> On Wed, Aug 05, 2015 at 09:55:57AM +0000, Devshatwar, Nikhil wrote:
>>> -----Original Message-----
>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>>> bounces@arago-project.org] On Behalf Of Stiffler, Jacob
>>> Sent: Tuesday, August 04, 2015 9:33 PM
>>> To: meta-arago@arago-project.org
>>> Subject: [meta-arago] [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config
>>> for ti-vip and ti-vpe
>>>
>>> * The module ti-vip and ti-vpe will use the device node /dev/video0
>>> and /dev/video1
>>> * Which module gets which device node depends on the order they are
>>> inserted.
>>> * The vpe gstreamer plugin has the assumption that ti-vpe is connected
>>> to /dev/video0
>>> * This conf file will ensure that ti-vpe is inserted first, and gets
>>> node /dev/video0
>>>
>>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>>> ---
>>> .../gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-video.conf | 1 +
>>> .../gst-plugins-ti/gstreamer1.0-plugins-vpe_git.bb | 11 +++++++++--
>>> 2 files changed, 10 insertions(+), 2 deletions(-) create mode 100644 meta-
>>> arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-plugins-vpe/ti-
>>> video.conf
>>>
>>> diff --git a/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-
>>> plugins-vpe/ti-video.conf b/meta-arago-extras/recipes-multimedia/gst-plugins-
>>> ti/gstreamer1.0-plugins-vpe/ti-video.conf
>>> new file mode 100644
>>> index 0000000..339a0c1
>>> --- /dev/null
>>> +++ b/meta-arago-extras/recipes-multimedia/gst-plugins-ti/gstreamer1.0-p
>>> +++ lugins-vpe/ti-video.conf
>>> @@ -0,0 +1 @@
>>> +install ti-vip /sbin/modprobe ti-vpe; /sbin/modprobe --ignore-install
>>> +ti-vip
>> This would work only if the VIP and VPE are compiled as modules
>> Also, it won't work with if the USB camera registers a video device.
>> The better way would be for plugin to not hard code the /dev/video0 device
>> If you want to hard code the device name, it better be /dev/v4l/by-path/<the vpe platform device name>
>> This won't change irrespective of other modules are there or not, and works
>> even in case VIP,VPE is compiled as builtin
> Jake,
>
> Do you plan to make changes suggested by Nikhil?
>
I am going to try to work with the source owners to get this fixed upstream.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-08-12 18:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-04 16:03 [PATCH] gstreamer1.0-plugins-vpe: Add modprobe config for ti-vip and ti-vpe Jacob Stiffler
2015-08-05 9:55 ` Devshatwar, Nikhil
2015-08-10 17:00 ` Denys Dmytriyenko
2015-08-12 18:19 ` Jacob Stiffler
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.