docs: fix Installation guide
[apps/agl-service-can-low-level.git] / docs / 2-Installation.md
index 70036b6..410bc4b 100644 (file)
@@ -6,27 +6,21 @@
 
 It will produce a _configuration-generated.cpp_ file to paste in the source, _src/_, directory.
 
-* Make sure you already set up the AGL SDK using the following [guide](http://docs.iot.bzh/docs/getting_started/en/dev/reference/setup-sdk-environment.html).
+* Make sure you already set up the AGL SDK using the following [SDK Quick Setup Guide](http://docs.iot.bzh/docs/getting_started/en/dev/reference/setup-sdk-environment.html). Alternatively, please refer to official guides available on [AGL Developer Site](http://docs.automotivelinux.org/docs/devguides/en/dev/#guides).
 
-To get the correct SDK version installed, you **must** prepare your environment with the **chinook-next** version. To do so, run the following command in your docker image:
+To get the correct SDK version installed, you **must** prepare your environment with the **chinook-next** version. To do so, run the following command in your docker image in the step 4 in place of `... [ prepare build environment ] ...`:
 
-> **NOTE** This command assume that proprietary graphic drivers for Renesas Porter board are located in `/home/devel/share/proprietary-renesas-rcar` directory.
+> **NOTE** These commands assume that proprietary graphic drivers for Renesas Porter board are located in `/home/devel/share/proprietary-renesas-rcar` directory.
 
 ```bash
 $ prepare_meta -f chinook-next -o /xdt -l /home/devel/mirror -p /home/devel/share/proprietary-renesas-rcar/ -t porter -e wipeconfig -e rm_work
-```
-
-* Check that you updated git submodules, executing the following commands from this repository:
-
-```bash
-$ git clone https://github.com/iotbzh/CAN_signaling
-$ cd CAN_signaling
-$ git submodule init
-$ git submodule update
+$ /xdt/build/agl-init-build-env
 ```
 
 * An [USB CAN adapter](http://shop.8devices.com/usb2can) connected to connector through the [right cable](http://www.mouser.fr/ProductDetail/EasySync/OBD-M-DB9-F-ES/)).
 
+<!-- pagebreak -->
+
 # Getting started
 
 ## Use of CAN config generator
@@ -55,10 +49,13 @@ You may want to install `libboost-all-dev` to get all boost components even if i
 
 ```bash
 $ export PATH=$PATH:/xdt/sdk/sysroots/x86_64-aglsdk-linux/usr/bin
+$ export WD=$(pwd)
 $ git clone https://github.com/iotbzh/can-config-generator.git
 $ cd can-config-generator
-$ mkdir build
+$ mkdir -p build
+$ cd build
 $ cmake -G "Unix Makefiles" ..
+$ make
 ```
 
 ### Naming convention
@@ -92,12 +89,16 @@ engine.torque
 ```
 
 > **NOTE** It's recommended that you follow this naming convention to named your CAN signals.
+>
 > There is only character `*` that is forbidden in names because it's used as wildcard for subscription and unsubscrition.
+>
 > This described in the below chapter.
 
 ### Generating JSON from Vector CANoe Database
 
-If you use Canoe to store your `gold standard` CAN signal definitions, you may be able to use the OpenXC  `xml_to_json.py` script to make your JSON for you. First, export the Canoe .dbc file as XML - you can do this with Vector CANdb++. Next, create a JSON file according to the format defined above, but only define:
+> **CAUTION** This chapter has not been tested since we haven't necessary automotive tools for that. 
+
+If you use CANoe to store your `gold standard` CAN signal definitions, you may be able to use the OpenXC `xml_to_json.py` script to make your JSON for you. First, export the Canoe .dbc file as XML - you can do this with Vector CANdb++. Next, create a JSON file according to the format defined above, but only define:
 
 - CAN messages.
 - Name of CAN signals within messages and their generic_name.
@@ -107,13 +108,13 @@ To install the OpenXC utilities and runs `xml_to_json.py` script:
 
 ```bash
 $ sudo pip install openxc
-$ cd /usr/local/lib/python2.7/dist-packages/openxc-0.13.0-py2.7.egg/openxc/generator
+$ cd /usr/local/lib/python2.7/dist-packages/openxc/generator
 ```
 
 Assuming the data exported from Vector is in `signals.xml` and your minimal mapping file is `mapping.json`, run the script:
 
 ```bash
-$ ./xml_to_json.py signals.xml mapping.json signals.json
+$ python -m openxc.utils ./xml_to_json.py signals.xml mapping.json signals.json
 ```
 
 The script scans `mapping.json` to identify the CAN messages and signals that you want to use from the XML file. It pulls the neccessary details of the messages (bit position, bit size, offset, etc) and outputs the resulting subset as JSON into the output file, `signals.json`.
@@ -125,7 +126,7 @@ The resulting file together with `mapping.json` will work as input to the code g
 To generate your config file you just have to run the generator using the `-m` option to specify your JSON file.
 
 ```bash
-$ can-config-generator -m ../tests/basic.json -o configuration-generated.cpp
+$ ./can-config-generator -m ../tests/basic.json -o configuration-generated.cpp
 ```
 
 If you omit the `-o` option, then code is generated on the stdout.  
@@ -145,21 +146,35 @@ This generator will follow OpenXC support status of the low level CAN signaling
 > **NOTE**: The `buses` item will not be supported by this generator because the binding use another way to declare and configure buses. Please refer to the binding's documentation.
 
 ## Compile and install the binding
+Clone the binding repository, copy the generated file and updated the git submodules.
 
-With an AGL SDK environment correctly configured and **sourced**, I suggest you to set the TARGET variable in the root CMakeLists.txt file if you have an AGL target already running in your network.
+Execute the following commands from this repository:
+
+```bash
+$ cd $WD
+$ git clone https://github.com/iotbzh/CAN_signaling
+$ cd CAN_signaling
+$ git submodule init
+$ git submodule update
+$ cp $WD/can-config-generator/build/configuration-generated.cpp src/
+```
+
+### Installation using *make install*
+
+With an AGL SDK environment correctly configured and **sourced**, I suggest you to set the TARGET variable in the CMakeLists.txt file located under _src_ directory if you have an AGL target already running in your network.
 
 Then you can directly build and install the binding and source directory on your target system.
 
-Execute these commands to get your binding compile :
+Execute these commands to get your binding compile:
 
 ```bash
-$ mkdir build
+$ mkdir -p build
 $ cd build
 $ cmake ..
 $ make
 ```
 
-And if you have set TARGET variable, you can install it on your AGL system :
+And if you have set TARGET variable, you can install it on your AGL system:
 
 ```bash
 $ make install
@@ -185,6 +200,8 @@ It's because installation remove the binding before installing it.
 
 If it is the first time that you make the installation then you'll have this message in place of _**true**_.
 
+### Manual Installation
+
 To install it manually, you need to copy the _low-can-binding.wgt_ file on your target, then from it execute the following commands :
 
 On your host, to copy over the network :
@@ -193,9 +210,9 @@ On your host, to copy over the network :
 $ scp low-can-binding.wgt root@<target_IP>:~
 ```
 
-On the target, assuming _**wgt**_ file is in the root home directory :
+On the target, assuming _**wgt**_ file is in the root home directory:
 
 ```bash
-~# afm-util install low-can-binding.wgt
+# afm-util install low-can-binding.wgt
 { "added": "low-can-binding@0.1" }
 ```