From: castet.matthieu@free.fr
To: Florian Fainelli <florian@openwrt.org>
Cc: matthieu castet <castet.matthieu@free.fr>,
linville@tuxdriver.com, Michael Buesch <mb@bu3sch.de>,
linux-mips@linux-mips.org, netdev@vger.kernel.org,
Daniel Mueller <daniel@danm.de>
Subject: Re: [PATCH] bc47xx : fix ssb irq setup
Date: Thu, 11 Jun 2009 11:23:45 +0200 [thread overview]
Message-ID: <1244712225.4a30cd2154cdf@imp.free.fr> (raw)
In-Reply-To: <200906111009.15080.florian@openwrt.org>
Quoting Florian Fainelli <florian@openwrt.org>:
> Le Thursday 11 June 2009 09:59:36 Florian Fainelli, vous avez écrit :
> > Hi Matthieu, Michael,
> >
> > Le Monday 25 May 2009 19:55:58 matthieu castet, vous avez écrit :
> > > Michael Buesch wrote:
> > > > On Tuesday 19 May 2009 00:06:12 matthieu castet wrote:
> > > >> Hi,
> > > >>
> > > >>
> > > >> [1] http://www.danm.de/files/src/bcm5365p/REPORTED_DEVICES
> > > >>
> > > >> Signed-off-by: Matthieu CASTET <castet.matthieu@free.fr>
> > > >
> > > > If this works on all devices, I'm OK with this. Please submit to
> > > > linville@tuxdriver.com You can add my ack.
> > >
> > > Well I have only a wl500gd.
> > > I have submit it on openwrt project in order to test in more devices.
> >
> > It makes the IPsec core work on my Netgear WGT634U and I did not see any
> > regression on a Linksys WRT54GS.
> >
> > Tested-by: Florian Fainelli <florian@openwrt.org>
>
> One minor thing, please remove the dump_irqs call, it is convenient for
> debugging to print the IRQ routing, but I find it a little too verbose for
> production. This can be a follow-up patch if you prefer not to respin it.
Well all x86 does something similar, with acpi dumping the pci interrupt Routing
Table even in production kernel.
But it can removed if people don't want it.
Matthieu
prev parent reply other threads:[~2009-06-11 9:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-18 22:06 [PATCH] bc47xx : fix ssb irq setup matthieu castet
2009-05-19 13:19 ` Michael Buesch
2009-05-25 17:55 ` matthieu castet
2009-06-11 7:59 ` Florian Fainelli
2009-06-11 8:09 ` Florian Fainelli
2009-06-11 9:23 ` castet.matthieu [this message]
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=1244712225.4a30cd2154cdf@imp.free.fr \
--to=castet.matthieu@free.fr \
--cc=daniel@danm.de \
--cc=florian@openwrt.org \
--cc=linux-mips@linux-mips.org \
--cc=linville@tuxdriver.com \
--cc=mb@bu3sch.de \
--cc=netdev@vger.kernel.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).