APB READ_TRANSFER

UVM
8

In reply to chr_sue:

This is my Driver class… so apb_cmd =1 is RD and,= 0 WR
case (tr.apb_cmd)
1’b1: drive_read(tr.addr, tr.data);
1’b0: drive_write(tr.addr, tr.data);
endcase
////////////////////////////////

i changed little bit of code, but now it is working fine. without scoreboard

9

In reply to PJ:

In reply to chr_sue:
This is my Driver class… so apb_cmd =1 is RD and,= 0 WR
case (tr.apb_cmd)
1’b1: drive_read(tr.addr, tr.data);
1’b0: drive_write(tr.addr, tr.data);
endcase
////////////////////////////////
i changed little bit of code, but now it is working fine. without scoreboard

can u pls share urs changed code

10

In reply to vlsique:

before change the code the uvm sequence like below ,i randomize penable,psel,pwrite directly here…
typedef enum {READ, WRITE} kind_e;
rand bit [31:0] addr; //Address
rand logic [31:0] data; //Data - For write or read response
rand logic penable;
rand logic psel;
rand logic pwrite;
rand kind_e apb_cmd;
so the sequence_item was :-
assert(rw_trans.randomize() with { addr==32’h6523_1221;data == 32’h1021_2321;apb_cmd == 1’b0;penable==1’b1;psel==1’b1;pwrite==1’b1});
finish_item(rw_trans);
but now,

i’m directly hard coded the psel,pwrite,penable values in uvm_driver only

case (tr.apb_cmd)
1’b1: drive_read(tr.addr, tr.data);
1’b0: drive_write(tr.addr, tr.data);
endcase
//Handshake DONE back to sequencer
seq_item_port.item_done();
end
endtask: run_phase
virtual protected task drive_write(input bit [31:0] addr,
input bit [31:0] data);

  this.vif.master_cb.paddr   <= addr;             
 @(this.vif.master_cb);
  this.vif.master_cb.pwdata  <= data;
  
 this.vif.master_cb.psel    <= '1;
 this.vif.master_cb.penable <= '0;
 this.vif.master_cb.pwrite  <= '1;

////////
k…

11

In reply to PJ:

I don’t understand why you have psel, pwrite and penable in your sequence item. I believe it does not make any sense to have them there. There is no need to randomize these items. They are simple control signals for your bus protocol.

12

In reply to chr_sue:

yes chr_sue, that’s what i changed the code

13

In reply to PJ:

Hi,

In waveform transfers of apb protocol working fine,but there is a mistake in scoreboard expected and actual values.

thanks,
pj

14

In reply to PJ:

In reply to PJ:
Hi,
In waveform transfers of apb protocol working fine,but there is a mistake in scoreboard expected and actual values.
apb_with_sb - EDA Playground
thanks,
pj

why is scoreboard giving error ,i m not getting the solution ,can anyone pls help

15

In reply to vlsique:

You should publish the EDAplayground page …

16

In reply to chr_sue:

Hi,

sorry for delay.

you can proceed now.

Thanks,
pj

17

In reply to PJ:
Your environment looks a little bit like quick & dirty. There are a few weaknesses:

(1) the ADDR_WIDTH is not consistent (8/32 bit)
(2) you are dealing in some components with the clocking block in others not
(3) if you want to compare x-values (undefined) you have to use the case equality (===). Changing this generates only passes.

I did not investigate where the x-values are coming from and if they are correct.

18

In reply to chr_sue:
Hi,

am getting zero when fifo rd operation,

Thanks,

19

In reply to PJ:

I had a closer look to your code. You are still doing strange things:
(1) in the driver you have on the left-hand side in the run_task the transaction object.
(2) in the monitor you are assigning the re in the same clock as the value from the vif.d_out to your transaction.

I do not know your protocol, but I believe there is more than 1 clock cycle in between. The d_out in the wave forms is always ‘x’.

20

In reply to PJ:

I was now looking in more detail.
You are reading from your uninitialized memory. This is really not useful, because it returns only x on your interface.
And I modified your monitor to:

task run_phase(uvm_phase phase);
    sequence_item transaction;
    transaction = sequence_item::type_id::create("transaction");
forever begin
  @ (posedge vif.clk);
  transaction.data_in = vif.din;
  transaction.wr_en = vif.wr_en;
  transaction.rd_en  = vif.rd_en;
  //transaction.data_ram = vif.mem;
  @ (posedge vif.clk);
  transaction.data_out = vif.dout;
  
  //$display("ram=%h",vif.mem);
  item_collected_port.write(transaction);
 `uvm_info("FIFO_MONITOR", $psprintf("Wrote transaction %s",transaction.convert2string()), UVM_LOW);
  end
   
  endtask:run_phase

See the additional clockedge befor vif.dout will be stored.

21

In reply to chr_sue:

Hi,

This is one tb block APB with FIFO,so now am working on it, FIFO working fine but apb get issue with read transfer it showing a zero value in scoreboard

Thanks,

22

In reply to PJ:

You see in the waveforms a ‘x’ and this is the indication something goes wrong with your design.
The mem is not initialized and then you are trying to read from this mem. First you should do some wr before reading.

23

In reply to chr_sue:

yah, i did it.in waveform prdata ok,but in scoreboard prdata not comming.

24

In reply to PJ:

You are extracting the rd-data at a wrong time. You have to adopt the run_phase to the right behavior.
BTW $Strobe does not exist but $strobe.

25

In reply to chr_sue:

okay if suppose ‘rdata’ wrong in time , it not coming into scoreboard,
it is showing zero in design also ,

26

In reply to PJ:

In the wave prdata is not always 0. During penable = 1, it contains the right data. Correct your monitor behavior, looking at the right time to the signal prdata.

27

In reply to chr_sue:

Hi,

i did the maximum changes in my monitor, but still getting one error ,when read at first time

Thanks,