[gem5-dev] Change in gem5/gem5[master]: dev-arm: A9SCU fixup

2019-07-10 Thread Tiago Mück (Gerrit)
Tiago Mück has submitted this change and it was merged. (  
https://gem5-review.googlesource.com/c/public/gem5/+/19408 )


Change subject: dev-arm: A9SCU fixup
..

dev-arm: A9SCU fixup

Shifting instead of expensive power.

Change-Id: I164933257db125e18721c5b8bcaf9702030ebf40
Signed-off-by: Tiago Mück 
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19408
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/dev/arm/a9scu.cc
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/dev/arm/a9scu.cc b/src/dev/arm/a9scu.cc
index 677101b..20a0c33 100644
--- a/src/dev/arm/a9scu.cc
+++ b/src/dev/arm/a9scu.cc
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010 ARM Limited
+ * Copyright (c) 2010,2019 ARM Limited
  * All rights reserved
  *
  * The license below extends only to copyright in the software and shall
@@ -73,7 +73,7 @@
 fatal("Too many CPUs (%d) for A9SCU!\n",  
sys->numContexts());

 }
 int smp_bits, core_cnt;
-smp_bits = power(2,sys->numContexts()) - 1;
+smp_bits = (1 << sys->numContexts()) - 1;
 core_cnt = sys->numContexts() - 1;
 pkt->setLE(smp_bits << 4 | core_cnt);
 break;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/19408
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I164933257db125e18721c5b8bcaf9702030ebf40
Gerrit-Change-Number: 19408
Gerrit-PatchSet: 2
Gerrit-Owner: Tiago Mück 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Tiago Mück 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

[gem5-dev] Change in gem5/gem5[master]: dev-arm: A9SCU fixup

2019-07-08 Thread Tiago Mück (Gerrit)
Tiago Mück has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/19408



Change subject: dev-arm: A9SCU fixup
..

dev-arm: A9SCU fixup

Shifting instead of expensive power.

Change-Id: I164933257db125e18721c5b8bcaf9702030ebf40
Signed-off-by: Tiago Mück 
---
M src/dev/arm/a9scu.cc
1 file changed, 2 insertions(+), 2 deletions(-)



diff --git a/src/dev/arm/a9scu.cc b/src/dev/arm/a9scu.cc
index 677101b..20a0c33 100644
--- a/src/dev/arm/a9scu.cc
+++ b/src/dev/arm/a9scu.cc
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010 ARM Limited
+ * Copyright (c) 2010,2019 ARM Limited
  * All rights reserved
  *
  * The license below extends only to copyright in the software and shall
@@ -73,7 +73,7 @@
 fatal("Too many CPUs (%d) for A9SCU!\n",  
sys->numContexts());

 }
 int smp_bits, core_cnt;
-smp_bits = power(2,sys->numContexts()) - 1;
+smp_bits = (1 << sys->numContexts()) - 1;
 core_cnt = sys->numContexts() - 1;
 pkt->setLE(smp_bits << 4 | core_cnt);
 break;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/19408
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I164933257db125e18721c5b8bcaf9702030ebf40
Gerrit-Change-Number: 19408
Gerrit-PatchSet: 1
Gerrit-Owner: Tiago Mück 
Gerrit-MessageType: newchange
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev