BasitMehmood
BasitMehmood
@jdonjdon kindly review it.
I am closing this and did a new PR of this in #21619.
I don't know why (Run DV smoke test) is failing?
> Something has got a little bit shuffled around. I think what we want is probably two commits: > > 1. Define `request_halt` and use it in places that might...
@rswarbrick we can first merge #21663 branch where we defined halt_task and use it in vseqs. Then we can remove extra files(vseqs) in this branch that also uses that task,...
There is a check in scoreboard line 363. Which compare the value that is written, against the csr.get_mirrored_value(). But this check failed because item.d_data has written value but in csr.get_mirrored_value()...
> This is unchecked, but can't we simplify this a bit? I'm imagining something like > > ```systemverilog > task progbuf_rw(int idx); > uvm_reg_data_t data = $urandom; > uvm_reg_data_t r_data;...
> I agree that this you're correctly describing what the code currently does :-) But I'm trying to say that this is currently testing a reasonably trivial property ("writes to...
@rswarbrick now what's your thoughts on this PR?
> Sorry for the slow reply. > > But I'm clearly missing something! Are you saying that it's impossible to do a CSR read over the DMI interface? I don't...