Mehrfache Instanziierungen

[Said (Gast)]

Hallo Leute, steige gerade frisch in VHDL ein und bräuchte da mal ein 
wenig Hilfe. Wie kann ich mehrere Instanzen in mein topmodul aufrufen?
In diesem Beispiel habe habe ich versucht zwei Instanzen aufzubauen, 
aber es wird ein Fehler angezeigt.

library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity top_level is
    Port ( SW     : in   STD_LOGIC_VECTOR(2 downto 0);
           LED1_B : out  STD_LOGIC
    );
end top_level;

architecture Behavioral of top_level is

------------Signale für and_gate_1-------------

signal  store_output_1    :   std_logic;
signal  store_output_2    :   std_logic;

begin    

--------------------Instanz für and_gate_1-----

   Inst_and_gate_1: entity work.and_gate_1 PORT MAP(
   
        input_1     =>  SW(0),
        input_2     =>  SW(1),
        output_1    =>  store_output_1   
   );
--------------------Ende Instanz and_gate_1----



--------------------Instanz and_gate_2---------

    Inst_and_gate_2: entity work.and_gate_1 PORT MAP(
    
        ip2     =>  store_output_1 ,
        ipc     =>  SW(2),
        out_2   =>  store_output_2
    );
-------------------Ende Instanz and_gate_2-----

LED1_B   <=  store_output_1 and not SW(2);

end Behavioral;

Und das ist das and_gate_1:

library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity and_gate_1 is
    Port ( input_1  : in     STD_LOGIC;
           input_2  : in     STD_LOGIC;
           ip2      : in     STD_LOGIC;
           ipc      : in     STD_LOGIC;
           out_2    : out    STD_LOGIC;
           output_1 : out STD_LOGIC);
end and_gate_1;

architecture Behavioral of and_gate_1 is

begin
   
    output_1    <=  input_1 and input_2;
    
    out_2       <=  ip2 or ipc;

end Behavioral;

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Said schrieb:
> aber es wird ein Fehler angezeigt.
Welcher denn?
Der, dass du in jeder Instanz jeweils die Hälfte der Ports der 
Komponente nicht verwendest?

Probiers mal so:

:
:
    Inst_or_gate_2: entity work.or_gate_1 PORT MAP(
    
        ip2     =>  store_output_1 ,
        ipc     =>  SW(2),
        out_2   =>  store_output_2
    );

:
:



library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity and_gate_1 is
    Port ( input_1  : in     STD_LOGIC;
           input_2  : in     STD_LOGIC;
           output_1 : out STD_LOGIC);
end and_gate_1;

architecture Behavioral of and_gate_1 is

begin
   
    output_1    <=  input_1 and input_2;
    
end Behavioral;




library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity or_gate_1 is
    Port ( ip2      : in     STD_LOGIC;
           ipc      : in     STD_LOGIC;
           out_2    : out    STD_LOGIC);
end or_gate_1;

architecture Behavioral of or_gate_1 is

begin
   
    out_2       <=  ip2 or ipc;

end Behavioral;

> steige gerade frisch in VHDL ein
Dann bitte weiterhin die Bedienungsanleitung beachten:

Antwort schreiben
Wichtige Regeln - erst lesen, dann posten!
    ....
    Längeren Sourcecode nicht im Text einfügen, sondern als Dateianhang

Formatierung (mehr Informationen...)
    ....
    [vhdl]VHDL-Code[/vhdl]

Die steht über jeder Eingabebox hier.

[Said (Gast)]

In  Inst_and_gate_1 sagt er ip2 has no actual or default value.

In  Inst_and_gate_2 sagt er input_1 has no actual or default value.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Wie gesagt: die Fehlermeldung passt zu deinem Code.
Jeder Port einer Instanz muss eine Zuweisung bekommen.

Du könntest sogar noch deinen Code "hinmurksen":

--------------------Instanz für and_gate_1-----

   Inst_and_gate_1: entity work.and_gate_1 PORT MAP(
   
        input_1     =>  SW(0),
        input_2     =>  SW(1),
        output_1    =>  store_output_1,
        ip2     =>  '0',
        ipc     =>  '0',
        out_2   =>  OPEN
   );
--------------------Ende Instanz and_gate_1----



--------------------Instanz and_gate_2---------

    Inst_and_gate_2: entity work.and_gate_1 PORT MAP(
    
        input_1     =>  '0',
        input_2     =>  '0',
        output_1    =>  OPEN,
        ip2     =>  store_output_1 ,
        ipc     =>  SW(2),
        out_2   =>  store_output_2
    );
-------------------Ende Instanz and_gate_2-----

Oder so:

library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity and_gate_1 is
    Port ( input_1  : in     STD_LOGIC := '0';
           input_2  : in     STD_LOGIC := '0';
           ip2      : in     STD_LOGIC := '0';
           ipc      : in     STD_LOGIC := '0';
           out_2    : out    STD_LOGIC;
           output_1 : out STD_LOGIC);
end and_gate_1;

architecture Behavioral of and_gate_1 is

begin
   
    output_1    <=  input_1 and input_2;
    
    out_2       <=  ip2 or ipc;

end Behavioral;



library IEEE;
use IEEE.STD_LOGIC_1164.ALL;


entity top_level is
    Port ( SW     : in   STD_LOGIC_VECTOR(2 downto 0);
           LED1_B : out  STD_LOGIC
    );
end top_level;

architecture Behavioral of top_level is

------------Signale für and_gate_1-------------

signal  store_output_1    :   std_logic;
signal  store_output_2    :   std_logic;

begin    

--------------------Instanz für and_gate_1-----

   Inst_and_gate_1: entity work.and_gate_1 PORT MAP(
   
        input_1     =>  SW(0),
        input_2     =>  SW(1),
        output_1    =>  store_output_1,
        ip2     =>  OPEN,
        ipc     =>  OPEN,
        out_2   =>  OPEN
   );
--------------------Ende Instanz and_gate_1----



--------------------Instanz and_gate_2---------

    Inst_and_gate_2: entity work.and_gate_1 PORT MAP(
    
        input_1     =>  OPEN,
        input_2     =>  OPEN,
        output_1    =>  OPEN,
        ip2     =>  store_output_1 ,
        ipc     =>  SW(2),
        out_2   =>  store_output_2
    );
-------------------Ende Instanz and_gate_2-----
LED1_B   <=  store_output_1 and not SW(2);

end Behavioral;

Aber eigentlich ist so ein "Obfuscated"-Code ein Kündigungsgrund!

[Said (Gast)]

Vielen Danke! Hat funktioniert!

[freduardo (Gast)]

Lothar M. schrieb:
>
>
>     Inst_and_gate_2: entity work.and_gate_1 PORT MAP(
>
>         input_1     =>  OPEN,
>         input_2     =>  OPEN,
>         output_1    =>  OPEN,
>         ip2     =>  store_output_1 ,
>         ipc     =>  SW(2),
>         out_2   =>  store_output_2
>     );
> -------------------Ende Instanz and_gate_2-----
> LED1_B   <=  store_output_1 and not SW(2);
>
> end Behavioral;
>


Bitte nicht als Klugscheisserei verstehen, das ist es nicht:

Ist die Zuweisung von inputs als open zulässig? Ich dachte bisher das 
geht nur mit Outputs (die man zumindest in Vivado/ISE bei der 
Instanziierung auch komplett weglassen kann).

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

freduardo schrieb:
> Ist die Zuweisung von inputs als open zulässig?
Nur, wenn wie in meinem Beispiel gleichzeitig ein Defaultwert in der 
Portdefinition angegeben ist.

Dann könnte man vermutlich sogar die Portzuweisung weglassen. Ich habe 
mich bisher noch nie auf dieses Glatteis begeben...  ;-)

[freduardo (Gast)]

Danke für die Info.

Da wir gerade bei der Frage sind, und ich keine IDE zur Hand habe, um es 
zu testen: Darf man dann bei der Instanziierung die Inputs auch komplett 
weglassen, wenn in der Entity Defaultwerte vergeben wurden?

[freduardo (Gast)]

Sorry, für den Doppelpost.

Meine Frage hat sich durch Lothars Edit auch schon beantwortet.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

freduardo schrieb:
> Darf man dann bei der Instanziierung die Inputs auch komplett weglassen,
> wenn in der Entity Defaultwerte vergeben wurden?
Geht tatsächlich, allerdings würden mich die vielen Warnungen stören:

WARNING:Xst:752 - Unconnected input port 'ip2' of component 'and_gate_1' is tied to default value.
WARNING:Xst:752 - Unconnected input port 'ipc' of component 'and_gate_1' is tied to default value.
WARNING:Xst:753 - Unconnected output port 'out_2' of component 'and_gate_1'.
WARNING:Xst:752 - Unconnected input port 'input_1' of component 'and_gate_1' is tied to default value.
WARNING:Xst:752 - Unconnected input port 'input_2' of component 'and_gate_1' is tied to default value.
WARNING:Xst:753 - Unconnected output port 'output_1' of component 'and_gate_1'.
WARNING:Xst:646 - Signal <store_output_2> is assigned but never used. This unconnected signal will be trimmed during the optimization process.
WARNING:Xst:524 - All outputs of the instance <Inst_and_gate_2> of the block <and_gate_1> are unconnected in block <top_level>.
   This instance will be removed from the design along with all underlying logic

Man sieht hier auch, dass eh' die Hälfte des Designs wegoptimiert wird, 
weil store_output_2 nicht verwendet wird...

[Vancouver (Gast)]

Es kommt oft vor, dass von einem komplexen Core nur ein Teil der Ports 
verwendet wird, der Rest ist auf einen Defaultwert geklemmt oder open. 
Die sauberste Lösung ist dann, einen Wrapper drumherum zu schreiben, der 
sich um die unbenutzten Ports kümmert und nur die tatsächlich benötigten 
Signale durchschleift. Die Instanz den Wrappers ist dann sehr kompakt 
und der Code bleibt verständlich. Gute Beispiele sind die Xilinx 
Blockrams oder MCMMs. Ohne Wrapper sind die kaum zu handeln, außer man 
steht auf wirklich undurchsichtigen HDL-Code.