[gem5-dev] Change in gem5/gem5[develop]: mem: Stop using SlavePort as a base class.

2021-10-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/51529 )


 (

1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the  
submitted one.

 )Change subject: mem: Stop using SlavePort as a base class.
..

mem: Stop using SlavePort as a base class.

There are other classes like "ExternalSlave" which still have the word
"Slave" in them, but at least this will make the build quit complaining
about the deprecated SlavePort.

Change-Id: I917c2880574cb77ea37c69dc2727ac5e84b83cd5
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51529
Reviewed-by: Jason Lowe-Power 
Reviewed-by: Daniel Carvalho 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/mem/external_slave.hh
1 file changed, 21 insertions(+), 3 deletions(-)

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




diff --git a/src/mem/external_slave.hh b/src/mem/external_slave.hh
index 51c884b..17ab42a 100644
--- a/src/mem/external_slave.hh
+++ b/src/mem/external_slave.hh
@@ -69,7 +69,7 @@
 {
   public:
 /** Derive from this class to create an external port interface */
-class ExternalPort : public SlavePort
+class ExternalPort : public ResponsePort
 {
   protected:
 ExternalSlave 
@@ -77,7 +77,7 @@
   public:
 ExternalPort(const std::string _,
 ExternalSlave _) :
-SlavePort(name_, _), owner(owner_)
+ResponsePort(name_, _), owner(owner_)
 { }

 ~ExternalPort() { }
@@ -90,7 +90,7 @@

 /* Handlers are specific to *types* of port not specific port
  * instantiations.  A handler will typically build a bridge to the
- * external port from gem5 and provide gem5 with a SlavePort that can  
be
+ * external port from gem5 and provide gem5 with a ResponsePort that  
can be

  * bound to for each call to Handler::getExternalPort.*/
 class Handler
 {

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/51529
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: I917c2880574cb77ea37c69dc2727ac5e84b83cd5
Gerrit-Change-Number: 51529
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Daniel Carvalho 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Nikos Nikoleris 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: mem: Stop using SlavePort as a base class.

2021-10-13 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/51529 )



Change subject: mem: Stop using SlavePort as a base class.
..

mem: Stop using SlavePort as a base class.

There are other classes like "ExternalSlave" which still have the word
"Slave" in them, but at least this will make the build quit complaining
about the deprecated SlavePort.

Change-Id: I917c2880574cb77ea37c69dc2727ac5e84b83cd5
---
M src/mem/external_slave.hh
1 file changed, 16 insertions(+), 3 deletions(-)



diff --git a/src/mem/external_slave.hh b/src/mem/external_slave.hh
index 51c884b..17ab42a 100644
--- a/src/mem/external_slave.hh
+++ b/src/mem/external_slave.hh
@@ -69,7 +69,7 @@
 {
   public:
 /** Derive from this class to create an external port interface */
-class ExternalPort : public SlavePort
+class ExternalPort : public ResponsePort
 {
   protected:
 ExternalSlave 
@@ -77,7 +77,7 @@
   public:
 ExternalPort(const std::string _,
 ExternalSlave _) :
-SlavePort(name_, _), owner(owner_)
+ResponsePort(name_, _), owner(owner_)
 { }

 ~ExternalPort() { }
@@ -90,7 +90,7 @@

 /* Handlers are specific to *types* of port not specific port
  * instantiations.  A handler will typically build a bridge to the
- * external port from gem5 and provide gem5 with a SlavePort that can  
be
+ * external port from gem5 and provide gem5 with a ResponsePort that  
can be

  * bound to for each call to Handler::getExternalPort.*/
 class Handler
 {

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/51529
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: I917c2880574cb77ea37c69dc2727ac5e84b83cd5
Gerrit-Change-Number: 51529
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s