public inbox for linux-can@vger.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Grandegger <wg@grandegger.com>
To: Oliver Hartkopp <socketcan@hartkopp.net>,
	Marc Kleine-Budde <mkl@pengutronix.de>,
	dev@sebastianhaas.info, linux-can@vger.kernel.org
Subject: Re: [RFC] can: Introducing CANFD for af_can & can-raw
Date: Thu, 22 Mar 2012 12:00:51 +0100	[thread overview]
Message-ID: <4F6B0663.1080405@grandegger.com> (raw)
In-Reply-To: <20120322103515.GE426@vandijck-laurijssen.be>

On 03/22/2012 11:35 AM, Kurt Van Dijck wrote:
> Hey Wolfgang,
> 
> On Thu, Mar 22, 2012 at 11:06:44AM +0100, Wolfgang Grandegger wrote:
>> Hi Kurt,
>>
>> On 03/22/2012 10:57 AM, Kurt Van Dijck wrote:
>>> On Wed, Mar 21, 2012 at 04:05:28PM +0100, Oliver Hartkopp wrote:
>>>> On 21.03.2012 15:56, Wolfgang Grandegger wrote:
>>>>
>>>>>>>
>>>>>>> I think it's much more tricky to find a proper solution here.
>>>>>> As I mentioned before, I just use CAN_RAW with this. Actually, I just
>>>>>> ran a real CAN program that I wrote years ago. It still operates!
>>>>>
>>>>> Would you program still run if the payload is bigger than 8 bytes?
>>>>> I agree with Oliver.
>>>>
>>> The other side would have noticed at design time that it cannot
>>> send >8 bytes :-)
>>>
>>> Did I miss something here? Is there a reason why existing programs
>>> may or should not work on a CANFD bus? I wanted to incorporate
>>> CANFD functionality _without_ putting an alternative API next to the
>>> current one, so to be able to use existing binaries on a future
>>> CANFD bus _without_ recompile (and thus not using the extras
>>> possibilities for that program).
>>
>> I believe that 99% of the existing apps, including candump, do work with
>> a (hard-coded) maximum length of 8 bytes instead of a variable length.
> I agree with that (99% may even be a bit low :-) )
> 
> I did manage to let such applications still function as expected.
> 
> The remaining 1% may or may not work. In case of proper coded programs,
> I'm sure they work. In other cases, I'm not sure.
> Thus the question is:
> Does the remaining 1% that we don't know, justify an alternate approach?

Well, I do not yet know. I first would like to see the data sheet of a
CANFD controller to understand better the possible use cases. And
without real use case, we will not add any CANFD support to the mainline
kernel anyway. There is still some time to think and discuss.

> My initial patch mainly illustrates that this may be a reasonable question.
> 
> As ethernet user, on a socket level, I also do not care about low-level
> technology (100Mbit, 1GBit, UTP, ...).

But that's because we use higher-level protocols like UDP and TCP which
do not care about the hardware frame size.

Wolfgang.


  reply	other threads:[~2012-03-22 11:01 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-21  9:10 [RFC] can: Introducing CANFD for af_can & can-raw Kurt Van Dijck
     [not found] ` <E1SAIM4-0007a6-Sf@smtprelay03.ispgateway.de>
2012-03-21 11:05   ` Kurt Van Dijck
2012-03-21 11:43     ` Marc Kleine-Budde
2012-03-21 12:08       ` Kurt Van Dijck
2012-03-21 12:32         ` Marc Kleine-Budde
2012-03-21 12:51           ` Kurt Van Dijck
2012-03-21 13:19             ` Marc Kleine-Budde
2012-03-21 13:21           ` Oliver Hartkopp
2012-03-21 13:53             ` Kurt Van Dijck
2012-03-21 14:49               ` Oliver Hartkopp
2012-03-21 15:26                 ` Oliver Hartkopp
2012-03-22  9:03                 ` Kurt Van Dijck
2012-03-21 14:56               ` Wolfgang Grandegger
2012-03-21 15:05                 ` Oliver Hartkopp
2012-03-22  9:24                   ` Kurt Van Dijck
2012-03-22  9:32                     ` Marc Kleine-Budde
2012-03-22  9:38                     ` Wolfgang Grandegger
2012-03-22 10:13                       ` Kurt Van Dijck
2012-03-23 11:01                         ` Wolfgang Grandegger
2012-03-22  9:57                   ` Kurt Van Dijck
2012-03-22 10:06                     ` Wolfgang Grandegger
2012-03-22 10:35                       ` Kurt Van Dijck
2012-03-22 11:00                         ` Wolfgang Grandegger [this message]
2012-03-22 12:25                           ` Oliver Hartkopp
2012-03-22 12:47                             ` Kurt Van Dijck
2012-03-21 13:29           ` Alexander Stein
2012-03-21 13:34             ` Kurt Van Dijck
2012-03-21 13:51             ` Marc Kleine-Budde
2012-03-21 15:47               ` Alexander Stein

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=4F6B0663.1080405@grandegger.com \
    --to=wg@grandegger.com \
    --cc=dev@sebastianhaas.info \
    --cc=linux-can@vger.kernel.org \
    --cc=mkl@pengutronix.de \
    --cc=socketcan@hartkopp.net \
    /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