diff mbox series

[03/17] sim: bfin: fix -Wunused-but-set-variable warnings

Message ID 20231207035937.14920-3-vapier@gentoo.org
State New
Headers
Series [01/17] sim: arm: fix -Wunused-but-set-variable warnings |

Commit Message

Mike Frysinger Dec. 7, 2023, 3:59 a.m. UTC 
  ---
 sim/bfin/bfin-sim.c     | 3 ++-
 sim/bfin/dv-bfin_emac.c | 3 ---
 sim/bfin/dv-bfin_mmu.c  | 4 +---
 sim/bfin/dv-bfin_pll.c  | 2 --
 sim/bfin/dv-bfin_rtc.c  | 2 --
 sim/bfin/dv-bfin_sic.c  | 8 --------
 6 files changed, 3 insertions(+), 19 deletions(-)
diff mbox series

Patch

diff --git a/sim/bfin/bfin-sim.c b/sim/bfin/bfin-sim.c
index a9b8fd09220f..12c7827f9ec4 100644
--- a/sim/bfin/bfin-sim.c
+++ b/sim/bfin/bfin-sim.c
@@ -1592,7 +1592,8 @@  decode_macfunc (SIM_CPU *cpu, int which, int op, int h0, int h1, int src0,
 
   if (op != 3)
     {
-      bu8 sgn0 = (acc >> 31) & 1;
+      /* TODO: Figure out how the 32-bit sign is used.  */
+      ATTRIBUTE_UNUSED bu8 sgn0 = (acc >> 31) & 1;
       bu8 sgn40 = (acc >> 39) & 1;
       bu40 nosat_acc;
 
diff --git a/sim/bfin/dv-bfin_emac.c b/sim/bfin/dv-bfin_emac.c
index 0288b7404999..1ab7cd63554d 100644
--- a/sim/bfin/dv-bfin_emac.c
+++ b/sim/bfin/dv-bfin_emac.c
@@ -539,11 +539,8 @@  bfin_emac_tap_init (struct hw *me)
 {
 #if WITH_TUN
   struct bfin_emac *emac = hw_data (me);
-  const hw_unit *unit;
   int flags;
 
-  unit = hw_unit_address (me);
-
   emac->tap = open ("/dev/net/tun", O_RDWR);
   if (emac->tap == -1)
     {
diff --git a/sim/bfin/dv-bfin_mmu.c b/sim/bfin/dv-bfin_mmu.c
index 71c7176b88bf..c14e2088c1f1 100644
--- a/sim/bfin/dv-bfin_mmu.c
+++ b/sim/bfin/dv-bfin_mmu.c
@@ -451,7 +451,7 @@  _mmu_check_addr (SIM_CPU *cpu, bu32 addr, bool write, bool inst, int size)
 {
   SIM_DESC sd = CPU_STATE (cpu);
   struct bfin_mmu *mmu;
-  bu32 *fault_status, *fault_addr, *mem_control, *cplb_addr, *cplb_data;
+  bu32 *mem_control, *cplb_addr, *cplb_data;
   bu32 faults;
   bool supv, do_excp, dag1;
   int i, hits;
@@ -469,8 +469,6 @@  _mmu_check_addr (SIM_CPU *cpu, bu32 addr, bool write, bool inst, int size)
     }
 
   mmu = MMU_STATE (cpu);
-  fault_status = inst ? &mmu->icplb_fault_status : &mmu->dcplb_fault_status;
-  fault_addr = inst ? &mmu->icplb_fault_addr : &mmu->dcplb_fault_addr;
   mem_control = inst ? &mmu->imem_control : &mmu->dmem_control;
   cplb_addr = inst ? &mmu->icplb_addr[0] : &mmu->dcplb_addr[0];
   cplb_data = inst ? &mmu->icplb_data[0] : &mmu->dcplb_data[0];
diff --git a/sim/bfin/dv-bfin_pll.c b/sim/bfin/dv-bfin_pll.c
index dd95013f2f49..863b07f49ec3 100644
--- a/sim/bfin/dv-bfin_pll.c
+++ b/sim/bfin/dv-bfin_pll.c
@@ -56,7 +56,6 @@  bfin_pll_io_write_buffer (struct hw *me, const void *source,
   bu32 mmr_off;
   bu32 value;
   bu16 *value16p;
-  bu32 *value32p;
   void *valuep;
 
   /* Invalid access mode is higher priority than missing register.  */
@@ -71,7 +70,6 @@  bfin_pll_io_write_buffer (struct hw *me, const void *source,
   mmr_off = addr - pll->base;
   valuep = (void *)((uintptr_t)pll + mmr_base() + mmr_off);
   value16p = valuep;
-  value32p = valuep;
 
   HW_TRACE_WRITE ();
 
diff --git a/sim/bfin/dv-bfin_rtc.c b/sim/bfin/dv-bfin_rtc.c
index b6ae0a537712..fdb63f3a4f9a 100644
--- a/sim/bfin/dv-bfin_rtc.c
+++ b/sim/bfin/dv-bfin_rtc.c
@@ -59,7 +59,6 @@  bfin_rtc_io_write_buffer (struct hw *me, const void *source,
   bu32 mmr_off;
   bu32 value;
   bu16 *value16p;
-  bu32 *value32p;
   void *valuep;
 
   /* Invalid access mode is higher priority than missing register.  */
@@ -74,7 +73,6 @@  bfin_rtc_io_write_buffer (struct hw *me, const void *source,
   mmr_off = addr - rtc->base;
   valuep = (void *)((uintptr_t)rtc + mmr_base() + mmr_off);
   value16p = valuep;
-  value32p = valuep;
 
   HW_TRACE_WRITE ();
 
diff --git a/sim/bfin/dv-bfin_sic.c b/sim/bfin/dv-bfin_sic.c
index 5210cf0a8eeb..ac302a7f8519 100644
--- a/sim/bfin/dv-bfin_sic.c
+++ b/sim/bfin/dv-bfin_sic.c
@@ -147,7 +147,6 @@  bfin_sic_52x_io_write_buffer (struct hw *me, const void *source, int space,
   struct bfin_sic *sic = hw_data (me);
   bu32 mmr_off;
   bu32 value;
-  bu16 *value16p;
   bu32 *value32p;
   void *valuep;
 
@@ -162,7 +161,6 @@  bfin_sic_52x_io_write_buffer (struct hw *me, const void *source, int space,
 
   mmr_off = addr - sic->base;
   valuep = (void *)((uintptr_t)sic + mmr_base() + mmr_off);
-  value16p = valuep;
   value32p = valuep;
 
   HW_TRACE_WRITE ();
@@ -261,7 +259,6 @@  bfin_sic_537_io_write_buffer (struct hw *me, const void *source, int space,
   struct bfin_sic *sic = hw_data (me);
   bu32 mmr_off;
   bu32 value;
-  bu16 *value16p;
   bu32 *value32p;
   void *valuep;
 
@@ -276,7 +273,6 @@  bfin_sic_537_io_write_buffer (struct hw *me, const void *source, int space,
 
   mmr_off = addr - sic->base;
   valuep = (void *)((uintptr_t)sic + mmr_base() + mmr_off);
-  value16p = valuep;
   value32p = valuep;
 
   HW_TRACE_WRITE ();
@@ -375,7 +371,6 @@  bfin_sic_54x_io_write_buffer (struct hw *me, const void *source, int space,
   struct bfin_sic *sic = hw_data (me);
   bu32 mmr_off;
   bu32 value;
-  bu16 *value16p;
   bu32 *value32p;
   void *valuep;
 
@@ -390,7 +385,6 @@  bfin_sic_54x_io_write_buffer (struct hw *me, const void *source, int space,
 
   mmr_off = addr - sic->base;
   valuep = (void *)((uintptr_t)sic + mmr_base() + mmr_off);
-  value16p = valuep;
   value32p = valuep;
 
   HW_TRACE_WRITE ();
@@ -482,7 +476,6 @@  bfin_sic_561_io_write_buffer (struct hw *me, const void *source, int space,
   struct bfin_sic *sic = hw_data (me);
   bu32 mmr_off;
   bu32 value;
-  bu16 *value16p;
   bu32 *value32p;
   void *valuep;
 
@@ -497,7 +490,6 @@  bfin_sic_561_io_write_buffer (struct hw *me, const void *source, int space,
 
   mmr_off = addr - sic->base;
   valuep = (void *)((uintptr_t)sic + mmr_base() + mmr_off);
-  value16p = valuep;
   value32p = valuep;
 
   HW_TRACE_WRITE ();