Vscode exporter improvements #3568

janjongboom · 2017-01-11T17:16:46Z

@sg- For some reason I cannot open this against your mbed OS fork...

Some stuff that still would need to be fixed to make this work properly:

Restrict intellisense to files listed microsoft/vscode-cpptools#374 - for Intellisense
miDebuggerPath should search PATH microsoft/vscode-cpptools#442 - for finding the pyOCD tools without relying on full paths
How to run GDB -target-download from the debugger microsoft/vscode-cpptools#443 - to load the binary when starting a debug session

janjongboom · 2017-01-11T17:19:08Z

Pretty warnings are pretty:

theotherjimmy · 2017-01-11T17:56:25Z

tools/export/cdt/__init__.py

@@ -14,7 +14,7 @@ def generate(self):
        super(Eclipse, self).generate()
        ctx = {
            'name': self.project_name,
-            'elf_location': join('BUILD',self.project_name)+'.elf',
+            'elf_location': join('BUILD', self.project_name)+'.elf',


This is nice and all, but not really related.

sg- · 2017-01-11T20:48:38Z

Lets get this on a feature branch https://github.com/ARMmbed/mbed-os/tree/feature-vscode

theotherjimmy · 2017-01-11T20:53:33Z

Done.

0xc0170 · 2017-01-12T11:32:01Z

tools/export/vscode/launch.tmpl

+            },
+            "osx": {
+                "MIMode": "gdb",
+                "MIDebuggerPath": "/Applications/yotta.app/Contents/Resources/prerequisites/gcc-arm-none-eabi-4_9-2015q3/bin/arm-none-eabi-gdb",


Is this path correct for any osx ? windows and linux have it generic compare to this one.

No, that's why we need microsoft/vscode-cpptools#442

0xc0170 · 2017-01-12T11:41:21Z

Please resolve the conflict.

Glad to see this contribution !!

theotherjimmy · 2017-01-12T17:48:13Z

Nice! It's great to see an exporter take care of flashing!

theotherjimmy · 2017-01-12T20:32:57Z

tools/export/vscode/launch.tmpl

+                "debugServerPath": "${workspaceRoot}\\.vscode\\bin\\windows\\gdb_server.exe",
+                "setupCommands": [
+                    { "text": "-target-select remote localhost:3333", "description": "connect to target", "ignoreFailures": false },
+                    { "text": "-file-exec-and-symbols z:\\\\mbed-os-example-blinky\\\\BUILD\\\\${workspaceRootFolderName}.elf", "description": "load file", "ignoreFailures": false},


Mmmm.... lotsa slashes...

Yep... GDB wants it's own escape for backslashes... And then escape in JavaScript, so that's 4. Very annoying.

theotherjimmy · 2017-01-13T20:02:47Z

tools/export/vscode/tasks.tmpl

+        "args": ["-j"]
+    },
+    "windows": {
+        "command": "${workspaceRoot}\\.vscode\\bin\\windows\\make.bat"


I take it that make.bat does not do parallel the same way that gnumake does?

@theotherjimmy It sets the PATH first to point to the cross-compilation toolchain, and then just invokes GNU Make (with -j).

Ah, cool the performance is comparable.

sg- · 2017-01-14T22:41:16Z

That didn't work right meaning the online resolve conflict tool created a merge commit :(

Resolved conflict and rebased master offline to keep the branch up to date. Should pass CI now.

Had to force push to @janjongboom branch 😨

sg- · 2017-01-14T22:58:29Z

/morph export-build

mbed-bot · 2017-01-14T23:41:55Z

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 82

All exports and builds passed!

sg- · 2017-01-15T17:23:01Z

retest uvisor

janjongboom · 2017-01-17T11:42:31Z

@sg- Fixed uvisor debugging by enabling semihosting. Not sure if it's wanted for all targets.

sg- · 2017-01-17T12:56:50Z

retest uvisor

bridadan · 2017-01-17T17:53:56Z

retest uvisor

sg- · 2017-01-17T22:45:07Z

@bridadan where is the status? #3568 (comment)

sg- · 2017-01-17T22:47:18Z

nvmd 🤓 @janjongboom pushed a commit. @c1728p9 anything about enabling semihosting for all targets that you think should be a problem?

@janjongboom what else before getting merged on the feature branch and iterating?

janjongboom · 2017-01-18T20:04:00Z

@sg- I'd love some way of integrating the cross-compilation tools into this, but for now I think we can go forward with this. Only issue I have right now is running make before debugging (as a pre-debug step) on Windows, don't remember whether I fixed that already or not. Couldn't get it to work at Ericsson today (but they use mbed CLI, not the makefile).

Other than that, using it daily, and it's awesome. Especially like the GDB console, already more productive with this than I've ever been with uVision.

c1728p9 · 2017-01-19T21:57:15Z

@sg- semihosting looks good to me. @janjongboom what behavior do you see if semihosting is off? Breakpoints constantly being hit?

janjongboom · 2017-01-20T11:24:38Z

@c1728p9 Yeah, w/o uVisor I keep getting trapped in uvisor_api:

1: (8019) ->*stopped,reason="signal-received",signal-name="SIGTRAP",signal-meaning="Trace/breakpoint trap",frame={addr="0x0000050c",func="uvisor_api",args=[]},thread-id="1",stopped-threads="all",core="0"
0x0000050c in uvisor_api ()
1: (8019) <-1018-stack-list-frames 0 1000
1: (8152) ->1018^done,stack=[frame={level="0",addr="0x0000050c",func="uvisor_api"}]
1: (8153) ->(gdb)
1: (8153) 1018: elapsed time 133
Execute debugger commands using "-exec <command>", for example "-exec info registers" will list registers in use (when GDB is the debugger)
C threads: null
C stackTrace: {"threadId":1,"startFrame":0,"levels":20}
1: (8180) <-1019-stack-list-arguments 2 0 0
1: (8180) ->1019^done,stack-args=[frame={level="0",args=[]}]
1: (8180) ->(gdb)
1: (8181) 1019: elapsed time 1

example specified in the supplied .json file, in a user specified fork. A pull request is then made from each fork to the ARMmbed master repo. New usage: python update.py <args> Where <args> are: [-c <file.json>] optional argument for example list, default example.json -U <github user with forked repos> -T <github authorisation token> tag

This saves about 100kB of flash when using ethernet only configuration. Otherwise only border router configuration was supporting ethernet.

For equeue_sema_wait, -1 is used to indicate an infinite wait. This wasn't handled in the nonrtos implementation and caused undefined/weird behaviour after an overflow on integer multiplication. On most boards, the infinite wait would return after ~50 days, on the STM32F4 the timeout killed all other timeouts for some reason.

Previously, when "length = 0" as failsafe, "return 0" is executed. But we changed so that only START bit and STOP bit are executed same as other devices.

CMSIS v2.3.0 => v2.3.1 STM32F0 HAL v1.4.0 => v1.5.0 LL Layer introduction for STM32F0

Following HAL update, this is needed to use the I2C API function rather than previously used MACRO. An assert would fail at compilation time otherwise.

Following STM32 HAL update, it is needed to use up-to-date MACROs. Otherwise, build would fail.

The UART state should be set to busy before starting the transfer Signed-off-by: Mahadevan Mahesh <[email protected]>

Update exporter grouping code to group by directories in the root of the project rather than by the parent directory of each file. This reduces the number of groups and allows all mbed-os code to reside in its own folder.

…lers are set Signed-off-by: Mahadevan Mahesh <[email protected]>

Turn on the vector checksum on all LPC4088 variants. This checksum is required for an application to boot.

When building with the debug profile, certain ST plaforms error with 'get_i2c_timing' not being defined. This is because the function is not defined as 'static inline', but just 'inline'.

…ured for linux and mac although the default installation locations needs attention. also missing windows default paths

sg- · 2017-02-01T19:17:49Z

Now replaced by #3679

theotherjimmy reviewed Jan 11, 2017

View reviewed changes

theotherjimmy approved these changes Jan 11, 2017

View reviewed changes

theotherjimmy changed the base branch from master to feature-vscode January 11, 2017 20:53

0xc0170 reviewed Jan 12, 2017

View reviewed changes

0xc0170 added the needs: work label Jan 12, 2017

theotherjimmy reviewed Jan 12, 2017

View reviewed changes

theotherjimmy reviewed Jan 13, 2017

View reviewed changes

sg- force-pushed the vscode_improvements branch from 4ecc6fd to 0f272cf Compare January 14, 2017 22:43

janjongboom force-pushed the vscode_improvements branch from c9073eb to 205cf49 Compare February 1, 2017 18:43

adbridge and others added 4 commits February 1, 2017 18:44

Eclipse: Symbols should also be loaded when starting a debug session

0437072

STM32L4 deepsleep improvement

fd14867

Add support for ethernet-only configuration with Nanostack.

2a08588

This saves about 100kB of flash when using ethernet only configuration. Otherwise only border router configuration was supporting ethernet.

geky and others added 26 commits February 1, 2017 18:44

Fix I2C driver for RZ/A1H

a39aa5e

Previously, when "length = 0" as failsafe, "return 0" is executed. But we changed so that only START bit and STOP bit are executed same as other devices.

Add NUCLEO_F412ZG in main.cpp files used for the tests

e1a50b3

Add CAN object structure

4b2c9ea

Add CAN feature and activate CAN tests for this platform

dcfa481

Remove automated flag for MBED_29/30 tests

c810eaa

STM32Cube_FW_F0_V1.7.0

f90825f

CMSIS v2.3.0 => v2.3.1 STM32F0 HAL v1.4.0 => v1.5.0 LL Layer introduction for STM32F0

STM32: HAL update, use I2C function instead of MACRO

2623c87

Following HAL update, this is needed to use the I2C API function rather than previously used MACRO. An assert would fail at compilation time otherwise.

STM32: Serial: Use up to date MACROs

d8730b3

Following STM32 HAL update, it is needed to use up-to-date MACROs. Otherwise, build would fail.

STM32: TIM: Initialize new TIM parameter

9efdf05

K64F UART Asynch API: Fix synchronization issue

207f3ca

The UART state should be set to busy before starting the transfer Signed-off-by: Mahadevan Mahesh <[email protected]>

exporters - group by directories in prj root

af266d4

Update exporter grouping code to group by directories in the root of the project rather than by the parent directory of each file. This reduces the number of groups and allows all mbed-os code to reside in its own folder.

MCUXpresso: Fix ENET driver to enable interrupts after interrupt hand…

d53b72d

…lers are set Signed-off-by: Mahadevan Mahesh <[email protected]>

Alphabetize UVision groups

c1c4c4c

LPC4088 - Fix vector checksum

837be15

Turn on the vector checksum on all LPC4088 variants. This checksum is required for an application to boot.

Asciiabetize the files within a group

be9b651

Change sorting to ignore case

f3908c6

Fixes linking errors when building with debug profile

7f0555b

When building with the debug profile, certain ST plaforms error with 'get_i2c_timing' not being defined. This is because the function is not defined as 'static inline', but just 'inline'.

adding vscode exporter. basic compiling as a task and somewhat config…

093561d

…ured for linux and mac although the default installation locations needs attention. also missing windows default paths

vscode: Build the application before starting a debug session

a4d5ca5

Add problemMatcher definitions

ee8bc9e

Flash before debugging

c6d33fa

Point arm-none-eabi-gdb and pyocd-gdbserver to /usr/local/bin on mac

512feb1

Debugging on Windows (although path issues)

d6508bb

make on clean Windows

e41e04e

Enable semihosting, required for uvisor

205cf49

sg- mentioned this pull request Feb 1, 2017

Feature vscode #3679

Merged

2 tasks

sg- closed this Feb 1, 2017

janjongboom mentioned this pull request Mar 9, 2017

Feature vscode #3915

Merged

Vscode exporter improvements #3568

Vscode exporter improvements #3568

Uh oh!

Conversation

janjongboom commented Jan 11, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

janjongboom commented Jan 11, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sg- commented Jan 11, 2017

Uh oh!

theotherjimmy commented Jan 11, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

0xc0170 commented Jan 12, 2017

Uh oh!

theotherjimmy commented Jan 12, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sg- commented Jan 14, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sg- commented Jan 14, 2017

Uh oh!

mbed-bot commented Jan 14, 2017

Output

Uh oh!

sg- commented Jan 15, 2017

Uh oh!

janjongboom commented Jan 17, 2017

Uh oh!

sg- commented Jan 17, 2017

Uh oh!

bridadan commented Jan 17, 2017

Uh oh!

sg- commented Jan 17, 2017

Uh oh!

sg- commented Jan 17, 2017

Uh oh!

janjongboom commented Jan 18, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

c1728p9 commented Jan 19, 2017

Uh oh!

janjongboom commented Jan 20, 2017

Uh oh!

sg- commented Feb 1, 2017

Uh oh!

Uh oh!

janjongboom commented Jan 11, 2017 •

edited

Loading

sg- commented Jan 14, 2017 •

edited

Loading

janjongboom commented Jan 18, 2017 •

edited

Loading