Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-06-02 Thread Alex Bennée
Peter Maydell writes: > On 2 June 2017 at 16:49, Alex Bennée wrote: >> Peter Maydell writes: >>> We seem to run into "CONFIG_FOO is defined for make but not >>> for the C preprocessor" from time to time, so maybe we should >>> fix that? >> >> Hmm is should be auto-generated from config-(host|t

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-06-02 Thread Peter Maydell
On 2 June 2017 at 16:49, Alex Bennée wrote: > Peter Maydell writes: >> We seem to run into "CONFIG_FOO is defined for make but not >> for the C preprocessor" from time to time, so maybe we should >> fix that? > > Hmm is should be auto-generated from config-(host|target).mak but I've > never quite

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-06-02 Thread Alex Bennée
Peter Maydell writes: > On 28 April 2017 at 14:58, Wei Huang wrote: >> Using #ifdef is indeed a cleaner solution. However I can't use "#ifdef >> CONFIG_ARM_V7M" & co inside target/arm/cpu.c file because CONFIG_xxx >> aren't available as preprocessor directives. From what I saw the >> CONFIG_xxx

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-06-02 Thread Peter Maydell
On 28 April 2017 at 14:58, Wei Huang wrote: > Using #ifdef is indeed a cleaner solution. However I can't use "#ifdef > CONFIG_ARM_V7M" & co inside target/arm/cpu.c file because CONFIG_xxx > aren't available as preprocessor directives. From what I saw the > CONFIG_xxx options can only control compi

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-04-28 Thread Wei Huang
On 04/28/2017 07:09 AM, Peter Maydell wrote: > On 28 April 2017 at 01:10, Wei Huang wrote: >> In our downstream code, we try to minimize the number of supported >> devices to reduce the maintenance effort. ARM v7M is not in the >> supported list. >> >> For most unsupported devices, we can remove

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-04-28 Thread Peter Maydell
On 28 April 2017 at 01:10, Wei Huang wrote: > In our downstream code, we try to minimize the number of supported > devices to reduce the maintenance effort. ARM v7M is not in the > supported list. > > For most unsupported devices, we can remove them by configuring > default-configs file, except fo

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-04-27 Thread Wei Huang
On 04/27/2017 04:13 PM, Peter Maydell wrote: > On 27 April 2017 at 18:38, Wei Huang wrote: >> CONFIG_ARM_V7M is used to control the compilation of NVIC and SysTick >> Timer in QEMU. These two devices are not available when CONFIG_ARM_V7M >> is un-defined. This can cause problems with ARMv7M code

Re: [Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-04-27 Thread Peter Maydell
On 27 April 2017 at 18:38, Wei Huang wrote: > CONFIG_ARM_V7M is used to control the compilation of NVIC and SysTick > Timer in QEMU. These two devices are not available when CONFIG_ARM_V7M > is un-defined. This can cause problems with ARMv7M code as it is tightly > integrated with NVIC for process

[Qemu-devel] [PATCH 0/2] Fix compilation dependency issues of CONFIG_ARM_V7M

2017-04-27 Thread Wei Huang
CONFIG_ARM_V7M is used to control the compilation of NVIC and SysTick Timer in QEMU. These two devices are not available when CONFIG_ARM_V7M is un-defined. This can cause problems with ARMv7M code as it is tightly integrated with NVIC for processing IRQ and exception. This patchset addresses the p