Skip to content

Rollup pt2: fixes for 5.12.1 #10116

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

Merged
merged 20 commits into from
Mar 15, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
070a412
BLE:Cordio:insert_descriptor check r/w properties
desmond-blue Mar 4, 2019
7ec9be6
Update Wi-SUN network default name
Mar 4, 2019
5d04df7
Fix lwip to compile if MBED_CONF_LWIP_DEBUG_ENABLED is defined
Mar 5, 2019
a830dbf
Cellular: removed manual registering state.
Mar 1, 2019
a5c6d42
tsi/main.cpp is using wrong `printf` format
janjongboom Mar 7, 2019
f3614ab
OdinWifiInterface is calling memcpy with a null pointer
janjongboom Mar 7, 2019
68399d7
Remove intervaltree from requirements.txt for mbed-os as it contradic…
linlingao Mar 7, 2019
a70f0e6
BLE:Fix cordio reset sequence
desmond-blue Mar 14, 2019
fc97a75
Remove #ifndef NO_GREENTEA from tests
Mar 3, 2019
421e8df
Remove fuzzywuzzy dependency
arekzaluski Mar 14, 2019
c1724e7
Merge branch 'remove-fuzzywuzzy' of https://github.com/arekzaluski/mb…
0xc0170 Mar 15, 2019
9960ea9
Merge branch 'fix-ble-cordio-reset-sequence' of https://github.com/de…
0xc0170 Mar 15, 2019
02f486d
Merge branch 'fix_intervaltree' of https://github.com/linlingao/mbed-…
0xc0170 Mar 15, 2019
4b358d6
Merge branch 'patch-10' of https://github.com/janjongboom/mbed-os int…
0xc0170 Mar 15, 2019
0597e33
Merge branch 'patch-9' of https://github.com/janjongboom/mbed-os into…
0xc0170 Mar 15, 2019
74f13dd
Merge branch 'fix_Cordio-descriptor-set-cback' of https://github.com/…
0xc0170 Mar 15, 2019
759627f
Merge branch 'plmn_stm_fix2' of https://github.com/jarvte/mbed-os int…
0xc0170 Mar 15, 2019
4a24bd1
Merge branch 'fix_lwip_debug_log_compile' of https://github.com/jarvt…
0xc0170 Mar 15, 2019
4cf7864
Merge branch 'wisun_network_default_name' of https://github.com/artok…
0xc0170 Mar 15, 2019
686095e
Merge branch 'remove_no_greentea' of https://github.com/kfnta/mbed-os…
0xc0170 Mar 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
2 changes: 0 additions & 2 deletions TESTS/mbed-crypto/sanity/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -438,9 +438,7 @@ utest::v1::status_t case_teardown_handler(const Case *const source, const size_t

utest::v1::status_t test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(120, "default_auto");
#endif
return verbose_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/mbed_hal/spm/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -150,9 +150,7 @@ Case cases[] = {

utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(20, "default_auto");
#endif
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/attestation/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,7 @@ using namespace utest::v1;

utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/crypto_access_control/COMPONENT_NSPE/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -468,9 +468,7 @@ utest::v1::status_t case_teardown_handler(const Case *const source, const size_t

utest::v1::status_t test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(120, "default_auto");
#endif
return verbose_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/crypto_init/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,7 @@ using namespace utest::v1;

utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/entropy_inject/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,7 @@ void run_entropy_inject_with_crypto_init()

utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif

/* fill seed in some data */
for (size_t i = 0; i < sizeof(seed); ++i) {
Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/its_ps/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -184,9 +184,7 @@ Case cases[] = {

utest::v1::status_t greentea_test_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/spm_client/COMPONENT_NSPE/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -475,9 +475,7 @@ Case cases[] = {
utest::v1::status_t test_setup(const size_t number_of_cases)
{
// Setup Greentea using a reasonable timeout in seconds
#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif
return verbose_test_setup_handler(number_of_cases);
}

Expand Down
2 changes: 0 additions & 2 deletions TESTS/psa/spm_server/COMPONENT_NSPE/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -213,9 +213,7 @@ utest::v1::status_t spm_setup(const size_t number_of_cases)
error("Could not open a connection with SERVER_TESTS_PART1_CONTROL ROT_SRV");
}

#ifndef NO_GREENTEA
GREENTEA_SETUP(60, "default_auto");
#endif
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
3 changes: 0 additions & 3 deletions TESTS/psa/spm_smoke/COMPONENT_NSPE/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,7 @@ void example_main(void)

utest::v1::status_t greentea_setup(const size_t number_of_cases)
{
#ifndef NO_GREENTEA
GREENTEA_SETUP(20, "default_auto");
#endif
// Call the default reporting function
return greentea_test_setup_handler(number_of_cases);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ enum UT_CellularState {
UT_STATE_DEVICE_READY,
UT_STATE_SIM_PIN,
UT_STATE_REGISTERING_NETWORK,
UT_STATE_MANUAL_REGISTERING_NETWORK,
UT_STATE_ATTACHING_NETWORK,
UT_STATE_MAX_FSM_STATE
};
Expand Down Expand Up @@ -392,8 +391,8 @@ TEST_F(TestCellularStateMachine, test_run_to_state)
ut.set_plmn("12345");
ASSERT_EQ(NSAPI_ERROR_OK, ut.run_to_device_registered());
(void)ut.get_current_status(current_state, target_state);
ASSERT_EQ(UT_STATE_MANUAL_REGISTERING_NETWORK, current_state);
ASSERT_EQ(UT_STATE_MANUAL_REGISTERING_NETWORK, target_state);
ASSERT_EQ(UT_STATE_REGISTERING_NETWORK, current_state);
ASSERT_EQ(UT_STATE_REGISTERING_NETWORK, target_state);
ut.cellular_event_changed((nsapi_event_t)CellularRegistrationStatusChanged, (intptr_t)&data);
ut.reset();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,11 @@ void CordioHCIDriver::handle_reset_sequence(uint8_t *pMsg)
} break;

case HCI_OPCODE_LE_WRITE_DEF_DATA_LEN:
/* send next command in sequence */
HciReadLocalVerInfoCmd();
break;

case HCI_OPCODE_READ_LOCAL_VER_INFO:
if (hciCoreCb.extResetSeq) {
/* send first extended command */
(*hciCoreCb.extResetSeq)(pMsg, opcode);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -488,6 +488,10 @@ ble_error_t GattServer::insert_descriptor(
#endif // BLE_FEATURE_SECURE_CONNECTIONS
#endif // BLE_FEATURE_SECURITY
}

if (properties & READ_PROPERTY) {
attribute_it->settings |= ATTS_SET_READ_CBACK;
}
}

// configure write permission
Expand Down Expand Up @@ -517,6 +521,10 @@ ble_error_t GattServer::insert_descriptor(
#endif // BLE_FEATURE_SECURE_CONNECTIONS
#endif // BLE_FEATURE_SECURITY
}

if (properties & WRITABLE_PROPERTIES) {
attribute_it->settings |= ATTS_SET_WRITE_CBACK;
}
}

attribute_it++;
Expand Down
7 changes: 6 additions & 1 deletion features/cellular/framework/AT/AT_CellularNetwork.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,9 @@ void AT_CellularNetwork::read_reg_params_and_compare(RegistrationType type)
reg_params._status == RegisteredRoaming)) {
if (previous_registration_status == RegisteredHomeNetwork ||
previous_registration_status == RegisteredRoaming) {
call_network_cb(NSAPI_STATUS_DISCONNECTED);
if (type != C_REG) {// we are interested only if we drop from packet network
_connection_status_cb(NSAPI_EVENT_CONNECTION_STATUS_CHANGE, NSAPI_STATUS_DISCONNECTED);
}
}
}
}
Expand Down Expand Up @@ -268,6 +270,9 @@ nsapi_error_t AT_CellularNetwork::set_registration(const char *plmn)
tr_debug("Manual network registration to %s", plmn);
_at.cmd_start("AT+COPS=1,2,");
_at.write_string(plmn);
if (_op_act != RAT_UNKNOWN) {
_at.write_int(_op_act);
}
_at.cmd_stop_read_resp();
}

Expand Down
Loading