X-Git-Url: http://rtime.felk.cvut.cz/gitweb/linux-lin.git/blobdiff_plain/aad94d17bb017b15814cbbc0e7fe4481f7050737..77e7c0235327e4ad651462afcea7fd9dc094f255:/sllin/sllin.c diff --git a/sllin/sllin.c b/sllin/sllin.c index 0e49aa5..e2d430a 100644 --- a/sllin/sllin.c +++ b/sllin/sllin.c @@ -1,4 +1,4 @@ -/* +/* * sllin.c - serial line LIN interface driver (using tty line discipline) * * This file is derived from drivers/net/can/slcan.c @@ -32,7 +32,7 @@ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH * DAMAGE. * - * Idea: Oliver Hartkopp + * Idea: Oliver Hartkopp * Copyright: (c) 2011 Czech Technical University in Prague * (c) 2011 Volkswagen Group Research * Authors: Pavel Pisa @@ -41,7 +41,7 @@ * Funded by: Volkswagen Group Research */ -#define DEBUG 1 /* Enables pr_debug() printouts */ +#define DEBUG 1 /* Enables pr_debug() printouts */ #include #include @@ -66,7 +66,7 @@ #include "linux/lin_bus.h" /* Should be in include/linux/tty.h */ -#define N_SLLIN 25 +#define N_SLLIN 25 /* -------------------------------- */ static __initdata const char banner[] = @@ -77,11 +77,11 @@ MODULE_DESCRIPTION("serial line LIN interface"); MODULE_LICENSE("GPL"); MODULE_AUTHOR("Pavel Pisa "); -#define SLLIN_MAGIC 0x53CA +#define SLLIN_MAGIC 0x53CA /* #define BREAK_BY_BAUD */ static int master = true; -static int baudrate = 0; /* Use LIN_DEFAULT_BAUDRATE when not set */ +static int baudrate; /* Use LIN_DEFAULT_BAUDRATE when not set */ module_param(master, bool, 0); MODULE_PARM_DESC(master, "LIN interface is Master device"); @@ -95,13 +95,13 @@ module_param(maxdev, int, 0); MODULE_PARM_DESC(maxdev, "Maximum number of sllin interfaces"); /* maximum buffer len to store whole LIN message*/ -#define SLLIN_DATA_MAX 8 +#define SLLIN_DATA_MAX 8 #define SLLIN_BUFF_LEN (1 /*break*/ + 1 /*sync*/ + 1 /*ID*/ + \ - SLLIN_DATA_MAX + 1 /*checksum*/) -#define SLLIN_BUFF_BREAK 0 -#define SLLIN_BUFF_SYNC 1 -#define SLLIN_BUFF_ID 2 -#define SLLIN_BUFF_DATA 3 + SLLIN_DATA_MAX + 1 /*checksum*/) +#define SLLIN_BUFF_BREAK 0 +#define SLLIN_BUFF_SYNC 1 +#define SLLIN_BUFF_ID 2 +#define SLLIN_BUFF_DATA 3 #define SLLIN_SAMPLES_PER_CHAR 10 #define SLLIN_CHARS_TO_TIMEOUT 24 @@ -142,12 +142,14 @@ struct sllin { int tx_cnt; /* number of already Tx bytes */ char lin_master; /* node is a master node */ int lin_baud; /* LIN baudrate */ - int lin_state; /* state */ - char id_to_send; /* there is ID to be sent */ + int lin_state; /* state */ + char id_to_send; /* there is ID to be sent */ char data_to_send; /* there are data to be sent */ char resp_len_known; /* Length of the response is known */ - char header_received;/* In Slave mode, set when header was already + char header_received;/* In Slave mode, set when header was already received */ + char rx_len_unknown; /* We are not sure how much data will be sent to us -- + we just guess the length */ unsigned long flags; /* Flag values/ mode etc */ #define SLF_INUSE 0 /* Channel in use */ @@ -176,14 +178,14 @@ static int sllin_configure_frame_cache(struct sllin *sl, struct can_frame *cf); /* Values of two parity bits in LIN Protected Identifier for each particular LIN ID */ const unsigned char sllin_id_parity_table[] = { - 0x80, 0xc0, 0x40, 0x00, 0xc0, 0x80, 0x00, 0x40, - 0x00, 0x40, 0xc0, 0x80, 0x40, 0x00, 0x80, 0xc0, - 0x40, 0x00, 0x80, 0xc0, 0x00, 0x40, 0xc0, 0x80, - 0xc0, 0x80, 0x00, 0x40, 0x80, 0xc0, 0x40, 0x00, - 0x00, 0x40, 0xc0, 0x80, 0x40, 0x00, 0x80, 0xc0, - 0x80, 0xc0, 0x40, 0x00, 0xc0, 0x80, 0x00, 0x40, - 0xc0, 0x80, 0x00, 0x40, 0x80, 0xc0, 0x40, 0x00, - 0x40, 0x00, 0x80, 0xc0, 0x00, 0x40, 0xc0, 0x80 + 0x80, 0xc0, 0x40, 0x00, 0xc0, 0x80, 0x00, 0x40, + 0x00, 0x40, 0xc0, 0x80, 0x40, 0x00, 0x80, 0xc0, + 0x40, 0x00, 0x80, 0xc0, 0x00, 0x40, 0xc0, 0x80, + 0xc0, 0x80, 0x00, 0x40, 0x80, 0xc0, 0x40, 0x00, + 0x00, 0x40, 0xc0, 0x80, 0x40, 0x00, 0x80, 0xc0, + 0x80, 0xc0, 0x40, 0x00, 0xc0, 0x80, 0x00, 0x40, + 0xc0, 0x80, 0x00, 0x40, 0x80, 0xc0, 0x40, 0x00, + 0x40, 0x00, 0x80, 0xc0, 0x00, 0x40, 0xc0, 0x80 }; /** @@ -429,6 +431,12 @@ static void sll_setup(struct net_device *dev) /****************************************** Routines looking at TTY side. ******************************************/ +#define SLL_RESPONSE_RECEIVED ((sl->header_received == true) && \ + ((sl->rx_cnt >= sl->rx_expect) || \ + ((sl->rx_len_unknown == true) && (count == 0)))) + +#define SLL_HEADER_RECEIVED ((sl->header_received == false) && \ + (sl->rx_cnt >= (SLLIN_BUFF_ID + 1))) static void sllin_receive_buf(struct tty_struct *tty, const unsigned char *cp, char *fp, int count) @@ -453,67 +461,70 @@ static void sllin_receive_buf(struct tty_struct *tty, } else { /* Received Break */ sl->rx_cnt = 0; sl->rx_expect = SLLIN_BUFF_ID + 1; + sl->rx_len_unknown = false; /* We do know exact length of the header */ sl->header_received = false; - return; } } - if (sl->rx_cnt < SLLIN_BUFF_LEN) { #ifndef BREAK_BY_BAUD - /* We didn't receive Break character -- fake it! */ - if ((sl->rx_cnt == SLLIN_BUFF_BREAK) && (*cp == 0x55)) { - pr_debug("sllin: LIN_RX[%d]: 0x00\n", sl->rx_cnt); - sl->rx_buff[sl->rx_cnt++] = 0x00; - } + /* We didn't receive Break character -- fake it! */ + if ((sl->rx_cnt == SLLIN_BUFF_BREAK) && (*cp == 0x55)) { + pr_debug("sllin: LIN_RX[%d]: 0x00\n", sl->rx_cnt); + sl->rx_buff[sl->rx_cnt++] = 0x00; + } #endif + if (sl->rx_cnt < SLLIN_BUFF_LEN) { pr_debug("sllin: LIN_RX[%d]: 0x%02x\n", sl->rx_cnt, *cp); sl->rx_buff[sl->rx_cnt++] = *cp++; } - } - if (sl->lin_master == true) { - if (sl->rx_cnt >= sl->rx_expect) { /* Probably whole frame was received */ - set_bit(SLF_RXEVENT, &sl->flags); - wake_up(&sl->kwt_wq); - pr_debug("sllin: sllin_receive_buf count %d, wakeup\n", sl->rx_cnt); - } else { - pr_debug("sllin: sllin_receive_buf count %d, waiting\n", sl->rx_cnt); - } - } else { /* LIN slave */ - int lin_id; - struct sllin_conf_entry *sce; - - pr_debug("sllin: rx_cnt = %u; header_received = %u\n", - sl->rx_cnt, sl->header_received); - - /* Whole header was received */ - if ((sl->rx_cnt >= (SLLIN_BUFF_ID + 1)) && - (sl->header_received == false)) - { - lin_id = sl->rx_buff[SLLIN_BUFF_ID] & LIN_ID_MASK; - sce = &sl->linfr_cache[lin_id]; - - /* Is the length of data set in frame cache? */ - if (sce->frame_fl & LIN_LOC_SLAVE_CACHE) - sl->rx_expect += sce->dlc; - else - sl->rx_expect += 2; /* 1 data byte + checksum */ - - sl->header_received = true; - sll_send_rtr(sl); - } + if (sl->lin_master == true) { + if (SLL_RESPONSE_RECEIVED) { + set_bit(SLF_RXEVENT, &sl->flags); + wake_up(&sl->kwt_wq); + pr_debug("sllin: sllin_receive_buf count %d, wakeup\n", sl->rx_cnt); + } else { + pr_debug("sllin: sllin_receive_buf count %d, waiting\n", sl->rx_cnt); + } + } else { /* LIN slave */ + int lin_id; + struct sllin_conf_entry *sce; + + pr_debug("sllin: rx_cnt = %u; header_received = %u\n", + sl->rx_cnt, sl->header_received); + + if (SLL_HEADER_RECEIVED) { + lin_id = sl->rx_buff[SLLIN_BUFF_ID] & LIN_ID_MASK; + sce = &sl->linfr_cache[lin_id]; + + spin_lock(&sl->lock); + /* Is the length of data set in frame cache? */ + if (sce->frame_fl & LIN_LOC_SLAVE_CACHE) { + sl->rx_expect += sce->dlc; + sl->rx_len_unknown = false; + } else { + sl->rx_expect += SLLIN_DATA_MAX + 1; /* + checksum */ + sl->rx_len_unknown = true; + } + spin_unlock(&sl->lock); - /* Probably whole frame was received */ - if ((sl->rx_cnt >= sl->rx_expect) && (sl->rx_cnt > SLLIN_BUFF_DATA)) { - sll_bump(sl); - pr_debug("sllin: Received LIN header & LIN response. " - "rx_cnt = %u, rx_expect = %u\n", sl->rx_cnt, - sl->rx_expect); - - /* Prepare for reception of new header */ - sl->rx_cnt = 0; - sl->rx_expect = SLLIN_BUFF_ID + 1; - sl->header_received = false; + sl->header_received = true; + sll_send_rtr(sl); + continue; + } + + if (SLL_RESPONSE_RECEIVED) { + sll_bump(sl); + pr_debug("sllin: Received LIN header & LIN response. " + "rx_cnt = %u, rx_expect = %u\n", sl->rx_cnt, + sl->rx_expect); + + /* Prepare for reception of new header */ + sl->rx_cnt = 0; + sl->rx_expect = SLLIN_BUFF_ID + 1; + sl->rx_len_unknown = false; /* We do know exact length of the header */ + sl->header_received = false; + } } } } @@ -523,7 +534,7 @@ static void sllin_receive_buf(struct tty_struct *tty, *****************************************/ /** * sllin_report_error() -- Report an error by sending CAN frame - * with particular error flag set in can_id + * with particular error flag set in can_id * * @sl: * @err: Error flag to be sent. @@ -531,17 +542,17 @@ static void sllin_receive_buf(struct tty_struct *tty, void sllin_report_error(struct sllin *sl, int err) { switch (err) { - case LIN_ERR_CHECKSUM: - sl->dev->stats.rx_crc_errors++; - break; + case LIN_ERR_CHECKSUM: + sl->dev->stats.rx_crc_errors++; + break; - case LIN_ERR_RX_TIMEOUT: - sl->dev->stats.rx_errors++; - break; + case LIN_ERR_RX_TIMEOUT: + sl->dev->stats.rx_errors++; + break; - case LIN_ERR_FRAMING: - sl->dev->stats.rx_frame_errors++; - break; + case LIN_ERR_FRAMING: + sl->dev->stats.rx_frame_errors++; + break; } sllin_send_canfr(sl, 0 | CAN_EFF_FLAG | @@ -554,6 +565,8 @@ void sllin_report_error(struct sllin *sl, int err) * @sl: * @cf: Pointer to CAN frame sent to this driver * holding configuration information + * + * Called with sl->lock held. */ static int sllin_configure_frame_cache(struct sllin *sl, struct can_frame *cf) { @@ -578,9 +591,9 @@ static int sllin_configure_frame_cache(struct sllin *sl, struct can_frame *cf) /** * sllin_checksum() -- Count checksum for particular data * - * @data: Pointer to the buffer containing whole LIN + * @data: Pointer to the buffer containing whole LIN * frame (i.e. including break and sync bytes). - * @length: Length of the buffer. + * @length: Length of the buffer. * @enhanced_fl: Flag determining whether Enhanced or Classic * checksum should be counted. */ @@ -783,8 +796,10 @@ static int sllin_rx_validate(struct sllin *sl) actual_id = sl->rx_buff[SLLIN_BUFF_ID] & LIN_ID_MASK; scf = &sl->linfr_cache[actual_id]; + spin_lock(&sl->lock); lin_dlc = scf->dlc; ext_chcks_fl = scf->frame_fl & LIN_CHECKSUM_EXTENDED; + spin_unlock(&sl->lock); if (sllin_checksum(sl->rx_buff, sl->rx_cnt - 1, ext_chcks_fl) != rec_chcksm) { @@ -830,7 +845,7 @@ int sllin_kwthread(void *ptr) if ((sl->lin_state == SLSTATE_IDLE) && sl->lin_master && sl->id_to_send) { - if(sllin_send_break(sl) < 0) { + if (sllin_send_break(sl) < 0) { /* error processing */ } } @@ -877,171 +892,171 @@ int sllin_kwthread(void *ptr) } switch (sl->lin_state) { - case SLSTATE_IDLE: - if (!test_bit(SLF_MSGEVENT, &sl->flags)) - break; + case SLSTATE_IDLE: + if (!test_bit(SLF_MSGEVENT, &sl->flags)) + break; - cf = (struct can_frame *)sl->tx_req_skb->data; + cf = (struct can_frame *)sl->tx_req_skb->data; - /* SFF RTR CAN frame -> LIN header */ - if (cf->can_id & CAN_RTR_FLAG) { - spin_lock(&sl->lock); - pr_debug("sllin: %s: RTR SFF CAN frame, ID = %x\n", - __FUNCTION__, cf->can_id & LIN_ID_MASK); - - /* Is there Slave response in linfr_cache to be sent? */ - if ((sl->linfr_cache[cf->can_id & LIN_ID_MASK].frame_fl & - LIN_LOC_SLAVE_CACHE) - && (sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc > 0)) { - - pr_debug("sllin: Sending LIN response from linfr_cache\n"); - lin_data = sl->linfr_cache[cf->can_id & LIN_ID_MASK].data; - lin_dlc = sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc; - if (lin_dlc > SLLIN_DATA_MAX) - lin_dlc = SLLIN_DATA_MAX; - memcpy(lin_data_buff, lin_data, lin_dlc); - lin_data = lin_data_buff; - } else { - lin_data = NULL; - lin_dlc = sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc; - } - spin_unlock(&sl->lock); - } else { /* SFF NON-RTR CAN frame -> LIN header + LIN response */ - pr_debug("sllin: %s: NON-RTR SFF CAN frame, ID = %x\n", - __FUNCTION__, (int)cf->can_id & LIN_ID_MASK); + /* SFF RTR CAN frame -> LIN header */ + if (cf->can_id & CAN_RTR_FLAG) { + spin_lock(&sl->lock); + pr_debug("sllin: %s: RTR SFF CAN frame, ID = %x\n", + __FUNCTION__, cf->can_id & LIN_ID_MASK); + + /* Is there Slave response in linfr_cache to be sent? */ + if ((sl->linfr_cache[cf->can_id & LIN_ID_MASK].frame_fl & + LIN_LOC_SLAVE_CACHE) + && (sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc > 0)) { - lin_data = cf->data; - lin_dlc = cf->can_dlc; + pr_debug("sllin: Sending LIN response from linfr_cache\n"); + lin_data = sl->linfr_cache[cf->can_id & LIN_ID_MASK].data; + lin_dlc = sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc; if (lin_dlc > SLLIN_DATA_MAX) lin_dlc = SLLIN_DATA_MAX; - tx_bytes = lin_dlc; + memcpy(lin_data_buff, lin_data, lin_dlc); + lin_data = lin_data_buff; + } else { + lin_data = NULL; + lin_dlc = sl->linfr_cache[cf->can_id & LIN_ID_MASK].dlc; } + spin_unlock(&sl->lock); + } else { /* SFF NON-RTR CAN frame -> LIN header + LIN response */ + pr_debug("sllin: %s: NON-RTR SFF CAN frame, ID = %x\n", + __FUNCTION__, (int)cf->can_id & LIN_ID_MASK); + + lin_data = cf->data; + lin_dlc = cf->can_dlc; + if (lin_dlc > SLLIN_DATA_MAX) + lin_dlc = SLLIN_DATA_MAX; + tx_bytes = lin_dlc; + } - if (sllin_setup_msg(sl, 0, cf->can_id & LIN_ID_MASK, - lin_data, lin_dlc) != -1) { + if (sllin_setup_msg(sl, 0, cf->can_id & LIN_ID_MASK, + lin_data, lin_dlc) != -1) { - sl->id_to_send = true; - sl->data_to_send = (lin_data != NULL) ? true : false; - sl->resp_len_known = (lin_dlc > 0) ? true : false; - sl->dev->stats.tx_packets++; - sl->dev->stats.tx_bytes += tx_bytes; - } + sl->id_to_send = true; + sl->data_to_send = (lin_data != NULL) ? true : false; + sl->resp_len_known = (lin_dlc > 0) ? true : false; + sl->dev->stats.tx_packets++; + sl->dev->stats.tx_bytes += tx_bytes; + } - clear_bit(SLF_MSGEVENT, &sl->flags); - kfree_skb(sl->tx_req_skb); - netif_wake_queue(sl->dev); - hrtimer_start(&sl->rx_timer, - ktime_add(ktime_get(), sl->rx_timer_timeout), - HRTIMER_MODE_ABS); - break; + clear_bit(SLF_MSGEVENT, &sl->flags); + kfree_skb(sl->tx_req_skb); + netif_wake_queue(sl->dev); + hrtimer_start(&sl->rx_timer, + ktime_add(ktime_get(), sl->rx_timer_timeout), + HRTIMER_MODE_ABS); + break; - case SLSTATE_BREAK_SENT: + case SLSTATE_BREAK_SENT: #ifdef BREAK_BY_BAUD - if (sl->rx_cnt <= SLLIN_BUFF_BREAK) - continue; + if (sl->rx_cnt <= SLLIN_BUFF_BREAK) + continue; - res = sltty_change_speed(tty, sl->lin_baud); + res = sltty_change_speed(tty, sl->lin_baud); #endif - sl->lin_state = SLSTATE_ID_SENT; - sllin_send_tx_buff(sl); - break; + sl->lin_state = SLSTATE_ID_SENT; + sllin_send_tx_buff(sl); + break; - case SLSTATE_ID_SENT: - hrtimer_cancel(&sl->rx_timer); - sl->id_to_send = false; - if (sl->data_to_send) { - sllin_send_tx_buff(sl); - sl->lin_state = SLSTATE_RESPONSE_SENT; - sl->rx_expect = sl->tx_lim; - goto slstate_response_sent; + case SLSTATE_ID_SENT: + hrtimer_cancel(&sl->rx_timer); + sl->id_to_send = false; + if (sl->data_to_send) { + sllin_send_tx_buff(sl); + sl->lin_state = SLSTATE_RESPONSE_SENT; + sl->rx_expect = sl->tx_lim; + goto slstate_response_sent; + } else { + if (sl->resp_len_known) { + sl->rx_expect = sl->rx_lim; } else { - if (sl->resp_len_known) { - sl->rx_expect = sl->rx_lim; - } else { - sl->rx_expect = SLLIN_BUFF_DATA + 2; - } - sl->lin_state = SLSTATE_RESPONSE_WAIT; - /* If we don't receive anything, timer will "unblock" us */ - hrtimer_start(&sl->rx_timer, - ktime_add(ktime_get(), sl->rx_timer_timeout), - HRTIMER_MODE_ABS); - goto slstate_response_wait; + sl->rx_expect = SLLIN_BUFF_DATA + 2; } - break; + sl->lin_state = SLSTATE_RESPONSE_WAIT; + /* If we don't receive anything, timer will "unblock" us */ + hrtimer_start(&sl->rx_timer, + ktime_add(ktime_get(), sl->rx_timer_timeout), + HRTIMER_MODE_ABS); + goto slstate_response_wait; + } + break; - case SLSTATE_RESPONSE_WAIT: - slstate_response_wait: - if (test_bit(SLF_MSGEVENT, &sl->flags)) { - unsigned char *lin_buff; - cf = (struct can_frame *)sl->tx_req_skb->data; - - lin_buff = (sl->lin_master) ? sl->tx_buff : sl->rx_buff; - if (cf->can_id == (lin_buff[SLLIN_BUFF_ID] & LIN_ID_MASK)) { - hrtimer_cancel(&sl->rx_timer); - pr_debug("sllin: received LIN response in a CAN frame.\n"); - if (sllin_setup_msg(sl, SLLIN_STPMSG_RESPONLY, - cf->can_id & LIN_ID_MASK, - cf->data, cf->can_dlc) != -1) { - - sl->rx_expect = sl->tx_lim; - sl->data_to_send = true; - sl->dev->stats.tx_packets++; - sl->dev->stats.tx_bytes += tx_bytes; - - if (!sl->lin_master) { - sl->tx_cnt = SLLIN_BUFF_DATA; - } - - sllin_send_tx_buff(sl); - clear_bit(SLF_MSGEVENT, &sl->flags); - kfree_skb(sl->tx_req_skb); - netif_wake_queue(sl->dev); - - sl->lin_state = SLSTATE_RESPONSE_SENT; - goto slstate_response_sent; - } - } else { - sl->lin_state = SLSTATE_RESPONSE_WAIT_BUS; - } - } + case SLSTATE_RESPONSE_WAIT: +slstate_response_wait: + if (test_bit(SLF_MSGEVENT, &sl->flags)) { + unsigned char *lin_buff; + cf = (struct can_frame *)sl->tx_req_skb->data; - /* Be aware, no BREAK here */ - case SLSTATE_RESPONSE_WAIT_BUS: - if (sl->rx_cnt < sl->rx_expect) - continue; + lin_buff = (sl->lin_master) ? sl->tx_buff : sl->rx_buff; + if (cf->can_id == (lin_buff[SLLIN_BUFF_ID] & LIN_ID_MASK)) { + hrtimer_cancel(&sl->rx_timer); + pr_debug("sllin: received LIN response in a CAN frame.\n"); + if (sllin_setup_msg(sl, SLLIN_STPMSG_RESPONLY, + cf->can_id & LIN_ID_MASK, + cf->data, cf->can_dlc) != -1) { + + sl->rx_expect = sl->tx_lim; + sl->data_to_send = true; + sl->dev->stats.tx_packets++; + sl->dev->stats.tx_bytes += tx_bytes; + + if (!sl->lin_master) { + sl->tx_cnt = SLLIN_BUFF_DATA; + } - hrtimer_cancel(&sl->rx_timer); - pr_debug("sllin: response received ID %d len %d\n", - sl->rx_buff[SLLIN_BUFF_ID], sl->rx_cnt - SLLIN_BUFF_DATA - 1); + sllin_send_tx_buff(sl); + clear_bit(SLF_MSGEVENT, &sl->flags); + kfree_skb(sl->tx_req_skb); + netif_wake_queue(sl->dev); - if (sllin_rx_validate(sl) == -1) { - pr_debug("sllin: RX validation failed.\n"); - sllin_report_error(sl, LIN_ERR_CHECKSUM); + sl->lin_state = SLSTATE_RESPONSE_SENT; + goto slstate_response_sent; + } } else { - /* Send CAN non-RTR frame with data */ - pr_debug("sllin: sending NON-RTR CAN" - "frame with LIN payload."); - sll_bump(sl); /* send packet to the network layer */ + sl->lin_state = SLSTATE_RESPONSE_WAIT_BUS; } + } - sl->id_to_send = false; - sl->lin_state = SLSTATE_IDLE; - break; - - case SLSTATE_RESPONSE_SENT: - slstate_response_sent: - if (sl->rx_cnt < sl->tx_lim) - continue; + /* Be aware, no BREAK here */ + case SLSTATE_RESPONSE_WAIT_BUS: + if (sl->rx_cnt < sl->rx_expect) + continue; - hrtimer_cancel(&sl->rx_timer); + hrtimer_cancel(&sl->rx_timer); + pr_debug("sllin: response received ID %d len %d\n", + sl->rx_buff[SLLIN_BUFF_ID], sl->rx_cnt - SLLIN_BUFF_DATA - 1); + + if (sllin_rx_validate(sl) == -1) { + pr_debug("sllin: RX validation failed.\n"); + sllin_report_error(sl, LIN_ERR_CHECKSUM); + } else { + /* Send CAN non-RTR frame with data */ + pr_debug("sllin: sending NON-RTR CAN" + "frame with LIN payload."); sll_bump(sl); /* send packet to the network layer */ - pr_debug("sllin: response sent ID %d len %d\n", - sl->rx_buff[SLLIN_BUFF_ID], sl->rx_cnt - SLLIN_BUFF_DATA - 1); + } - sl->id_to_send = false; - sl->lin_state = SLSTATE_IDLE; - break; + sl->id_to_send = false; + sl->lin_state = SLSTATE_IDLE; + break; + + case SLSTATE_RESPONSE_SENT: +slstate_response_sent: + if (sl->rx_cnt < sl->tx_lim) + continue; + + hrtimer_cancel(&sl->rx_timer); + sll_bump(sl); /* send packet to the network layer */ + pr_debug("sllin: response sent ID %d len %d\n", + sl->rx_buff[SLLIN_BUFF_ID], sl->rx_cnt - SLLIN_BUFF_DATA - 1); + + sl->id_to_send = false; + sl->lin_state = SLSTATE_IDLE; + break; } }