[U-Boot] [PATCH 1/3] mpc85xx: Add support for the MPC8572DS reference board
Kumar Gala
galak at kernel.crashing.org
Tue Aug 12 18:27:50 CEST 2008
On Aug 12, 2008, at 11:23 AM, Scott Wood wrote:
> Kumar Gala wrote:
>> + for (i = 0; i < CONFIG_CHIP_SELECTS_PER_CTRL; i++) {
>> + unsigned int bnds = 0;
>> + unsigned int config = 0;
>> + unsigned int config_2 = 0;
>> + unsigned int *pbnds = NULL;
>> + unsigned int *pconfig = NULL;
>> + unsigned int *pconfig_2 = NULL;
>> +
>> + if (i == 0) {
>> + bnds = ddr->cs0_bnds;
>> + config = ddr->cs0_config;
>> + config_2 = ddr->cs0_config_2;
>> + pbnds = (unsigned int *)&ddr->cs0_bnds;
>> + pconfig = (unsigned int *)&ddr->cs0_config;
>> + pconfig_2= (unsigned int *)&ddr->cs0_config_2;
>> +
>> + } else if (i == 1) {
>> + bnds = ddr->cs1_bnds;
>> + config = ddr->cs1_config;
>> + config_2 = ddr->cs1_config_2;
>> + pbnds = (unsigned int *)&ddr->cs1_bnds;
>> + pconfig = (unsigned int *)&ddr->cs1_config;
>> + pconfig_2= (unsigned int *)&ddr->cs1_config_2;
>> +
>> + } else if (i == 2) {
>> + bnds = ddr->cs2_bnds;
>> + config = ddr->cs2_config;
>> + config_2 = ddr->cs2_config_2;
>> + pbnds = (unsigned int *)&ddr->cs2_bnds;
>> + pconfig = (unsigned int *)&ddr->cs2_config;
>> + pconfig_2= (unsigned int *)&ddr->cs2_config_2;
>> +
>> + } else if (i == 3) {
>> + bnds = ddr->cs3_bnds;
>> + config = ddr->cs3_config;
>> + config_2 = ddr->cs3_config_2;
>> + pbnds = (unsigned int *) &ddr->cs3_bnds;
>> + pconfig = (unsigned int *) &ddr->cs3_config;
>> + pconfig_2= (unsigned int *) &ddr->cs3_config_2;
>> +
>> + } else {
>> + /*
>> + * FIXME what happens if CONFIG_CHIP_SELECTS_PER_CTRL > 4
>> + */
>> + }
>> +
>> + printf("cs%u_bnds = %08X\t%p\n", i, bnds, pbnds);
>> + printf("cs%u_config = %08X\t%p\n", i, config, pconfig);
>> + printf("cs%u_config_2 = %08X\t%p\n",
>> + i, config_2, pconfig_2);
>> + }
>
> Couldn't we use arrays for this sort of thing?
this is historic in how the immap is defined for these parts. As
patch is welcome to do such cleanup. I don't think it should hold up
acceptance.
- k
More information about the U-Boot
mailing list