From: Oliver Hartkopp <socketcan@hartkopp.net>
To: Marek Vasut <marex@denx.de>, Mirza Krak <mirza.krak@hostmobility.com>
Cc: "linux-can@vger.kernel.org" <linux-can@vger.kernel.org>,
Marc Kleine-Budde <mkl@pengutronix.de>,
Wolfgang Grandegger <wg@grandegger.com>
Subject: Re: [RFC] K-Line protocol via SocketCAN
Date: Sun, 5 Jun 2016 14:07:17 +0200 [thread overview]
Message-ID: <575415F5.3070306@hartkopp.net> (raw)
In-Reply-To: <574E47C8.4000709@denx.de>
On 06/01/2016 04:26 AM, Marek Vasut wrote:
>>> What would be the advantages to put this into kernel space (or into the
>>> CAN networking infrastructure)?
>>
>> I see two for putting this into the kernel:
>> - At faster bus speeds, you can achieve more precise timing if this is
>> in the kernel, both of the inter-byte delay and also for the
>> timestamps. Having this in the kernel even allows usage of the
>> realtime extensions if needed.
A topic for the serial driver infrastructure?
>> - Dedicated hardware driver can plug into such K-Line infrastructure.
>> Such hardware might be needed to support the faster modes to further
>> increase the timing precision.
Which K-Line infrastructure?
>> And two for using socketcan/network interface:
>> - The addressing support of the network stack can be mapped to K-Line
>> bus addresses.
The K-Line addressing is totally different from CAN addressing. Why do
you think the stuff in linux/net/can has any conjunction to K-Line
addressing?
>> - The rtnl can be used as an extensible interface for configuring the
>> K-Line parameters.
The idea of SocketCAN using the Linux networking infrastructure and
network interfaces is the multi-user capability which is enabled through
this layering.
IIRC K-Line is ONE point-to-point connection which is not multi-user
capable - so what would be the benefit to implement a K-Line serial
driver as network device and then configure it via rtnl?
> What do you think about putting the KLine support into socketcan stack ?
If I would think about your mentioned requirements and ideas I would
start with pyOBD http://www.obdtester.com/pyobd and probably move this
to be a PREEMPT_RT task and I would add an interface to specify
inter-byte delays into the serial driver infrastructure.
Even when trying to think very positively about your question I don't
see any conjunction to SocketCAN.
Best regards,
Oliver
next prev parent reply other threads:[~2016-06-05 12:07 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-19 23:15 [RFC] K-Line protocol via SocketCAN Marek Vasut
2016-05-20 6:04 ` Mirza Krak
2016-05-20 6:28 ` Oliver Hartkopp
2016-05-20 11:59 ` Marek Vasut
2016-05-22 20:27 ` Patrick Menschel
2016-05-22 21:11 ` Marek Vasut
2016-06-01 2:26 ` Marek Vasut
2016-06-05 12:07 ` Oliver Hartkopp [this message]
2016-06-09 15:00 ` Marek Vasut
2016-06-09 18:29 ` Oliver Hartkopp
2016-06-09 19:21 ` Marek Vasut
2016-06-09 20:12 ` Oliver Hartkopp
2016-06-11 19:42 ` Marek Vasut
2016-06-12 19:28 ` Oliver Hartkopp
2016-06-13 22:07 ` Marek Vasut
2016-06-14 6:10 ` Oliver Hartkopp
2016-06-15 3:42 ` Marek Vasut
2016-06-15 6:57 ` Oliver Hartkopp
2016-06-15 11:05 ` Marek Vasut
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=575415F5.3070306@hartkopp.net \
--to=socketcan@hartkopp.net \
--cc=linux-can@vger.kernel.org \
--cc=marex@denx.de \
--cc=mirza.krak@hostmobility.com \
--cc=mkl@pengutronix.de \
--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 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.