[U-Boot] [RFC] Make sure the chip reset and init right
Wolfgang Denk
wd at denx.de
Mon Jan 10 22:44:44 CET 2011
Dear Yanjun Yang,
In message <AANLkTimOs07p88QYRpWL5wzD+eiVreBiGRqt2Q8DTgr2 at mail.gmail.com> you wrote:
> It seems that the chip can only be reset into a known
> state by using attribute space. The smc_reset and
> smc_enable function also need more lines to make the
> chip work.
>
> Signed-off-by: YanJun Yang <yangyj.ee at gmail.com>
> ---
> drivers/net/lan91c96.c | 26 ++++++++++++++++++++++----
> 1 files changed, 22 insertions(+), 4 deletions(-)
Your patch is white space corrupted again; we cannot apply it for
testing. Please fix.
> + SMC_outw(dev, LAN91C96_CTR_TE_ENABLE | LAN91C96_CTR_BIT_8,
> LAN91C96_CONTROL);
YOur patch is also line-wrapped.
> + udelay( 750 );
...
> + udelay( 750 );
> + attaddr[LAN91C96_ECOR] &= ~LAN91C96_ECOR_SRESET;
> + udelay( 750 );
> + attaddr[LAN91C96_ECOR] |= LAN91C96_ECOR_ENABLE;
> + udelay( 750 );
>
> + /* force 16-bit mode */
> + attaddr[LAN91C96_ECSR] &= ~LAN91C96_ECSR_IOIS8;
> + udelay( 750 );
Please omit these bogus spaces arount the function arguments.
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: wd at denx.de
"If God had wanted us to use the metric system, Jesus would have had
10 apostles."
More information about the U-Boot
mailing list