Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cellular refactor power and get_default_instance, also remove deprecated #9273

Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
573b6ce
Cellular: Fixed CellularStateMachine to stop on all target states.
Nov 28, 2018
bc8515b
Cellular: Removed get_extended_signal_quality and changed get_signal_…
Nov 30, 2018
176067b
Removed CellularSIM interface.
Nov 28, 2018
2a9bedd
cellular: eps ciot optimization network support check
Nov 9, 2018
ec7d13e
cellular: eps ciot optimization network support check astyle fix
Nov 12, 2018
f52c9e3
Merge pull request #8891 from jarvte/remove_sim_interface
Dec 5, 2018
60fe3c5
Merge pull request #8979 from TeemuKultala/cellular_ciot_eps_on_featu…
Dec 7, 2018
1f744ef
Cellular: Move eDRX setting from power to network
Nov 27, 2018
30bd353
Cellular: Move PSM setting from power to device
Nov 28, 2018
ef2a091
Cellular: Move device_ready from power to device
Nov 28, 2018
6ec5f79
Cellular: AT remove_urc_handler changed to set_urc_handler(prefix, 0)
Nov 28, 2018
7f7ffbe
Cellular: Move ready_cb from power to device
Nov 28, 2018
09e2a7e
Cellular: Unify set_at_mode and init_module into init()
Nov 29, 2018
5b2dc38
Cellular: Removed set_power_level()
Nov 29, 2018
65da29d
Cellular: Moved reset from power to device
Nov 29, 2018
d09ee0d
Cellular: Added shutdown()
Nov 29, 2018
0f1e08f
Cellular: changed support features to CellularProperty array.
Dec 5, 2018
a57bacf
Merge pull request #8901 from AriParkkila/cell-refactor-power-api
Dec 12, 2018
c4138f4
Merge pull request #9050 from jarvte/support_feature_to_key_value
Dec 12, 2018
22d0d90
Cellular: Removed unnecessary checks after new
Dec 13, 2018
104700c
Merge pull request #9090 from jarvte/remove_new_null_checks
Dec 13, 2018
f76b21c
Cellular: Change AT_CellularNetwork to use CellularProperties.
Dec 11, 2018
e30c20b
Cellular: Remove target files inheriting from AT_CellularNetwork
Dec 12, 2018
40db17e
Cellular: change stack_type_supported to get_property
Dec 13, 2018
9e51217
Cellular: Added generic cellular modem
Dec 17, 2018
a743b4a
Cellular: fix possible crash in state machine
Dec 3, 2018
e107599
Cellular: Add State machine unit tests.
Dec 3, 2018
f1691e4
Merge pull request #9120 from jarvte/create_generic_modem
0xc0170 Dec 18, 2018
e4956ce
Cellular: added setting of data carrier support for UART.
Dec 4, 2018
a7f4388
Merge pull request #8971 from jarvte/add_datacarrier_detect
0xc0170 Dec 19, 2018
8129f78
Cellular: Non-IP socket and PDP context for EPS control plane data de…
Nov 16, 2018
2af12ae
Merge pull request #9121 from mirelachirica/feature-cellular-refactor…
Dec 20, 2018
e04161c
Cellular: Move power on/off to device and remove CellularPower
Dec 18, 2018
d852c88
Cellular: Remove netsocket/generic_modem_driver
Dec 19, 2018
248f193
Cellular: Add get_target_default_instance in CellularDevice
Dec 20, 2018
d3b4caf
Cellular: Default modem drivers with FF_ARDUINO
Jan 15, 2019
84216a5
Cellular: Power API updated to match onboard_modem_api
Jan 15, 2019
a42aa44
Cellular: Update API description to match better onboard_modem_api
Jan 15, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
#include "gtest/gtest.h"
#include "AT_CellularBase.h"
#include "EventQueue.h"
#include "AT_CellularBase.h"
#include "AT_CellularNetwork.h"
#include "ATHandler_stub.h"
#include "FileHandle_stub.h"
#include <string.h>
Expand All @@ -33,28 +33,25 @@ class my_base : public AT_CellularBase {
}
bool check_not_supported()
{
static const AT_CellularBase::SupportedFeature unsupported_features[] = {
AT_CellularBase::AT_CGSN_WITH_TYPE,
AT_CellularBase::SUPPORTED_FEATURE_END_MARK
static const intptr_t cellular_properties[AT_CellularBase::PROPERTY_MAX] = {
AT_CellularNetwork::RegistrationModeDisable,// C_EREG
AT_CellularNetwork::RegistrationModeLAC, // C_GREG
AT_CellularNetwork::RegistrationModeLAC, // C_REG
0, // AT_CGSN_WITH_TYPE
1, // AT_CGDATA
1, // AT_CGAUTH
1, // PROPERTY_IPV4_STACK
0, // PROPERTY_IPV6_STACK
0, // PROPERTY_IPV4V6_STACK
};
set_unsupported_features(unsupported_features);
return is_supported(AT_CGSN_WITH_TYPE);
}

bool check_supported()
{
set_unsupported_features(NULL);
return is_supported(AT_CGSN_WITH_TYPE);
set_cellular_properties(cellular_properties);
return get_property(PROPERTY_AT_CGSN_WITH_TYPE);
}

bool check_supported_not_found()
bool check_supported()
{
static const AT_CellularBase::SupportedFeature unsupported_features[] = {
AT_CellularBase::AT_CGSN_WITH_TYPE,
AT_CellularBase::SUPPORTED_FEATURE_END_MARK
};
set_unsupported_features(unsupported_features);
return is_supported(SUPPORTED_FEATURE_END_MARK);
return get_property(PROPERTY_AT_CGDATA);
}
};

Expand Down Expand Up @@ -109,19 +106,25 @@ TEST_F(TestAT_CellularBase, test_AT_CellularBase_get_device_error)
ATHandler_stub::device_err_value.errCode = 0;
}

TEST_F(TestAT_CellularBase, test_AT_CellularBase_set_unsupported_features)
TEST_F(TestAT_CellularBase, test_AT_CellularBase_set_cellular_properties)
{
EventQueue eq;
FileHandle_stub fh;
ATHandler ah(&fh, eq, 0, ",");
AT_CellularBase at(ah);

static const AT_CellularBase::SupportedFeature unsupported_features[] = {
AT_CellularBase::AT_CGSN_WITH_TYPE,
AT_CellularBase::SUPPORTED_FEATURE_END_MARK
static const intptr_t cellular_properties[AT_CellularBase::PROPERTY_MAX] = {
AT_CellularNetwork::RegistrationModeDisable,// C_EREG
AT_CellularNetwork::RegistrationModeLAC, // C_GREG
AT_CellularNetwork::RegistrationModeLAC, // C_REG
0, // AT_CGSN_WITH_TYPE
1, // AT_CGDATA
1, // AT_CGAUTH
1, // PROPERTY_IPV4_STACK
0, // PROPERTY_IPV6_STACK
0, // PROPERTY_IPV4V6_STACK
};

at.set_unsupported_features(unsupported_features);
at.set_cellular_properties(cellular_properties);
}

TEST_F(TestAT_CellularBase, test_AT_CellularBase_is_supported)
Expand All @@ -131,7 +134,6 @@ TEST_F(TestAT_CellularBase, test_AT_CellularBase_is_supported)
ATHandler ah(&fh, eq, 0, ",");
my_base my_at(ah);

EXPECT_TRUE(true == my_at.check_supported());
EXPECT_TRUE(true == my_at.check_supported_not_found());
EXPECT_TRUE(false == my_at.check_not_supported());
EXPECT_EQ(true, my_at.check_supported());
EXPECT_EQ(false, my_at.check_not_supported());
}
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
#include "Semaphore_stub.h"
#include "CellularDevice_stub.h"
#include "equeue_stub.h"
#include "CellularSIM.h"
#include "AT_CellularBase_stub.h"

using namespace mbed;
using namespace events;
Expand Down Expand Up @@ -115,12 +115,12 @@ class my_stack : public AT_CellularStack {
class my_AT_CTX : public AT_CellularContext {
public:
my_AT_CTX(ATHandler &at, CellularDevice *device, const char *apn = MBED_CONF_NSAPI_DEFAULT_CELLULAR_APN) :
AT_CellularContext(at, device, apn), _st(at) {}
virtual ~my_AT_CTX() {}
virtual bool stack_type_supported(nsapi_ip_stack_t stack_type)
AT_CellularContext(at, device, apn), _st(at)
{
return false;
AT_CellularBase_stub::supported_bool = false;
}
virtual ~my_AT_CTX() {}

virtual NetworkStack *get_stack()
{
return &_st;
Expand Down Expand Up @@ -528,7 +528,7 @@ TEST_F(TestAT_CellularContext, set_sim_ready)
cell_callback_data_t data;
data.error = NSAPI_ERROR_OK;
ctx.cellular_callback((nsapi_event_t)CellularDeviceReady, (intptr_t)&data);
data.status_data = CellularSIM::SimStateReady;
data.status_data = CellularDevice::SimStateReady;
ctx.cellular_callback((nsapi_event_t)CellularSIMStatusChanged, (intptr_t)&data);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ set(unittest-includes ${unittest-includes}
../features/cellular/framework/common
../features/cellular/framework/AT
../features/cellular/framework/device
../features/netsocket/cellular
)

# Source files
Expand Down Expand Up @@ -36,4 +37,6 @@ set(unittest-test-sources
stubs/randLIB_stub.cpp
stubs/Semaphore_stub.cpp
stubs/us_ticker_stub.cpp
stubs/UARTSerial_stub.cpp
stubs/SerialBase_stub.cpp
)
Loading