Merge pull request #12211 from hugueskamba/hk-replace-uartserial-cellular

Cellular: Replace UARTSerial references with BufferedSerial

Author: 0xc0170

UNITTESTS/features/cellular/framework/AT/at_cellularcontext/at_cellularcontexttest.cpp

@@ -531,7 +531,7 @@ TEST_F(TestAT_CellularContext, set_file_handle)
     AT_CellularContext ctx(at, &dev);
     ctx.set_file_handle(&fh1);
 
-    UARTSerial ss(NC, NC);
+    BufferedSerial ss(NC, NC);
 
     ctx.set_file_handle(&ss, PTC0, true);
     ctx.enable_hup(true);

UNITTESTS/features/cellular/framework/AT/at_cellularcontext/unittest.cmake

@@ -37,7 +37,7 @@ set(unittest-test-sources
   stubs/randLIB_stub.cpp
   stubs/Semaphore_stub.cpp
   stubs/us_ticker_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/CellularContext_stub.cpp
   stubs/CellularUtil_stub.cpp

UNITTESTS/features/cellular/framework/AT/at_cellulardevice/unittest.cmake

@@ -38,7 +38,7 @@ set(unittest-test-sources
   stubs/NetworkStack_stub.cpp
   stubs/AT_CellularContext_stub.cpp
   stubs/Semaphore_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/CellularStateMachine_stub.cpp
   stubs/CellularContext_stub.cpp

UNITTESTS/features/cellular/framework/device/athandler/unittest.cmake

@@ -24,7 +24,7 @@ set(unittest-test-sources
   stubs/EventQueue_stub.cpp
   stubs/FileHandle_stub.cpp
   stubs/us_ticker_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/mbed_assert_stub.cpp
   stubs/mbed_poll_stub.cpp

UNITTESTS/features/cellular/framework/device/cellularcontext/cellularcontexttest.cpp

@@ -165,7 +165,7 @@ class testContext : public CellularContext
 
 	}
 #if (DEVICE_SERIAL && DEVICE_INTERRUPTIN) || defined(DOXYGEN_ONLY)
-    virtual void set_file_handle(UARTSerial *serial, PinName dcd_pin = NC, bool active_high = false)
+    virtual void set_file_handle(BufferedSerial *serial, PinName dcd_pin = NC, bool active_high = false)
     {
 
     }

UNITTESTS/features/cellular/framework/device/cellularcontext/unittest.cmake

@@ -29,7 +29,7 @@ set(unittest-test-sources
   stubs/CellularStateMachine_stub.cpp
   stubs/EventQueue_stub.cpp
   stubs/mbed_assert_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/ATHandler_stub.cpp
   stubs/AT_CellularNetwork_stub.cpp

UNITTESTS/features/cellular/framework/device/cellulardevice/unittest.cmake

@@ -23,7 +23,7 @@ set(unittest-test-sources
   stubs/CellularStateMachine_stub.cpp
   stubs/EventQueue_stub.cpp
   stubs/mbed_assert_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/ATHandler_stub.cpp
   stubs/AT_CellularNetwork_stub.cpp

UNITTESTS/features/cellular/framework/device/cellularstatemachine/unittest.cmake

@@ -23,7 +23,7 @@ set(unittest-test-sources
   stubs/CellularDevice_stub.cpp
   stubs/EventQueue_stub.cpp
   stubs/mbed_assert_stub.cpp
-  stubs/UARTSerial_stub.cpp
+  stubs/BufferedSerial_stub.cpp
   stubs/SerialBase_stub.cpp
   stubs/ATHandler_stub.cpp
   stubs/AT_CellularNetwork_stub.cpp

UNITTESTS/stubs/AT_CellularContext_stub.cpp

@@ -51,7 +51,7 @@ AT_CellularContext::~AT_CellularContext()
 {
 }
 
-void AT_CellularContext::set_file_handle(UARTSerial *serial, PinName dcd_pin, bool active_high)
+void AT_CellularContext::set_file_handle(BufferedSerial *serial, PinName dcd_pin, bool active_high)
 {
 }
 

UNITTESTS/stubs/AT_CellularDevice_stub.cpp

@@ -65,7 +65,7 @@ nsapi_error_t AT_CellularDevice::release_at_handler(ATHandler *at_handler)
     }
 }
 
-CellularContext *AT_CellularDevice::create_context(UARTSerial *serial, const char *const apn, PinName dcd_pin,
+CellularContext *AT_CellularDevice::create_context(BufferedSerial *serial, const char *const apn, PinName dcd_pin,
                                                    bool active_high, bool cp_req, bool nonip_req)
 {
 }

UNITTESTS/stubs/BufferedSerial_stub.cpp

@@ -0,0 +1,132 @@
+/*
+ * Copyright (c) 2018, Arm Limited and affiliates.
+ * SPDX-License-Identifier: Apache-2.0
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include "drivers/BufferedSerial.h"
+
+namespace mbed {
+
+
+BufferedSerial::BufferedSerial(PinName tx, PinName rx, int baud) : SerialBase(tx, rx, baud)
+{
+}
+
+BufferedSerial::~BufferedSerial()
+{
+}
+
+ssize_t BufferedSerial::read(void *buffer, size_t length)
+{
+    return 0;
+}
+
+ssize_t BufferedSerial::write(const void *buffer, size_t length)
+{
+    return 0;
+}
+
+off_t BufferedSerial::seek(off_t offset, int whence)
+{
+    return -ESPIPE;
+}
+
+int BufferedSerial::close()
+{
+    return 0;
+}
+
+void BufferedSerial::dcd_irq()
+{
+}
+
+void BufferedSerial::set_baud(int baud)
+{
+}
+
+void BufferedSerial::set_data_carrier_detect(PinName dcd_pin, bool active_high)
+{
+}
+
+void BufferedSerial::set_format(int bits, Parity parity, int stop_bits)
+{
+}
+
+int BufferedSerial::isatty()
+{
+    return 1;
+}
+
+int BufferedSerial::sync()
+{
+    return 0;
+}
+
+void BufferedSerial::sigio(Callback<void()> func)
+{
+}
+
+ssize_t BufferedSerial::write_unbuffered(const char *buf_ptr, size_t length)
+{
+    return 0;
+}
+
+bool BufferedSerial::hup() const
+{
+}
+
+void BufferedSerial::wake()
+{
+}
+
+short BufferedSerial::poll(short events) const
+{
+    return 0;
+}
+
+void BufferedSerial::api_lock(void)
+{
+}
+
+void BufferedSerial::api_unlock(void)
+{
+}
+
+void BufferedSerial::rx_irq(void)
+{
+}
+
+void BufferedSerial::tx_irq(void)
+{
+}
+
+int BufferedSerial::enable_input(bool enabled)
+{
+    return 0;
+}
+
+int BufferedSerial::enable_output(bool enabled)
+{
+    return 0;
+}
+
+#if DEVICE_SERIAL_FC
+void BufferedSerial::set_flow_control(mbed::SerialBase::Flow, PinName, PinName)
+{
+
+}
+#endif
+
+}

UNITTESTS/stubs/UARTSerial_stub.cpp

@@ -39,7 +39,7 @@ class myCellularContext : public CellularContext {
         }
     }
 
-    void set_file_handle(UARTSerial *serial, PinName dcd_pin, bool active_high)
+    void set_file_handle(BufferedSerial *serial, PinName dcd_pin, bool active_high)
     {
     };
 

UNITTESTS/target_h/myCellularContext.h

@@ -23,7 +23,7 @@
 #include "ATHandler_stub.h"
 #include "AT_CellularContext.h"
 #include "gtest/gtest.h"
-#include "UARTSerial.h"
+#include "drivers/BufferedSerial.h"
 
 using namespace events;
 
@@ -53,7 +53,7 @@ class myCellularDevice : public AT_CellularDevice {
         return NSAPI_ERROR_OK;
     }
 
-    virtual CellularContext *create_context(UARTSerial *serial, const char *const apn, PinName dcd_pin,
+    virtual CellularContext *create_context(BufferedSerial *serial, const char *const apn, PinName dcd_pin,
                                             bool active_high, bool cp_req = false, bool nonip_req = false)
     {
         if (_context_list) {