From: Pavel Pisa <pisa@cmp.felk.cvut.cz>
Date: Fri, 17 Jan 2014 00:31:47 +0000 (+0100)
Subject: Adjust sllin configuration utility to specify external expected response length.
X-Git-Url: http://rtime.felk.cvut.cz/gitweb/linux-lin.git/commitdiff_plain/5d7ddb29cf05cc4c6c49460bdc12b28d71c7b168?ds=sidebyside

Adjust sllin configuration utility to specify external expected response length.

The setup of the length has meaning not only for sent frames
but even for received responses. The new kernel driver
code uses DLC value stored in the cache for received frames.
The LIN_CACHE_RESPONSE is used only for IDs which
should be sent by the kernel driver.

Signed-off-by: Pavel Pisa <pisa@cmp.felk.cvut.cz>
---

diff --git a/lin_config/src/sllin_config.c b/lin_config/src/sllin_config.c
index dc704d8..c4b4084 100644
--- a/lin_config/src/sllin_config.c
+++ b/lin_config/src/sllin_config.c
@@ -123,24 +123,24 @@ int sllin_cache_config(struct linc_lin_state *linc_lin_state,
 	}
 
 	for (i = 0; i < 0x3F; i++) {
+		frame.can_dlc = linc_lin_state->frame_entry[i].data_len;
+		frame.can_id = i; /* LIN ID */
+		frame.data[0] = linc_lin_state->frame_entry[i].data[0]; /* Data */
+		frame.data[1] = linc_lin_state->frame_entry[i].data[1]; /* Data */
+		frame.data[2] = linc_lin_state->frame_entry[i].data[2]; /* Data */
+		frame.data[3] = linc_lin_state->frame_entry[i].data[3]; /* Data */
+		frame.data[4] = linc_lin_state->frame_entry[i].data[4]; /* Data */
+		frame.data[5] = linc_lin_state->frame_entry[i].data[5]; /* Data */
+		frame.data[6] = linc_lin_state->frame_entry[i].data[6]; /* Data */
+		frame.data[7] = linc_lin_state->frame_entry[i].data[7]; /* Data */
+		frame.can_id |= LIN_CTRL_FRAME;
 		if (linc_lin_state->frame_entry[i].status == 1) { /* Is active */
-			frame.can_dlc = linc_lin_state->frame_entry[i].data_len;
-			frame.can_id = i; /* LIN ID */
-			frame.data[0] = linc_lin_state->frame_entry[i].data[0]; /* Data */
-			frame.data[1] = linc_lin_state->frame_entry[i].data[1]; /* Data */
-			frame.data[2] = linc_lin_state->frame_entry[i].data[2]; /* Data */
-			frame.data[3] = linc_lin_state->frame_entry[i].data[3]; /* Data */
-			frame.data[4] = linc_lin_state->frame_entry[i].data[4]; /* Data */
-			frame.data[5] = linc_lin_state->frame_entry[i].data[5]; /* Data */
-			frame.data[6] = linc_lin_state->frame_entry[i].data[6]; /* Data */
-			frame.data[7] = linc_lin_state->frame_entry[i].data[7]; /* Data */
-
-			frame.can_id |= LIN_CTRL_FRAME | LIN_CACHE_RESPONSE;
-			ret = write(s, &frame, sizeof(frame));
-			printf("configuring frame cache; ret = %d\n", ret);
-			//if (ret ...)
-			//read_response(tty);
+			frame.can_id |= LIN_CACHE_RESPONSE;
 		}
+		ret = write(s, &frame, sizeof(frame));
+		printf("configuring frame cache; ret = %d\n", ret);
+		//if (ret ...)
+		//read_response(tty);
 	}
 
 	close(s);