Tidying up the boiler plate code

A little tidying up was needed as there was some boiler plate code that
could be shoved into a corner and reused.

Author: Hasnain Virk

features/cellular/framework/targets/QUECTEL/EC2X/QUECTEL_EC2X.cpp

@@ -66,29 +66,27 @@ CellularDevice *CellularDevice::get_default_instance()
     return &device;
 }
 
-nsapi_error_t QUECTEL_EC2X::hard_power_on()
+nsapi_error_t QUECTEL_EC2X::press_power_button(uint32_t timeout)
 {
     if (_pwr_key.is_connected()) {
         _pwr_key = 1;
-        ThisThread::sleep_for(600);
+        ThisThread::sleep_for(timeout);
         _pwr_key = 0;
         ThisThread::sleep_for(100);
     }
 
     return NSAPI_ERROR_OK;
 }
 
+nsapi_error_t QUECTEL_EC2X::hard_power_on()
+{
+    return press_power_button(600);
+}
+
 nsapi_error_t QUECTEL_EC2X::hard_power_off()
 
 {
-    if (_pwr_key.is_connected()) {
-        _pwr_key = 1;
-        ThisThread::sleep_for(750);
-        _pwr_key = 0;
-        ThisThread::sleep_for(100);
-    }
-
-    return NSAPI_ERROR_OK;
+    return press_power_button(750);
 }
 
 nsapi_error_t QUECTEL_EC2X::soft_power_on()
@@ -119,14 +117,7 @@ nsapi_error_t QUECTEL_EC2X::soft_power_on()
 
 nsapi_error_t QUECTEL_EC2X::soft_power_off()
 {
-    if (_pwr_key.is_connected()) {
-        _pwr_key = 1;
-        ThisThread::sleep_for(750);
-        _pwr_key = 0;
-        ThisThread::sleep_for(100);
-    }
-
-    return NSAPI_ERROR_OK;
+    return hard_power_off();
 }
 
 #endif

features/cellular/framework/targets/QUECTEL/EC2X/QUECTEL_EC2X.h

@@ -41,6 +41,7 @@ class QUECTEL_EC2X : public AT_CellularDevice {
     virtual nsapi_error_t soft_power_off();
 
 private:
+    nsapi_error_t press_power_button(uint32_t timeout);
     DigitalOut _pwr_key;
     DigitalOut _rst;
 };