diff --git a/driver/charger/rt946x.c b/driver/charger/rt946x.c index e2957a1fca..590b542977 100644 --- a/driver/charger/rt946x.c +++ b/driver/charger/rt946x.c @@ -699,19 +699,17 @@ static void rt946x_init(void) CPRINTF("RT946X incorrect ID: 0x%02x\n", reg); return; } - CPRINTF("Device ID(0x%02X) found\n", RT946X_VENDOR_ID); - /* Check revision id */ - if (rt946x_chip_rev(®) || reg < RT946X_CHIP_REV) { - CPRINTF("RT946X incorrect CHIP REV: 0x%02x\n", reg); + if (rt946x_chip_rev(®)) { + CPRINTF("Failed to read RT946X CHIP REV\n"); return; } + CPRINTF("RT946X CHIP REV: 0x%02x\n", reg); if (rt946x_init_setting()) return; - - CPRINTF("Device ID(0x%02X) initialized\n", RT946X_VENDOR_ID); + CPRINTF("RT946X initialized\n"); } DECLARE_HOOK(HOOK_INIT, rt946x_init, HOOK_PRIO_LAST); diff --git a/driver/charger/rt946x.h b/driver/charger/rt946x.h index 9341f52246..027de6f15e 100644 --- a/driver/charger/rt946x.h +++ b/driver/charger/rt946x.h @@ -235,12 +235,10 @@ #if defined(CONFIG_CHARGER_RT9466) #define RT946X_CHARGER_NAME "rt9466" #define RT946X_VENDOR_ID 0x80 - #define RT946X_CHIP_REV 0x05 #define RT946X_ADDR (0x53 << 1) #elif defined(CONFIG_CHARGER_RT9467) #define RT946X_CHARGER_NAME "rt9467" #define RT946X_VENDOR_ID 0x90 - #define RT946X_CHIP_REV 0x05 #define RT946X_ADDR (0x5B << 1) #endif