From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1 v2][Net] Convert CS8900 Ethernet driver to CONFIG_NET_MULTI API
Date: Thu, 13 Aug 2009 10:07:38 +0200 [thread overview]
Message-ID: <20090813080738.22A42833DBD2@gemini.denx.de> (raw)
In-Reply-To: <1250145251-23438-1-git-send-email-biggerbadderben@gmail.com>
Dear Ben Warren,
In message <1250145251-23438-1-git-send-email-biggerbadderben@gmail.com> you wrote:
> All in-tree boards that use this controller have CONFIG_NET_MULTI added
> Also:
> - changed CONFIG_DRIVER_CS8900 to CONFIG_CS8900
> - changed CS8900_BASE to CONFIG_CS8900_BASE
> - changed CS8900_BUS?? to CONFIG_CS8900_BUS??
> - cleaned up line lengths
> - modified VCMA9 command function that accesses the device
> - removed MAC address initialization from lib_arm/board.c
>
> Signed-off-by: Ben Warren <biggerbadderben@gmail.com>
> ---
>
> v2: Fixed typo: CS8900_BUS32 -> CONFIG_CS8900_BUS32
> Added more descriptive printf in cs8900_init as suggested by Wolfgang
> Initialize 'priv' data structure in cs8900_initialize()
Thanks. That's better, but no cigar yet.
I now get this:
TRAB # run load
Using CS8900-0 device
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: data abort
pc : [<0df6b254>] lr : [<0df4c694>]
sp : 0df1bb74 ip : 00000004 fp : 0000083c
r10: 0df79ca0 r9 : 000000c0 r8 : 0df1bfd8
r7 : 0df1c100 r6 : 0df1c140 r5 : 0000002a r4 : 0000005c
r3 : 07000300 r2 : 0000002a r1 : 0df79ca0 r0 : 0df1c100
Flags: nzCv IRQs off FIQs off Mode SVC_32
Resetting CPU ...
Decoding PC and LR gives:
PC: 0x0df6b254 -- 0x0df6b228 + 0x002c cs8900_send
LR: 0x0df4c694 -- 0x0df4c668 + 0x002c eth_send
Placing a few printf()s indicates that it crashes in the writel()
following the rety: label. Looking at the code, the
"&priv->regs->txcmd" looks a bit fishy to me. I played around a bit
(guessing for "priv->regs->txcmd" and "&(priv->regs->txcmd)" but this
doesn't change behaviour.
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
"We have the right to survive!"
"Not be killing others."
-- Deela and Kirk, "Wink of An Eye", stardate 5710.5
next prev parent reply other threads:[~2009-08-13 8:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-13 6:34 [U-Boot] [PATCH 1/1 v2][Net] Convert CS8900 Ethernet driver to CONFIG_NET_MULTI API Ben Warren
2009-08-13 8:07 ` Wolfgang Denk [this message]
2009-08-14 6:36 ` Ben Warren
2009-08-20 9:56 ` Wolfgang Denk
2009-08-20 17:36 ` Ben Warren
2009-08-13 9:41 ` Wolfgang Denk
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=20090813080738.22A42833DBD2@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox