linux-can.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "j.p.lammertink" <j.p.lammertink@kubicas.com>
To: Wolfgang Grandegger <wg@grandegger.com>
Cc: linux-can@vger.kernel.org
Subject: Re: Evaluating the c_can driver for EG20T
Date: Thu, 25 Apr 2013 10:55:07 +0200	[thread overview]
Message-ID: <20130425085507.3EE222B1334F5@bmail03.one.com> (raw)
In-Reply-To: <5178E0BD.30501@grandegger.com>


<> Hi Wolfgang,

<>< Hi Jeroen,
<><> Hi Wolfgang,

<><> I think I have a patch for the "Loosing CAN messages problem"
<><> See at the bottom Appendix 7 Sugested patch.
<><> I realize that this patch has a performance penalty, but I didn't dare
<><> to make more rigorous changes.
<><> I've put the patch on top of Patch 1, of the patches that you sugested (see
<><> text below)

<><> I'm interested in all feedback.
<>< Well, we want to get rid of the pch_can driver sooner than later.
<>< Therefore I would appreciate if you could test and improve the c_can
<>< driver instead.
<>< Wolfgang.

<> I tried once before but I did not succeed. I probably did something
<> wrong. Can you tell me how I can replace the pch_can driver by the
<> c_can driver?
<> We have reduced confidence in the pch_can driver and therefor we
<> are also intrested in evaluating the c_can driver.

< Good, anyway, could you sent me your patches pch_can.c to understand
< better your lates problems.

<> Note, after applying patch 1/7 to the pch_can driver, I also
<> attemped to
<> apply the patches 2/7-7/7 which mainly/only(?) effect the c_can
<> driver.
<> 1/7 went succesfull, 2/7-7/7 not. Do I still need to apply those
<> patches?

< What exact kernel disto and kernel version are you using. With that
< knowledge I could provid new patches.
< Wolfgang.

Hi Wolfgang,

I've included the patch in the E-mail: 
"Losing CAN messages with socket-CAN" of 24 apr 2013 11:11. (Appendix 7)
It also contains kernel disto and kernel version and also
the problem description.

You replied to above mentioned E-mail: 
"Therefore I would appreciate if you could test and improve the c_can
driver instead."

Can you please indicate how to activate the c_can driver 
(instead of the pch_driver)?

Best regards,

Jeroen

  reply	other threads:[~2013-04-25  8:55 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-25  7:43 Evaluating the c_can driver for EG20T j.p.lammertink
2013-04-25  7:52 ` Wolfgang Grandegger
2013-04-25  8:55   ` j.p.lammertink [this message]
2013-04-25  9:23     ` Wolfgang Grandegger
2013-04-25 10:11       ` j.p.lammertink
2013-04-26 11:42         ` Wolfgang Grandegger
2013-04-25 13:25       ` Wolfgang Grandegger
  -- strict thread matches above, loose matches on Subject: below --
2013-04-26  7:23 j.p.lammertink
2013-04-26 11:45 ` Wolfgang Grandegger

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=20130425085507.3EE222B1334F5@bmail03.one.com \
    --to=j.p.lammertink@kubicas.com \
    --cc=linux-can@vger.kernel.org \
    --cc=wg@grandegger.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 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).