linux-can.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Oliver Hartkopp <socketcan@hartkopp.net>
To: Stephane Grosjean <s.grosjean@peak-system.com>
Cc: linux-can Mailing List <linux-can@vger.kernel.org>
Subject: Re: [PATCH 0/3 v5] can/usb: Add PEAK-System PCAN USB adapters driver
Date: Wed, 22 Feb 2012 17:40:33 +0100	[thread overview]
Message-ID: <4F451A81.4070009@hartkopp.net> (raw)
In-Reply-To: <4F4518CF.6070207@peak-system.com>

On 22.02.2012 17:33, Stephane Grosjean wrote:

> Le 22/02/2012 17:01, Oliver Hartkopp a écrit :
> 
>> Pls try the following:
>>
>> PC1:
>> can0 ->  usb-pro ch1 ----+
>> can1 ->  usb-pro ch2 -+  |
>> can2 ->  usb ---------+  |
>>                       |  | 1Mpbs
>> PC2:                 |  |
>> can0 ->  pcmcia ch1 --+  |
>> can1 ->  pcmcia ch2 -----+
>>
>> PC1:
>> cangen -g0 -i can0
>> cangen -g0 -i can1
>> candump any
>>
>> PC2:
>> cangen -g0 -i can0
>> cangen -g0 -i can1
>> candump any
>>
>> and then pull the usb-pro from PC1 ...
>>
>> That's really unfriendly 8-)
> 
> 
> Ok, I found another cable and set up your tesbed: now the usb-pro is pulled
> and both "candump" always are displaying data sent by "cangen can0" from PC2
> and read from pcan-usb on PC1... Both drivers can be removed without any problem.


cangen can0 only on PC2?

pls put the cangen on PC2 _and_ PC1 as shown above.


>>
>> Please check this use-case above. And when it works with your fixed driver you
>> can probalby check the 'old' v5 version to see if you can follow my bug report
>> on your setup.
> 
> ???
> I'm sorry I don't understand what you mean: why should I check the v5 version?


Didn't you write that you made some changes before the test??

> And what is (to) "follow your bug report on my setup" ???


I just wanted to make sure that you can see the same effect with your setup
that i had with version 5.

Regards,
Oliver

  reply	other threads:[~2012-02-22 16:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-17 12:24 [PATCH 0/3 v5] can/usb: Add PEAK-System PCAN USB adapters driver Stephane Grosjean
2012-02-17 12:24 ` [PATCH 1/3 v5] can/usb: PEAK-System Technik USB adapters driver core Stephane Grosjean
2012-02-22  7:05 ` [PATCH 0/3 v5] can/usb: Add PEAK-System PCAN USB adapters driver Oliver Hartkopp
2012-02-22  7:30   ` Oliver Hartkopp
2012-02-22 15:19     ` Stephane Grosjean
2012-02-22 16:01       ` Oliver Hartkopp
2012-02-22 16:33         ` Stephane Grosjean
2012-02-22 16:40           ` Oliver Hartkopp [this message]
2012-02-22 17:03             ` Stephane Grosjean
2012-02-22 17:50               ` Oliver Hartkopp
2012-02-22 11:03 ` Marc Kleine-Budde

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=4F451A81.4070009@hartkopp.net \
    --to=socketcan@hartkopp.net \
    --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 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).