[modbus] clean logs

pyheatpump/modbus.py

@@ -46,9 +46,7 @@ def read_coils(start, end):
     address = -1
     qty = -1
 
-    logger.info('Read coils [{}, {}]'.format(
-        start, end
-    ))
+    logger.info('read_coils: [%s, %s]', start, end)
     try:
         for address in range(start, end + 1, 125):
             qty = 125 if (end - address) >= 125 else (end - address)
@@ -66,6 +64,7 @@ def read_coils(start, end):
         print(e)
         print(f'{address} {qty}')
 
+    logger.debug('read_coils [%s, %s] result: %s', start, end, res)
     return res
 
 
@@ -78,9 +77,9 @@ def read_holding_registers(start, end):
     address = -1
     qty = -1
 
-    logger.info('Read registers [{}, {}]'.format(
+    logger.debug('read_holding_registers [%s, %s]',
         start, end
-    ))
+    )
 
     try:
         for address in range(start, end + 1, 125):
@@ -99,15 +98,16 @@ def read_holding_registers(start, end):
         print(e)
         print(f'{address} {qty}')
 
+    logger.debug('read_holding_registers [%s, %s] result: %s', start, end, res)
     return res
 
 def write_coil(var_value):
     global serial_conn
     connect()
 
-    logger.info('Write coil at address {} with value {}]'.format(
+    logger.debug('write_coil address: %s, value: %s',
         var_value.address, var_value.value
-    ))
+    )
 
     try:
         req_adu = rtu.write_single_coil(
@@ -115,7 +115,10 @@ def write_coil(var_value):
             address=var_value.address,
             value=var_value.value)
         resp = rtu.send_message(req_adu, serial_conn)
-        logger.info('Modbus response : {}'.format(resp))
+
+        logger.debug('write_coil address: %s, response: %s',
+                var_value.address, resp)
+
         if resp != var_value.value:
             return False
         return True
@@ -127,9 +130,9 @@ def write_holding_register(var_value):
     global serial_conn
     connect()
 
-    logger.info('Write register at address {} with value {}]'.format(
+    logger.debug('write_holding_register address: %s, value: %s',
         var_value.address, var_value.value
-    ))
+    )
 
     try:
         req_adu = rtu.write_single_register(
@@ -137,7 +140,8 @@ def write_holding_register(var_value):
             address=var_value.address,
             value=var_value.value)
         resp = rtu.send_message(req_adu, serial_conn)
-        logger.info('Modbus response : {}'.format(resp))
+        logger.debug('write_holding_register, addres: %s, response: %s',
+                var_value.address, resp)
         if resp != var_value.value:
             return False
         return True