реклама на сайте
подробности

 
 
> получил предупреждения
serg_k1
сообщение Aug 29 2013, 13:48
Сообщение #1


Частый гость
**

Группа: Участник
Сообщений: 115
Регистрация: 21-03-07
Пользователь №: 26 368



Здравстуйте,помогите разобраться.
получил предупреждения
WARNING:PhysDesignRules:372 - Gated clock. Clock net
Ch_M_BC/GND_12_o_GND_12_o_AND_159_o is sourced by a combinatorial pin. This
is not good design practice. Use the CE pin to control the loading of data
into the flip-flop.
как найти причину и насколько это важно?
методом исключений из описания получил , что вроде бы это получается из следующего куска кода. таких есть несколько. получается , что нужно упрощать?
Код
//----------- 2   3   4   5-е слово  -------------------------------------            
if ( reg_nomer_slova == 8'h02 | reg_nomer_slova == 8'h03 | reg_nomer_slova == 8'h04 | reg_nomer_slova == 8'h05)
  begin
casez (nomer_bit)
        5'b00???:   begin     //0..7                                 
            reg_CM_BC <=  (reg_nomer_slova == 8'h02) ?
                ( ( reg_kod_oper  == paket_znak ) ?  reg_kolich_znakov[nomer_bit]  :
                            ( ( reg_kod_oper  == paket_linia ) ?  reg_nach_koord_linii_Y[nomer_bit]  : 1'bz  ) ) :
                    (reg_nomer_slova == 8'h03) ?          
                        ( ( reg_kod_oper  == paket_znak ) ?  reg_koord_perv_znaka_Y[nomer_bit]  :  
                            ( ( reg_kod_oper  == paket_linia )  ?  reg_nach_koord_linii_X[nomer_bit]  : 1'bz  ) ) :
                        (reg_nomer_slova == 8'h04) ?        
                            ( ( reg_kod_oper  == paket_znak ) ?  reg_koord_perv_znaka_X[nomer_bit]  :  
                                ( ( reg_kod_oper  == paket_linia )  ?  reg_proekcii_linii_Y[nomer_bit]  : 1'bz  ) ) :        
                                ( ( reg_kod_oper  == paket_znak ) ?  reg_kod_znaka_bit_ml[nomer_bit]  :  
                                    ( ( reg_kod_oper  == paket_linia )  ?  reg_proekcii_linii_X[nomer_bit]  : 1'bz  ) );
                nomer_bit <= nomer_bit + 1'b1;    
            end
    5'b0100?:   begin     //8,9
            reg_podgotovki_smeni_slova <= 1'b0;
            reg_CM_BC <=  (reg_nomer_slova == 8'h02) ?
                        ( ( reg_kod_oper  == paket_znak ) ?  reg_kolich_znakov[nomer_bit]  :
                            ( ( reg_kod_oper  == paket_linia ) ?  reg_nach_koord_linii_Y[nomer_bit]  : 1'bz  ) ) :
                        (reg_nomer_slova == 8'h03) ?          
                            ( ( reg_kod_oper  == paket_znak ) ?  reg_koord_perv_znaka_Y[nomer_bit]  :  
                                ( ( reg_kod_oper  == paket_linia )  ?  reg_nach_koord_linii_X[nomer_bit]  : 1'bz  ) ) :
                            (reg_nomer_slova == 8'h04) ?        
                                ( ( reg_kod_oper  == paket_znak ) ?  reg_koord_perv_znaka_X[nomer_bit]  :  
                                ( ( reg_kod_oper  == paket_linia )  ?  reg_proekcii_linii_Y[nomer_bit]  : 1'bz  ) ):        
                                        ( ( reg_kod_oper  == paket_znak ) ?  reg_kod_znaka_bit_st[nomer_bit-8]  :  
                                    ( ( reg_kod_oper  == paket_linia )  ?  reg_proekcii_linii_X[nomer_bit]  : 1'bz  ) );
                    nomer_bit <= nomer_bit + 1'b1;    
                end                              
        5'b0101?:   begin     //10,11
        reg_CM_BC <=  (reg_nomer_slova == 8'h02 | reg_nomer_slova == 8'h03 | reg_nomer_slova == 8'h04) ?
                        ( ( reg_kod_oper  == paket_znak ) ?   1'b0  :
                            ( ( reg_kod_oper  == paket_linia ) ?   1'b0   : 1'bz  ) ) :
                                ( ( reg_kod_oper  == paket_znak ) ?  reg_kod_znaka_bit_st[nomer_bit-8]  :  
                                    ( ( reg_kod_oper  == paket_linia )  ?   1'b0  : 1'bz  ) );
                    nomer_bit <= nomer_bit + 1'b1;    
                end    
        5'b0110?:   begin      //12,13
            reg_CM_BC <=
                                       ...
                nomer_bit <= nomer_bit + 1'b1;    
            end    
        5'b01110:   begin      //14
            reg_CM_BC <=  ...

                nomer_bit <= nomer_bit + 1'b1;    
            end    
    5'b01111:   begin      //15
            reg_CM_BC <=
                                         ...
                nomer_bit <= nomer_bit + 1'b1;    
            end
    5'b10000:   begin      
                reg_CM_BC <=  1'b0;
                nomer_bit <= nomer_bit + 1'b1;    
             end          
    5'b10001:   begin
                reg_CM_BC <=  reg_kontr_razr;
                nomer_bit <= nomer_bit + 1'b1;                                
                            ...    
                                        
            end                          
                    default :   begin
                                
                                nomer_bit <= 5'b00000;
                                end      
        endcase
end    //reg_nomer_slova == 8'h02

и еще одно предупреждение.
PhysDesignRules:372 - Gated clock. Clock net Ch_M_BC/reg_IZ_BC_CIB_BC is
sourced by a combinatorial pin. This is not good design practice. Use the CE
pin to control the loading of data into the flip-flop.
история такая.
есть два входных сигнала импульс IZ_BC и еще несколько импульсов CIB_BC. непересекающихся. под частоту в always получен общий reg_IZ_BC_CIB_BC. и далее на нем сделан свой always.
Go to the top of the page
 
+Quote Post
 
Start new topic
Ответов
Maverick
сообщение Aug 29 2013, 20:28
Сообщение #2


я только учусь...
******

Группа: Модераторы
Сообщений: 3 447
Регистрация: 29-01-07
Из: Украина
Пользователь №: 24 839



Цитата(serg_k1 @ Aug 29 2013, 16:48) *
Здравстуйте,помогите разобраться.
получил предупреждения
WARNING:PhysDesignRules:372 - Gated clock. Clock net
Ch_M_BC/GND_12_o_GND_12_o_AND_159_o is sourced by a combinatorial pin. This
is not good design practice. Use the CE pin to control the loading of data
into the flip-flop.
как найти причину и насколько это важно?
методом исключений из описания получил , что вроде бы это получается из следующего куска кода. таких есть несколько. получается , что нужно упрощать?
и еще одно предупреждение.
PhysDesignRules:372 - Gated clock. Clock net Ch_M_BC/reg_IZ_BC_CIB_BC is
sourced by a combinatorial pin. This is not good design practice. Use the CE
pin to control the loading of data into the flip-flop.
история такая.
есть два входных сигнала импульс IZ_BC и еще несколько импульсов CIB_BC. непересекающихся. под частоту в always получен общий reg_IZ_BC_CIB_BC. и далее на нем сделан свой always.

Вы должны отталкиваться от тактовой частоты в ПЛИС. Следовательно, все входные сигналы Вы должны "привязать"(синхронизировать) с Вашей тактовой частотой в ПЛИС. Потом уже производить обработку этих сигналов на тактовой частоте в ПЛИС.
Иначе у Вас будет асинхронная логика...


--------------------
If it doesn't work in simulation, it won't work on the board.

"Ты живешь в своих поступках, а не в теле. Ты — это твои действия, и нет другого тебя" Антуан де Сент-Экзюпери повесть "Маленький принц"
Go to the top of the page
 
+Quote Post
serg_k1
сообщение Aug 30 2013, 05:11
Сообщение #3


Частый гость
**

Группа: Участник
Сообщений: 115
Регистрация: 21-03-07
Пользователь №: 26 368



Цитата(Maverick @ Aug 30 2013, 00:28) *
Вы должны отталкиваться от тактовой частоты в ПЛИС. Следовательно, все входные сигналы Вы должны "привязать"(синхронизировать) с Вашей тактовой частотой в ПЛИС. Потом уже производить обработку этих сигналов на тактовой частоте в ПЛИС.
Иначе у Вас будет асинхронная логика...

структура такая
Код
always @(posedge clk_100_MHz )    
begin
    if( IZ_BC ==1 )
       reg_IZ_BC <= 1'b1;        
    else        
       reg_IZ_BC <= 1'b0;        
    if( CIB_BC ==1 )    
        reg_CIB_BC <= 1'b1;        
    else
        reg_CIB_BC <= 1'b0;        
    if( IZ_BC ==1 | CIB_BC ==1)            
        reg_IZ_BC_CIB_BC <= 1'b1;        
    else        
        reg_IZ_BC_CIB_BC <= 1'b0;                
end
always @( posedge reg_CIB_BC or posedge reg_IZ_BC  )    // reg_IZ_BC  в роли сброса
   begin    
    if ( reg_IZ_BC == 1 )
        begin
...
            end
       else
            begin
...
            end
  end

always @(  negedge reg_IZ_BC_CIB_BC )
begin    
  ...
   здесь приведенный ранее код
end
Go to the top of the page
 
+Quote Post
Maverick
сообщение Aug 30 2013, 10:44
Сообщение #4


я только учусь...
******

Группа: Модераторы
Сообщений: 3 447
Регистрация: 29-01-07
Из: Украина
Пользователь №: 24 839



Цитата(serg_k1 @ Aug 30 2013, 08:11) *
структура такая
Код
always @(posedge clk_100_MHz )    
begin
    if( IZ_BC ==1 )
       reg_IZ_BC <= 1'b1;        
    else        
       reg_IZ_BC <= 1'b0;        
    if( CIB_BC ==1 )    
        reg_CIB_BC <= 1'b1;        
    else
        reg_CIB_BC <= 1'b0;        
    if( IZ_BC ==1 | CIB_BC ==1)            
        reg_IZ_BC_CIB_BC <= 1'b1;        
    else        
        reg_IZ_BC_CIB_BC <= 1'b0;                
end
always @( posedge reg_CIB_BC or posedge reg_IZ_BC  )    // reg_IZ_BC  в роли сброса
   begin    
    if ( reg_IZ_BC == 1 )
        begin
...
            end
       else
            begin
...
            end
  end

always @(  negedge reg_IZ_BC_CIB_BC )
begin    
  ...
   здесь приведенный ранее код
end

почитайте про асинхронные частоты, пересечение клоковых доменов и синхронизацию


--------------------
If it doesn't work in simulation, it won't work on the board.

"Ты живешь в своих поступках, а не в теле. Ты — это твои действия, и нет другого тебя" Антуан де Сент-Экзюпери повесть "Маленький принц"
Go to the top of the page
 
+Quote Post
serg_k1
сообщение Sep 2 2013, 07:32
Сообщение #5


Частый гость
**

Группа: Участник
Сообщений: 115
Регистрация: 21-03-07
Пользователь №: 26 368



Цитата(Maverick @ Aug 30 2013, 14:44) *
почитайте про асинхронные частоты, пересечение клоковых доменов и синхронизацию[/url]

Спасибо, почитал. добавил на входные сигналы следующий модуль синхронизации (на другие- аналогичные) , а также синхронизатор сброса
Код
module synchronizer_CIB_BC (
    input clk,
    input reset,
    input din,
    output dout,
     output dout1
);
    reg d,dout,dout1;
    always @(posedge clk, posedge reset) begin
        if (reset) begin
            d <= 0;
            dout <= 0;
        dout1 <= 0;
        end
     else
        begin
            d <= din;
            dout <= d;
        dout1 <= dout;
        end
    end
endmodule

//===============  синхронизатор сброса   ==============
module async_reset (
input clk,
input reset_in,
output rst_out
);
reg rst_out, rff1;
always @(posedge clk or posedge reset_in)
if (reset_in)
    begin
        rst_out <= 1'b1;
        rff1  <= 1'b1;
    end
else
    begin
        rff1 <= 1'b0;
        rst_out <= rff1;
    end
endmodule

dout1 для определения фронта. но осталось такое предупреждение
PhysDesignRules:372 - Gated clock. Clock net async_reset/rst_out is
sourced by a combinatorial pin. This is not good design practice. Use the CE
pin to control the loading of data into the flip-flop.
Go to the top of the page
 
+Quote Post

Сообщений в этой теме


Reply to this topicStart new topic
3 чел. читают эту тему (гостей: 3, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 29th July 2025 - 23:26
Рейтинг@Mail.ru


Страница сгенерированна за 0.01472 секунд с 7
ELECTRONIX ©2004-2016