HFR 검수

1. 부팅시 죽는 문제 수정 CrcLength 오버 플로우 로인한 죽음 수정
2.SelfTest On off 시 Atten 0 과 이전값을 기억 수정 완료
3.Carrier OFF 시 path 제어 안되 도록 수정

Bluecell_Src/Bluecell_operate.c

@@ -80,6 +80,9 @@ extern volatile uint32_t Alarm_UL_Level_TimerOnCnt ;
 
 extern volatile uint32_t SelfTestLifeCnt[4];
 
+uint8_t Selftest_DL_PrevAttenSave[8] = {0,};
+uint8_t Selftest_UL_PrevAttenSave[8] = {0,};
+
 
 /***************************************************************************************/
 /*                                  Function                                           */
@@ -180,6 +183,12 @@ typedef enum{
     Atten_L,        
 };
    
+typedef enum{
+    SelfTest1 = 0,
+    SelfTest2,
+    SelfTest3,
+    SelfTest4,    
+};
 
 
 #if 0 // PYJ.2020.05.21_BEGIN -- 
@@ -764,6 +773,11 @@ void DL_AttenOperate(uint8_t* Atten,uint8_t* PrevAtten,uint8_t* UserAtten,uint8_
 
 
 }
+typedef enum{
+    UserAtten_H = 0,
+    UserAtten_L,        
+};
+
 void CompareAttenData(BLUESTATUS_st Curr,BLUESTATUS_st Prev){
      uint8_t val = 0;
      uint16_t MBIC_Val = 0;
@@ -2506,6 +2520,8 @@ bool Bluecell_Operate(uint8_t* data){
 //        printf("Function : %s Line %d \r\n",__func__,__LINE__);
 		break;
 	case ATT_SelfTest1		 :
+#if 0 // PYJ.2020.07.05_BEGIN -- 
+        
         if(data[BLUECELL_DATA]==0){
             HAL_GPIO_WritePin(_PATH_SW1_GPIO_Port,_PATH_SW1_Pin,GPIO_PIN_RESET);//CLOCK
             HAL_GPIO_WritePin(PATH_SW1_GPIO_Port,PATH_SW1_Pin,GPIO_PIN_SET);//CLOCK
@@ -2516,6 +2532,28 @@ bool Bluecell_Operate(uint8_t* data){
         }
         bluecell_Currdatastatus.Selftest1 = data[BLUECELL_DATA];
 //		printf("Function : %s Line %d \r\n",__func__,__LINE__);
+#else
+#if 0 // PYJ.2020.07.05_BEGIN -- 
+        if(data[BLUECELL_DATA] == 0){
+            bluecell_Currdatastatus.ATT_DL1_H = Selftest_DL_PrevAttenSave[0]; 
+            bluecell_Currdatastatus.ATT_DL1_L = Selftest_DL_PrevAttenSave[1];  
+            bluecell_Currdatastatus.ATT_UL1_H = Selftest_UL_PrevAttenSave[0]; 
+            bluecell_Currdatastatus.ATT_UL1_L = Selftest_UL_PrevAttenSave[1];
+        }else{
+            Selftest_DL_PrevAttenSave[0] = bluecell_Currdatastatus.ATT_DL1_H;
+            Selftest_DL_PrevAttenSave[1] = bluecell_Currdatastatus.ATT_DL1_L;            
+            Selftest_UL_PrevAttenSave[0] = bluecell_Currdatastatus.ATT_UL1_H;
+            Selftest_UL_PrevAttenSave[1] = bluecell_Currdatastatus.ATT_UL1_L;            
+            bluecell_Currdatastatus.ATT_DL1_H = 0;
+            bluecell_Currdatastatus.ATT_DL1_L = 0;        
+            bluecell_Currdatastatus.ATT_UL1_H = 0;
+            bluecell_Currdatastatus.ATT_UL1_L = 0;              
+        }
+        CompareAttenData(bluecell_Currdatastatus,bluecell_Prevdatastatus);
+#endif // PYJ.2020.07.05_END -- 
+        SelfTest_Ctrl(SelfTest1,data[BLUECELL_DATA],&bluecell_Currdatastatus.Selftest1,&bluecell_Currdatastatus.ATT_DL1_H,&bluecell_Currdatastatus.ATT_UL1_H);
+
+#endif // PYJ.2020.07.05_END -- 
 		break;
 	case ATT_DL2_PATH		 :
 #if 0 // PYJ.2020.07.03_BEGIN -- 
@@ -2572,6 +2610,7 @@ bool Bluecell_Operate(uint8_t* data){
 #endif // PYJ.2020.07.03_END -- 
 		break;
 	case ATT_SelfTest2		 :
+#if 0 // PYJ.2020.07.05_BEGIN -- 
         if(data[BLUECELL_DATA]==0){
             HAL_GPIO_WritePin(_PATH_SW2_GPIO_Port,_PATH_SW2_Pin,GPIO_PIN_RESET);//CLOCK
             HAL_GPIO_WritePin(PATH_SW2_GPIO_Port,PATH_SW2_Pin,GPIO_PIN_SET);//CLOCK
@@ -2580,6 +2619,10 @@ bool Bluecell_Operate(uint8_t* data){
             HAL_GPIO_WritePin(_PATH_SW2_GPIO_Port,_PATH_SW2_Pin,GPIO_PIN_SET);//CLOCK
         }
         bluecell_Currdatastatus.Selftest2 = data[BLUECELL_DATA];
+#else
+        SelfTest_Ctrl(SelfTest2,data[BLUECELL_DATA],&bluecell_Currdatastatus.Selftest2,&bluecell_Currdatastatus.ATT_DL2_H,&bluecell_Currdatastatus.ATT_UL2_H);
+
+#endif // PYJ.2020.07.05_END -- 
 //		printf("Function : %s Line %d \r\n",__func__,__LINE__);
 		break;
 	case ATT_DL3_PATH :
@@ -2633,6 +2676,7 @@ bool Bluecell_Operate(uint8_t* data){
 #endif // PYJ.2020.07.03_END -- 
 		break;
 	case ATT_SelfTest3 :
+#if 0 // PYJ.2020.07.05_BEGIN -- 
         if(data[BLUECELL_DATA]==0){
             HAL_GPIO_WritePin(_PATH_SW3_GPIO_Port,_PATH_SW3_Pin,GPIO_PIN_RESET);//CLOCK
             HAL_GPIO_WritePin(PATH_SW3_GPIO_Port,PATH_SW3_Pin,GPIO_PIN_SET);//CLOCK
@@ -2642,6 +2686,10 @@ bool Bluecell_Operate(uint8_t* data){
             HAL_GPIO_WritePin(_PATH_SW3_GPIO_Port,_PATH_SW3_Pin,GPIO_PIN_SET);//CLOCK
         }
         bluecell_Currdatastatus.Selftest3 = data[BLUECELL_DATA];
+#else
+        SelfTest_Ctrl(SelfTest3,data[BLUECELL_DATA],&bluecell_Currdatastatus.Selftest3,&bluecell_Currdatastatus.ATT_DL3_H,&bluecell_Currdatastatus.ATT_UL3_H);
+
+#endif // PYJ.2020.07.05_END -- 
 //		printf("Function : %s Line %d \r\n",__func__,__LINE__);
 		break;
 	case ATT_DL4_PATH :
@@ -2698,6 +2746,7 @@ bool Bluecell_Operate(uint8_t* data){
 //		printf("Function : %s Line %d \r\n",__func__,__LINE__);
 		break;
 	case ATT_SelfTest4 :
+#if 0 // PYJ.2020.07.05_BEGIN -- 
 	  if(data[BLUECELL_DATA]==0){
 	    HAL_GPIO_WritePin(_PATH_SW4_GPIO_Port,_PATH_SW4_Pin,GPIO_PIN_RESET);//CLOCK
 	    HAL_GPIO_WritePin(PATH_SW4_GPIO_Port,PATH_SW4_Pin,GPIO_PIN_SET);//CLOCK
@@ -2706,6 +2755,10 @@ bool Bluecell_Operate(uint8_t* data){
 	    HAL_GPIO_WritePin(PATH_SW4_GPIO_Port,PATH_SW4_Pin,GPIO_PIN_RESET);//CLOCK
 	  }
       bluecell_Currdatastatus.Selftest4 = data[BLUECELL_DATA];
+#else
+        SelfTest_Ctrl(SelfTest4,data[BLUECELL_DATA],&bluecell_Currdatastatus.Selftest4,&bluecell_Currdatastatus.ATT_DL4_H,&bluecell_Currdatastatus.ATT_UL4_H);
+
+#endif // PYJ.2020.07.05_END -- 
 //		printf("Function : %s Line %d \r\n",__func__,__LINE__);
 		break;
     case ATT_TableSet:
@@ -2860,7 +2913,7 @@ bool Bluecell_Operate(uint8_t* data){
 
 
         
-        printf("Temp Alarm %d \r\n");
+        printf("Temp Alarm %d \r\n",bluecell_Currdatastatus.ALARM_TEMP_HIGH);
         printf("ALC  Alarm 1: %d 2:  %d 3: %d 4: %d\r\n",
             bluecell_Currdatastatus.ALARM_ULO_ALC_Alarm & ALARM_ALC_P1,
             bluecell_Currdatastatus.ALARM_ULO_ALC_Alarm & ALARM_ALC_P2,
@@ -3379,10 +3432,14 @@ typedef enum{
     Turn off all paths at boot
  */
    
-   bluecell_Currdatastatus.Type =  HAL_GPIO_ReadPin(UNIT_ID0_GPIO_Port,UNIT_ID0_Pin);
+   bluecell_Currdatastatus.Type =   HAL_GPIO_ReadPin(UNIT_ID0_GPIO_Port,UNIT_ID0_Pin);
    bluecell_Currdatastatus.Type +=  HAL_GPIO_ReadPin(UNIT_ID1_GPIO_Port,UNIT_ID1_Pin) << 1;
    bluecell_Currdatastatus.Type +=  HAL_GPIO_ReadPin(UNIT_ID2_GPIO_Port,UNIT_ID2_Pin) << 2;
    bluecell_Currdatastatus.Type +=  HAL_GPIO_ReadPin(UNIT_ID3_GPIO_Port,UNIT_ID3_Pin) << 3;
+   printf("0: %d \r\n",HAL_GPIO_ReadPin(UNIT_ID0_GPIO_Port,UNIT_ID0_Pin));
+   printf("1: %d \r\n",HAL_GPIO_ReadPin(UNIT_ID1_GPIO_Port,UNIT_ID1_Pin));
+   printf("2: %d \r\n",HAL_GPIO_ReadPin(UNIT_ID2_GPIO_Port,UNIT_ID2_Pin));
+   printf("3: %d \r\n",HAL_GPIO_ReadPin(UNIT_ID3_GPIO_Port,UNIT_ID3_Pin));   
     switch(bluecell_Currdatastatus.Type){
      case BLUECELL_3_7G: bluecell_Currdatastatus.Type = 0x80;bluecell_Currdatastatus.Carrier_ID = 0x01 ;printf("Type BLUECELL_3_7G  Ready....\r\n"); break;
      case BLUECELL_3_8G: bluecell_Currdatastatus.Type = 0x81;bluecell_Currdatastatus.Carrier_ID = 0x02; printf("Type BLUECELL_3_8G  Ready....\r\n");break;
@@ -3391,7 +3448,7 @@ typedef enum{
      case BLUECELL_4_1G: bluecell_Currdatastatus.Type = 0x84;bluecell_Currdatastatus.Carrier_ID = 0x02; printf("Type BLUECELL_4_1G  Ready....\r\n");break;
      case BLUECELL_4_6G: bluecell_Currdatastatus.Type = 0x85;bluecell_Currdatastatus.Carrier_ID = 0x01; printf("Type BLUECELL_4_6G  Ready....\r\n");break;
      case BLUECELL_4_7G: bluecell_Currdatastatus.Type = 0x86;bluecell_Currdatastatus.Carrier_ID = 0x00; printf("Type BLUECELL_4_7G  Ready....\r\n");break;
-     case BLUECELL_4_8G: bluecell_Currdatastatus.Type = 0x87;bluecell_Currdatastatus.Carrier_ID = 0x00; printf("Type BLUECELL_3_7G  Ready....\r\n");break;
+     case BLUECELL_4_8G: bluecell_Currdatastatus.Type = 0x87;bluecell_Currdatastatus.Carrier_ID = 0x00; printf("Type BLUECELL_4_8G  Ready....\r\n");break;
      case BLUECELL_4_9G: bluecell_Currdatastatus.Type = 0x88;bluecell_Currdatastatus.Carrier_ID = 0x00; printf("Type BLUECELL_4_9G  Ready....\r\n");break;
      case BLUECELL_5_0G: bluecell_Currdatastatus.Type = 0x89;bluecell_Currdatastatus.Carrier_ID = 0x00; printf("Type BLUECELL_5_0G  Ready....\r\n");break;
      case BLUECELL_DCM : bluecell_Currdatastatus.Type = 0x8A;bluecell_Currdatastatus.Carrier_ID = 0x01;printf("Type BLUECELL_DCM  Ready....\r\n"); break;
@@ -4533,6 +4590,9 @@ void Factory_Set(){
 void DL_Path_OnOff(uint8_t Index,uint8_t value,uint8_t* path,uint8_t* PrevAtten,uint8_t* retrycnt,uint8_t* PrevRetryCnt){
     GPIO_TypeDef *Port = 0;
     uint16_t Pin = 0;    
+    if(bluecell_Currdatastatus.Carrier_ON_OFF == false)
+        return;
+        
     switch(Index){
         case Path1_OnOff:
         Port = PATH_EN_DL1_GPIO_Port;
@@ -4570,6 +4630,8 @@ void DL_Path_OnOff(uint8_t Index,uint8_t value,uint8_t* path,uint8_t* PrevAtten,
 void UL_Path_OnOff(uint8_t Index,uint8_t value,uint8_t* path,uint8_t* PrevAtten,uint8_t* retrycnt,uint8_t* PrevRetryCnt){
     GPIO_TypeDef *Port = 0;
     uint16_t Pin = 0;    
+     if(bluecell_Currdatastatus.Carrier_ON_OFF == false)
+            return;    
     switch(Index){
         case Path1_OnOff:
         Port = PATH_EN_UL1_GPIO_Port;
@@ -5427,6 +5489,7 @@ bool MBIC_Operate(uint8_t* data){
 //                bluecell_Currdatastatus.ULO_Level_High_Threshold_default = data[MBIC_PAYLOADSTART + 3]; 
                 break;
             case ULO_SelfTest1:
+#if 0 // PYJ.2020.07.05_BEGIN -- 
                 if(data[MBIC_PAYLOADSTART + 3]==0){
                     HAL_GPIO_WritePin(_PATH_SW1_GPIO_Port,_PATH_SW1_Pin,GPIO_PIN_RESET);//CLOCK
                     HAL_GPIO_WritePin(PATH_SW1_GPIO_Port,PATH_SW1_Pin,GPIO_PIN_SET);//CLOCK
@@ -5436,8 +5499,13 @@ bool MBIC_Operate(uint8_t* data){
                     HAL_GPIO_WritePin(PATH_SW1_GPIO_Port,PATH_SW1_Pin,GPIO_PIN_RESET);//CLOCK
                 }
                 bluecell_Currdatastatus.Selftest1 = data[MBIC_PAYLOADSTART + 3];                
+#else
+                SelfTest_Ctrl(SelfTest1,data[MBIC_PAYLOADSTART + 3],&bluecell_Currdatastatus.Selftest1,&bluecell_Currdatastatus.ATT_DL1_H,&bluecell_Currdatastatus.ATT_UL1_H);
+
+#endif // PYJ.2020.07.05_END -- 
                 break;
             case ULO_SelfTest2:
+#if 0 // PYJ.2020.07.05_BEGIN -- 
                 if(data[MBIC_PAYLOADSTART + 3]==0){
                     HAL_GPIO_WritePin(_PATH_SW2_GPIO_Port,_PATH_SW2_Pin,GPIO_PIN_RESET);//CLOCK
                     HAL_GPIO_WritePin(PATH_SW2_GPIO_Port,PATH_SW2_Pin,GPIO_PIN_SET);//CLOCK
@@ -5447,8 +5515,13 @@ bool MBIC_Operate(uint8_t* data){
                     HAL_GPIO_WritePin(PATH_SW2_GPIO_Port,PATH_SW2_Pin,GPIO_PIN_RESET);//CLOCK
                 }
                 bluecell_Currdatastatus.Selftest2 = data[MBIC_PAYLOADSTART + 3];                
+#else
+                SelfTest_Ctrl(SelfTest2,data[MBIC_PAYLOADSTART + 3],&bluecell_Currdatastatus.Selftest2,&bluecell_Currdatastatus.ATT_DL2_H,&bluecell_Currdatastatus.ATT_UL2_H);
+
+#endif // PYJ.2020.07.05_END -- 
                 break;
             case ULO_SelfTest3:
+#if 0 // PYJ.2020.07.05_BEGIN -- 
                 if(data[MBIC_PAYLOADSTART + 3]==0){
                     HAL_GPIO_WritePin(_PATH_SW3_GPIO_Port,_PATH_SW3_Pin,GPIO_PIN_RESET);//CLOCK
                     HAL_GPIO_WritePin(PATH_SW3_GPIO_Port,PATH_SW3_Pin,GPIO_PIN_SET);//CLOCK
@@ -5458,8 +5531,13 @@ bool MBIC_Operate(uint8_t* data){
                     HAL_GPIO_WritePin(PATH_SW3_GPIO_Port,PATH_SW3_Pin,GPIO_PIN_RESET);//CLOCK
                 }
                 bluecell_Currdatastatus.Selftest3 = data[MBIC_PAYLOADSTART + 3];                
+#else         
+                SelfTest_Ctrl(SelfTest3,data[MBIC_PAYLOADSTART + 3],&bluecell_Currdatastatus.Selftest3,&bluecell_Currdatastatus.ATT_DL3_H,&bluecell_Currdatastatus.ATT_UL3_H);
+
+#endif // PYJ.2020.07.05_END -- 
                 break;
             case ULO_SelfTest4:
+#if 0 // PYJ.2020.07.05_BEGIN -- 
                 if(data[MBIC_PAYLOADSTART + 3]==0){
                     HAL_GPIO_WritePin(_PATH_SW4_GPIO_Port,_PATH_SW4_Pin,GPIO_PIN_RESET);//CLOCK
                     HAL_GPIO_WritePin(PATH_SW4_GPIO_Port,PATH_SW4_Pin,GPIO_PIN_SET);//CLOCK
@@ -5469,6 +5547,10 @@ bool MBIC_Operate(uint8_t* data){
                     HAL_GPIO_WritePin(PATH_SW4_GPIO_Port,PATH_SW4_Pin,GPIO_PIN_RESET);//CLOCK
                 }
                 bluecell_Currdatastatus.Selftest4 = data[MBIC_PAYLOADSTART + 3];                
+#else
+                SelfTest_Ctrl(SelfTest4,data[MBIC_PAYLOADSTART + 3],&bluecell_Currdatastatus.Selftest4,&bluecell_Currdatastatus.ATT_DL4_H,&bluecell_Currdatastatus.ATT_UL4_H);
+
+#endif // PYJ.2020.07.05_END -- 
                 break;
             case  ULO_ALC_ON_OFF                   :
                 bluecell_Currdatastatus.ULO_ALC_ON_OFF = data[MBIC_PAYLOADSTART + 3]; 
@@ -5484,9 +5566,9 @@ bool MBIC_Operate(uint8_t* data){
             case  ULO_ALC_Threshold_Default               :
 			    tempdata = ULO_ALC_THREADHOLD_DEFAULT;
 			    bluecell_Currdatastatus.ULO_ALC_Threshold_H = ((tempdata & 0xFF00) >> 8);    
-			    bluecell_Currdatastatus.ULO_ALC_Threshold_L = ((tempdata & 0x00FF));  				
+			    bluecell_Currdatastatus.ULO_ALC_Threshold_L = ((tempdata & 0x00FF));  	
+                printf("ALC DEFAULT VALUE SETTING COMPLETE \r\n");
 //                bluecell_Currdatastatus.ULO_ALC_Threshold_Default = data[MBIC_PAYLOADSTART + 3]; 
-
                 break;
             case  ULO_Shutdown_ON_OFF                     :
                 bluecell_Currdatastatus.ULO_Shutdown_ON_OFF = data[MBIC_PAYLOADSTART + 3]; 
@@ -9464,12 +9546,7 @@ void FRBT_Tracking_Package(uint8_t Index, uint8_t* FRBT,uint8_t* DL_MainAtten,ui
     FRBT[FRBT_L] = DL_MainAtten[FRBT_L];        
 
 }
-typedef enum{
-    SelfTest1 = 0,
-    SelfTest2,
-    SelfTest3,
-    SelfTest4,    
-};
+
 void SelfTest_TimerOff(uint8_t num,uint8_t* selftest){
     GPIO_TypeDef *Port = 0;
     uint16_t Pin = 0;
@@ -9513,6 +9590,69 @@ void SelfTest_TimerOff(uint8_t num,uint8_t* selftest){
     HAL_GPIO_WritePin(Port,Pin,GPIO_PIN_SET);//CLOCK
     
 }
+
+void SelfTest_Ctrl(uint8_t num,uint8_t val,uint8_t* selftest,uint8_t* DL_Atten,uint8_t* UL_Atten){
+    GPIO_TypeDef *Port = 0;
+    uint16_t Pin = 0;
+    GPIO_TypeDef *_Port = 0;
+    uint16_t _Pin = 0;   
+    
+    switch(num){
+        case SelfTest1:
+        _Port = _PATH_SW1_GPIO_Port;
+        _Pin = _PATH_SW1_Pin;
+        
+        Port = PATH_SW1_GPIO_Port;
+        Pin =  PATH_SW1_Pin;   
+        break;
+        case SelfTest2:
+        _Port = _PATH_SW2_GPIO_Port;
+        _Pin = _PATH_SW2_Pin;
+        
+        Port = PATH_SW2_GPIO_Port;
+        Pin =  PATH_SW2_Pin;        
+        break;
+        case SelfTest3:
+        _Port = _PATH_SW3_GPIO_Port;
+        _Pin = _PATH_SW3_Pin;
+        
+        Port = PATH_SW3_GPIO_Port;
+        Pin =  PATH_SW3_Pin;        
+        break;
+        case SelfTest4:
+        _Port = _PATH_SW4_GPIO_Port;
+        _Pin = _PATH_SW4_Pin;
+        
+        Port = PATH_SW4_GPIO_Port;
+        Pin =  PATH_SW4_Pin;        
+        break;        
+    }
+
+        
+    if(val == true){
+        Selftest_DL_PrevAttenSave[num * 2]         = DL_Atten[Atten_H];
+        Selftest_DL_PrevAttenSave[((num * 2) + 1)] = DL_Atten[Atten_L];
+        Selftest_UL_PrevAttenSave[num * 2]         = UL_Atten[Atten_H];
+        Selftest_UL_PrevAttenSave[((num * 2) + 1)] = UL_Atten[Atten_L];
+        DL_Atten[Atten_H] = 0;
+        DL_Atten[Atten_L] = 0;
+        UL_Atten[Atten_H] = 0;
+        UL_Atten[Atten_L] = 0;          
+        HAL_GPIO_WritePin(_Port,_Pin,GPIO_PIN_SET);//CLOCK
+        HAL_GPIO_WritePin(Port,Pin,GPIO_PIN_RESET);//CLOCK
+    }else{
+        DL_Atten[Atten_H] = Selftest_DL_PrevAttenSave[num * 2];
+        DL_Atten[Atten_L] = Selftest_DL_PrevAttenSave[((num * 2) + 1)];    
+        UL_Atten[Atten_H] = Selftest_UL_PrevAttenSave[num * 2];
+        UL_Atten[Atten_L] = Selftest_UL_PrevAttenSave[((num * 2) + 1)];              
+        HAL_GPIO_WritePin(_Port,_Pin,GPIO_PIN_RESET);//CLOCK
+        HAL_GPIO_WritePin(Port,Pin,GPIO_PIN_SET);//CLOCK
+    }
+    CompareAttenData(bluecell_Currdatastatus,bluecell_Prevdatastatus);       
+    *selftest = val;
+    
+}
+
 void SelfTestTimer_Operate()
 {
     SelfTest_TimerOff(SelfTest1,&bluecell_Currdatastatus.Selftest1);

Bluecell_Src/flash.c

@@ -28,6 +28,25 @@ void Jump_App(void){
     jump_to_app();
 }
 
+uint8_t CurrFlash_BankCheck(){
+    uint32_t  CurrApiAddress = 0,Bank1Address=0,Bank2Address = 0;
+    uint32_t i = 0;
+    uint8_t ret = 0;
+    CurrApiAddress = FLASH_MBICUSER_START_ADDR;
+    Bank1Address = FLASH_USER_BANK1_START_ADDR;
+    Bank2Address = FLASH_USER_BANK2_START_ADDR;
+    
+    uint8_t* Currdata = (uint8_t*)CurrApiAddress;
+    uint8_t* Bank1data = (uint8_t*)Bank1Address;
+    uint8_t* Bank2data = (uint8_t*)Bank2Address;
+
+    for(i = 0; i < MBIC_BOOT_DATA; i++){
+        if(Currdata[MBIC_BOOT_MARK + i] != Bank2data[MBIC_BOOT_MARK + i]){
+            return HFR_BANK1_SEL;
+        }
+    }
+    return HFR_BANK2_SEL;
+}
 
 
 void Flash_InitRead() // ?占쏙옙湲고븿?占쏙옙
@@ -177,7 +196,7 @@ void Flash_InitRead() // ?占쏙옙湲고븿?占쏙옙
     | Bank1data[MBIC_BOOT_LENGTH + 1]<<16 
     | Bank1data[MBIC_BOOT_LENGTH + 2]<<8   
     | Bank1data[MBIC_BOOT_LENGTH + 3]);
-    if(CrcLength > 0 )
+    if(CrcLength > 0 && CrcLength <= 0x1FFFF )
         crcret = crc32(&Bank1data[MBIC_BOOT_DATA], CrcLength);
     if(crcret != FileCrc){
         bluecell_Currdatastatus.CPU_Bank1_Image_Version1 = 0xFF;
@@ -254,7 +273,7 @@ void Flash_InitRead() // ?占쏙옙湲고븿?占쏙옙
     | Bank2data[MBIC_BOOT_LENGTH + 1]<<16 
     | Bank2data[MBIC_BOOT_LENGTH + 2]<<8   
     | Bank2data[MBIC_BOOT_LENGTH + 3]);       
-     if(CrcLength > 0 )
+     if(CrcLength > 0 && CrcLength <= 0x1FFFF )
         crcret = crc32(&Bank2data[MBIC_BOOT_DATA], CrcLength);
     if(crcret != FileCrc){
    

Src/main.c

@@ -304,8 +304,10 @@ int main(void)  {
         | Bank1data[MBIC_BOOT_LENGTH + 2]<<8   
         | Bank1data[MBIC_BOOT_LENGTH + 3]);    
 
-        if(CrcLength > 0)
+        if(CrcLength > 0 && CrcLength <= 0x1FFFF ){
+            printf("Bank 1 Crc ret : %x  Length : %x :  %d \r\n",crcret ,CrcLength,CrcLength);
             crcret = CRC16_Generate(Bank1data, CrcLength + 128);
+        }
         CrcLength += 128;
         printf("Bank 1 Crc ret : %x  Length : %x :  %d \r\n",crcret ,CrcLength,CrcLength);
         
@@ -318,6 +320,7 @@ int main(void)  {
         
         printf("Bank 2 Crc ret : %x  Length : %x :  %d  \r\n", crcret,CrcLength,CrcLength );
         CrcLength += 128;
+        if(CrcLength > 0 && CrcLength <= 0x1FFFF )
          crcret = CRC16_Generate(Bank2data, CrcLength );
         printf("Bank 2 Crc ret : %x  Length : %x :  %d  \r\n", crcret,CrcLength,CrcLength );