All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Katu Txakur <katutxakurra@gmail.com>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>,
	"yocto@yoctoproject.org" <yocto@yoctoproject.org>
Subject: Re: using module_autoload
Date: Thu, 30 May 2013 13:34:05 +0200	[thread overview]
Message-ID: <20130530113405.GA24725@jama> (raw)
In-Reply-To: <CAM1ZcoU5-YGrxQnYZQLHS7Ej-7t0C0yTuOYj-BCPDYd=H4FU_w@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 5125 bytes --]

On Thu, May 30, 2013 at 09:49:13AM +0100, Katu Txakur wrote:
> Thanks again Martin,
> I tried with that kernel-module-split.bbclass but although I got no errors,
> it didn't work :(

Did you just copy that file from master/dylan to danny? It wont work,
because it depends also on other changes, e.g.

http://git.openembedded.org/openembedded-core/commit/meta/classes/package.bbclass?id=5aa52c6882d2929b2e530c4fa297c6d3e97d4f9f

This version of that class is compatible with danny:
https://github.com/openwebos/meta-webos/commit/c5a3676e69ab54d62f29d7bbeb6a4ac517eb1c33

and this is the usage:
https://github.com/openwebos/meta-webos/commit/c9f91ecf31ebb602a8a86c8e2017ca431e2623e8

> Can I create the file spike.conf and tell the recipe or the Makefile to
> copy it to /etc/modules-load.d ?
> how can I do that?

Yes you can do the same in recipe as what that bbclass does in more
generic way, depends on how many modules you need to fix..


> 
> Thanks again, I would be completely stuck without your help
> Katu
> 
> 
> 2013/5/29 Martin Jansa <martin.jansa@gmail.com>
> 
> > On Wed, May 29, 2013 at 04:48:58PM +0100, Katu Txakur wrote:
> > > Guys,
> > > I'm using the gumstix repo
> > > https://github.com/gumstix/Gumstix-YoctoProject-Repo and I don't have
> > any
> > > module-split bbclass.
> > > The module.bbclass inherits module-base.bbclass and module_strip.bbclass,
> > > but the latest is empty.
> > > Do you think my problem could be related with that? module_strip is also
> > > inherited by kernel.bbclass
> >
> > That looks like danny release, so you don't have this commit
> >
> > http://git.openembedded.org/openembedded-core/commit/meta/classes/kernel-module-split.bbclass?id=51928b6b5ca0a46a9dcd754483a19af58b95fa18
> >
> > You can backport it if you want, but without this module_autoload won't
> > work for external modules.
> >
> > > 2013/5/29 Bruce Ashfield <bruce.ashfield@windriver.com>
> > >
> > > > On 13-05-29 11:32 AM, Katu Txakur wrote:
> > > >
> > > >> Martin, Paul, thanks for your help.
> > > >> Yes, the recipe does inherit module.
> > > >> Any other problem that you can think off?
> > > >>
> > > >
> > > > Note: if you are on master, Martin said "module-split", not
> > > > "module".
> > > >
> > > > Cheers,
> > > >
> > > > Bruce
> > > >
> > > >
> > > >> Thanks a lot in advance,
> > > >> Katu
> > > >>
> > > >>
> > > >> 2013/5/29 Paul Eggleton <paul.eggleton@linux.intel.com
> > > >> <mailto:paul.eggleton@linux.**intel.com <
> > paul.eggleton@linux.intel.com>>>
> > > >>
> > > >>
> > > >>     On Wednesday 29 May 2013 16:39:48 Martin Jansa wrote:
> > > >>      > On Wed, May 29, 2013 at 02:01:11PM +0100, Katu Txakur wrote:
> > > >>      > > Hi,
> > > >>      > > I'm failing to load a kernel module named spike doing this:
> > > >>      > > 1) add the recipe (inherit module), based in a Makefile with
> > > >>      > > modules_install:
> > > >>      > > $(MAKE) INSTALL_MOD_DIR=kernel/**drivers/spike -C
> > $(KERNEL_SRC)
> > > >>     M=$(SRC)
> > > >>      > > modules_install
> > > >>      > > 2) Add the package to the image with IMAGE_INSTALL += "spike"
> > > >>     in the
> > > >>      > > local.conf file
> > > >>      > > 3) In the machine/mymachine.conf I've added
> > > >>      > >
> > > >>      > >         MACHINE_ESSENTIAL_EXTRA_**RRECOMMENDS += "spike"
> > > >>      > >         module_autoload_spike = "spike"
> > > >>      > >
> > > >>      > > The driver gets added to the image and I can load it with
> > > >>     insmod spike,
> > > >>      > > however, I would like to load it at boot time.
> > > >>      > > A workaround is to create a file with the name spike.conf and
> > > >>     write spike
> > > >>      > > inside. Copying that file to /etc/modules-load.d/spike.conf
> > > >>      > > loads the driver correctly, but I need to do it from a
> > recipe.
> > > >>      > >
> > > >>      > > Can anyone spot what am I doing wrong?
> > > >>      >
> > > >>      > module_autoload won't work with external module unless you
> > > >>     inherit also
> > > >>      > kernel-module-split
> > > >>
> > > >>     Note that module.bbclass already does this, so if the external
> > > >>     module recipe
> > > >>     already does "inherit module" then that is not the problem.
> > > >>
> > > >>     Cheers,
> > > >>     Paul
> > > >>
> > > >>     --
> > > >>
> > > >>     Paul Eggleton
> > > >>     Intel Open Source Technology Centre
> > > >>
> > > >>
> > > >>
> > > >>
> > > >> ______________________________**_________________
> > > >> yocto mailing list
> > > >> yocto@yoctoproject.org
> > > >> https://lists.yoctoproject.**org/listinfo/yocto<
> > https://lists.yoctoproject.org/listinfo/yocto>
> > > >>
> > > >>
> > > >
> >
> > > _______________________________________________
> > > yocto mailing list
> > > yocto@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/yocto
> >
> >
> > --
> > Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> >

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

  reply	other threads:[~2013-05-30 11:34 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-29 13:01 using module_autoload Katu Txakur
2013-05-29 14:39 ` Martin Jansa
2013-05-29 14:55   ` Paul Eggleton
2013-05-29 15:32     ` Katu Txakur
2013-05-29 15:34       ` Bruce Ashfield
2013-05-29 15:48         ` Katu Txakur
2013-05-29 16:14           ` Martin Jansa
2013-05-30  8:49             ` Katu Txakur
2013-05-30 11:34               ` Martin Jansa [this message]
2013-05-30 16:44                 ` Katu Txakur
2013-05-30 17:12               ` Khem Raj
2013-05-30 23:03                 ` Katu Txakur

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130530113405.GA24725@jama \
    --to=martin.jansa@gmail.com \
    --cc=katutxakurra@gmail.com \
    --cc=paul.eggleton@linux.intel.com \
    --cc=yocto@yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.