Hi Liu Ying,

On Sat, Oct 6, 2012 at 7:32 AM, Liu Ying <ying....@freescale.com> wrote:

> @@ -397,6 +397,9 @@ void ipu_reset(void)
>         value = __raw_readl(reg);
>         value = value | SW_IPU_RST;
>         __raw_writel(value, reg);
> +
> +       while (__raw_readl(reg) & SW_IPU_RST)
> +               ;

Ok, but if the reset fails we would hand the whole system.

Wouldn't it be better to add a timeout here?

Regards,

Fabio Estevam
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to