apps/low-level-can-service.git
7 years agoDefine predefined OBD2 signals by default.
Romain Forlot [Wed, 8 Mar 2017 12:37:18 +0000 (13:37 +0100)]
Define predefined OBD2 signals by default.

Change-Id: Ide948c64fe2085b98acf7cdb70e6baf7b6a669ed
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMiss removed header file include
Romain Forlot [Wed, 8 Mar 2017 12:36:30 +0000 (13:36 +0100)]
Miss removed header file include

Change-Id: Iea45260ff78fadced7c9eb3ccefbc05341be25e4
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix missing constructor declaration
Romain Forlot [Wed, 8 Mar 2017 12:21:54 +0000 (13:21 +0100)]
Fix missing constructor declaration

Change-Id: I22dcc2a52fcbe6e37c820fe378bfb7dbbea8c25b
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoSeparate methods declarations and implementations.
Romain Forlot [Wed, 8 Mar 2017 21:53:22 +0000 (22:53 +0100)]
Separate methods declarations and implementations.
Get out examples objects to test into another cpp that
will be used to be the template for the futur generated
code.

Change-Id: Ic630478b5419e7a18e4e6446edbac7fbe68299b9
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
Conflicts:
src/configuration.hpp

7 years agoFix : wrong attribute type
Romain Forlot [Wed, 8 Mar 2017 12:17:03 +0000 (13:17 +0100)]
Fix : wrong attribute type

Change-Id: I8a87c295d4d76a5551fbb67b04b9fee69644eb00
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: missing method declaration
Romain Forlot [Wed, 8 Mar 2017 12:15:47 +0000 (13:15 +0100)]
Fix: missing method declaration

Change-Id: Iac5872ffe9f4b26e7c966e5ea29516f193ba547d
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoRemove now unused function that has moved into configuration class
Romain Forlot [Wed, 8 Mar 2017 11:53:59 +0000 (12:53 +0100)]
Remove now unused function that has moved into configuration class

Change-Id: I1f16a7acb32407420b2c5f5deb5b815829b0b610
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMake a global variable to a configuration object.
Romain Forlot [Wed, 8 Mar 2017 11:55:47 +0000 (12:55 +0100)]
Make a global variable to a configuration object.

With afb interface, it will be the last global variable
needed as it regroup all needed stuff to other class and
functions.

Change-Id: I3d82b6f748ff22a9a975d15ab9df51f63005737f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoRemove moved functions into configuration class
Romain Forlot [Wed, 8 Mar 2017 21:22:41 +0000 (22:22 +0100)]
Remove moved functions into configuration class

Change-Id: Ia8aa6a9f6211c2e040f53972ba689aecdeda9011
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: wrong method signature.
Romain Forlot [Wed, 8 Mar 2017 11:53:27 +0000 (12:53 +0100)]
Fix: wrong method signature.

Change-Id: Id836fac13d66443c313403a572ab840eecb8321d
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMake dumb poco class from old C struct
Romain Forlot [Wed, 8 Mar 2017 11:52:46 +0000 (12:52 +0100)]
Make dumb poco class from old C struct

Change-Id: I1cae74b33e1351bb84d002e86f93c9329f8389f0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdd movable constructor to the class
Romain Forlot [Wed, 8 Mar 2017 21:21:15 +0000 (22:21 +0100)]
Add movable constructor to the class

Change-Id: I4d050fda9b189e4d88393013c2b6c893063f24db
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdded a warning that it is an example code
Romain Forlot [Wed, 8 Mar 2017 08:44:53 +0000 (09:44 +0100)]
Added a warning that it is an example code

Change-Id: Ib66e49960a348896020fba56f7d98910623af83f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoseparation of can_bus_t and can_bus_dev_t and use of the new socket class.
Loïc Collignon [Wed, 8 Mar 2017 13:47:38 +0000 (14:47 +0100)]
separation of can_bus_t and can_bus_dev_t and use of the new socket class.

Change-Id: I5f292ab18737fa48259f357d660fed27fb8fb4be
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded a socket class to wrap the C socket API.
Loïc Collignon [Wed, 8 Mar 2017 13:40:24 +0000 (14:40 +0100)]
added a socket class to wrap the C socket API.

Change-Id: Id90c60caba93b65011def0cc01232a7a2d7dbac4
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded a parent CMakeLists.txt to allow build in root dir and use by IDEs.
Loïc Collignon [Wed, 8 Mar 2017 13:39:48 +0000 (14:39 +0100)]
added a parent CMakeLists.txt to allow build in root dir and use by IDEs.

Change-Id: I1d5e3a85c582ed2c711d3138df0030655ce09d5e
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded some ignore to skip KDevelop files.
Loïc Collignon [Wed, 8 Mar 2017 13:37:26 +0000 (14:37 +0100)]
added some ignore to skip KDevelop files.

Change-Id: I602b92268f82250730818027c846b569db7f2b59
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoChange doxygen comments
Loïc Collignon [Wed, 8 Mar 2017 11:44:50 +0000 (12:44 +0100)]
Change doxygen comments

Change-Id: Ia635a7483e2fcda3b98fd3d3b3f7e996d26926d0
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoBeginning of work of get central configuration object
Romain Forlot [Wed, 8 Mar 2017 01:41:56 +0000 (02:41 +0100)]
Beginning of work of get central configuration object
that will be used by the binding to access generated
elements. It will hold generated structure/objects and
make a resume to the binding configuration.

Idea is to have an object that can be inherited
in the generated code to extend its functionnalities
or change some of its behoviors.

Change-Id: If2ce5cbe2eb98a74a8e3f13000ee02855674216f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoIt isn't needed to initialize diag manager
Romain Forlot [Tue, 7 Mar 2017 23:21:21 +0000 (00:21 +0100)]
It isn't needed to initialize diag manager
for each can_bus_dev_t instance
Indeed, generator will write the diag manager objects statically with the
correct can_bus_dev_t instance to use.

Change-Id: Ib52d93bff576ef962dc141f5936133e878977017

7 years agoIntroducing active-diagnostic-message that will be hold
Romain Forlot [Tue, 7 Mar 2017 14:37:10 +0000 (15:37 +0100)]
Introducing active-diagnostic-message that will be hold
by Diagnostic manager in its queue and list.

Change-Id: I565d06634a55b99cc97f36ec6d695b3b0bf21dca
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: typo, and dirty fix to get compiled ftm.
Romain Forlot [Tue, 7 Mar 2017 14:36:18 +0000 (15:36 +0100)]
Fix: typo, and dirty fix to get compiled ftm.

Change-Id: I74042051346a0f0f4de7152e2a668c041edaebde
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoIntroducing diagnostic manager class.
Romain Forlot [Tue, 7 Mar 2017 14:35:37 +0000 (15:35 +0100)]
Introducing diagnostic manager class.

It will hold communication through uds-c lib allowing
to communication with diagnostic protocol obd2. It is
attached to can_bus_dev_t class 'cause it must regularly
send CAN message through it.

Change-Id: I2d9d8dfaca10e9865bf82b0ae83e65490ca982f8
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMake an extern get_signal_id() function plug to get_pid()
Romain Forlot [Tue, 7 Mar 2017 14:31:20 +0000 (15:31 +0100)]
Make an extern get_signal_id() function plug to get_pid()
member function from class obd2_signals class.

Change-Id: I5a0f1cfd0c2fb9374cc3928861ca4e8a52911942
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: wrong function signature and get out from class
Romain Forlot [Tue, 7 Mar 2017 08:30:50 +0000 (09:30 +0100)]
Fix: wrong function signature and get out from class
temporarly to get compiled project.

Change-Id: I7235f503c42a3e31d09094c42882b74e5e809493
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoInclude file changes, typo and renaming operation.
Romain Forlot [Thu, 16 Mar 2017 16:08:57 +0000 (17:08 +0100)]
Include file changes, typo and renaming operation.

Change-Id: I6643333cfcc8bea120496c53f1cd3b8596e33398
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: typo
Romain Forlot [Mon, 6 Mar 2017 21:39:41 +0000 (22:39 +0100)]
Fix: typo

Change-Id: I882595f4a0f28c60b12f8c6ed0bff11a80af29af
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFinalize new directories organization.
Romain Forlot [Mon, 6 Mar 2017 21:39:31 +0000 (22:39 +0100)]
Finalize new directories organization.

Change include statements and CMakeLists.txt accordingly
to the new layout.

Change-Id: Ief0821f7f6636b072cf26c7d8d8fcc16fe43ab01
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoBeginning of work on obd2 object and
Romain Forlot [Mon, 6 Mar 2017 21:28:50 +0000 (22:28 +0100)]
Beginning of work on obd2 object and
files reorganization.

Change-Id: Idaa7ad05c45d734ce771506fd6e41f1a09a6ac66
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoRaw import openXC function to detect obd2 response.
Romain Forlot [Mon, 6 Mar 2017 18:16:30 +0000 (19:16 +0100)]
Raw import openXC function to detect obd2 response.

Change-Id: Ieea0396979302329f5884ebef2064bdc7015a45f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMerge pull request #250 from conz27/master
Petteri Aimonen [Thu, 16 Mar 2017 14:48:26 +0000 (16:48 +0200)]
Merge pull request #250 from conz27/master

CMAKE: remove use of relative paths

7 years agoadded somes files and folders to ignore in .gitignore.
Loïc Collignon [Thu, 16 Mar 2017 09:51:07 +0000 (10:51 +0100)]
added somes files and folders to ignore in .gitignore.

Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded visual studio 2017 linux cross compile projects.
Loïc Collignon [Thu, 16 Mar 2017 09:49:31 +0000 (10:49 +0100)]
added visual studio 2017 linux cross compile projects.

Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded a simpler json for testing.
Loïc Collignon [Thu, 16 Mar 2017 09:48:38 +0000 (10:48 +0100)]
added a simpler json for testing.

Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agofix in generated braced initializing lists.
Loïc Collignon [Thu, 16 Mar 2017 09:48:04 +0000 (10:48 +0100)]
fix in generated braced initializing lists.

Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agowip: generation of a configuration for can_signaling.
Loïc Collignon [Thu, 16 Mar 2017 08:54:02 +0000 (09:54 +0100)]
wip: generation of a configuration for can_signaling.

Change-Id: I35b31a98e87943a4e2fcad06561fec7a347b8483
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoadded a .gitignore.
Loïc Collignon [Thu, 16 Mar 2017 08:50:00 +0000 (09:50 +0100)]
added a .gitignore.

Change-Id: I9916bd0aee0dcce89300d1ed47320954f2c4acdf
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoinitial commit.
Loïc Collignon [Wed, 15 Mar 2017 01:36:22 +0000 (02:36 +0100)]
initial commit.

Change-Id: I5e639061c99b7cc6da3da06ccf19bde683a69e9f
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoInitial commit
Loïc Collignon [IoT.bzh] [Wed, 15 Mar 2017 01:23:41 +0000 (02:23 +0100)]
Initial commit

7 years agoFix potential out-of-bounds read with more than 64 required fields
Petteri Aimonen [Sun, 12 Mar 2017 10:18:32 +0000 (12:18 +0200)]
Fix potential out-of-bounds read with more than 64 required fields

7 years agoImprove varint unittest coverage for error cases
Petteri Aimonen [Sun, 12 Mar 2017 09:44:24 +0000 (11:44 +0200)]
Improve varint unittest coverage for error cases

7 years agoFix: wrong type not compatible with signature in
Romain Forlot [Thu, 9 Mar 2017 09:02:17 +0000 (10:02 +0100)]
Fix: wrong type not compatible with signature in
isotp-c libs.

Change-Id: Ie5709e5c0b7f25c4a1eab876a695c2d24f7fe936
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoComment pass to cpp file instead of hpp
Romain Forlot [Mon, 6 Mar 2017 18:08:32 +0000 (19:08 +0100)]
Comment pass to cpp file instead of hpp

Change-Id: I13f3b3d81485ec59a636f11140074d9f25f87428
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoSort and comment the code
Romain Forlot [Mon, 6 Mar 2017 17:45:35 +0000 (18:45 +0100)]
Sort and comment the code

Change-Id: I1860c0d5fb03ce2217d5553159a090298596d939
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: clear pointer vector before fill it.
Romain Forlot [Mon, 6 Mar 2017 16:00:12 +0000 (17:00 +0100)]
Fix: clear pointer vector before fill it.

Clearer indentation

Change-Id: I9fdea8d03bf33bf0ada3504770b52a21f30b57a0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoInclude new signals.xpp files to the compilation
Romain Forlot [Mon, 6 Mar 2017 15:58:04 +0000 (16:58 +0100)]
Include new signals.xpp files to the compilation

Change-Id: If105a195f24f90d8a75793e127488563ca299851
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoChange way to find signals to make it a little bit more generic.
Romain Forlot [Mon, 6 Mar 2017 15:57:14 +0000 (16:57 +0100)]
Change way to find signals to make it a little bit more generic.

Now we have basically 3 functions :
 - one for subscription which return signal name into a vector
 - one to find from CanSignal vector
 - a last to find from Obd2Pid vector

Name of some function has been changed.

May be reworked later.

Change-Id: Ie72969fd20503fc1f9291958dc2ac5d827a6be59
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFixing install issue due to relative paths
Constantine Grantcharov [Sun, 5 Mar 2017 18:02:43 +0000 (13:02 -0500)]
Fixing install issue due to relative paths

7 years agoRemove use of relative paths
Constantine Grantcharov [Sun, 5 Mar 2017 17:29:09 +0000 (12:29 -0500)]
Remove use of relative paths

When doing out of source builds:

mkdir build
cd build && cmake -G "Unix Makefiles" ../ && make

The build script tripped up on relative paths of /generator/proto and
the files under generator/proto/*.proto.

By using ${PROJECT_SOURCE_DIR}, the paths become absolute and the issue
disappears.

7 years agoForgot to fill in the dates
Petteri Aimonen [Sun, 5 Mar 2017 16:40:43 +0000 (18:40 +0200)]
Forgot to fill in the dates

7 years agoSetting version to 0.3.9-dev
Petteri Aimonen [Sun, 5 Mar 2017 16:39:40 +0000 (18:39 +0200)]
Setting version to 0.3.9-dev

7 years agoPublishing nanopb-0.3.8
Petteri Aimonen [Sun, 5 Mar 2017 16:28:23 +0000 (18:28 +0200)]
Publishing nanopb-0.3.8

7 years agoUpdate changelog
Petteri Aimonen [Sun, 5 Mar 2017 16:27:44 +0000 (18:27 +0200)]
Update changelog

7 years agochange all genericName to generic_name. Now all
Romain Forlot [Fri, 3 Mar 2017 14:12:17 +0000 (15:12 +0100)]
change all genericName to generic_name. Now all
objects use the same variable name

Change-Id: I8a232bc8fd7e580bd8088e6b2ae7a90666ef9c62
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoChange TARGET variable name and remove CMakeTools
Romain Forlot [Fri, 3 Mar 2017 14:11:33 +0000 (15:11 +0100)]
Change TARGET variable name and remove CMakeTools
helpers modules requirements

Change-Id: Ifaa03b13671c62f877e391e7cd6f0ff0b40fd356
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agotbf
Romain Forlot [Fri, 3 Mar 2017 07:49:00 +0000 (08:49 +0100)]
tbf

Change-Id: I4e46542258b8dab0c0e266adddf9bfc7a4111376
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoClean up uneeded members and functions
Romain Forlot [Thu, 2 Mar 2017 22:23:55 +0000 (23:23 +0100)]
Clean up uneeded members and functions

Change-Id: Idc9ebe9cfc6e348c378c1489971600b8f125e6e1
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: unsynchronization between queues.
Romain Forlot [Thu, 2 Mar 2017 22:23:32 +0000 (23:23 +0100)]
Fix: unsynchronization between queues.
Ressources not enough protected with mutex, I extend
the scope of the lock to make sure that event flow go
through synchronously.

Change-Id: I77a80ab69eb768a211c537e0f33580379832a465
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: no json data for subcribed events
Romain Forlot [Thu, 2 Mar 2017 22:16:56 +0000 (23:16 +0100)]
Fix: no json data for subcribed events

Change-Id: I9c260559d28bfe4bf26a77403254cd46a84a7d6f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: Make sure can_message_t.data_ member has been
Romain Forlot [Thu, 2 Mar 2017 22:06:22 +0000 (23:06 +0100)]
Fix: Make sure can_message_t.data_ member has been
clear before use it again.

Change-Id: Ib073d67651cd3272190ef7464e806cf1f22b4cf3
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoDebugging message cook
Romain Forlot [Thu, 2 Mar 2017 22:05:50 +0000 (23:05 +0100)]
Debugging message cook

Change-Id: If93bf9bf7c6e2f429a4313a7f6500ded950229c9
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdding DEBUG messages to track error on STL container
Romain Forlot [Thu, 2 Mar 2017 21:35:22 +0000 (22:35 +0100)]
Adding DEBUG messages to track error on STL container
Fix: iteration around signal vector.

Change-Id: I188cfa8be452ba5268f236b5a8cd1f2bce89058b
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: Wrong set up of vector.
Romain Forlot [Thu, 2 Mar 2017 21:34:23 +0000 (22:34 +0100)]
Fix: Wrong set up of vector.

Change-Id: Id9fa04541a62714802a64492e3981b204756212e
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoReserve new space to data_ can_message_t member only if the
Romain Forlot [Thu, 2 Mar 2017 21:33:38 +0000 (22:33 +0100)]
Reserve new space to data_ can_message_t member only if the
new incoming can frame if bigger than the old one.

Change-Id: I39634cc0d01b300ea471b358e13d704b8eff351c
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: multiple subscription and maintain subscribed_signals coherence
Romain Forlot [Thu, 2 Mar 2017 21:32:53 +0000 (22:32 +0100)]
Fix: multiple subscription and maintain subscribed_signals coherence
across usage.

- Transmission of a reference instead of copy.
- Don't use anymore iterator on subscribed_signals map

Change-Id: I5e5b7b0bb8598be3bb0ec59c29418ee937ddcc9e
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoInitialize all boolean value of DynamicField. Avoiding mistake.
Romain Forlot [Thu, 2 Mar 2017 21:24:32 +0000 (22:24 +0100)]
Initialize all boolean value of DynamicField. Avoiding mistake.

Change-Id: I9cd5551186debd153e838b6d4156430553d5235f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoClean and imrove log messages.
Romain Forlot [Thu, 2 Mar 2017 21:21:31 +0000 (22:21 +0100)]
Clean and imrove log messages.

Change-Id: I77c74851ec436de0c7a7c47993badb7d1e2c01ca
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoTests with CmakeTools extension for VSCode. Can install on target
Romain Forlot [Thu, 2 Mar 2017 21:20:10 +0000 (22:20 +0100)]
Tests with CmakeTools extension for VSCode. Can install on target
from VSCode.

Change-Id: I464d3379fe061605fcc84ba432e7a8a8b6ec6ec7
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix alltypes_callback testcase
Petteri Aimonen [Thu, 2 Mar 2017 20:40:15 +0000 (22:40 +0200)]
Fix alltypes_callback testcase

The test case was erroneously comparing whole submsg structures,
which could result in false errors when padding bytes differed.

7 years agoFix callback pointer corruption in proto3 mode (issue #249)
Petteri Aimonen [Thu, 2 Mar 2017 20:14:00 +0000 (22:14 +0200)]
Fix callback pointer corruption in proto3 mode (issue #249)

7 years agoAdd regression test for issue #249
Petteri Aimonen [Thu, 2 Mar 2017 20:04:02 +0000 (22:04 +0200)]
Add regression test for issue #249

7 years agoAdd callback test for proto3 mode
Petteri Aimonen [Thu, 2 Mar 2017 20:03:50 +0000 (22:03 +0200)]
Add callback test for proto3 mode

7 years agoFix bug in alltypes_callback test case
Petteri Aimonen [Thu, 2 Mar 2017 19:51:31 +0000 (21:51 +0200)]
Fix bug in alltypes_callback test case

7 years agoChange id to an hexadecimal value.
Romain Forlot [Wed, 1 Mar 2017 18:29:31 +0000 (19:29 +0100)]
Change id to an hexadecimal value.

Change-Id: Iecf8179e2885a7d1392a854bcd56e731fed5bba0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: subscription and improve code readabilty
Romain Forlot [Wed, 1 Mar 2017 18:28:20 +0000 (19:28 +0100)]
Fix: subscription and improve code readabilty

There was probleme between char and string
conversion. Also, I get rid of iterator and
use direct access using operator[].

Change-Id: I235ef0dd3ef9d9cebb624491d86383392e9eb8d8
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoNew way to return canfd_frame with now number of read bytes.
Romain Forlot [Wed, 1 Mar 2017 16:42:52 +0000 (17:42 +0100)]
New way to return canfd_frame with now number of read bytes.
This way, we can detect correctly the max data length and
set the data correctly.

Change-Id: I346df154df76dbd4a463d1fabfd9edd56faca506
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: wrong format to display can_message_t information
Romain Forlot [Wed, 1 Mar 2017 16:41:16 +0000 (17:41 +0100)]
Fix: wrong format to display can_message_t information

Change-Id: I6c17c68d8e995022886750f2ef05348b544cbde5
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMerge branch 'master' of github.com:iotbzh/CAN_signaling
Romain Forlot [Wed, 1 Mar 2017 16:07:49 +0000 (17:07 +0100)]
Merge branch 'master' of github.com:iotbzh/CAN_signaling

7 years agoWrong call to thread.join cause deadlock and thread termination.
Romain Forlot [Wed, 1 Mar 2017 16:07:20 +0000 (17:07 +0100)]
Wrong call to thread.join cause deadlock and thread termination.
Positionning the controlling boolean value will be enough for now

Change-Id: I28e2d175e23370746cc30c2403829ebdeec7c2bb
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: miss 'if' brackets to get incomplete CAN frames correctly
Romain Forlot [Wed, 1 Mar 2017 16:06:17 +0000 (17:06 +0100)]
Fix: miss 'if' brackets to get incomplete CAN frames correctly
Fix: displaying can message frame with hexa format.

Change-Id: Iff658ac38f21a15945ed82509d4e291f0fda7503
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoUpdated the README.md to clarify the use of git submodule.
Loïc Collignon [Wed, 1 Mar 2017 15:19:25 +0000 (16:19 +0100)]
Updated the README.md to clarify the use of git submodule.

Change-Id: Iceec81be36b217f69565452955da2f4ea43e10e9
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoChanged submodules URL to use https instead of ssh.
Loïc Collignon [Wed, 1 Mar 2017 15:14:29 +0000 (16:14 +0100)]
Changed submodules URL to use https instead of ssh.

Change-Id: I8614cca0ad386bdf6d2339e014a85d790da04d87
Signed-off-by: Loïc Collignon <loic.collignon@iot.bzh>
7 years agoChange data_ member of can_message_t object
Romain Forlot [Wed, 1 Mar 2017 14:58:42 +0000 (15:58 +0100)]
Change data_ member of can_message_t object
to a vector of uint8_t instead of an array

This is more flexible to allocate dynamically
the vector depending if we process classic CAN
frame or CAN FD frame which are 64bytes long.

Change-Id: I698002139d612e3aaaa33f0f5a895e16ff655f5d
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: improve can_message read
Romain Forlot [Wed, 1 Mar 2017 14:05:18 +0000 (15:05 +0100)]
Fix: improve can_message read

Change-Id: If0c52c6df3f68e9b79fb851d64a729946cac6ef6
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoDelete for good obsolete code
Romain Forlot [Wed, 1 Mar 2017 08:20:46 +0000 (09:20 +0100)]
Delete for good obsolete code

Change-Id: Ic56fadcc7e6265072e4e3413f914b735b56e903d
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agotbf
Romain Forlot [Wed, 1 Mar 2017 07:41:29 +0000 (08:41 +0100)]
tbf

Change-Id: I30e16cd45d613db1aaf7d1064bfaf5c6013d991a
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoImplement a new method returning the can device
Romain Forlot [Tue, 28 Feb 2017 20:29:17 +0000 (21:29 +0100)]
Implement a new method returning the can device
map.

Change-Id: Ie44bb825269034f9f071bd286c7830a77ba5c479
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: can bus object life is now expanded and don't
Romain Forlot [Tue, 28 Feb 2017 20:09:26 +0000 (21:09 +0100)]
Fix: can bus object life is now expanded and don't
die prematuraly.

can_bus_handler object is now a pointer living
forever.
can_bus_dev_t object is now stored as a shared
pointer into a variable member into can_bus_t.

So everyone survive now.

Change-Id: I52768dd7fe3c203a5f679f59afd1bcf330f3af35
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: wrong argument passed to start_reading().
Romain Forlot [Tue, 28 Feb 2017 15:21:19 +0000 (16:21 +0100)]
Fix: wrong argument passed to start_reading().
Not needed to specify std::ref...

Change-Id: I0e5ae02554f8a0f07ec1d40670b9b089453804e0
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdded debugging messages.
Romain Forlot [Tue, 28 Feb 2017 15:06:15 +0000 (16:06 +0100)]
Added debugging messages.

Change-Id: I42d2576355f69ac167b170a2994c7360d8449fb7
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoMake the thread function members of can_bus_t and can_bus_dev_t objects.
Romain Forlot [Tue, 28 Feb 2017 15:05:37 +0000 (16:05 +0100)]
Make the thread function members of can_bus_t and can_bus_dev_t objects.

Change-Id: I3cf06998c6ff6d859c7fdf6bf52a9b6ff061c556
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoRe-arranging objects splitting all objects over
Romain Forlot [Tue, 28 Feb 2017 11:00:43 +0000 (12:00 +0100)]
Re-arranging objects splitting all objects over
separated files.

Change-Id: Ifbdb2ea08704324e505a48ba3124df6f2af63c6b
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: start reading the can bus if it was correctly
Romain Forlot [Mon, 27 Feb 2017 22:17:43 +0000 (23:17 +0100)]
Fix: start reading the can bus if it was correctly
opened.

Change-Id: If285791aee51d4a57e17270e8ac458cb6869f970
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdded some DEBUG log messages around reading conf
Romain Forlot [Mon, 27 Feb 2017 22:03:55 +0000 (23:03 +0100)]
Added some DEBUG log messages around reading conf
file

Change-Id: Iec52801df6ce0c47c24412ac46332c301d5c46d6
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: wrong conf file name specified
Romain Forlot [Mon, 27 Feb 2017 22:03:12 +0000 (23:03 +0100)]
Fix: wrong conf file name specified
Comestic change about binding description and
prefix

Change-Id: Iffb3e0d13a81461db0f1300674c4fef3a6bde34f
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix: mismatch function signature and implementation
Romain Forlot [Mon, 27 Feb 2017 22:02:25 +0000 (23:02 +0100)]
Fix: mismatch function signature and implementation

Change-Id: I3e43ffda762117eac1b13da0d5a652338c6ad1f1
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix decoder_t methods to get it compile
Romain Forlot [Mon, 27 Feb 2017 21:23:34 +0000 (22:23 +0100)]
Fix decoder_t methods to get it compile

Change-Id: I283480bf434b00ade15bcb1dbc9d097c338caab4
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoAdded missing pthread lib
Romain Forlot [Mon, 27 Feb 2017 21:23:15 +0000 (22:23 +0100)]
Added missing pthread lib

Change-Id: I34c43af949a5f47334fa606acfe5b2bcccf59083
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoFix wrong returned value about getCanBuses.
Romain Forlot [Mon, 27 Feb 2017 20:30:05 +0000 (21:30 +0100)]
Fix wrong returned value about getCanBuses.

Why I keep this function anyway...

Change-Id: Ifcbc00625fcb065b73afed51fc712067aa9db412
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
7 years agoIn class mutex and condition variable except for subscribed_signals map
Romain Forlot [Mon, 27 Feb 2017 20:29:07 +0000 (21:29 +0100)]
In class mutex and condition variable except for subscribed_signals map
For now...

Change-Id: I133deb39fcd0660064b3b3c2a52f86ad37cb29c2
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>