ARMmbed / wifi-ism43362

ISM43362 WiFi driver
17 stars 22 forks source link

Removed timeout code as it was not useful #14

Closed jeromecoutant closed 6 years ago

jeromecoutant commented 6 years ago

In the Inventek UM, timeout value are defined only for send and receive data once the Transport Protocol has been defined and either the server or client mode has been enabled

So other timeout values seem to be not useful

Patch:

Test

+-------------------------+---------------------+------------------------------------+----------------------------------------+--------+--------+--------+--------------------+ | target | platform_name | test suite | test case | passed | failed | result | elapsed_time (sec) | +-------------------------+---------------------+------------------------------------+----------------------------------------+--------+--------+--------+--------------------+ | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-connectivity | Bringing the network up and down | 1 | 0 | OK | 4.2 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-connectivity | Bringing the network up and down twice | 1 | 0 | OK | 7.63 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-gethostbyname | DNS literal | 1 | 0 | OK | 0.1 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-gethostbyname | DNS preference literal | 1 | 0 | OK | 0.13 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-gethostbyname | DNS preference query | 1 | 0 | OK | 0.15 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-gethostbyname | DNS query | 1 | 0 | OK | 0.18 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Hollowed IPv6 address | 1 | 0 | OK | 0.05 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Left-weighted IPv4 address | 1 | 0 | OK | 0.07 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Left-weighted IPv6 address | 1 | 0 | OK | 0.05 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Null IPv4 address | 1 | 0 | OK | 0.04 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Null IPv6 address | 1 | 0 | OK | 0.05 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Right-weighted IPv4 address | 1 | 0 | OK | 0.07 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Right-weighted IPv6 address | 1 | 0 | OK | 0.04 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Simple IPv4 address | 1 | 0 | OK | 0.03 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-ip_parsing | Simple IPv6 address | 1 | 0 | OK | 0.05 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-socket_sigio | Socket Attach Test | 1 | 0 | OK | 1.26 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-socket_sigio | Socket Detach Test | 1 | 0 | OK | 5.4 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-socket_sigio | Socket Reattach Test | 1 | 0 | OK | 0.59 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-tcp_echo | TCP echo | 1 | 0 | OK | 14.87 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-tcp_hello_world | TCP hello world | 1 | 0 | OK | 5.52 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-udp_dtls_handshake | UDP DTLS handshake | 0 | 0 | ERROR | 0.0 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-netsocket-udp_echo | UDP echo | 1 | 0 | OK | 86.32 | +-------------------------+---------------------+------------------------------------+----------------------------------------+--------+--------+--------+--------------------+

+-------------------------+---------------------+--------------------+------------------------------------+--------+--------+--------+--------------------+ | target | platform_name | test suite | test case | passed | failed | result | elapsed_time (sec) | +-------------------------+---------------------+--------------------+------------------------------------+--------+--------+--------+--------------------+ | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT | 1 | 0 | OK | 1.31 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-DISCONNECT-REPEAT | 1 | 0 | OK | 13.32 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-NOCREDENTIALS | 1 | 0 | OK | 0.04 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-PARAMS-CHANNEL | 1 | 0 | OK | 0.11 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-PARAMS-CHANNEL-FAIL | 1 | 0 | OK | 0.11 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-PARAMS-NULL | 1 | 0 | OK | 0.06 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-PARAMS-VALID-SECURE | 1 | 0 | OK | 3.69 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-PARAMS-VALID-UNSECURE | 1 | 0 | OK | 1.36 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-SECURE | 1 | 0 | OK | 3.75 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONNECT-SECURE-FAIL | 1 | 0 | OK | 21.36 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-CONSTRUCTOR | 1 | 0 | OK | 0.57 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-GET-RSSI | 1 | 0 | OK | 2.13 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-SCAN | 1 | 0 | OK | 2.81 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-SCAN-NULL | 1 | 0 | OK | 2.8 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-SET-CHANNEL | 1 | 0 | OK | 0.1 | | DISCO_L475VG_IOT01A-ARM | DISCO_L475VG_IOT01A | tests-network-wifi | WIFI-SET-CREDENTIAL | 1 | 0 | OK | 0.05 | +-------------------------+---------------------+--------------------+------------------------------------+--------+--------+--------+--------------------+

jeromecoutant commented 6 years ago

Patch is updated.

Default Read Transport Timeout is set to 1ms.

MBED-OS socket tcp tests with EXTENDED option is OK with all the coming PR.

jeromecoutant commented 6 years ago

Comment should be implemented now. Thx for review