Giacomo Travaglini has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/31057 )

Change subject: dev-arm: Style fixes for src/dev/arm/gic_v2.hh
......................................................................

dev-arm: Style fixes for src/dev/arm/gic_v2.hh

JIRA: https://gem5.atlassian.net/browse/GEM5-667

Change-Id: I80dce7b72775beabafa3b54e915a369571f2e4c9
Signed-off-by: Giacomo Travaglini <[email protected]>
---
M src/dev/arm/gic_v2.hh
1 file changed, 39 insertions(+), 14 deletions(-)



diff --git a/src/dev/arm/gic_v2.hh b/src/dev/arm/gic_v2.hh
index 54c95f9..088d31e 100644
--- a/src/dev/arm/gic_v2.hh
+++ b/src/dev/arm/gic_v2.hh
@@ -217,7 +217,9 @@
      * 1b per interrupt, 32 bits per word, 31 words */
     uint32_t intEnabled[INT_BITS_MAX-1];

-    uint32_t& getIntEnabled(ContextID ctx, uint32_t ix) {
+    uint32_t&
+    getIntEnabled(ContextID ctx, uint32_t ix)
+    {
         if (ix == 0) {
             return getBankedRegs(ctx).intEnabled;
         } else {
@@ -230,7 +232,9 @@
      * 1b per interrupt, 32 bits per word, 31 words */
     uint32_t pendingInt[INT_BITS_MAX-1];

-    uint32_t& getPendingInt(ContextID ctx, uint32_t ix) {
+    uint32_t&
+    getPendingInt(ContextID ctx, uint32_t ix)
+    {
         assert(ix < INT_BITS_MAX);
         if (ix == 0) {
             return getBankedRegs(ctx).pendingInt;
@@ -244,7 +248,9 @@
      * 1b per interrupt, 32 bits per word, 31 words */
     uint32_t activeInt[INT_BITS_MAX-1];

-    uint32_t& getActiveInt(ContextID ctx, uint32_t ix) {
+    uint32_t&
+    getActiveInt(ContextID ctx, uint32_t ix)
+    {
         assert(ix < INT_BITS_MAX);
         if (ix == 0) {
             return getBankedRegs(ctx).activeInt;
@@ -258,7 +264,9 @@
      * 1b per interrupt, 32 bits per word, 31 words */
     uint32_t intGroup[INT_BITS_MAX-1];

-    uint32_t& getIntGroup(ContextID ctx, uint32_t ix) {
+    uint32_t&
+    getIntGroup(ContextID ctx, uint32_t ix)
+    {
         assert(ix < INT_BITS_MAX);
         if (ix == 0) {
             return getBankedRegs(ctx).intGroup;
@@ -276,7 +284,9 @@
      */
     uint8_t intPriority[GLOBAL_INT_LINES];

-    uint8_t& getIntPriority(ContextID ctx, uint32_t ix) {
+    uint8_t&
+    getIntPriority(ContextID ctx, uint32_t ix)
+    {
         assert(ix < INT_LINES_MAX);
         if (ix < SGI_MAX + PPI_MAX) {
             return getBankedRegs(ctx).intPriority[ix];
@@ -297,7 +307,9 @@
      * @param ix interrupt word index
      * @returns the interrupt config word
      */
-    uint32_t& getIntConfig(ContextID ctx, uint32_t ix) {
+    uint32_t&
+    getIntConfig(ContextID ctx, uint32_t ix)
+    {
         assert(ix < INT_BITS_MAX*2);
         if (ix < 2) {
             /** SGIs and PPIs **/
@@ -312,7 +324,9 @@
      */
     uint8_t cpuTarget[GLOBAL_INT_LINES];

-    uint8_t getCpuTarget(ContextID ctx, uint32_t ix) {
+    uint8_t
+    getCpuTarget(ContextID ctx, uint32_t ix) const
+    {
         assert(ctx < sys->threads.numRunning());
         assert(ix < INT_LINES_MAX);
         if (ix < SGI_MAX + PPI_MAX) {
@@ -335,7 +349,9 @@
         }
     }

-    bool isLevelSensitive(ContextID ctx, uint32_t int_num) {
+    bool
+    isLevelSensitive(ContextID ctx, uint32_t int_num)
+    {
         if (int_num == SPURIOUS_INT) {
             return false;
         } else {
@@ -345,7 +361,9 @@
         }
     }

-    bool isGroup0(ContextID ctx, uint32_t int_num) {
+    bool
+    isGroup0(ContextID ctx, uint32_t int_num)
+    {
         const uint32_t group_reg = getIntGroup(ctx, intNumToWord(int_num));
         return !bits(group_reg, intNumToBit(int_num));
     }
@@ -360,7 +378,9 @@
* 2) GICC_CTLR.FIQEn: controls whether the CPU interface signals Group 0
      * interrupts to a target processor using the FIQ or the IRQ signal
      */
-    bool isFiq(ContextID ctx, uint32_t int_num) {
+    bool
+    isFiq(ContextID ctx, uint32_t int_num)
+    {
         const bool is_group0 = isGroup0(ctx, int_num);
         const bool use_fiq = cpuControl[ctx].fiqEn;

@@ -374,7 +394,9 @@
     /** CPU enabled:
      * Checks if GICC_CTLR.EnableGrp0 or EnableGrp1 are set
      */
-    bool cpuEnabled(ContextID ctx) const {
+    bool
+    cpuEnabled(ContextID ctx) const
+    {
         return cpuControl[ctx].enableGrp0 ||
                cpuControl[ctx].enableGrp1;
     }
@@ -496,7 +518,9 @@
     Tick readDistributor(PacketPtr pkt);
     uint32_t readDistributor(ContextID ctx, Addr daddr,
                              size_t resp_sz);
-    uint32_t readDistributor(ContextID ctx, Addr daddr) override {
+    uint32_t
+    readDistributor(ContextID ctx, Addr daddr) override
+    {
         return readDistributor(ctx, daddr, 4);
     }

@@ -512,8 +536,9 @@
     Tick writeDistributor(PacketPtr pkt);
     void writeDistributor(ContextID ctx, Addr daddr,
                           uint32_t data, size_t data_sz);
-    void writeDistributor(ContextID ctx, Addr daddr,
-                                  uint32_t data) override {
+    void
+    writeDistributor(ContextID ctx, Addr daddr, uint32_t data) override
+    {
         return writeDistributor(ctx, daddr, data, 4);
     }


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

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I80dce7b72775beabafa3b54e915a369571f2e4c9
Gerrit-Change-Number: 31057
Gerrit-PatchSet: 1
Gerrit-Owner: Giacomo Travaglini <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to