Dear Stefan,

In message <[email protected]> Dave 
Mitchell wrote:
...
> --- a/board/amcc/canyonlands/canyonlands.c
> +++ b/board/amcc/canyonlands/canyonlands.c
> @@ -177,8 +177,11 @@ int board_early_init_f(void)
>       /* Remove NOR-FLASH, NAND-FLASH & EEPROM hardware write protection */
>       out_8((void *)CONFIG_SYS_BCSR_BASE + 5, 0);
>  
> -     /* Enable USB host & USB-OTG */
> +     /* Enable USB host & USB-OTG;force assert,then de-assert PHY reset */
> +     out_8((void *)CONFIG_SYS_BCSR_BASE + 7, 1);
> +     mdelay(100);
>       out_8((void *)CONFIG_SYS_BCSR_BASE + 7, 0);
> +     mdealy(100);

We should get rid of this "CONFIG_SYS_BCSR_BASE + offset" stuff and
use a proper C struct for this.  What do you think?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
Testing can show the presense of bugs, but not their absence.
                                                   -- Edsger Dijkstra
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to