In reply to chr_sue:
Hi,
Please observe the waveform, it is perfect with the write transfer,but error in read transfer EPWave Waveform Viewer
and
Scoreboard not working…
Your sequence Body Task looks a Little bit strange.
task body();
//Create 10 random APB read/write transaction and send to driver
rw_trans = apb_rw::type_id::create("rw_trans");
//apb_rw::type_id::create(.name("rw_trans"),.contxt(get_full_name()));
for(int i=0;i<5;i++) begin
start_item(rw_trans);
//assert (rw_trans.randomize());
assert(rw_trans.randomize() with { addr==32'h6523_1221;data == 32'h1021_2321;apb_cmd == 1'b0;});
finish_item(rw_trans);
start_item(rw_trans);
//assert (rw_trans.randomize());
assert(rw_trans.randomize() with { addr==32'h6523_1221;data == 32'h1021_2321;apb_cmd == 1'b1;});
finish_item(rw_trans);
start_item(rw_trans);
//assert (rw_trans.randomize());
assert(rw_trans.randomize() with { addr==32'h6526_1551;data == 32'h5555_0202;apb_cmd == 1'b0;});
finish_item(rw_trans);
start_item(rw_trans);
//assert (rw_trans.randomize());
assert(rw_trans.randomize() with { addr==32'h6526_1551;data == 32'h5555_0202;apb_cmd == 1'b1;});
finish_item(rw_trans);
end
endtask
Could you please explain which apb_cmd is RD and which is WR. The RD needs only the command and the address, but no data in the sequence.
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
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
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);
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.
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
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.
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’.
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:
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
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.
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.