Re: [Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-13 Thread Leon Alrae
On 13/05/2015 06:48, Liviu Ionescu wrote: are these patches finally in? if so, what repo/branch should I use to get them? Note that v3 was sent quite recently and there should be some time to give everyone a chance to comment. I was planning to send it out later, together with UHI via

Re: [Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-13 Thread Peter Maydell
On 13 May 2015 at 10:29, Leon Alrae leon.al...@imgtec.com wrote: On 13/05/2015 06:48, Liviu Ionescu wrote: are these patches finally in? if so, what repo/branch should I use to get them? Note that v3 was sent quite recently and there should be some time to give everyone a chance to comment.

Re: [Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-12 Thread Liviu Ionescu
On 06 May 2015, at 19:12, Leon Alrae leon.al...@imgtec.com wrote: ... I'm going to follow up with UHI patch series only. I don't have anything set up to test other semihosting interfaces, thus I don't plan to do any changes in these areas in near future. Therefore feel free to send your

Re: [Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-06 Thread Leon Alrae
On 06/05/2015 16:22, Liviu Ionescu wrote: apparently your patch does not fix the arm semihosting problems. do you plan a separate patch for this? This patchset doesn't contain any target semihosting specific changes (just a clean up and new arg option). I'm going to follow up with UHI patch

Re: [Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-06 Thread Liviu Ionescu
apparently your patch does not fix the arm semihosting problems. do you plan a separate patch for this? bool semihosting_enabled(void) { return semihosting.allowed; } any particular reason for naming the structure member .allowed and the getter function _enabled()? regards, Liviu

[Qemu-devel] [PATCH 0/4] semihosting: clean up and add --semihosting-config arg

2015-05-06 Thread Leon Alrae
Hi, This patch series adds arg= sub-option to --semihosting-config group. It allows building up a list of input arguments as it can appear multiple times in the command line. This is a flexible solution for creating argc/argv for the guest program (needed by UHI semihosting for example). RFC