You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2022/09/21 18:22:24 UTC

[incubator-nuttx] branch master updated: armv7-r/tms570: fix build break

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new fb4c593e89 armv7-r/tms570: fix build break
fb4c593e89 is described below

commit fb4c593e890bf28d585341c537af142fd59472e9
Author: chao an <an...@xiaomi.com>
AuthorDate: Wed Sep 21 22:08:50 2022 +0800

    armv7-r/tms570: fix build break
    
    1.
    make[1]: *** No rule to make target 'tms570_spi.c', needed by '.depend'.  Stop.
    
    2.
    In file included from armv7-r/arm_gicv2.c:36:
    armv7-r/gic.h: In function 'arm_gic_nlines':
    armv7-r/mpcore.h:63:29: error: 'CHIP_MPCORE_VBASE' undeclared (first use in this function)
       63 | #define MPCORE_ICD_VBASE   (CHIP_MPCORE_VBASE+MPCORE_ICD_OFFSET)
          |                             ^~~~~~~~~~~~~~~~~
    
    Signed-off-by: chao an <an...@xiaomi.com>
---
 arch/arm/src/tms570/Make.defs                         | 4 +++-
 arch/arm/src/tms570/tms570_clockconfig.c              | 2 +-
 arch/arm/src/tms570/tms570_esm.c                      | 2 +-
 arch/arm/src/tms570/tms570_serial.c                   | 4 ++--
 boards/arm/tms570/launchxl-tms57004/scripts/Make.defs | 1 -
 5 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/arch/arm/src/tms570/Make.defs b/arch/arm/src/tms570/Make.defs
index 3abfbef822..eaf0867457 100644
--- a/arch/arm/src/tms570/Make.defs
+++ b/arch/arm/src/tms570/Make.defs
@@ -23,7 +23,7 @@ include armv7-r/Make.defs
 # SAMA5-specific C source files
 
 CHIP_CSRCS  = tms570_boot.c tms570_clockconfig.c tms570_esm.c tms570_gio.c
-CHIP_CSRCS += tms570_irq.c tms570_lowputc.c tms570_serial.c tms570_spi.c
+CHIP_CSRCS += tms570_irq.c tms570_lowputc.c tms570_serial.c
 
 # Configuration dependent C and assembly language files
 
@@ -38,3 +38,5 @@ endif
 ifeq ($(CONFIG_TMS570_GIO_IRQ),y)
 CHIP_CSRCS += tms570_gioirq.c
 endif
+
+CMN_CSRCS := $(filter-out arm_gicv2.c, $(CMN_CSRCS))
diff --git a/arch/arm/src/tms570/tms570_clockconfig.c b/arch/arm/src/tms570/tms570_clockconfig.c
index 7d57dca9e3..f6304edee5 100644
--- a/arch/arm/src/tms570/tms570_clockconfig.c
+++ b/arch/arm/src/tms570/tms570_clockconfig.c
@@ -203,7 +203,7 @@ static uint32_t check_frequency(uint32_t cnt1_clksrc)
 
 uint32_t _errata_sswf021_45_both_plls(uint32_t count)
 {
-  uint32_t failcode;
+  uint32_t failcode = 0u;
   uint32_t retries;
   uint32_t clkcntrlsave;
   uint32_t regval;
diff --git a/arch/arm/src/tms570/tms570_esm.c b/arch/arm/src/tms570/tms570_esm.c
index 2fff8c7f29..59e09fa1ad 100644
--- a/arch/arm/src/tms570/tms570_esm.c
+++ b/arch/arm/src/tms570/tms570_esm.c
@@ -154,7 +154,7 @@ int tms570_esm_interrupt(int irq, void *context, void *arg)
 
   /* Crash -- possibly showing diagnostic debug information. */
 
-  _err("ERROR: ESM Interrupt. PC: %08x\n", CURRENT_REGS[REG_PC]);
+  _err("ERROR: ESM Interrupt. PC: %08" PRIx32 "\n", CURRENT_REGS[REG_PC]);
   PANIC();
   return OK; /* To keep the compiler happy */
 }
diff --git a/arch/arm/src/tms570/tms570_serial.c b/arch/arm/src/tms570/tms570_serial.c
index 01767d85c3..c81cce447c 100644
--- a/arch/arm/src/tms570/tms570_serial.c
+++ b/arch/arm/src/tms570/tms570_serial.c
@@ -128,7 +128,7 @@ static int  tms570_attach(struct uart_dev_s *dev);
 static void tms570_detach(struct uart_dev_s *dev);
 static int  tms570_interrupt(int irq, void *context, void *arg);
 static int  tms570_ioctl(struct file *filep, int cmd, unsigned long arg);
-static int  tms570_receive(struct uart_dev_s *dev, uint32_t *status);
+static int  tms570_receive(struct uart_dev_s *dev, unsigned int *status);
 static void tms570_rxint(struct uart_dev_s *dev, bool enable);
 static bool tms570_rxavailable(struct uart_dev_s *dev);
 static void tms570_send(struct uart_dev_s *dev, int ch);
@@ -692,7 +692,7 @@ static int tms570_ioctl(struct file *filep, int cmd, unsigned long arg)
  *
  ****************************************************************************/
 
-static int tms570_receive(struct uart_dev_s *dev, uint32_t *status)
+static int tms570_receive(struct uart_dev_s *dev, unsigned int *status)
 {
   struct tms570_dev_s *priv = (struct tms570_dev_s *)dev->priv;
 
diff --git a/boards/arm/tms570/launchxl-tms57004/scripts/Make.defs b/boards/arm/tms570/launchxl-tms57004/scripts/Make.defs
index 82c2abe0a8..ea8a3849b1 100644
--- a/boards/arm/tms570/launchxl-tms57004/scripts/Make.defs
+++ b/boards/arm/tms570/launchxl-tms57004/scripts/Make.defs
@@ -25,7 +25,6 @@ include $(TOPDIR)/arch/arm/src/armv7-r/Toolchain.defs
 LDSCRIPT = flash-sram.ld
 ARCHSCRIPT += $(BOARD_DIR)$(DELIM)scripts$(DELIM)$(LDSCRIPT)
 
-ARCHCPUFLAGS = -mcpu=cortex-r4 -mbig-endian
 ARCHPICFLAGS = -fpic -msingle-pic-base -mpic-register=r10
 
 CFLAGS := $(ARCHCFLAGS) $(ARCHOPTIMIZATION) $(ARCHCPUFLAGS) $(ARCHINCLUDES) $(ARCHDEFINES) $(EXTRAFLAGS) -pipe