[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20110929061402.GA322@e-circ.dyndns.org>
Date: Thu, 29 Sep 2011 08:14:02 +0200
From: Kurt Van Dijck <kurt.van.dijck@....be>
To: Oliver Hartkopp <socketcan@...tkopp.net>
Cc: David Miller <davem@...emloft.net>,
Linux Netdev List <netdev@...r.kernel.org>,
Wolfgang Grandegger <wg@...ndegger.com>,
Pavel Píša <pisa@....felk.cvut.cz>,
SocketCAN Core Mailing List <socketcan-core@...ts.berlios.de>
Subject: Re: [PATCH net-next v2] candev: allow SJW user setting for
bittiming calculation
On Thu, Sep 22, 2011 at 12:28:54PM +0200, Oliver Hartkopp wrote:
> This patch adds support for SJW user settings to not set the synchronization
> jump width (SJW) to 1 in any case when using the in-kernel bittiming
> calculation.
>
> The ip-tool from iproute2 already supports to pass the user defined SJW
> value. The given SJW value is sanitized with the controller specific sjw_max
> and the calculated tseg2 value. As the SJW can have values up to 4 providing
> this value will lead to the maximum possible SJW automatically. A higher SJW
> allows higher controller oscillator tolerances.
>
Since both iproute2 & the bittiming calculation seem to be ready for it,
activating the SJW setting looks like a good idea.
Thanks,
Kurt
Acked-by: Kurt Van Dijck <kurt.van.dijck@....be>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists