From: Romain Forlot Date: Tue, 16 May 2017 10:41:10 +0000 (+0200) Subject: Fix: listening on broadcast CAN ID. X-Git-Tag: 5.0.2~223 X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=commitdiff_plain;h=046af0b8441ff890f644da722507bc77cd3270f7;p=apps%2Fagl-service-can-low-level.git Fix: listening on broadcast CAN ID. Wrong test about CAN ID, it isn't the response ID that is passed as argument but the broadcast. Little mistake... Change-Id: I2622c23356802c3108dbf8e6ede07456294834ee Signed-off-by: Romain Forlot --- diff --git a/CAN-binder/low-can-binding/diagnostic/diagnostic-manager.cpp b/CAN-binder/low-can-binding/diagnostic/diagnostic-manager.cpp index fb62dfb0..75396342 100644 --- a/CAN-binder/low-can-binding/diagnostic/diagnostic-manager.cpp +++ b/CAN-binder/low-can-binding/diagnostic/diagnostic-manager.cpp @@ -109,7 +109,7 @@ int diagnostic_manager_t::add_rx_filter(uint32_t can_id) bcm_msg.msg_head.ival2.tv_usec = freq.tv_usec; // If it isn't an OBD2 CAN ID then just add a simple RX_SETUP job - if(can_id != OBD2_FUNCTIONAL_RESPONSE_START) + if(can_id != OBD2_FUNCTIONAL_BROADCAST_ID) { bcm_msg.msg_head.can_id = can_id;