Skip to content

Commit 76da1f3

Browse files
committed
[!] minor modification to avoid double assignment into variables path_id. No functional changes but somehow it should help with issue #46
1 parent 426af3d commit 76da1f3

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

sc/rtl/riverlib/cache/cache_top.cpp

+2-4
Original file line numberDiff line numberDiff line change
@@ -389,8 +389,8 @@ void CacheTop::comb() {
389389
vb_ctrl_bus = 0ull;
390390
vb_data_bus = 0ull;
391391
vb_queue_bus = 0ull;
392-
ctrl_path_id = 0;
393-
data_path_id = 0;
392+
ctrl_path_id = CTRL_PATH;
393+
data_path_id = DATA_PATH;
394394
v_queue_we = 0;
395395
v_queue_re = 0;
396396
v_req_mem_path_o = 0;
@@ -407,13 +407,11 @@ void CacheTop::comb() {
407407
v_queue_re = i_req_mem_ready;
408408
v_queue_we = (i.req_mem_valid || d.req_mem_valid);
409409

410-
ctrl_path_id = CTRL_PATH;
411410
vb_ctrl_bus = (ctrl_path_id,
412411
i.req_mem_type,
413412
i.req_mem_size,
414413
i.req_mem_addr);
415414

416-
data_path_id = DATA_PATH;
417415
vb_data_bus = (data_path_id,
418416
d.req_mem_type,
419417
d.req_mem_size,

sv/rtl/riverlib/cache/cache_top.sv

+2-4
Original file line numberDiff line numberDiff line change
@@ -266,8 +266,8 @@ begin: comb_proc
266266
vb_ctrl_bus = 55'h00000000000000;
267267
vb_data_bus = 55'h00000000000000;
268268
vb_queue_bus = 55'h00000000000000;
269-
ctrl_path_id = 0;
270-
data_path_id = 0;
269+
ctrl_path_id = CTRL_PATH;
270+
data_path_id = DATA_PATH;
271271
v_queue_we = 0;
272272
v_queue_re = 0;
273273
v_req_mem_path_o = 0;
@@ -284,13 +284,11 @@ begin: comb_proc
284284
v_queue_re = i_req_mem_ready;
285285
v_queue_we = (i.req_mem_valid || d.req_mem_valid);
286286

287-
ctrl_path_id = CTRL_PATH;
288287
vb_ctrl_bus = {ctrl_path_id,
289288
i.req_mem_type,
290289
i.req_mem_size,
291290
i.req_mem_addr};
292291

293-
data_path_id = DATA_PATH;
294292
vb_data_bus = {data_path_id,
295293
d.req_mem_type,
296294
d.req_mem_size,

0 commit comments

Comments
 (0)