mbox series

[Semihosting,Tests,v2,0/3] add SYS_GET_CMDLINE test

Message ID 20240530112332.1439238-1-alex.bennee@linaro.org (mailing list archive)
Headers show
Series add SYS_GET_CMDLINE test | expand

Message

Alex Bennée May 30, 2024, 11:23 a.m. UTC 
Hi Peter,

Looking at bug #2322 I wanted to make sure SYS_GET_CMDLINE works as I
expected. While at it I needed to fix a compile error with headers
which I guess we got away with on earlier compilers.

I've added an editorconfig for good measure.

v2
  - addressed review comments

Alex Bennée (3):
  .editorconfig: add code conventions for tooling
  update includes for bare metal compiling
  add SYS_GET_CMDLINE test

 .editorconfig | 29 +++++++++++++++++++++++++++++
 Makefile      | 22 +++++++++++-----------
 semihosting.c |  4 ++--
 semihosting.h |  2 +-
 string.c      |  2 +-
 usertest.c    | 33 ++++++++++++++++++++++++++++++++-
 6 files changed, 76 insertions(+), 16 deletions(-)
 create mode 100644 .editorconfig

Comments

Peter Maydell May 30, 2024, 2:31 p.m. UTC | #1 
On Thu, 30 May 2024 at 12:23, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Hi Peter,
>
> Looking at bug #2322 I wanted to make sure SYS_GET_CMDLINE works as I
> expected. While at it I needed to fix a compile error with headers
> which I guess we got away with on earlier compilers.
>
> I've added an editorconfig for good measure.
>
> v2
>   - addressed review comments
>
> Alex Bennée (3):
>   .editorconfig: add code conventions for tooling
>   update includes for bare metal compiling
>   add SYS_GET_CMDLINE test

Applied to the semihosting-tests git repo, thanks.

-- PMM