All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically
@ 2013-09-12  2:20 Denys Dmytriyenko
  2013-09-16 19:39 ` [meta-ti] " Cooper Jr., Franklin
  0 siblings, 1 reply; 4+ messages in thread
From: Denys Dmytriyenko @ 2013-09-12  2:20 UTC (permalink / raw)
  To: meta-ti

From: Denys Dmytriyenko <denys@ti.com>

The new method is to automatically package modules into kernel-module-<modname>
and make the custom name depend on the standard unified one. Messing with
FILES_${PN} usually causes problems, so avoid it and let module.bbclass do
it's magic.

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb b/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
index c7fdfd1..1a418d7 100644
--- a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
+++ b/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
@@ -7,7 +7,7 @@ COMPATIBLE_MACHINE = "omapl138"
 
 INHIBIT_PACKAGE_STRIP = "1"
 
-MACHINE_KERNEL_PR_append = "a+svnr${SRCPV}"
+MACHINE_KERNEL_PR_append = "b+svnr${SRCPV}"
 PR = "${MACHINE_KERNEL_PR}"
 
 SRC_URI = "svn://gforge.ti.com/svn/pru_sw/;module=trunk;protocol=https;user=anonymous;pswd=''"
@@ -28,5 +28,3 @@ do_install () {
         install -d ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru
         install -m 0755 ${S}/edmautils.ko ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/
 }
-
-FILES_${PN} = "/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/edmautils.ko"
-- 
1.8.3.2



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically
  2013-09-12  2:20 [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically Denys Dmytriyenko
@ 2013-09-16 19:39 ` Cooper Jr., Franklin
  2013-09-16 19:49   ` Denys Dmytriyenko
  0 siblings, 1 reply; 4+ messages in thread
From: Cooper Jr., Franklin @ 2013-09-16 19:39 UTC (permalink / raw)
  To: Denys Dmytriyenko, meta-arago@arago-project.org

Base on this patch and a similar patch for ocf-crypto-module are their patches that need to be added in meta-arago to make sure the kernel modules are included in the file system?  

> -----Original Message-----
> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> bounces@yoctoproject.org] On Behalf Of Denys Dmytriyenko
> Sent: Wednesday, September 11, 2013 9:20 PM
> To: meta-ti@yoctoproject.org
> Subject: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass package
> modules automatically
> 
> From: Denys Dmytriyenko <denys@ti.com>
> 
> The new method is to automatically package modules into kernel-module-
> <modname> and make the custom name depend on the standard unified one.
> Messing with FILES_${PN} usually causes problems, so avoid it and let
> module.bbclass do it's magic.
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb b/recipes-
> bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> index c7fdfd1..1a418d7 100644
> --- a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> +++ b/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> @@ -7,7 +7,7 @@ COMPATIBLE_MACHINE = "omapl138"
> 
>  INHIBIT_PACKAGE_STRIP = "1"
> 
> -MACHINE_KERNEL_PR_append = "a+svnr${SRCPV}"
> +MACHINE_KERNEL_PR_append = "b+svnr${SRCPV}"
>  PR = "${MACHINE_KERNEL_PR}"
> 
>  SRC_URI =
> "svn://gforge.ti.com/svn/pru_sw/;module=trunk;protocol=https;user=anonym
> ous;pswd=''"
> @@ -28,5 +28,3 @@ do_install () {
>          install -d ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru
>          install -m 0755 ${S}/edmautils.ko
> ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/
>  }
> -
> -FILES_${PN} =
> "/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/edmautils.ko"
> --
> 1.8.3.2
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically
  2013-09-16 19:39 ` [meta-ti] " Cooper Jr., Franklin
@ 2013-09-16 19:49   ` Denys Dmytriyenko
  2013-09-16 20:05     ` Cooper Jr., Franklin
  0 siblings, 1 reply; 4+ messages in thread
From: Denys Dmytriyenko @ 2013-09-16 19:49 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

I thought that was already done. There's RDEPENDS on ti-ocf-crypto-module in 
crypto packagegroup and ti-pru-sw-examples RDEPENDS on ti-pru-sw-edma-driver.

-- 
Denys


On Mon, Sep 16, 2013 at 07:39:32PM +0000, Cooper Jr., Franklin wrote:
> Base on this patch and a similar patch for ocf-crypto-module are their 
> patches that need to be added in meta-arago to make sure the kernel modules 
> are included in the file system?
> 
> > -----Original Message-----
> > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > bounces@yoctoproject.org] On Behalf Of Denys Dmytriyenko
> > Sent: Wednesday, September 11, 2013 9:20 PM
> > To: meta-ti@yoctoproject.org
> > Subject: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass package
> > modules automatically
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > The new method is to automatically package modules into kernel-module-
> > <modname> and make the custom name depend on the standard unified one.
> > Messing with FILES_${PN} usually causes problems, so avoid it and let
> > module.bbclass do it's magic.
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> > 
> > diff --git a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb b/recipes-
> > bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > index c7fdfd1..1a418d7 100644
> > --- a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > +++ b/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > @@ -7,7 +7,7 @@ COMPATIBLE_MACHINE = "omapl138"
> > 
> >  INHIBIT_PACKAGE_STRIP = "1"
> > 
> > -MACHINE_KERNEL_PR_append = "a+svnr${SRCPV}"
> > +MACHINE_KERNEL_PR_append = "b+svnr${SRCPV}"
> >  PR = "${MACHINE_KERNEL_PR}"
> > 
> >  SRC_URI =
> > "svn://gforge.ti.com/svn/pru_sw/;module=trunk;protocol=https;user=anonym
> > ous;pswd=''"
> > @@ -28,5 +28,3 @@ do_install () {
> >          install -d ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru
> >          install -m 0755 ${S}/edmautils.ko
> > ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/
> >  }
> > -
> > -FILES_${PN} =
> > "/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/edmautils.ko"
> > --
> > 1.8.3.2
> > 
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
> _______________________________________________
> 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: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically
  2013-09-16 19:49   ` Denys Dmytriyenko
@ 2013-09-16 20:05     ` Cooper Jr., Franklin
  0 siblings, 0 replies; 4+ messages in thread
From: Cooper Jr., Franklin @ 2013-09-16 20:05 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org



> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, September 16, 2013 2:49 PM
> To: Cooper Jr., Franklin
> Cc: Denys Dmytriyenko; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [meta-ti] [PATCH] ti-pru-sw-edma-driver: let
> module.bbclass package modules automatically
> 
> I thought that was already done. There's RDEPENDS on ti-ocf-crypto-module in
> crypto packagegroup and ti-pru-sw-examples RDEPENDS on ti-pru-sw-edma-
> driver.
[Franklin] Ok I was confused by the commit message. I thought it meant a new package was being generated which would require some additional changes.
> 
> --
> Denys
> 
> 
> On Mon, Sep 16, 2013 at 07:39:32PM +0000, Cooper Jr., Franklin wrote:
> > Base on this patch and a similar patch for ocf-crypto-module are their
> > patches that need to be added in meta-arago to make sure the kernel modules
> > are included in the file system?
> >
> > > -----Original Message-----
> > > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > > bounces@yoctoproject.org] On Behalf Of Denys Dmytriyenko
> > > Sent: Wednesday, September 11, 2013 9:20 PM
> > > To: meta-ti@yoctoproject.org
> > > Subject: [meta-ti] [PATCH] ti-pru-sw-edma-driver: let module.bbclass
> package
> > > modules automatically
> > >
> > > From: Denys Dmytriyenko <denys@ti.com>
> > >
> > > The new method is to automatically package modules into kernel-module-
> > > <modname> and make the custom name depend on the standard unified
> one.
> > > Messing with FILES_${PN} usually causes problems, so avoid it and let
> > > module.bbclass do it's magic.
> > >
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > ---
> > >  recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb | 4 +---
> > >  1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> > > diff --git a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb b/recipes-
> > > bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > > index c7fdfd1..1a418d7 100644
> > > --- a/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > > +++ b/recipes-bsp/pru/ti-pru-sw-edma-driver_1.00.00.bb
> > > @@ -7,7 +7,7 @@ COMPATIBLE_MACHINE = "omapl138"
> > >
> > >  INHIBIT_PACKAGE_STRIP = "1"
> > >
> > > -MACHINE_KERNEL_PR_append = "a+svnr${SRCPV}"
> > > +MACHINE_KERNEL_PR_append = "b+svnr${SRCPV}"
> > >  PR = "${MACHINE_KERNEL_PR}"
> > >
> > >  SRC_URI =
> > >
> "svn://gforge.ti.com/svn/pru_sw/;module=trunk;protocol=https;user=anonym
> > > ous;pswd=''"
> > > @@ -28,5 +28,3 @@ do_install () {
> > >          install -d ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru
> > >          install -m 0755 ${S}/edmautils.ko
> > > ${D}/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/
> > >  }
> > > -
> > > -FILES_${PN} =
> > > "/lib/modules/${KERNEL_VERSION}/kernel/drivers/pru/edmautils.ko"
> > > --
> > > 1.8.3.2
> > >
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti
> > _______________________________________________
> > 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

end of thread, other threads:[~2013-09-16 20:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-12  2:20 [PATCH] ti-pru-sw-edma-driver: let module.bbclass package modules automatically Denys Dmytriyenko
2013-09-16 19:39 ` [meta-ti] " Cooper Jr., Franklin
2013-09-16 19:49   ` Denys Dmytriyenko
2013-09-16 20:05     ` Cooper Jr., Franklin

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.