Dear Stefan Roese,
In message 1247834006-3005-1-git-send-email-sr@denx.de you wrote:
This patch adds a generic command for programming I2C bootstrap eeproms on PPC4xx. An implementation for Canyonlands board is included.
The command name is intentionally chosen not to be PPC4xx specific. This way other CPU's/SoC's can implement a similar command under the same name, perhaps with a different syntax.
Usage on Canyonlands:
=> cpu_config Available configurations (I2C address 0x52): 600-nor - NOR CPU: 600 PLB: 200 OPB: 100 EBC: 100 600-nand - NAND CPU: 600 PLB: 200 OPB: 100 EBC: 100 800-nor - NOR CPU: 800 PLB: 200 OPB: 100 EBC: 100 800-nand - NAND CPU: 800 PLB: 200 OPB: 100 EBC: 100 1000-nor - NOR CPU:1000 PLB: 200 OPB: 100 EBC: 100 1000-nand - NAND CPU:1000 PLB: 200 OPB: 100 EBC: 100 1066-nor - NOR CPU:1066 PLB: 266 OPB: 88 EBC: 88 *** 1066-nand - NAND CPU:1066 PLB: 266 OPB: 88 EBC: 88
I like that!
diff --git a/cpu/ppc4xx/cmd_cpu_config.c b/cpu/ppc4xx/cmd_cpu_config.c new file mode 100644 index 0000000..9701dbb --- /dev/null +++ b/cpu/ppc4xx/cmd_cpu_config.c
...
+static int do_cpu_config(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[]) +{
...
- if (cur_config_nr == -1) {
printf("Warning: The I2C bootstrap EEPROM values don't match any"" of the available config options!\n");
This message doesn't fit into a single line. Please shorten or make 2 lines.
Best regards,
Wolfgang Denk