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

[Denys Dmytriyenko <denys@ti.com>]

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"
> > 
>