Commit 69deb749 authored by Robert Schmidt's avatar Robert Schmidt

Merge remote-tracking branch 'safwan2mo/fix/sss-nid1-nid2-mismatch' into integration_2026_w29

fix(PHY): compare NID2 instead of NID1 in rx_sss_nr() coherence check (#304)

target_Nid_cell was being decomposed with GET_NID1() and compared
against pss->nid2 (the detected PSS NID2). Since Nid1 != Nid2 in almost
all cases, the coherence check always failed, forcing an exhaustive
336-hypothesis SSS search instead of the intended single-candidate fast
path when validating a known neighbor PCI, and spamming LOG_E on every
such call.

Closes: #296
Reviewed-by: default avatarFrancesco Mani <email@francescomani.it>
parents 80cfd087 a2685262
...@@ -217,7 +217,7 @@ sss_detection_result_t rx_sss_nr(nr_sss_params_t *params, ...@@ -217,7 +217,7 @@ sss_detection_result_t rx_sss_nr(nr_sss_params_t *params,
int Nid1_start = 0; int Nid1_start = 0;
int Nid1_end = N_ID_1_NUMBER; int Nid1_end = N_ID_1_NUMBER;
if (target_Nid_cell != -1) { if (target_Nid_cell != -1) {
if (GET_NID1(target_Nid_cell) != pss->nid2) { if (GET_NID2(target_Nid_cell) != pss->nid2) {
LOG_E(PHY, "calling sss detection with incoherent context %d, %d\n", pss->nid2, target_Nid_cell); LOG_E(PHY, "calling sss detection with incoherent context %d, %d\n", pss->nid2, target_Nid_cell);
} else { } else {
Nid1_start = GET_NID1(target_Nid_cell); Nid1_start = GET_NID1(target_Nid_cell);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment