From: Oliver Hartkopp <oliver.hartkopp@volkswagen.de>
To: Stephane Grosjean <s.grosjean@peak-system.com>
Cc: linux-can Mailing List <linux-can@vger.kernel.org>
Subject: Re: About can/usb v4 patches serie
Date: Tue, 07 Feb 2012 14:13:46 +0100 [thread overview]
Message-ID: <4F31238A.9060305@volkswagen.de> (raw)
In-Reply-To: <4F30F691.5070307@peak-system.com>
On 07.02.2012 11:01, Stephane Grosjean wrote:
> Hi Oliver,
>
> Thanks for all of your tests.
> Can you please install the serie of patches I just post which contains last
> version of the can/usb divers (according to your logs, you don't use it).
Hi Stephane,
i thought i used the latest version you sent with the tar-ball.
But having a clean version from the mailing list is always a better starting
point for tests ;-)
> This
> version should handle restart mechanism in a (more) correct way, for both
> pcan-usb/pcan-us pro adapters now.
ok.
> Meanwhile, I have a look to the deadlock issue in the pcmcia driver timer
> function first...
Yes. Just to be sure this eject problem can generally be solved i replaced the
PEAK PCMCIA card with the EMS PCMCIA card in my setup, where i only get two
removal notifications when pulling the card under full rx load.
Just one remark upon the USB driver:
I did a
cangen -g0 -i can5
to flood the USB-Pro adapter with outgoing traffic.
When doing an "ifconfig can5 down" i get:
peak_usb 2-2:1.0: can5: Tx URB aborted (-2)
ten times (what is probably the netdev queue length)
So on shutdown and/or unplugging there's something to do too.
Best regards,
Oliver
next prev parent reply other threads:[~2012-02-07 13:14 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-03 15:12 [PATCH v2] can/sja1000: add support for PEAK-System PCMCIA card Stephane Grosjean
2012-02-03 16:44 ` Oliver Hartkopp
2012-02-06 16:03 ` Oliver Hartkopp
2012-02-06 17:05 ` Sven Geggus
2012-02-06 17:41 ` Oliver Hartkopp
[not found] ` <4F30F691.5070307@peak-system.com>
2012-02-07 13:13 ` Oliver Hartkopp [this message]
2012-02-07 13:43 ` About can/usb v4 patches serie Stephane Grosjean
2012-02-07 13:51 ` Oliver Hartkopp
2012-02-15 14:41 ` Stephane Grosjean
2012-02-15 20:05 ` Oliver Hartkopp
2012-02-10 11:00 ` can/sja1000: potential issue in sja1000.c? Stephane Grosjean
2012-02-10 11:16 ` Marc Kleine-Budde
2012-02-10 11:44 ` Wolfgang Grandegger
2012-02-10 15:32 ` Sebastian Haas
2012-02-10 15:37 ` Marc Kleine-Budde
2012-02-10 11:56 ` Wolfgang Grandegger
2012-02-10 15:39 ` Oliver Hartkopp
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=4F31238A.9060305@volkswagen.de \
--to=oliver.hartkopp@volkswagen.de \
--cc=linux-can@vger.kernel.org \
--cc=s.grosjean@peak-system.com \
/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.