From: Michael Ellerman <michael@ellerman.id.au>
To: Guillaume Dargaud <dargaud@lpsc.in2p3.fr>
Cc: linuxppc-dev@lists.ozlabs.org
Subject: Re: Getting the IRQ number (Was: Basic driver devel questions ?)
Date: Mon, 13 Dec 2010 08:46:11 +1100 [thread overview]
Message-ID: <1292190371.14908.4.camel@concordia> (raw)
In-Reply-To: <201012101721.03689.dargaud@lpsc.in2p3.fr>
[-- Attachment #1: Type: text/plain, Size: 1356 bytes --]
On Fri, 2010-12-10 at 17:21 +0100, Guillaume Dargaud wrote:
> Hello all,
>
> > Also turn on CONFIG_DEBUG_DRIVER and see if that gives you anything
> > interesting.
>
> I see an unset CONFIG_DEBUG_KERNEL but no CONFIG_DEBUG_DRIVER in the .config of
> my current kernel.
Right you'd need to turn on CONFIG_DEBUG_KERNEL first, and then
CONFIG_DEBUG_DRIVER will appear.
> Does it need to be changed in the .config with a full kernel recompilation, or
> can I still benefit from it in my module only by doing a #define at the begining
> of my code ? Some #defines like DEBUG seem to work locally on things like
> dev_dbg but I doubt the CONFIG_* work the same...
No you'd need to rebuild your kernel. DEBUG is the only #define I can
think of that has an effect locally. Using different values for a
CONFIG_ in the kernel vs modules will not work.
> > Er. Not sure sorry. I can't see anything obviously wrong. Maybe post
> > your driver code again.
>
> Err... I ran it again this morning and it worked farther. I now get into the
> probe function and can now register my interrupt, yeah!, but I don't see the
> ISR being called. I'm currently checking if it can be a hardware problem
> before coming back here for more questions !
Great!
> BTW, is errno/strerror used within the kernel ?
Nope.
cheers
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2010-12-12 21:46 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-01 10:15 Basic driver devel questions ? Guillaume Dargaud
2010-12-01 12:19 ` Michael Ellerman
2010-12-01 16:35 ` Getting the IRQ number (Was: Basic driver devel questions ?) Guillaume Dargaud
2010-12-01 18:29 ` Philipp Ittershagen
2010-12-01 18:41 ` Scott Wood
2010-12-02 15:36 ` Guillaume Dargaud
2010-12-02 16:17 ` Timur Tabi
2010-12-02 17:47 ` Guillaume Dargaud
2010-12-02 20:22 ` Benjamin Herrenschmidt
2010-12-03 14:58 ` Guillaume Dargaud
2010-12-03 15:37 ` Martyn Welch
2010-12-06 5:29 ` Michael Ellerman
2010-12-06 6:35 ` Jeremy Kerr
2010-12-06 11:56 ` Michael Ellerman
2010-12-06 9:58 ` Guillaume Dargaud
2010-12-06 12:07 ` Michael Ellerman
2010-12-06 14:44 ` Guillaume Dargaud
2010-12-06 14:47 ` David Laight
2010-12-08 1:03 ` Michael Ellerman
2010-12-08 10:18 ` Guillaume Dargaud
2010-12-08 13:45 ` Michael Ellerman
2010-12-08 15:52 ` Guillaume Dargaud
2010-12-09 0:22 ` Michael Ellerman
2010-12-10 16:21 ` Guillaume Dargaud
2010-12-12 21:46 ` Michael Ellerman [this message]
2010-12-08 19:20 ` Joachim Förster
2010-12-06 12:17 ` David Laight
2010-12-07 12:46 ` Guillaume Dargaud
2010-12-08 0:50 ` Michael Ellerman
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=1292190371.14908.4.camel@concordia \
--to=michael@ellerman.id.au \
--cc=dargaud@lpsc.in2p3.fr \
--cc=linuxppc-dev@lists.ozlabs.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).