mac: reverse PMI bits in CSI for test_mode UE

frankist · frankist · commit 87e0c09aecdd · 2023-06-19T21:45:12.000+02:00
diff --git a/lib/du_high/mac_test_mode_adapter.cpp b/lib/du_high/mac_test_mode_adapter.cpp
@@ -172,8 +172,9 @@ class test_cell_adapter : public mac_cell_control_information_handler
       payload.resize(CQI_BITLEN);
     } else {
       payload.resize(CQI_BITLEN + RI_BITLEN + PMI_BITLEN);
-      packed_bits = packed_bits << (RI_BITLEN + PMI_BITLEN);
-      packed_bits += (test_ue_cfg.ri - 1) + (test_ue_cfg.pmi << RI_BITLEN);
+      packed_bits = packed_bits << (RI_BITLEN + PMI_BITLEN); // shift to give space for RI and PMI.
+      packed_bits += (bit_reverse(test_ue_cfg.pmi) >> (64U - PMI_BITLEN)) << RI_BITLEN;
+      packed_bits += (test_ue_cfg.ri - 1);
     }
     payload.from_uint64(packed_bits);
   }

Original file line number	Diff line number	Diff line change
`@@ -172,8 +172,9 @@ class test_cell_adapter : public mac_cell_control_information_handler`
`172`	`172`	`payload.resize(CQI_BITLEN);`
`173`	`173`	`} else {`
`174`	`174`	`payload.resize(CQI_BITLEN + RI_BITLEN + PMI_BITLEN);`
`175`		`- packed_bits = packed_bits << (RI_BITLEN + PMI_BITLEN);`
`176`		`- packed_bits += (test_ue_cfg.ri - 1) + (test_ue_cfg.pmi << RI_BITLEN);`
	`175`	`+ packed_bits = packed_bits << (RI_BITLEN + PMI_BITLEN); // shift to give space for RI and PMI.`
	`176`	`+ packed_bits += (bit_reverse(test_ue_cfg.pmi) >> (64U - PMI_BITLEN)) << RI_BITLEN;`
	`177`	`+ packed_bits += (test_ue_cfg.ri - 1);`
`177`	`178`	`}`
`178`	`179`	`payload.from_uint64(packed_bits);`
`179`	`180`	`}`