-
Notifications
You must be signed in to change notification settings - Fork 3k
Modified Wiced drivers EMAC instance get #7413
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
Conversation
Removed EMAC get_default_instance() since WLAN drivers are not default EMAC drivers. Moved EMAC static declaration inside get_instance().
cc @pan |
/morph build |
@0xc0170 Kari is on vacation. @kjbracey-arm Please review this. |
/morph build |
Build : SUCCESSBuild number : 2523 Triggering tests/morph test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is more bearable.
+------------------+-------+-------+------+
| Module | .text | .data | .bss |
+------------------+-------+-------+------+
| [fill] | 100 | 4 | 21 |
| [lib]\c.a | 25093 | 2472 | 89 |
| [lib]\gcc.a | 3168 | 0 | 0 |
| [lib]\misc | 252 | 16 | 28 |
| main.o | 31 | 0 | 0 |
| mbed-os\drivers | 192 | 4 | 100 |
| mbed-os\features | 204 | 12 | 48 |
| mbed-os\hal | 1653 | 4 | 68 |
| mbed-os\platform | 2764 | 260 | 133 |
| mbed-os\rtos | 9337 | 168 | 6073 |
| mbed-os\targets | 7328 | 4 | 428 |
| Subtotals | 50122 | 2944 | 6988 |
+------------------+-------+-------+------+
Build : SUCCESSBuild number : 2527 Triggering tests/morph test |
Test : SUCCESSBuild number : 2285 |
Exporter Build : SUCCESSBuild number : 2171 |
Exporter Build : SUCCESSBuild number : 2175 |
Test : SUCCESSBuild number : 2289 |
Description
Removed EMAC get_default_instance() since WLAN drivers are not default EMAC drivers. Moved EMAC static declaration inside get_instance().
Changes has been tested using ip-core nightly Jenkins jobs.
Fixes #7372
Pull request type