X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=low-can-binding%2Fbinding%2Flow-can-subscription.cpp;h=3cbcb6c93de25c25f7b8f4f8521baf77a09cf93a;hb=24bb25c9f16ecace05b64142493dce596f5a9e37;hp=05367b79c56ee6050f76990ebcf73418cd0f2d0b;hpb=779da0a54b29db5dd4c12387970026a264d2e6cb;p=apps%2Fagl-service-can-low-level.git

diff --git a/low-can-binding/binding/low-can-subscription.cpp b/low-can-binding/binding/low-can-subscription.cpp
index 05367b79..3cbcb6c9 100644
--- a/low-can-binding/binding/low-can-subscription.cpp
+++ b/low-can-binding/binding/low-can-subscription.cpp
@@ -370,7 +370,7 @@ int low_can_subscription_t::open_socket(low_can_subscription_t &subscription, co
 			if( subscription.signal_ )
 				ret = subscription.socket_->open(subscription.signal_->get_message()->get_bus_device_name());
 			else if(! subscription.diagnostic_message_.empty())
-				ret = subscription.socket_->open(application_t::instance().get_diagnostic_manager().get_bus_name());
+				ret = subscription.socket_->open(application_t::instance().get_diagnostic_manager().get_bus_device_name());
 			else if(! bus_name.empty())
 				ret = subscription.socket_->open(bus_name);
 
@@ -648,7 +648,6 @@ int low_can_subscription_t::create_rx_filter(std::shared_ptr<diagnostic_message_
 	diagnostic_message_.push_back(sig);
 
 	struct timeval freq = frequency_clock_t(event_filter_.frequency).get_timeval_from_period();
-	//struct timeval timeout = frequency_clock_t(10).get_timeval_from_period();
 	struct timeval timeout = {0, 0};
 
 	struct bcm_msg bcm_msg =  make_bcm_head(RX_SETUP, OBD2_FUNCTIONAL_BROADCAST_ID, SETTIMER | RX_NO_AUTOTIMER | RX_FILTER_ID, timeout, freq);
@@ -669,13 +668,11 @@ int low_can_subscription_t::create_rx_filter_bcm(low_can_subscription_t &subscri
 	if(subscription.open_socket(subscription, "", CAN_PROTOCOL) < 0)
 		return -1;
 
-	// If it's not an OBD2 CAN ID then just add a simple RX_SETUP job
-	// else monitor all standard 8 CAN OBD2 ID response.
-
 	can_message_t msg = can_message_t();
-
 	msg.set_bcm_msg(bcm_msg);
 
+	// If it's not an OBD2 CAN ID then just add a simple RX_SETUP job
+	// else monitor all standard 8 CAN OBD2 ID response.
 	if(bcm_msg.msg_head.can_id != OBD2_FUNCTIONAL_BROADCAST_ID)
 	{
 		subscription.socket_->write_message(msg);
@@ -714,8 +711,6 @@ int low_can_subscription_t::tx_send(low_can_subscription_t &subscription, messag
 
 	struct bcm_msg &bcm_cm = cm->get_bcm_msg();
 
-
-
 	if(cfd_vect.size() > 1)
 	{
 		AFB_ERROR("Multi frame BCM not implemented");