From c93dcc048493467029d1898857c539511c7da7ab Mon Sep 17 00:00:00 2001
From: francescomani <email@francescomani.it>
Date: Wed, 28 Aug 2024 17:11:57 +0200
Subject: [PATCH] cleanup of encode_parity_check_part_optim including asserting
 for case not handled

---
 .../nrLDPC_encoder/ldpc_encode_parity_check.c | 233 ++++++++----------
 1 file changed, 107 insertions(+), 126 deletions(-)

diff --git a/openair1/PHY/CODING/nrLDPC_encoder/ldpc_encode_parity_check.c b/openair1/PHY/CODING/nrLDPC_encoder/ldpc_encode_parity_check.c
index f4632c61d7..55e02eea6a 100644
--- a/openair1/PHY/CODING/nrLDPC_encoder/ldpc_encode_parity_check.c
+++ b/openair1/PHY/CODING/nrLDPC_encoder/ldpc_encode_parity_check.c
@@ -85,138 +85,119 @@
 #include "ldpc_BG2_Zc80_byte.c"
 #include "ldpc_BG2_Zc72_byte.c"
 
-
-
-static void encode_parity_check_part_optim(uint8_t *cc,uint8_t *d, short BG,short Zc,short Kb, int simd_size, int ncols)
+static void encode_parity_check_part_optim(uint8_t *cc, uint8_t *d, short BG,short Zc,short Kb, int simd_size, int ncols)
 {
-  unsigned char c[2*22*Zc*simd_size] __attribute__((aligned(32)));      //double size matrix of c
-  for (int i1=0; i1 < ncols; i1++)   {
-    memcpy(&c[2*i1*Zc], &cc[i1*Zc], Zc*sizeof(unsigned char));
-    memcpy(&c[(2*i1+1)*Zc], &cc[i1*Zc], Zc*sizeof(unsigned char));
+  unsigned char c[2 * 22 * Zc * simd_size] __attribute__((aligned(32))); //double size matrix of c
+  for (int i1 = 0; i1 < ncols; i1++)   {
+    memcpy(&c[2 * i1 * Zc], &cc[i1 * Zc], Zc * sizeof(unsigned char));
+    memcpy(&c[(2 * i1 + 1) * Zc], &cc[i1 * Zc], Zc * sizeof(unsigned char));
   }
-  for (int i1=1;i1<simd_size;i1++) {
-    memcpy(&c[(2*ncols*Zc*i1)], &c[i1], (2*ncols*Zc*sizeof(unsigned char))-i1);
+  for (int i1 = 1; i1 < simd_size; i1++) {
+    memcpy(&c[(2 * ncols * Zc * i1)], &c[i1], (2 * ncols * Zc * sizeof(unsigned char)) - i1);
   }
   
-  if (BG==1)
-  {
-    switch (Zc)
-    {
-    case 2: break;
-    case 3: break;
-    case 4: break;
-    case 5: break;
-    case 6: break;
-    case 7: break;
-    case 8: break;
-    case 9: break;
-    case 10: break;
-    case 11: break;
-    case 12: break;
-    case 13: break;
-    case 14: break;
-    case 15: break;
-    case 16: break;
-    case 18: break;
-    case 20: break;
-    case 22: break;
-    case 24: break;      
-    case 26: break;
-    case 28: break;
-    case 30: break;
-    case 32: break;
-    case 36: break;
-    case 40: break;
-    case 44: break;
-    case 48: break;
-    case 52: break;
-    case 56: break;
-    case 60: break;
-    case 64: break;
-    case 72: break;
-    case 80: break;   
-    case 88: break;   
-    case 96: break;
-    case 104: break;
-    case 112: break;
-    case 120: break;
-    case 128: break;
-    case 144: break;
-    case 160: break;
-    case 176: ldpc176_byte(c,d); break;
-    case 192: ldpc192_byte(c,d); break;
-    case 208: ldpc208_byte(c,d); break;
-    case 224: ldpc224_byte(c,d); break;
-    case 240: ldpc240_byte(c,d); break;
-    case 256: ldpc256_byte(c,d); break;
-    case 288: ldpc288_byte(c,d); break;
-    case 320: ldpc320_byte(c,d); break;
-    case 352: ldpc352_byte(c,d); break;
-    case 384: ldpc384_byte(c,d); break;
-    default: AssertFatal(0,"BG %d Zc %d is not supported yet\n",BG,Zc); break;
+  if (BG == 1) {
+    switch (Zc) {
+      case 176:
+        ldpc176_byte(c, d);
+        break;
+      case 192:
+        ldpc192_byte(c, d);
+        break;
+      case 208:
+        ldpc208_byte(c, d);
+        break;
+      case 224:
+        ldpc224_byte(c, d);
+        break;
+      case 240:
+        ldpc240_byte(c, d);
+        break;
+      case 256:
+        ldpc256_byte(c, d);
+        break;
+      case 288:
+        ldpc288_byte(c, d);
+        break;
+      case 320:
+        ldpc320_byte(c, d);
+        break;
+      case 352:
+        ldpc352_byte(c, d);
+        break;
+      case 384:
+        ldpc384_byte(c, d);
+        break;
+      default:
+        AssertFatal(false, "BG %d Zc %d is not supported yet\n", BG, Zc);
     }
-  }
-  else if (BG==2) {
-    switch (Zc)
-    {
-    case 2: break;
-    case 3: break;
-    case 4: break;
-    case 5: break;
-    case 6: break;
-    case 7: break;
-    case 8: break;
-    case 9: break;
-    case 10: break;
-    case 11: break;
-    case 12: break;
-    case 13: break;
-    case 14: break;
-    case 15: break;
-    case 16: break;
-    case 18: break;
-    case 20: break;
-    case 22: break;
-    case 24: break;      
-    case 26: break;
-    case 28: break;
-    case 30: break;
-    case 32: break;
-    case 36: break;
-    case 40: break;
-    case 44: break;
-    case 48: break;
-    case 52: break;
-    case 56: break;
-    case 60: break;
-    case 64: break;
-    case 72: ldpc_BG2_Zc72_byte(c,d); break;
-    case 80: ldpc_BG2_Zc80_byte(c,d); break;   
-    case 88: ldpc_BG2_Zc88_byte(c,d); break;   
-    case 96: ldpc_BG2_Zc96_byte(c,d); break;
-    case 104: ldpc_BG2_Zc104_byte(c,d); break;
-    case 112: ldpc_BG2_Zc112_byte(c,d); break;
-    case 120: ldpc_BG2_Zc120_byte(c,d); break;
-    case 128: ldpc_BG2_Zc128_byte(c,d); break;
-    case 144: ldpc_BG2_Zc144_byte(c,d); break;
-    case 160: ldpc_BG2_Zc160_byte(c,d); break;
-    case 176: ldpc_BG2_Zc176_byte(c,d); break;
-    case 192: ldpc_BG2_Zc192_byte(c,d); break;
-    case 208: ldpc_BG2_Zc208_byte(c,d); break;
-    case 224: ldpc_BG2_Zc224_byte(c,d); break;
-    case 240: ldpc_BG2_Zc240_byte(c,d); break;
-    case 256: ldpc_BG2_Zc256_byte(c,d); break;
-    case 288: ldpc_BG2_Zc288_byte(c,d); break;
-    case 320: ldpc_BG2_Zc320_byte(c,d); break;
-    case 352: ldpc_BG2_Zc352_byte(c,d); break;
-    case 384: ldpc_BG2_Zc384_byte(c,d); break;
-    default: AssertFatal(0,"BG %d Zc %d is not supported yet\n",BG,Zc); break;
+  } else if (BG == 2) {
+    switch (Zc) {
+      case 72:
+        ldpc_BG2_Zc72_byte(c, d);
+        break;
+      case 80:
+        ldpc_BG2_Zc80_byte(c, d);
+        break;
+      case 88:
+        ldpc_BG2_Zc88_byte(c, d);
+        break;
+      case 96:
+        ldpc_BG2_Zc96_byte(c, d);
+        break;
+      case 104:
+        ldpc_BG2_Zc104_byte(c, d);
+        break;
+      case 112:
+        ldpc_BG2_Zc112_byte(c, d);
+        break;
+      case 120:
+        ldpc_BG2_Zc120_byte(c, d);
+        break;
+      case 128:
+        ldpc_BG2_Zc128_byte(c, d);
+        break;
+      case 144:
+        ldpc_BG2_Zc144_byte(c, d);
+        break;
+      case 160:
+        ldpc_BG2_Zc160_byte(c, d);
+        break;
+      case 176:
+        ldpc_BG2_Zc176_byte(c, d);
+        break;
+      case 192:
+        ldpc_BG2_Zc192_byte(c, d);
+        break;
+      case 208:
+        ldpc_BG2_Zc208_byte(c, d);
+        break;
+      case 224:
+        ldpc_BG2_Zc224_byte(c, d);
+        break;
+      case 240:
+        ldpc_BG2_Zc240_byte(c, d);
+        break;
+      case 256:
+        ldpc_BG2_Zc256_byte(c, d);
+        break;
+      case 288:
+        ldpc_BG2_Zc288_byte(c, d);
+        break;
+      case 320:
+        ldpc_BG2_Zc320_byte(c, d);
+        break;
+      case 352:
+        ldpc_BG2_Zc352_byte(c, d);
+        break;
+      case 384:
+        ldpc_BG2_Zc384_byte(c, d);
+        break;
+      default:
+        AssertFatal(false , "BG %d Zc %d is not supported yet\n", BG, Zc);
     }
-  }
-  else {
-    AssertFatal(0,"BG %d is not supported yet\n",BG);
-  } 
-
+  } else
+    AssertFatal(false, "BG %d is not supported\n", BG);
 }