[morty/master][PATCH] ti-ipc: add module dependency between omap_remoteproc and virtio_rpmsg_bus

[morty/master][PATCH] ti-ipc: add module dependency between omap_remoteproc and virtio_rpmsg_bus

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@ti.com>

This is required to workaround the remoteproc crash due to watchdog interrupt
conflicts - LCPD-9490

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
Cc: Sam Nelson <sam.nelson@ti.com>
Cc: Suman Anna <s-anna@ti.com>
---
 recipes-ti/ipc/ti-ipc/omap_remoteproc.conf | 1 +
 recipes-ti/ipc/ti-ipc_git.bb               | 4 ++++
 2 files changed, 5 insertions(+)
 create mode 100644 recipes-ti/ipc/ti-ipc/omap_remoteproc.conf

diff --git a/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
new file mode 100644
index 0000000..6b5034c
--- /dev/null
+++ b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
@@ -0,0 +1 @@
+softdep omap_remoteproc pre: virtio_rpmsg_bus
diff --git a/recipes-ti/ipc/ti-ipc_git.bb b/recipes-ti/ipc/ti-ipc_git.bb
index 5878b63..b536f61 100644
--- a/recipes-ti/ipc/ti-ipc_git.bb
+++ b/recipes-ti/ipc/ti-ipc_git.bb
@@ -9,6 +9,7 @@ PR = "${INC_PR}.0"
 DEPENDS += "virtual/kernel"
 
 SRC_URI += "file://tiipclad-daemon.sh \
+            file://omap_remoteproc.conf \
             file://0001-Add-kernel-build-dir.patch \
            "
 
@@ -41,6 +42,9 @@ do_install_append() {
     # lad daemon executable.
     sed -i -e "s/__LAD_DAEMON__/${DAEMON}/" ${WORKDIR}/tiipclad-daemon.sh
     install -c -m 755 ${WORKDIR}/tiipclad-daemon.sh ${D}${sysconfdir}/init.d/${INITSCRIPT_NAME}
+
+    install -d ${D}${sysconfdir}/modprobe.d/
+    install -c -m 644 ${WORKDIR}/omap_remoteproc.conf ${D}${sysconfdir}/modprobe.d/
 }
 
 PACKAGES =+ "${PN}-test"
-- 
2.7.4

Re: [morty/master][PATCH] ti-ipc: add module dependency between omap_remoteproc and virtio_rpmsg_bus

[Suman Anna]

Hi Denys,

On 03/27/2017 11:41 AM, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ti.com>
> 
> This is required to workaround the remoteproc crash due to watchdog interrupt
> conflicts - LCPD-9490
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> Cc: Sam Nelson <sam.nelson@ti.com>
> Cc: Suman Anna <s-anna@ti.com>
> ---
>  recipes-ti/ipc/ti-ipc/omap_remoteproc.conf | 1 +
>  recipes-ti/ipc/ti-ipc_git.bb               | 4 ++++
>  2 files changed, 5 insertions(+)
>  create mode 100644 recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> 
> diff --git a/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> new file mode 100644
> index 0000000..6b5034c
> --- /dev/null
> +++ b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> @@ -0,0 +1 @@
> +softdep omap_remoteproc pre: virtio_rpmsg_bus
> diff --git a/recipes-ti/ipc/ti-ipc_git.bb b/recipes-ti/ipc/ti-ipc_git.bb
> index 5878b63..b536f61 100644
> --- a/recipes-ti/ipc/ti-ipc_git.bb
> +++ b/recipes-ti/ipc/ti-ipc_git.bb
> @@ -9,6 +9,7 @@ PR = "${INC_PR}.0"
>  DEPENDS += "virtual/kernel"
>  
>  SRC_URI += "file://tiipclad-daemon.sh \
> +            file://omap_remoteproc.conf \
>              file://0001-Add-kernel-build-dir.patch \
>             "
>  
> @@ -41,6 +42,9 @@ do_install_append() {
>      # lad daemon executable.
>      sed -i -e "s/__LAD_DAEMON__/${DAEMON}/" ${WORKDIR}/tiipclad-daemon.sh
>      install -c -m 755 ${WORKDIR}/tiipclad-daemon.sh ${D}${sysconfdir}/init.d/${INITSCRIPT_NAME}
> +
> +    install -d ${D}${sysconfdir}/modprobe.d/
> +    install -c -m 644 ${WORKDIR}/omap_remoteproc.conf ${D}${sysconfdir}/modprobe.d/

Does this install even on K2 builds/FS? It is ideally not required on K2
builds, though I believe this will be a no-op on K2.

Otherwise, LGTM.

regards
Suman

>  }
>  
>  PACKAGES =+ "${PN}-test"
> 

Re: [morty/master][PATCH] ti-ipc: add module dependency between omap_remoteproc and virtio_rpmsg_bus

[Denys Dmytriyenko]

On Mon, Mar 27, 2017 at 11:46:30AM -0500, Suman Anna wrote:
> Hi Denys,
> 
> On 03/27/2017 11:41 AM, Denys Dmytriyenko wrote:
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > This is required to workaround the remoteproc crash due to watchdog interrupt
> > conflicts - LCPD-9490
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > Cc: Sam Nelson <sam.nelson@ti.com>
> > Cc: Suman Anna <s-anna@ti.com>
> > ---
> >  recipes-ti/ipc/ti-ipc/omap_remoteproc.conf | 1 +
> >  recipes-ti/ipc/ti-ipc_git.bb               | 4 ++++
> >  2 files changed, 5 insertions(+)
> >  create mode 100644 recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> > 
> > diff --git a/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> > new file mode 100644
> > index 0000000..6b5034c
> > --- /dev/null
> > +++ b/recipes-ti/ipc/ti-ipc/omap_remoteproc.conf
> > @@ -0,0 +1 @@
> > +softdep omap_remoteproc pre: virtio_rpmsg_bus
> > diff --git a/recipes-ti/ipc/ti-ipc_git.bb b/recipes-ti/ipc/ti-ipc_git.bb
> > index 5878b63..b536f61 100644
> > --- a/recipes-ti/ipc/ti-ipc_git.bb
> > +++ b/recipes-ti/ipc/ti-ipc_git.bb
> > @@ -9,6 +9,7 @@ PR = "${INC_PR}.0"
> >  DEPENDS += "virtual/kernel"
> >  
> >  SRC_URI += "file://tiipclad-daemon.sh \
> > +            file://omap_remoteproc.conf \
> >              file://0001-Add-kernel-build-dir.patch \
> >             "
> >  
> > @@ -41,6 +42,9 @@ do_install_append() {
> >      # lad daemon executable.
> >      sed -i -e "s/__LAD_DAEMON__/${DAEMON}/" ${WORKDIR}/tiipclad-daemon.sh
> >      install -c -m 755 ${WORKDIR}/tiipclad-daemon.sh ${D}${sysconfdir}/init.d/${INITSCRIPT_NAME}
> > +
> > +    install -d ${D}${sysconfdir}/modprobe.d/
> > +    install -c -m 644 ${WORKDIR}/omap_remoteproc.conf ${D}${sysconfdir}/modprobe.d/
> 
> Does this install even on K2 builds/FS? It is ideally not required on K2
> builds, though I believe this will be a no-op on K2.

I can limit it to "dra7xx" SOC family only - shouldn't be a problem since this 
package is already machine-specific.


> Otherwise, LGTM.
> 
> regards
> Suman
> 
> >  }
> >  
> >  PACKAGES =+ "${PN}-test"
> > 
>