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.

Sign up for GitHub

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

Jump to bottom

Update Segger SysView files to v3.10 #437

Merged
merged 6 commits into from
Mar 6, 2020
Merged

Update Segger SysView files to v3.10 #437

merged 6 commits into from
Mar 6, 2020

Commits on Feb 11, 2020

  1. Ignore local board/platform overrides 

    boards.local.txt and platform.local.txt should
never exist in main branch.
    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    bc94b48 View commit details
    Browse the repository at this point in the history

  2. Update SystemView to v3.10 

    Note: This does not compile, which is expected.
Need to apply patch file for RTOS v10 and restore
existing configuration modifications.
    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    84b65a8 View commit details
    Browse the repository at this point in the history

  3. Avoid link error from using ASM 

    When linking, get errors about undefined references
to `SEGGER_RTT_ASM_WriteSkipNoLock`
    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    beb5c1b View commit details
    Browse the repository at this point in the history

  4. Apply FreeRTOSV10_Core.patch 

    Of course, had to apply them manually, as the patch file
did not apply directly.  This is because some files were
renamed / moved relative to the original distribution,
while others were deleted because they did not apply.
    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    5ed66d2 View commit details
    Browse the repository at this point in the history

  5. Remove contents when SysView is not enabled.

    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    1cb51ff View commit details
    Browse the repository at this point in the history

  6. Fix compilation warning. 

    size_t is unsigned; the #defined constant was treated as an integer,
which caused the following warning:

In file included from cores\nRF5\Uart.cpp:21:0:
cores\nRF5\Arduino.h: In instantiation of
```C++
  decltype ( ((b < a) ? b :  a) )
   min(const T&, const L&)
   [with
    T = unsigned int;
    L = int;
    decltype (((b < a) ? b :  a)) = unsigned int
   ]:
```

cores\nRF5\Uart.cpp:228:54: required from here
cores\nRF5\Arduino.h:92:15: warning:
comparison between signed and unsigned integer expressions [-Wsign-compare]
     return (b < a) ? b : a;
    @henrygab
    henrygab committed Feb 11, 2020
    Configuration menu
    Copy the full SHA
    7c990c4 View commit details
    Browse the repository at this point in the history