jh.chun/VesiScan-Basic-firmware-test
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: BLE TX 먹통 해결 및 메모리 안전성 개선

Browse Source 
- binary_tx_handler를 dr_binary_tx_safe로 전체 교체 (APP_ERROR_CHECK 제거)
- data_tx_handler APP_ERROR_CHECK → DBG_PRINTF 교체
- memset/memcpy 하드코딩 크기를 define 상수로 교체 (버퍼 오버런 수정)
- SERIAL_NO_LENGTH, HW_NO_LENGTH, PASSKEY_LENGTH를 main.h로 통합
- 미사용 HW 드라이버/EEPROM 코드 삭제, TWI를 i2c_manager.c로 통합
- EEPROM → FDS 전환, 코드 리뷰 현황 문서 추가

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
jhchun
2026-03-16 16:39:26 +09:00
parent 836ebe5878
commit a1ad2a4b5b
76 changed files with 43253 additions and 16370 deletions
Download Patch File Download Diff File Expand all files Collapse all files

473
data/CODE_REVIEW_STATUS.md Normal file
View File

@@ -0,0 +1,473 @@
# VesiScan-Basic 코드 리뷰 현황 보고서
작성일: 2026-03-16
대상: VesiScan BASIC 펌웨어 전체 (nRF52840 + S140 SoftDevice)
기준 문서: CODE_REVIEW.pdf (2026-03-15)
현재 빌드: DEBUG_MINIMAL_BOOT=1, BLE_DEV_MODE=1
## 목차
1. 삭제 대상: 미사용 하드웨어 드라이버
2. 삭제 대상: 미사용 코드
3. DEBUG_MINIMAL_BOOT 조건부 컴파일 정리
4. Keil 프로젝트 파일 정리
5. 버그 및 오류 수정
6. 저전력 최적화
7. 코드 품질 개선
8. 작업 우선순위 요약
---
## 1. 삭제 대상: 미사용 하드웨어 드라이버
### 1.1 회로 기반 분석
변경 없음. VB0HW0100 회로 기준 IC 목록은 CODE_REVIEW.pdf 1.1절과 동일.
### 1.2 회로에 없는 IC의 드라이버 (삭제 대상)
| 파일 | IC/기능 | 판정 | 현재 상태 |
|------|--------|------|----------|
| ad5272_i2c.c/h | AD5272 디지털 포텐시오미터 | 삭제 | **완료** |
| ada2200_spi.c/h | ADA2200 동기 복조기 | 삭제 | **완료** |
| mcp4725_i2c.c/h | MCP4725 DAC | 삭제 | **완료** |
| mcp4725_adc.c/h | MCP4725 ADC 피드백 | 삭제 | **완료** |
| ir_i2c.c/h | IR LED 센서 (0x50~0x57) | 삭제 | **완료** |
| LED_Parse.c/h | LED ROM 파싱 | 삭제 | **완료** |
### 1.3 기능적으로 미사용인 드라이버 (삭제 대상)
| 파일 | 기능 | 판정 | 현재 상태 |
|------|------|------|----------|
| measurements.c/h | 레거시 측정 루프 | 삭제 | **완료** |
| measurements_20.c/h | 20-샘플 측정 변형 | 삭제 | **완료** |
| meas_pd_imm.c/h | 즉시 PD 측정 | 삭제 | **완료** |
| meas_pd_buff.c/h | 버퍼드 PD 측정 | 삭제 | **완료** |
| meas_pd_voltage_simple.c/h | 단순 전압 측정 | 삭제 | **완료** |
| meas_pd_voltage_half.c/h | 반파 전압 측정 | 삭제 | **완료** |
| meas_pd_voltage_full.c/h | 전파 전압 측정 | 삭제 | **완료** |
| meas_pd_voltage_custom.c/h | 커스텀 전압 측정 | 삭제 | **완료** |
| full_agc.c/h | 자동 이득 제어 | 삭제 | **완료** |
| debuf_print.h | 오타 파일 | 삭제 | **완료** |
### 1.4 삭제 시 의존성 영향
cmd_parse.h에서 제거할 include 목록:
| include | 현재 상태 |
|---------|----------|
| #include "ir_i2c.h" | **완료** (제거됨) |
| #include "ad5272_i2c.h" | **완료** (제거됨) |
| #include "ada2200_spi.h" | **완료** (제거됨) |
| #include "mcp4725_i2c.h" | **완료** (제거됨) |
| #include "measurements.h" | **완료** (제거됨) |
| #include "meas_pd_voltage_simple.h" | **완료** (제거됨) |
| #include "meas_pd_voltage_half.h" | **완료** (제거됨) |
| #include "meas_pd_voltage_full.h" | **완료** (제거됨) |
| #include "full_agc.h" | **완료** (제거됨) |
| #include "meas_pd_imm.h" | **완료** (제거됨) |
### 1.5 cat_interface.c (EEPROM 드라이버) 삭제 분석
#### 1.5.1 결론: EEPROM 삭제 완료, TWI는 i2c_manager.c로 통합
현재 상태: **완료** - jhChun 26.03.16
cat_interface.c 삭제(EEPROM/AES 코드 전체 삭제). TWI 인스턴스(m_twi) 및 초기화/해제 함수는 i2c_manager.c로 통합.
#### 1.5.2 cat_interface.c가 제공하는 것
| 기능 | 판정 | 현재 상태 |
|------|------|----------|
| m_eeprom TWI 인스턴스 | i2c_manager.c로 이동, m_twi로 리네이밍 | **완료** |
| eeprom_initialize() / uninitialize() | i2c_manager.c로 이동, twi_initialize/uninitialize로 리네이밍 (static) | **완료** |
| eeprom_read/write_byte/word/page/bytes | 삭제 | **완료** |
| eeprom_read/write_uint16_array | 삭제 | **완료** |
| eeprom_read/write_uint32 | 삭제 | **완료** |
| eeprom_write_encrypted() / read_decrypted() | 삭제 | **완료** |
| encrypt_data() / decrypt_data() | 삭제 | **완료** |
| aes_key[16], aes_iv[16] | 삭제 | **완료** |
#### 1.5.3 권장 작업
| # | 작업 | 현재 상태 |
|---|------|----------|
| 1 | TWI 인스턴스/init/uninit → i2c_manager.c로 통합 | **완료** |
| 2 | m_eeprom → m_twi 리네이밍 | **완료** |
| 3 | EEPROM read/write 전체 삭제, AES 전체 삭제 | **완료** |
| 4 | cmd_parse.c EEPROM 호출 -> FDS 전환 | **완료** |
| 5 | main.c 버튼 롱프레스 리셋: EEPROM -> FDS | **완료** |
| 6 | power_control.c: cat_interface.h include 제거 | **완료** |
#### 1.5.4 cmd_parse.c EEPROM -> FDS 전환 목록
| EEPROM 호출 | 위치 | FDS 대체 방법 | 현재 상태 |
|------------|------|------------|----------|
| eeprom_init_values_read() | cmd_parse.c | 삭제 (FDS config_load 대체) | **완료** |
| eeprom_write_byte(0x0070, m_pd_adc_cnt) | cmd_parse.c | m_config.pd_adc_cnt = val; config_save(); | **완료** |
| eeprom_write_word(0x0080, m_pd_delay_us) | cmd_parse.c | m_config.pd_delay_us = val; config_save(); | **완료** |
| eeprom_write_byte(0x0060, bond_data_delete) | cmd_parse.c | m_config.bond_data_delete = val; config_save(); | **완료** |
| eeprom_write_byte(0x0065, m_reset_status) | cmd_parse.c | m_config.reset_status = val; config_save(); | **완료** |
| eeprom_write_encrypted(0x0020, passkey, 6) | cmd_parse.c | memcpy(m_config.static_passkey, val, 6); config_save(); | **완료** |
| eeprom_read_decrypted(0x0020, passkey, 6) | cmd_parse.c | memcpy(buf, m_config.static_passkey, 6); | **완료** |
| eeprom_write_uint32(0x0090, m_life_cycle) | cmd_parse.c | m_config.life_cycle = val; config_save(); (FDS 필드 추가 완료) | **완료** |
| eeprom_read_uint32(0x0090, &m_life_cycle) | cmd_parse.c | m_life_cycle = m_config.life_cycle; | **완료** |
| eeprom_write/read_uint16_array(0x0480, ...) | cmd_parse.c | AGC gain — HW 제거됨, 명령 비활성화 | **완료** |
---
## 2. 삭제 대상: 미사용 코드
### 2.1 AES 암호화 (cat_interface.c)
판정 변경: 전체 삭제
| 항목 | 위치 | 판정 | 현재 상태 |
|------|------|------|----------|
| aes_key[16] 하드코딩 | cat_interface.c | 삭제 | **완료** |
| aes_iv[16] (고정 IV = 0) | cat_interface.c | 삭제 | **완료** |
| encrypt_data() | cat_interface.c | 삭제 | **완료** |
| decrypt_data() | cat_interface.c | 삭제 | **완료** |
| eeprom_write_encrypted() | cat_interface.c | 삭제 | **완료** |
| eeprom_read_decrypted() | cat_interface.c | 삭제 | **완료** |
### 2.2 Serial Number / Passkey / HW Version 중복 선언 분석
| 데이터 | 전역 변수 (런타임) | FDS (내장 Flash) | EEPROM (외부, 제거됨) | 현재 상태 |
|--------|----------------|----------------|-------------------|----------|
| Serial Number | SERIAL_NO[12] | m_config.serial_no[12] | ~~addr 0x0030~~ | **EEPROM 경로 삭제 완료** |
| Passkey | m_static_passkey[6] | m_config.static_passkey[6] | ~~addr 0x0020~~ | **EEPROM 경로 삭제 완료** |
| HW Version | HW_NO[12] | m_config.hw_no[12] | ~~addr 0x0010~~ | **HW_NO 미삭제** |
권장 작업 현황:
| # | 작업 | 현재 상태 |
|---|------|----------|
| 1 | EEPROM 경로 삭제 -> eeprom_init_values_read() 삭제 | **완료** |
| 2 | HW_NO[12] 삭제 -> m_config.hw_no 직접 사용 | **미완료** |
| 3 | SERIAL_NO, m_static_passkey 유지 (BLE 스택 필요) | 해당없음 |
| 4 | 데이터 흐름 단순화: FDS -> 전역변수 -> BLE 스택 | **미완료** |
### 2.3 미사용 전역 변수
| 변수 | 위치 | 판정 | 현재 상태 |
|------|------|------|----------|
| roles_str[] | main.c:189 | 삭제 | **미완료** |
| m_encrypted_text[16] | main.c:194 | 삭제 | **미완료** |
| m_encrypted_text2[16] | main.c:195 | 삭제 | **미완료** |
| m_decrypted_text[16] | main.c:196 | 삭제 | **미완료** |
| DEVICE_NAME | cmd_parse.c | 삭제 | 확인 필요 |
| HW_NO[12] | cmd_parse.c | 삭제 | **미완료** |
| AES_KEY_SIZE | main.c:154 | 삭제 | **미완료** |
| AES_BLOCK_SIZE | main.c:155 | 삭제 | **미완료** |
| suppress_unused_warnings() | main.c:239 | 삭제 | **미완료** |
| c_addr[6] | main.c:187 | 삭제 | **미완료** |
| led_pd_dac_v[LED_NUM] | 참조 다수 | 삭제 | 확인 필요 |
### 2.4 주석 처리된 코드
| 위치 | 내용 | 판정 | 현재 상태 |
|------|------|------|----------|
| main_timer.c:37-88 | FEATURE_DETAIL_VALUE_FULL 주석 블록 | 삭제 | **미완료** |
| cmd_parse.c:282-317 | serial_to_passkey_hash, test_eeprom_page_rw | 삭제 | **미완료** |
| cmd_parse.c:87-88 | pd_adc_half/full_a_start extern | 삭제 | 확인 필요 |
| battery_saadc.c:22 | //#include "fstorage.h" | 삭제 | **미완료** |
---
## 3. DEBUG_MINIMAL_BOOT 조건부 컴파일 정리
현재 DEBUG_MINIMAL_BOOT=1로 MINIMAL 모드만 사용. Full boot 경로는 실행되지 않음.
### 3.1 제거 대상 #if 블록
| 파일 | 블록 | 판정 | 현재 상태 |
|------|------|------|----------|
| main.c:280-281 | #if !DEBUG_MINIMAL_BOOT full_gpio_init() | 삭제 (함수 전체) | **미완료** |
| main.c:354-355 | #if !DEBUG_MINIMAL_BOOT load_eeprom_config() | 삭제 (함수 전체, EEPROM 제거 완료) | **완료** (함수 삭제됨) |
| main.c:1437-1439 | Full boot 경로 (ada2200_init, mcp4725_init 등) | 삭제 | **미완료** |
| main.c GPIO init 분기 | GPIO init 분기 | MINIMAL 코드만 유지 | **미완료** |
| power_control.c | Full boot power sequence | 삭제 (minimal 경로만 유지) | **미완료** |
### 3.3 DEBUG_MINIMAL_BOOT 제거 시 함께 정리
| 항목 | 현재 상태 |
|------|----------|
| #define DEBUG_MINIMAL_BOOT 1 삭제 (main.c:96) | **미완료** (여전히 활성) |
| 모든 #if 조건 제거, MINIMAL 코드만 유지 | **미완료** |
| BLE_DEV_MODE define 유지 | 해당없음 |
---
## 4. Keil 프로젝트 파일 정리
### 4.1 삭제할 소스 참조 (uvprojx)
현재 상태: **정리 불필요**
ble_app_bladder_patch_s140.uvprojx 확인 결과, 아래 삭제 대상 파일들은 이미 프로젝트에 등록되어 있지 않음:
- ad5272_i2c.c, ada2200_spi.c, mcp4725_i2c.c, mcp4725_adc.c
- ir_i2c.c, LED_Parse.c, measurements.c, measurements_20.c
- full_agc.c, meas_pd_voltage_custom.c, cat_interface.c
uvprojx 수정 작업 없음.
---
## 5. 버그 및 오류 수정
### 5.1 심각도 높음 (HIGH)
#### B1: BLE TX 큐 풀 시 APP_ERROR_CHECK로 인한 시스템 먹통 현상
파일: main.c (구 binary_tx_handler)
현재 상태: **완료**
- `binary_tx_handler()` 함수 본체 삭제
- 모든 호출처를 `dr_binary_tx_safe()`로 교체 완료
- cmd_parse.c (22곳 + 함수 포인터 1곳)
- battery_saadc.c (2곳)
- app_raw.c (2곳)
- tmp235_q1.c (1곳)
- parser.c (17곳)
- main.h 선언 제거, dr_util.c/dr_adc121s051.c의 옛날 extern 선언 제거
- `dr_binary_tx_safe()`는 NRF_ERROR_RESOURCES 시 최대 100회 재시도 후 패킷 드롭, 연결 에러 시 graceful 리턴
- `data_tx_handler()`의 APP_ERROR_CHECK도 DBG_PRINTF로 교체 — 예상치 못한 에러 시에도 패킷 드롭 후 정상 동작 유지
#### B2: nrf_delay_ms(10) 블로킹 대기 (BLE TX)
파일: main.c:1221-1228
현재 상태: **완료**
조건문 로직이 정상 수정됨. NRF_ERROR_RESOURCES가 아닌 경우에만 delay 실행.
#### B3: extern + 초기화 동시 사용
파일: cmd_parse.c
현재 상태: **완료**
`extern int8_t pd_adc_counts[4] = {8, 16, 24, 32};` 선언이 제거됨.
#### B4: memset/memcpy 버퍼 크기 불일치 및 매직 넘버
파일: main.c config_apply_to_globals()
현재 상태: **완료**
- memset 오버런 수정: 하드코딩 `16` → `sizeof(SERIAL_NO)`, `PASSKEY_LENGTH` 사용
- memcpy 매직 넘버 수정: 하드코딩 `12` → `sizeof(m_config.hw_no)`, `sizeof(m_config.serial_no)`, `sizeof(SERIAL_NO)` 등으로 교체
- m_static_passkey memset/memcpy: `PASSKEY_LENGTH` define 활용
스택/힙 오버플로우로 인한 메모리 손상 가능성이 있는 심각한 버그.
### 5.2 심각도 중간 (MEDIUM)
#### B6: battery_event_handler에서 floating-point 연산
파일: battery_saadc.c:167
현재 상태: **미완료**
여전히 부동소수점 연산 사용 중:
```c
batt_lvl_in_milli_volt_1 = (batt_lvl_in_milli_volt_0) *1.42;
```
수정 필요:
```c
batt_lvl_in_milli_volt_1 = (batt_lvl_in_milli_volt_0 * 142) / 100;
```
#### B7: main_timer 단발 모드 문제
현재 상태: **해당없음** (의도적 설계로 판단)
#### B8: processing_start_tick 미사용
파일: cmd_parse.c:57
현재 상태: **완료** (해결됨)
변수가 5초 타임아웃 추적에 활용되고 있음:
- cmd_parse.c:483에서 초기값 확인
- cmd_parse.c:489에서 5초 경과 비교
#### B9: config_load() goto 기반 흐름
파일: fstorage.c:157-261
현재 상태: **미완료**
`goto cfg_load_start` 패턴이 여전히 존재. 3개의 goto 문과 2개의 레이블로 복잡한 흐름 유지 중. FDS 초기화 실패 시 무한 루프 가능성 여전.
### 5.3 심각도 낮음 (LOW)
#### B11: debuf_print.h 파일명 오타
현재 상태: **완료** (파일 삭제)
#### B12: 함수 중복 선언
파일: main.h:44-54
현재 상태: **미완료**
static 함수 프로토타입 6개가 헤더에 여전히 선언됨:
- advertising_start (FEATURE_SECURE_CONNECTION 내)
- t_power_off_timeout_handler
- power_control_handler
- main_s
- PM_s
- main_routine_handler
#### B13: LED_ALLOFF, PD_ALLOFF 매크로 정의 위치
현재 상태: 확인 필요
---
## 6. 저전력 최적화
### 6.1 현재 전력 구조 분석
변경 없음. CODE_REVIEW.pdf 6.1절과 동일.
### 6.2 개선 항목
#### P1: UART 비활성화 (가장 큰 절약 -- 예상 ~1mA 절감)
파일: main.c:1427
현재 상태: **미완료**
`uart_init()`이 무조건 호출됨. `#if BLE_DEV_MODE` 조건으로 감싸져 있지 않음.
수정 필요:
```c
#if BLE_DEV_MODE
uart_init();
#endif
```
#### P2: SEGGER RTT 비활성화 (프로덕션)
파일: debug_print.h
현재 상태: **미완료**
`ENABLE_PRINTF`가 `1`로 하드코딩되어 있음. `BLE_DEV_MODE`와 연동되지 않음.
수정 필요:
```c
#if BLE_DEV_MODE
#define ENABLE_PRINTF 1
#else
#define ENABLE_PRINTF 0
#endif
```
#### P3: 배터리 타이머 주기 최적화
파일: battery_saadc.c:48
현재 상태: **미완료**
`BATTERY_LOOP_INTERVAL = 5000ms` 고정. 동적 조절 로직 없음.
#### P4: IMU 타이머 조건부 시작
파일: main.c BLE 이벤트 핸들러
현재 상태: **미완료**
BLE_GAP_EVT_CONNECTED/DISCONNECTED 핸들러에 IMU 타이머 start/stop 로직 없음. BLE 미연결 시에도 IMU를 읽을 수 있는 상태.
#### P5: main_loop() 타이머 폴링 효율화
현재 상태: **미완료**
플래그 기반 폴링 패턴 그대로 유지 중.
#### P6: NRF_LOG 비활성화 고려
현재 상태: **미완료** (확인 필요)
### 6.3 예상 절감 효과
| 최적화 | 예상 절감 | 난이도 | 현재 상태 |
|--------|----------|--------|----------|
| P1: UART 비활성화 | ~1mA | 쉬움 | **미완료** |
| P2: RTT 비활성화 | ~0.1-0.5mA | 쉬움 | **미완료** |
| P4: IMU 타이머 조건부 시작 | ~41uA (미연결 시) | 쉬움 | **미완료** |
| P3: 배터리 타이머 동적 | ~5uA | 보통 | **미완료** |
| P5-A: 단발->직접호출 | 응답 개선 | 쉬움 | **미완료** |
| P5-B: app_scheduler | 코드 품질 + 확장성 | 보통 | **미완료** |
| P5-C: IMU FIFO+인터럽트 | 연속측정 시 ~50uA | 어려움 | **미완료** |
| P6: NRF_LOG 비활성화 | ~10uA | 쉬움 | **미완료** |
---
## 7. 코드 품질 개선
### 7.1 구조 개선
| # | 항목 | 현재 상태 |
|---|------|----------|
| Q1 | main.h에서 static 함수 선언 제거 | **미완료** (6개 잔존) |
| Q2 | cat_interface.c -> i2c_manager.c로 TWI 통합 | **완료** |
| Q3 | extern 변수 과다 사용 | **미완료** (30개 이상) |
| Q4 | main_timer.c main_loop() 플래그 기반 폴링 | **미완료** |
| Q5 | 매직 넘버 제거 | **미완료** |
### 7.2 코딩 스타일
변경 없음. CODE_REVIEW.pdf 7.2절과 동일.
---
## 8. 작업 우선순위 요약
### 즉시 수정 (안전/안정성)
| # | 항목 | 심각도 | 현재 상태 |
|---|------|--------|----------|
| 1 | B4: memset/memcpy 버퍼 크기 불일치 및 매직 넘버 | CRITICAL | **완료** |
| 2 | B1: BLE TX 큐 풀 시 시스템 먹통 (binary_tx_handler → dr_binary_tx_safe 교체) | HIGH | **완료** |
| 3 | B2: BLE TX 에러 핸들링 로직 반전 | HIGH | **완료** |
| 4 | B3: extern + 초기화 동시 사용 | HIGH | **완료** |
### 1차 정리 (미사용 코드/EEPROM 삭제)
| # | 항목 | 현재 상태 |
|---|------|----------|
| 5 | 미사용 HW 드라이버 삭제 (1.2절) | **완료** |
| 6 | cat_interface.c EEPROM 기능 삭제 + TWI를 i2c_manager.c로 통합 (1.5절) | **완료** |
| 7 | cmd_parse.c EEPROM 호출 -> FDS 전환 (1.5.4절) | **완료** |
| 8 | 미사용 전역변수 + HW_NO 삭제 (2.2-2.3절) | **미완료** |
| 9 | DEBUG_MINIMAL_BOOT 조건 제거 (3절) | **미완료** |
| 10 | Keil uvprojx 소스 참조 제거 (4절) | 확인 필요 |
| 11 | 주석 처리된 코드 삭제 (2.4절) | **미완료** |
### 2차 정리 (저전력 최적화)
| # | 항목 | 예상 절감 | 현재 상태 |
|---|------|----------|----------|
| 12 | P1: UART 비활성화 | ~1mA | **미완료** |
| 13 | P2: RTT 비활성화 | ~0.1-0.5mA | **미완료** |
| 14 | P4: IMU 타이머 BLE 연결 시에만 동작 | ~41uA | **미완료** |
| 15 | P5-A: main_loop 단발 이벤트 직접 호출로 전환 | 응답 개선 | **미완료** |
| 16 | P5-B: app_scheduler 도입 (선택) | 코드 품질 | **미완료** |
| 17 | P3: 배터리 타이머 동적 조절 | ~5uA | **미완료** |
### 3차 정리 (코드 품질)
| # | 항목 | 현재 상태 |
|---|------|----------|
| 18 | Q1: main.h static 함수 제거 | **미완료** |
| 19 | Q5: 매직 넘버 define화 | **미완료** |
| 20 | B9: config_load() goto 제거 | **미완료** |