[PATCH] cryptodev-module: fix autoloading issue with dashed module name

[PATCH] cryptodev-module: fix autoloading issue with dashed module name

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@ti.com>

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 .../recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend            | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend b/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend
index 00d7919..7be7536 100644
--- a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend
+++ b/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend
@@ -1,6 +1,6 @@
-PR_append = "-arago1"
+PR_append = "-arago2"
 
-module_autoload_cryptodev-module = "cryptodev-module"
+module_autoload_cryptodev = "cryptodev-module"
 
 FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
 
-- 
2.0.4

Re: [PATCH] cryptodev-module: fix autoloading issue with dashed module name

[Cooper Jr., Franklin]

Acked-by: Franklin Cooper Jr. <fcooper@ti.com>

I never noticed that you could use "module_autoload". A lot better than the approach I was thinking of :)


> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> Sent: Wednesday, August 27, 2014 4:03 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] cryptodev-module: fix autoloading issue with
> dashed module name
> 
> From: Denys Dmytriyenko <denys@ti.com>
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  .../recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend            | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> module_1.6.bbappend b/meta-arago-distro/recipes-
> kernel/cryptodev/cryptodev-module_1.6.bbappend
> index 00d7919..7be7536 100644
> --- a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> module_1.6.bbappend
> +++ b/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> module_1.6.bbappend
> @@ -1,6 +1,6 @@
> -PR_append = "-arago1"
> +PR_append = "-arago2"
> 
> -module_autoload_cryptodev-module = "cryptodev-module"
> +module_autoload_cryptodev = "cryptodev-module"
> 
>  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> 
> --
> 2.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [PATCH] cryptodev-module: fix autoloading issue with dashed module name

[Denys Dmytriyenko]

On Wed, Aug 27, 2014 at 09:42:34PM +0000, Cooper Jr., Franklin wrote:
> Acked-by: Franklin Cooper Jr. <fcooper@ti.com>
> 
> I never noticed that you could use "module_autoload". A lot better than the 
> approach I was thinking of :)

Well, two things to keep in mind with this:

1. There seems to be a bug with modules having a dash in the name when it's 
used in the left side of the assignment, i.e. the variable name.
2. The syntax has completely changed in master, so Daisy is the last place 
where it would work as is.

-- 
Denys


> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Wednesday, August 27, 2014 4:03 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] cryptodev-module: fix autoloading issue with
> > dashed module name
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  .../recipes-kernel/cryptodev/cryptodev-module_1.6.bbappend            | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> > module_1.6.bbappend b/meta-arago-distro/recipes-
> > kernel/cryptodev/cryptodev-module_1.6.bbappend
> > index 00d7919..7be7536 100644
> > --- a/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> > module_1.6.bbappend
> > +++ b/meta-arago-distro/recipes-kernel/cryptodev/cryptodev-
> > module_1.6.bbappend
> > @@ -1,6 +1,6 @@
> > -PR_append = "-arago1"
> > +PR_append = "-arago2"
> > 
> > -module_autoload_cryptodev-module = "cryptodev-module"
> > +module_autoload_cryptodev = "cryptodev-module"
> > 
> >  FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > 
> > --
> > 2.0.4
> > 
> > _______________________________________________
> > 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