Commit Graph

35 Commits (c2f7f8e9b2f15431fb801eab479bd0196031c524)

Author SHA1 Message Date
Alexander Shiyan 0c4609f54f libsocketcan: Remove excess gotos.
Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2014-02-07 10:47:59 +01:00
Alexander Shiyan 24500fcaca libsocketcan: Fix return value for get{set}_link() function
This patch makes return value negative when open_nl_sock() function
fail and removes excess labels.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2014-02-06 11:45:55 +01:00
Alexander Shiyan 9a28d69183 Optimize can_do_restart() function
This patch optimizes can_do_restart() function by reusing some part
of code from existing set_link() function.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2014-01-31 11:12:20 +01:00
Alexander Shiyan d9469850ac Use "%u" printf-format specifier for unsigned variable
Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2014-01-31 11:12:14 +01:00
Alexander Stein 94094cc4ee Use strcmp instead of sizeof on char* string
This bug was detected by the clang warning:
libsocketcan.c:384:16: warning: argument to 'sizeof' in 'strncmp' call
is the same expression as the source; did you mean to provide an
explicit length? [-Wsizeof-pointer-memaccess]
sizeof(name)) != 0)
~~~~~~~^~~~~~

Signed-off-by: Alexander Stein <alexander.stein@systec-electronic.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2012-11-28 13:11:04 +01:00
Andrew Beard 5ba2310d85 Modified do_get_nl_link to potentially process multiple netlink dump replies.
Prior to this commit, do_get_nl_link assumed there would only be a single
reply for a dump request. On my system with multiple can interfaces, this
would cause only half of them to be listed in the dump reply. This change
checks for NLMSG_DONE dump terminator and stops receiving messages once
received.

Signed-off-by: Andrew Beard <abeard@ovro.caltech.edu>
[mkl: fix indention, move variable init out of loop]
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2012-08-07 09:40:50 +02:00
Kurt Van Dijck 6db5d772b4 add can_get_device_stats()
When using libsocketcan on git://git.pengutronix.de/git/tools/libsocketcan.git,
I kind of missed a method to fetch the can device statistics.
This patch adds that functionality.
I think this is the proper library to put such function.

Signed-off-by: Kurt Van Dijck <kurt.van.dijck@eia.be>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2011-10-18 13:26:48 +02:00
Marc Kleine-Budde d99010f50b libsocketcan: remove trailing whitespace
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
2010-10-25 13:36:49 +02:00
Luotao Fu 298e6efc3e add get error counter call
Signed-off-by: Luotao Fu <luotao@alea.(none)>
2010-02-14 18:12:01 +01:00
Luotao Fu 6f0eac7ab9 add doxygen documentation
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-12-08 15:54:53 +01:00
Wolfgang Grandegger 463ac873c7 Add can_get_bittiming_const()
Signed-off-by: Wolfgang Grandegger <wg@grandegger.com>
Acked-by: Luotao Fu <l.fu@pengutronix.de>
2009-12-01 10:20:41 +01:00
Luotao Fu 96b6e75947 lindent, remove debug print
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-27 16:28:36 +01:00
Luotao Fu b2306d4979 splitted up set_bitrate
we now have can_set_bitrate and can_set_bitrate_samplepoint. The second is for
convenience for the developer. We tend to let the kernel use CIA recommnended
sample point value, which can_set_bitrate simply does

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-27 13:27:11 +01:00
Luotao Fu 257877f315 rename library and API prefix
library is now called libsocketcan, API carry the prefix can_

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-27 11:53:45 +01:00
Luotao Fu 1751269c33 remove internal netif_up/down callbacks and export them
We don't do if_up/down internally any longer, since it is depreacated. Instead
we add now can_start/stop callbacks and let the user do the job themselves in
threir applications.

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-26 16:16:09 +01:00
Luotao Fu 32c72a4868 removed duplicated code in get_ callbacks
added a get_link internal callback to eliminate the duplicated code in _get
functions.

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-26 16:09:57 +01:00
Luotao Fu 5b2ef2f2d3 add get clock callbacks
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-25 15:21:05 +01:00
Luotao Fu 6f73bd10bc add set samplepoint value to set_bitrate
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-25 15:20:36 +01:00
Luotao Fu 14bb1061bc API change
* parse struct can_ctrlmode directly to set_ctrl_mode
* add set_bittiming, change set_bitrate internally
* rename set_restart to do_restart
* rerangement

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-25 11:29:42 +01:00
Luotao Fu 4a8daa7258 changed licence to LGPL
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 16:22:37 +01:00
Luotao Fu c73035239f move internally used defines away from header
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 15:38:07 +01:00
Luotao Fu dc8f25b61a switch netif_up/down to internal static function
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 15:36:38 +01:00
Luotao Fu fb19f0edcb change get_state/restart_ms function prototype
This way we can verify the return value correctly

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 15:32:32 +01:00
Luotao Fu ba5cc5b4c8 add set_bitrate callback
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 14:53:17 +01:00
Luotao Fu 022be45eb1 fix set_restart callback
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 14:52:48 +01:00
Luotao Fu bbac00bbc7 move internally used defines away from header
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 13:05:42 +01:00
Luotao Fu 7e041b9af7 remove own copy of can/netlink.h file
include this from the header file now to make sure that application using this
library will include this properly

Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 13:01:58 +01:00
Luotao Fu 0c3e5a710b fix printf format
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 12:35:08 +01:00
Luotao Fu 9c5c1090d1 change fixed char size in strncmp
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 11:11:37 +01:00
Luotao Fu 12a155aa20 rename API functions
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 11:03:09 +01:00
Luotao Fu 9b332909b3 add get/set raw ctrlmode callbacks
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 11:02:24 +01:00
Luotao Fu 8c25d8d4b0 Lindent
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 09:06:52 +01:00
Luotao Fu 0e3319df98 update copyright, remove obsolete prints
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-23 09:05:40 +01:00
Luotao Fu 64c23c603c rename set_link and switch to static. remove internally used structs from header
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-20 16:59:04 +01:00
Luotao Fu d9b6577d1a initial create libsocketcan-nl project
Signed-off-by: Luotao Fu <l.fu@pengutronix.de>
2009-11-20 16:13:02 +01:00