Message ID | 1303294755-2679-1-git-send-email-peter.maydell@linaro.org |
---|---|
State | Accepted |
Commit | 7cb4db8f41d20d2c9d9c9a6830a362eebaeb42ed |
Headers | show |
On Wed, Apr 20, 2011 at 11:19:15AM +0100, Peter Maydell wrote: > The REG_PC constant used in the ARM nwfpe code is fine in the kernel > but when used in qemu can clash with a definition in the host system > include files (in particular on Ubuntu Lucid SPARC, including signal.h > will define a REG_PC). Rename the constant to avoid this issue. > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > linux-user/arm/nwfpe/fpa11.c | 2 +- > linux-user/arm/nwfpe/fpa11.h | 2 +- > linux-user/arm/nwfpe/fpa11_cpdt.c | 8 ++++---- > 3 files changed, 6 insertions(+), 6 deletions(-) Thanks, applied. > diff --git a/linux-user/arm/nwfpe/fpa11.c b/linux-user/arm/nwfpe/fpa11.c > index 0a87c43..eebd93f 100644 > --- a/linux-user/arm/nwfpe/fpa11.c > +++ b/linux-user/arm/nwfpe/fpa11.c > @@ -144,7 +144,7 @@ unsigned int EmulateAll(unsigned int opcode, FPA11* qfpa, CPUARMState* qregs) > > #if 0 > fprintf(stderr,"emulating FP insn 0x%08x, PC=0x%08x\n", > - opcode, qregs[REG_PC]); > + opcode, qregs[ARM_REG_PC]); > #endif > fpa11 = GET_FPA11(); > > diff --git a/linux-user/arm/nwfpe/fpa11.h b/linux-user/arm/nwfpe/fpa11.h > index f17647b..002b3cb 100644 > --- a/linux-user/arm/nwfpe/fpa11.h > +++ b/linux-user/arm/nwfpe/fpa11.h > @@ -111,7 +111,7 @@ static inline void writeConditionCodes(unsigned int x) > cpsr_write(user_registers,x,CPSR_NZCV); > } > > -#define REG_PC 15 > +#define ARM_REG_PC 15 > > unsigned int EmulateAll(unsigned int opcode, FPA11* qfpa, CPUARMState* qregs); > > diff --git a/linux-user/arm/nwfpe/fpa11_cpdt.c b/linux-user/arm/nwfpe/fpa11_cpdt.c > index b12e27d..3e7a938 100644 > --- a/linux-user/arm/nwfpe/fpa11_cpdt.c > +++ b/linux-user/arm/nwfpe/fpa11_cpdt.c > @@ -220,7 +220,7 @@ static unsigned int PerformLDF(const unsigned int opcode) > //printk("PerformLDF(0x%08x), Fd = 0x%08x\n",opcode,getFd(opcode)); > > pBase = readRegister(getRn(opcode)); > - if (REG_PC == getRn(opcode)) > + if (ARM_REG_PC == getRn(opcode)) > { > pBase += 8; > write_back = 0; > @@ -256,7 +256,7 @@ static unsigned int PerformSTF(const unsigned int opcode) > SetRoundingMode(ROUND_TO_NEAREST); > > pBase = readRegister(getRn(opcode)); > - if (REG_PC == getRn(opcode)) > + if (ARM_REG_PC == getRn(opcode)) > { > pBase += 8; > write_back = 0; > @@ -289,7 +289,7 @@ static unsigned int PerformLFM(const unsigned int opcode) > target_ulong pBase, pAddress, pFinal; > > pBase = readRegister(getRn(opcode)); > - if (REG_PC == getRn(opcode)) > + if (ARM_REG_PC == getRn(opcode)) > { > pBase += 8; > write_back = 0; > @@ -322,7 +322,7 @@ static unsigned int PerformSFM(const unsigned int opcode) > target_ulong pBase, pAddress, pFinal; > > pBase = readRegister(getRn(opcode)); > - if (REG_PC == getRn(opcode)) > + if (ARM_REG_PC == getRn(opcode)) > { > pBase += 8; > write_back = 0; > -- > 1.7.1 > > >
diff --git a/linux-user/arm/nwfpe/fpa11.c b/linux-user/arm/nwfpe/fpa11.c index 0a87c43..eebd93f 100644 --- a/linux-user/arm/nwfpe/fpa11.c +++ b/linux-user/arm/nwfpe/fpa11.c @@ -144,7 +144,7 @@ unsigned int EmulateAll(unsigned int opcode, FPA11* qfpa, CPUARMState* qregs) #if 0 fprintf(stderr,"emulating FP insn 0x%08x, PC=0x%08x\n", - opcode, qregs[REG_PC]); + opcode, qregs[ARM_REG_PC]); #endif fpa11 = GET_FPA11(); diff --git a/linux-user/arm/nwfpe/fpa11.h b/linux-user/arm/nwfpe/fpa11.h index f17647b..002b3cb 100644 --- a/linux-user/arm/nwfpe/fpa11.h +++ b/linux-user/arm/nwfpe/fpa11.h @@ -111,7 +111,7 @@ static inline void writeConditionCodes(unsigned int x) cpsr_write(user_registers,x,CPSR_NZCV); } -#define REG_PC 15 +#define ARM_REG_PC 15 unsigned int EmulateAll(unsigned int opcode, FPA11* qfpa, CPUARMState* qregs); diff --git a/linux-user/arm/nwfpe/fpa11_cpdt.c b/linux-user/arm/nwfpe/fpa11_cpdt.c index b12e27d..3e7a938 100644 --- a/linux-user/arm/nwfpe/fpa11_cpdt.c +++ b/linux-user/arm/nwfpe/fpa11_cpdt.c @@ -220,7 +220,7 @@ static unsigned int PerformLDF(const unsigned int opcode) //printk("PerformLDF(0x%08x), Fd = 0x%08x\n",opcode,getFd(opcode)); pBase = readRegister(getRn(opcode)); - if (REG_PC == getRn(opcode)) + if (ARM_REG_PC == getRn(opcode)) { pBase += 8; write_back = 0; @@ -256,7 +256,7 @@ static unsigned int PerformSTF(const unsigned int opcode) SetRoundingMode(ROUND_TO_NEAREST); pBase = readRegister(getRn(opcode)); - if (REG_PC == getRn(opcode)) + if (ARM_REG_PC == getRn(opcode)) { pBase += 8; write_back = 0; @@ -289,7 +289,7 @@ static unsigned int PerformLFM(const unsigned int opcode) target_ulong pBase, pAddress, pFinal; pBase = readRegister(getRn(opcode)); - if (REG_PC == getRn(opcode)) + if (ARM_REG_PC == getRn(opcode)) { pBase += 8; write_back = 0; @@ -322,7 +322,7 @@ static unsigned int PerformSFM(const unsigned int opcode) target_ulong pBase, pAddress, pFinal; pBase = readRegister(getRn(opcode)); - if (REG_PC == getRn(opcode)) + if (ARM_REG_PC == getRn(opcode)) { pBase += 8; write_back = 0;
The REG_PC constant used in the ARM nwfpe code is fine in the kernel but when used in qemu can clash with a definition in the host system include files (in particular on Ubuntu Lucid SPARC, including signal.h will define a REG_PC). Rename the constant to avoid this issue. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- linux-user/arm/nwfpe/fpa11.c | 2 +- linux-user/arm/nwfpe/fpa11.h | 2 +- linux-user/arm/nwfpe/fpa11_cpdt.c | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-)