Hi Peter

Thanks for your feedback, but ..

Am Mittwoch 23 September 2009 17:59:10 schrieb Peter Tyser:
> Hi Niklaus,
> 
<snip>
> > +                   case 0x350:
> > +                           value = mfdcr(0x350);
> > +                           break;
> > +                   case 0x026:
> > +                           value = mfdcr(0x026);
> > +                           break;
> 
> Replace all the above "value = ...; break;" with 1 "value =
> mfdcr(ppc440epx_reg[i].address); break;"?
This cannot be done, as mfdcr is assembler mfdcr register, ID_OF_DCR
ID_OF_DCR cannot be a variable.

> 
> <snip>
> 
> > +
> > +/* define do_reghcu5 as u-boot command */
> > +U_BOOT_CMD(reghcu5, 2, 1, do_reghcu5,
> > +      "reghcu5 - print register information for HCU5\n",);
> 
> This command's help message won't be printed correctly.
> 
> Best,
> Peter
All the rest will show up in the coming next patch.

Best regards

-- 
Niklaus Giger
Netstal Maschinen AG
CH-8752 Naefels
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to