All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1][Net] Convert CS8900 Ethernet driver to CONFIG_NET_MULTI API
Date: Thu, 13 Aug 2009 00:50:57 +0200	[thread overview]
Message-ID: <20090812225057.F0C2E833DBD2@gemini.denx.de> (raw)
In-Reply-To: <4A8342BE.7050600@gmail.com>

Dear Ben Warren,

In message <4A8342BE.7050600@gmail.com> you wrote:
> 
> It looks like the 'trab' board uses 16-bit accesses 
> (CONFIG_CS8900_BUS16).  The function 'get_reg_init_bus()' does some 
> funny initialization to get the chip into 16-bit mode.  I wonder if 
> maybe that's not working the same as the original.  That could explain 
> the crash too.

Yes...

> BTW - it shouldn't matter in this case, but there's a typo in cs8900.c 
> that this should fix:

I confirm that this change has no impact on the current issue.


Note however that your modification was probably not the (only)
culprit. With current mainline version I get this:

TRAB # run load
TFTP from server 192.168.1.1; our IP address is 192.168.3.68
Filename 'trab/u-boot.bin-wd'.
Load address: 0xc100000
Loading: T T T T T T T T T T 
Retry count exceeded; starting again
TFTP from server 192.168.1.1; our IP address is 192.168.3.68
Filename 'trab/u-boot.bin-wd'.
Load address: 0xc100000
Loading: T T T T T T T T T T 
Retry count exceeded; starting again
TFTP from server 192.168.1.1; our IP address is 192.168.3.68
Filename 'trab/u-boot.bin-wd'.
Load address: 0xc100000
Loading: T T T T T T T T T T 
Retry count exceeded; starting again

with the timeouts being *way* too fast.

It seems that 1.2.0 was the last version that has been actually
tested on trab / SMDK2400.

I'll try to run a git bisect tomorrow; need to get some sleep now.

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
As long as we're going to reinvent the wheel again, we might as  well
try making it round this time.                        - Mike Dennison

  reply	other threads:[~2009-08-12 22:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12  4:12 [U-Boot] [PATCH 0/1][Net] Convert CS8900 Ethernet driver to CONFIG_NET_MULTI API Ben Warren
2009-08-12  4:12 ` [U-Boot] [PATCH 1/1][Net] " Ben Warren
2009-08-12 22:17   ` Wolfgang Denk
2009-08-12 22:31     ` Ben Warren
2009-08-12 22:50       ` Wolfgang Denk [this message]
2009-08-12 22:53 Wolfgang Denk
2009-08-12 22:55 ` Ben Warren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090812225057.F0C2E833DBD2@gemini.denx.de \
    --to=wd@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.