public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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