diff options
author | Mulin Chao <MLChao@nuvoton.com> | 2018-04-24 11:48:10 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-04-27 12:22:39 -0700 |
commit | 9dd10a768521918d96af8d831f7b840ae6cb68bb (patch) | |
tree | 7eea288a2fa0e6b488b77d276d1a9ec62d154c71 /util | |
parent | 1820ecce31e6a23e5ab79f708f66a8655da6a161 (diff) | |
download | chrome-ec-9dd10a768521918d96af8d831f7b840ae6cb68bb.tar.gz |
npcx: change chip variants of npcx7 series for better clarification.
In this CL, we changed chip variants npcx7m6xb to npcx7m6fb and npcx7m7w
to npcx7m7wb for better clafiication since it introduced new parameter
"b" for chip generation in the same family series.
In new npcx7 series naming rule, it follows:
Format: NPCX7(M)(N)(G/K/F)(B/C)
param M: 8: 128-pins package, 9: 144-pins package
param N: 5: 128KB RAM Size, 6: 256KB RAM Size, 7: 384KB RAM Size
param G/K/F/W: Google EC depends on specific features.
param B/C: Chip generation in npcx7. (Generation A is ignored. It
follows nameing rule in npcx5.)
The all chip variants of npcx7 used in boards are also listed below:
npcx7m6g - for npcx7 ec without internal flash on npcx_evb.
npcx7m6f - for npcx7 ec with internal flash.
npcx7m6fb - for npcx7 ec with internal flash, enhanced features.
npcx7m7wb - for npcx7 ec with internal flash, enhanced features + WOV.
BRANCH=none
BUG=none
TEST=No build errors for npcx7 series.
Change-Id: I896ee33209efa5d7157c90515005db5f36318c76
Signed-off-by: Mulin Chao <mlchao@nuvoton.com>
Reviewed-on: https://chromium-review.googlesource.com/1025471
Reviewed-by: Jett Rink <jettrink@chromium.org>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
Diffstat (limited to 'util')
-rwxr-xr-x | util/ecst.c | 29 |
1 files changed, 13 insertions, 16 deletions
diff --git a/util/ecst.c b/util/ecst.c index cf4752041c..773f0d270c 100755 --- a/util/ecst.c +++ b/util/ecst.c @@ -43,6 +43,10 @@ struct chip_info chip_info[] = {{NPCX5M5G_RAM_ADDR, NPCX5M5G_RAM_SIZE}, {NPCX7M6X_RAM_ADDR, NPCX7M6X_RAM_SIZE}, {NPCX7M7X_RAM_ADDR, NPCX7M7X_RAM_SIZE},}; +/* Support chips name strings */ +const char *supported_chips = "npcx5m5g, npcx5m6g, npcx7m5g, npcx7m6g, " + "npcx7m6f, npcx7m6fb or npcx7m7wb"; + static unsigned int calc_api_csum_bin(void); static unsigned int initialize_crc_32(void); static unsigned int update_crc_32(unsigned int crc, char c); @@ -192,15 +196,12 @@ int main(int argc, char *argv[]) (sscanf(hdr_args[arg_ind], "%s", main_str_temp) != 1)) { - my_printf(TERR, "\nCannot read chip name, "); - my_printf(TERR, "npcx7m7w"); - my_printf(TERR, "npcx7m6xb, npcx7m6f"); - my_printf(TERR, ", npcx7m6g, npcx7m5g"); - my_printf(TERR, ", npcx5m5g or npcx5m6g.\n"); + my_printf(TERR, "\nCannot read chip name %s.\n", + supported_chips); main_status = FALSE; } else { if (str_cmp_no_case(main_str_temp, - "npcx7m7w") == 0) { + "npcx7m7wb") == 0) { if ((bin_params.bin_params & BIN_FW_LOAD_START_ADDR) == 0x00000000) @@ -220,7 +221,7 @@ int main(int argc, char *argv[]) } else if ((str_cmp_no_case(main_str_temp, "npcx7m6f") == 0) || (str_cmp_no_case(main_str_temp, - "npcx7m6xb") == 0) || + "npcx7m6fb") == 0) || (str_cmp_no_case(main_str_temp, "npcx7m6g") == 0)) { if ((bin_params.bin_params @@ -302,11 +303,8 @@ int main(int argc, char *argv[]) my_printf(TERR, "\nInvalid chip name (%s) ", main_str_temp); - my_printf(TERR, "should be npcx7m7w, "); - my_printf(TERR, "npcx7m6xb, "); - my_printf(TERR, "npcx7m6f, npcx7m6g, "); - my_printf(TERR, "npcx7m5g, npcx5m5g, "); - my_printf(TERR, "or npcx5m6g."); + my_printf(TERR, ", it should be %s.\n", + supported_chips); main_status = FALSE; } @@ -841,10 +839,9 @@ void exit_with_usage(void) my_printf(TUSG, "(default is out_<input_filename>.bin)"); my_printf(TUSG, "\n -argfile <filename> - Arguments file name; "); my_printf(TUSG, "includes multiple flags"); - my_printf(TUSG, "\n -chip <name> - EC Chip Name: "); - my_printf(TUSG, "npcx7m7w|npcx7m6f|npcx7m6xb|npcx7m6g|npcx7m5g|"); - my_printf(TUSG, "npcx5m5g|npcx5m6g"); - my_printf(TUSG, " (default is npcx5m5g)"); + my_printf(TUSG, "\n -chip <name> - Supported EC Chip Name: "); + my_printf(TUSG, "%s. ", supported_chips); + my_printf(TUSG, "(default is npcx5m5g)"); my_printf(TUSG, "\n -v - Verbose; prints "); my_printf(TUSG, "information messages"); my_printf(TUSG, "\n -vv - Super Verbose; prints "); |