Cellular: Moved reset from power to device

Author: Ari Parkkila

UNITTESTS/features/cellular/framework/AT/at_cellulardevice/at_cellulardevicetest.cpp

@@ -239,6 +239,12 @@ TEST_F(TestAT_CellularDevice, test_AT_CellularDevice_init)
     EXPECT_EQ(dev.init(), NSAPI_ERROR_OK);
 }
 
+TEST_F(TestAT_CellularDevice, test_AT_CellularDevice_reset)
+{
+    FileHandle_stub fh1;
+    AT_CellularDevice dev(&fh1);
+    EXPECT_EQ(dev.reset(), NSAPI_ERROR_OK);
+}
 
 TEST_F(TestAT_CellularDevice, test_AT_CellularDevice_is_ready)
 {

UNITTESTS/features/cellular/framework/AT/at_cellularpower/at_cellularpowertest.cpp

@@ -74,17 +74,3 @@ TEST_F(TestAT_CellularPower, test_AT_CellularPower_off)
     AT_CellularPower pow(at);
     EXPECT_TRUE(NSAPI_ERROR_UNSUPPORTED == pow.off());
 }
-
-TEST_F(TestAT_CellularPower, test_AT_CellularPower_reset)
-{
-    EventQueue que;
-    FileHandle_stub fh1;
-    ATHandler at(&fh1, que, 0, ",");
-
-    AT_CellularPower pow(at);
-    ATHandler_stub::nsapi_error_value = NSAPI_ERROR_OK;
-    EXPECT_TRUE(NSAPI_ERROR_OK == pow.reset());
-
-    ATHandler_stub::nsapi_error_value = NSAPI_ERROR_DEVICE_ERROR;
-    EXPECT_TRUE(NSAPI_ERROR_DEVICE_ERROR == pow.reset());
-}

UNITTESTS/stubs/AT_CellularDevice_stub.cpp

@@ -171,6 +171,11 @@ nsapi_error_t AT_CellularDevice::init()
     return NSAPI_ERROR_OK;
 }
 
+nsapi_error_t AT_CellularDevice::reset()
+{
+    return NSAPI_ERROR_OK;
+}
+
 nsapi_error_t AT_CellularDevice::set_pin(const char *sim_pin)
 {
     return NSAPI_ERROR_OK;

UNITTESTS/stubs/AT_CellularPower_stub.cpp

@@ -39,8 +39,3 @@ nsapi_error_t AT_CellularPower::off()
 {
     return NSAPI_ERROR_OK;
 }
-
-nsapi_error_t AT_CellularPower::reset()
-{
-    return NSAPI_ERROR_OK;
-}

UNITTESTS/target_h/myCellularDevice.h

@@ -114,6 +114,11 @@ class myCellularDevice : public CellularDevice {
         return NSAPI_ERROR_OK;
     }
 
+    virtual nsapi_error_t reset()
+    {
+        return NSAPI_ERROR_OK;
+    }
+
     virtual nsapi_error_t is_ready()
     {
         return NSAPI_ERROR_OK;

features/cellular/TESTS/api/cellular_power/main.cpp

@@ -81,10 +81,6 @@ static void test_power_interface()
     TEST_ASSERT(err == NSAPI_ERROR_OK || err == NSAPI_ERROR_UNSUPPORTED);
     wait_for_power(pwr);
 
-    err = pwr->reset();
-    TEST_ASSERT(err == NSAPI_ERROR_OK);
-    wait_for_power(pwr);
-
     err = pwr->off();
     TEST_ASSERT(err == NSAPI_ERROR_OK || err == NSAPI_ERROR_UNSUPPORTED);
 

features/cellular/framework/API/CellularDevice.h

@@ -262,6 +262,13 @@ class CellularDevice {
      */
     virtual nsapi_error_t init() = 0;
 
+    /** Reset and wake-up cellular device.
+     *
+     *  @return         NSAPI_ERROR_OK on success
+     *                  NSAPI_ERROR_DEVICE_ERROR on failure
+     */
+    virtual nsapi_error_t reset() = 0;
+
     /** Check whether the device is ready to accept commands.
      *
      *  @return         NSAPI_ERROR_OK on success

features/cellular/framework/API/CellularPower.h

@@ -57,13 +57,6 @@ class CellularPower {
      *                  NSAPI_ERROR_UNSUPPORTED if not overridden by the target modem
      */
     virtual nsapi_error_t off() = 0;
-
-    /** Reset and wake-up cellular device.
-     *
-     *  @return         NSAPI_ERROR_OK on success
-     *                  NSAPI_ERROR_DEVICE_ERROR on failure
-     */
-    virtual nsapi_error_t reset() = 0;
 };
 
 } // namespace mbed

features/cellular/framework/AT/AT_CellularDevice.cpp

@@ -378,6 +378,17 @@ nsapi_error_t AT_CellularDevice::init()
     return _at->unlock_return_error();
 }
 
+nsapi_error_t AT_CellularDevice::reset()
+{
+    _at->lock();
+    if (_state_machine) {
+        _state_machine->reset();
+    }
+    _at->cmd_start("AT+CFUN=1,1");// reset to full power levels
+    _at->cmd_stop_read_resp();
+    return _at->unlock_return_error();
+}
+
 nsapi_error_t AT_CellularDevice::is_ready()
 {
     _at->lock();

features/cellular/framework/AT/AT_CellularDevice.h

@@ -72,6 +72,8 @@ class AT_CellularDevice : public CellularDevice {
 
     virtual nsapi_error_t init();
 
+    virtual nsapi_error_t reset();
+
     virtual nsapi_error_t is_ready();
 
     virtual nsapi_error_t set_ready_cb(Callback<void()> callback);