From: stephane Domas <stephane.domas@univ-fcomte.fr>
Date: Fri, 5 May 2017 12:30:12 +0000 (+0200)
Subject: added creation of control ifaces
X-Git-Url: https://bilbo.iut-bm.univ-fcomte.fr/and/gitweb/blast.git/commitdiff_plain/dd3fe103df79a5a4c2962e2f05fec9a9ed58580d?ds=sidebyside;hp=--cc

added creation of control ifaces
---

dd3fe103df79a5a4c2962e2f05fec9a9ed58580d
diff --git a/AbstractInterface.cpp b/AbstractInterface.cpp
index e651f58..dde5930 100644
--- a/AbstractInterface.cpp
+++ b/AbstractInterface.cpp
@@ -10,6 +10,7 @@ AbstractInterface::AbstractInterface(AbstractBlock* _owner) {
   direction = Input;
   purpose = Data;  
   type = Boolean;
+  associatedIface = NULL;
 
 }
 
@@ -21,6 +22,7 @@ AbstractInterface::AbstractInterface(AbstractBlock* _owner, const QString& _name
   direction = _direction;
   purpose = _purpose;
   type = typeFromString(_type);
+  associatedIface = NULL;
 }
 
 AbstractInterface::AbstractInterface(AbstractInterface* other) {
@@ -30,6 +32,7 @@ AbstractInterface::AbstractInterface(AbstractInterface* other) {
   width = other->width;
   direction = other->direction;
   purpose = other->purpose;
+  associatedIface = NULL;
 }
 
 AbstractInterface::~AbstractInterface() {
@@ -126,6 +129,14 @@ void AbstractInterface::setDirection(int _direction) {
   }
 }
 
+bool AbstractInterface::setAssociatedIface(AbstractInterface* iface) {
+  if (purpose != Control) return false;
+  if (iface->purpose != Data) return false;
+  associatedIface = iface;
+  iface->associatedIface = this;  
+  return true;
+}
+
 
 int AbstractInterface::getIntDirection(QString str) {
     if(str == "input") return Input;
diff --git a/AbstractInterface.h b/AbstractInterface.h
index 5c0b83a..29be729 100644
--- a/AbstractInterface.h
+++ b/AbstractInterface.h
@@ -46,6 +46,7 @@ public :
   inline int getDirection() { return direction;}
   QString getDirectionString();  
   inline AbstractBlock *getOwner() { return owner;}
+  inline AbstractInterface* getAssociatedIface() { return associatedIface; }
 
   double getDoubleWidth() throw(QException);
 
@@ -63,7 +64,8 @@ public :
   inline void setType(int _type) { type = _type;}
   inline void setType(const QString& _type) { type = typeFromString(_type);}
   void setPurpose(int _purpose);
-  void setDirection(int _direction);  
+  void setDirection(int _direction);
+  bool setAssociatedIface(AbstractInterface* iface);
 
   // testers
   virtual bool isReferenceInterface();
@@ -95,6 +97,15 @@ protected:
   int direction;
 
   AbstractBlock* owner;
+  /*!
+   * \brief associatedIface the control (resp. data) interface that is bounded to this data (resp. control) interface
+   * If a reference block is designed to be fully integrated in Blast, nearly every data interface is bounded
+   * to a control interface that signals if the value presented on the interface is valid or not. associatedIface
+   * references this control interface if this is a data interface, and a data interface is this is a control interface.
+   * Note that the association is done by the call of setAssociatedIface() that must be done only for a control interface.
+   * (NB: a test is done in the method to prevent the other case).
+   */
+  AbstractInterface* associatedIface;
 };
 
 
diff --git a/ConnectedInterface.h b/ConnectedInterface.h
index 3a1fb4f..fc89e22 100644
--- a/ConnectedInterface.h
+++ b/ConnectedInterface.h
@@ -64,16 +64,7 @@ protected:
    * there may be a single interface owned by another block (functional or group) that is connected to
    * this interface. connecteFrom references such an interface if it exists.
    */
-  ConnectedInterface* connectedFrom;
-  /*!
-   * \brief associatedIface the control (resp. data) interface that is bounded to this data (resp. control) interface
-   * If a reference block is designed to be fully integrated in Blast, nearly every data interface is bounded
-   * to a control interface that signals if the value presented on the interface is valid or not. associatedIface
-   * references this control interface if this is a data interface, and a data interface is this is a control interface.
-   * Note that the association is done by the call of setAssociatedIface() that must be done only for a control interface.
-   * (NB: a test is done in the method to prevent the other case).
-   */
-  ConnectedInterface* associatedIface;
+  ConnectedInterface* connectedFrom;  
 };
 
 
diff --git a/FunctionalBlock.cpp b/FunctionalBlock.cpp
index f547084..cb8ceba 100644
--- a/FunctionalBlock.cpp
+++ b/FunctionalBlock.cpp
@@ -46,25 +46,44 @@ void FunctionalBlock::populate() {
   BlockParameter* p;
   AbstractInterface* inter;
 
+  // create parameters from reference block
   QList<BlockParameter*> lstParam = reference->getParameters();
   for(i=0;i<lstParam.size();i++) {
     p = lstParam.at(i)->clone();
     addParameter(p);
   }
 
-  QList<AbstractInterface *> lstInter = reference->getInterfaces();
-  for(i=0;i<lstInter.size();i++) {
+  // create interfaces from reference block
+  QList<AbstractInterface *> lstRef = reference->getInterfaces();
+  // store relation between functional and reference
+  QHash<AbstractInterface *, AbstractInterface *> hashIface;
+  for(i=0;i<lstRef.size();i++) {
     try {
-      inter = new FunctionalInterface(this, (ReferenceInterface*)lstInter.at(i));
+      inter = new FunctionalInterface(this, AI_TO_REF(lstRef.at(i)));
     }
     catch(Exception e) {
       cerr << "Abnormal case: " << qPrintable(e.getDefaultMessage()) << endl << "Aborting execution." << endl;
       exit(1);
     }
+    hashIface.insert(lstRef.at(i),inter);
 
     addInterface(inter);
   }
-
+    
+  AbstractInterface* funCtlIface = NULL;
+  AbstractInterface* funDataIface = NULL;
+  
+  for(i=0;i<lstRef.size();i++) {    
+    AbstractInterface* refIface = lstRef.at(i);    
+    if (refIface->getPurpose() == AbstractInterface::Control) {
+      funCtlIface = hashIface.value(refIface);
+      funDataIface = hashIface.value(refIface->getAssociatedIface());
+      if (! funCtlIface->setAssociatedIface(funDataIface)) {
+        cerr << "Abnormal case when associating a control interface to data" << endl << "Aborting execution." << endl;
+        exit(1);
+      }       
+    }
+  }
 }
 
 
diff --git a/FunctionalInterface.cpp b/FunctionalInterface.cpp
index 1b8ff07..d34cc8d 100644
--- a/FunctionalInterface.cpp
+++ b/FunctionalInterface.cpp
@@ -20,7 +20,7 @@ FunctionalInterface::FunctionalInterface(AbstractBlock* _owner, ReferenceInterfa
   width = reference->getWidth();
   direction = reference->getDirection();
   purpose = reference->getPurpose();  
-  connectedFrom = NULL;
+  connectedFrom = NULL;  
 }
 
 bool FunctionalInterface::isFunctionalInterface() {
diff --git a/ReferenceBlock.cpp b/ReferenceBlock.cpp
index 722fc12..e09b118 100644
--- a/ReferenceBlock.cpp
+++ b/ReferenceBlock.cpp
@@ -224,6 +224,7 @@ void ReferenceBlock::loadInterfaces(QDomElement &elt) throw(Exception) {
   if ((elt.isNull()) || (elt.tagName() != "interfaces")) throw (Exception(BLOCKFILE_CORRUPTED));
 
   QDomElement eltInputs = elt.firstChildElement("inputs");
+  // getting each input
   QDomNodeList listNodeInputs = eltInputs.elementsByTagName("input");
   for(int i=0;i<listNodeInputs.size();i++) {
     QDomNode node = listNodeInputs.at(i);
@@ -242,7 +243,21 @@ void ReferenceBlock::loadInterfaces(QDomElement &elt) throw(Exception) {
     inter = new ReferenceInterface(this,nameStr,typeStr,widthStr,AbstractInterface::Input, purpose, mult);
     inputs.append(inter);
   }
-
+  // getting each control
+  QDomNodeList listNodeInCtl = eltInputs.elementsByTagName("control");
+  for(int i=0;i<listNodeInCtl.size();i++) {
+    QDomNode node = listNodeInCtl.at(i);
+    QDomElement eltInput = node.toElement();
+    nameStr = eltInput.attribute("iface","none");
+    AbstractInterface* dataIface = getIfaceFromName(nameStr);
+    if (dataIface == NULL) throw (Exception(BLOCKFILE_CORRUPTED));
+    nameStr = dataIface->getName()+"_ctl";
+    inter = new ReferenceInterface(this,nameStr,"boolean","1",AbstractInterface::Input, AbstractInterface::Control, 1);
+    if (!inter->setAssociatedIface(dataIface)) {
+      throw (Exception(BLOCKFILE_CORRUPTED));      
+    }
+    inputs.append(inter);
+  }
   QDomElement eltOutputs = eltInputs.nextSiblingElement("outputs");
   QDomNodeList listNodeOutputs = eltOutputs.elementsByTagName("output");
   for(int i=0;i<listNodeOutputs.size();i++) {
@@ -259,6 +274,21 @@ void ReferenceBlock::loadInterfaces(QDomElement &elt) throw(Exception) {
     inter = new ReferenceInterface(this,nameStr,typeStr,widthStr,AbstractInterface::Output, purpose, mult);
     outputs.append(inter);
   }
+  // getting each control
+  QDomNodeList listNodeOutCtl = eltOutputs.elementsByTagName("control");
+  for(int i=0;i<listNodeOutCtl.size();i++) {
+    QDomNode node = listNodeOutCtl.at(i);
+    QDomElement eltOutput = node.toElement();
+    nameStr = eltOutput.attribute("iface","none");
+    AbstractInterface* dataIface = getIfaceFromName(nameStr);
+    if (dataIface == NULL) throw (Exception(BLOCKFILE_CORRUPTED));
+    nameStr = dataIface->getName()+"_ctl";
+    inter = new ReferenceInterface(this,nameStr,"boolean","1",AbstractInterface::Output, AbstractInterface::Control, 1);
+    if (!inter->setAssociatedIface(dataIface)) {
+      throw (Exception(BLOCKFILE_CORRUPTED));      
+    }
+    outputs.append(inter);
+  }
 
   QDomElement eltBidirs = eltInputs.nextSiblingElement("bidirs");
   QDomNodeList listNodeBidirs = eltBidirs.elementsByTagName("bidir");
diff --git a/ReferenceInterface.cpp b/ReferenceInterface.cpp
index faa51e8..a0f50ca 100644
--- a/ReferenceInterface.cpp
+++ b/ReferenceInterface.cpp
@@ -17,6 +17,13 @@ throw (Exception) : AbstractInterface(_owner, _name, _type, _width, _direction,
   if (_owner->isReferenceBlock()) throw(Exception(BLOCK_INVALID_TYPE));
 
   multiplicity = _multiplicity;
+  
+  if (purpose == Control) {
+    // override some attributes with forced values
+    type = Boolean;
+    width = "1";
+    multiplicity = 1;
+  }  
   if (direction == InOut) {
     multiplicity = 1;
   }
@@ -42,9 +49,9 @@ int ReferenceInterface::translatePurpose(const QString& txt) {
   else if (txt == "reset") {
     return Reset;
   }
-  if (txt == "wb") {
+  else if (txt == "wb") {
     return Wishbone;
-  }
+  }  
   return Data;
 }
 
diff --git a/blast.creator.user b/blast.creator.user
index 95c4b8a..570664a 100755
--- a/blast.creator.user
+++ b/blast.creator.user
@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE QtCreatorProject>
-<!-- Written by QtCreator 3.2.1, 2017-05-04T21:30:29. -->
+<!-- Written by QtCreator 3.2.1, 2017-05-05T14:29:41. -->
 <qtcreator>
  <data>
   <variable>EnvironmentId</variable>
-  <value type="QByteArray">{c8006d66-d34f-42be-ad10-d0207752286d}</value>
+  <value type="QByteArray">{1d077e47-e3a1-47fd-8b12-4de650e39df5}</value>
  </data>
  <data>
   <variable>ProjectExplorer.Project.ActiveTarget</variable>
@@ -60,12 +60,12 @@
   <valuemap type="QVariantMap">
    <value type="QString" key="ProjectExplorer.ProjectConfiguration.DefaultDisplayName">Desktop</value>
    <value type="QString" key="ProjectExplorer.ProjectConfiguration.DisplayName">Desktop</value>
-   <value type="QString" key="ProjectExplorer.ProjectConfiguration.Id">{2c9bf876-3476-44eb-8065-1f0844704dda}</value>
+   <value type="QString" key="ProjectExplorer.ProjectConfiguration.Id">{451ee8a3-56ff-4aba-8a8e-3da882cc142e}</value>
    <value type="int" key="ProjectExplorer.Target.ActiveBuildConfiguration">0</value>
    <value type="int" key="ProjectExplorer.Target.ActiveDeployConfiguration">0</value>
    <value type="int" key="ProjectExplorer.Target.ActiveRunConfiguration">0</value>
    <valuemap type="QVariantMap" key="ProjectExplorer.Target.BuildConfiguration.0">
-    <value type="QString" key="ProjectExplorer.BuildConfiguration.BuildDirectory">/home/sdomas/Projet/Blast/code/blast</value>
+    <value type="QString" key="ProjectExplorer.BuildConfiguration.BuildDirectory">/localhome/sdomas/Projet/Blast/code/blast</value>
     <valuemap type="QVariantMap" key="ProjectExplorer.BuildConfiguration.BuildStepList.0">
      <valuemap type="QVariantMap" key="ProjectExplorer.BuildStepList.Step.0">
       <valuelist type="QVariantList" key="GenericProjectManager.GenericMakeStep.BuildTargets">
diff --git a/lib/references/references.bmf b/lib/references/references.bmf
index a5ec5ae..594beb0 100644
Binary files a/lib/references/references.bmf and b/lib/references/references.bmf differ