ipmi plugin: Remove duplicate/unused code and unneeded brackets.
authorMariusz Szafranski <mariuszx.szafranski@intel.com>
Mon, 5 Mar 2018 15:04:10 +0000 (15:04 +0000)
committerMariusz Szafranski <mariuszx.szafranski@intel.com>
Tue, 6 Mar 2018 09:29:27 +0000 (09:29 +0000)
Change-Id: Ia4298f75f59e697df25d05091adbb556b64869e1
Signed-off-by: Mariusz Szafranski <mariuszx.szafranski@intel.com>
src/ipmi.c

index a880586..59710c3 100644 (file)
@@ -387,14 +387,6 @@ static int sensor_list_add(c_ipmi_instance_t *st, ipmi_sensor_t *sensor) {
         ipmi_sensor_get_sensor_direction(sensor),
         ipmi_sensor_get_event_support(sensor));
 
-  /* Both `ignorelist' and `sensor_name_ptr' may be NULL. */
-  if (ignorelist_match(st->ignorelist, sensor_name_ptr) != 0) {
-    INFO("ipmi plugin: sensor_list_add: Ignore sensor `%s` of `%s`, "
-         "because it is on ignore list.",
-         sensor_name_ptr, st->name);
-    return -1;
-  }
-
   /* FIXME: Use rate unit or base unit to scale the value */
 
   sensor_type = ipmi_sensor_get_sensor_type(sensor);
@@ -804,10 +796,9 @@ entity_sensor_update_handler(enum ipmi_update_e op,
         status = ipmi_sensor_add_discrete_event_handler(
             sensor, sensor_discrete_event_handler, st);
 
-      if (status) {
+      if (status)
         ERROR("Unable to add sensor %s event handler, status: %d", sensor_name,
               status);
-      }
     }
   } else if (op == IPMI_DELETED) {
     sensor_list_remove(st, sensor);