Thread (8 messages) 8 messages, 4 authors, 2009-07-09

Re: [PATCH]: [MPC5200] Add ATA DMA support

From: Grant Likely <hidden>
Date: 2008-12-20 07:15:29

On Tue, Nov 25, 2008 at 8:19 AM, Matt Sealey [off-list ref] wrote:

On Tue, Nov 25, 2008 at 8:45 AM, Lehmann, Hans (Ritter Elektronik)
[off-list ref] wrote:
quoted
Tim, Grant,

just an info.

Very often the Bestcomm-FEC crashed without any error logs if I initiate a
transaction over FEC and save the file to disk (I rememeber I have read
something like that). A restart of FEC don't work.
But no I figured out, if I connect to the other ethernet port of our board
(natsemi) the FEC will awake to life again, if I initiate a new transaction
over natsemi. This seems a little oddly to me.
Hi guys,

We tried to get the SUSE guys to push the patch into openSUSE 11.1 release
and the tests came back negative here too with regards to FEC support; for
some odd reason, it does this;

https://bugzilla.novell.com/show_bug.cgi?id=445856#c10

There is some weird interaction here, but I can't imagine what it might be.
Tim, did you ever see any ethernet problems like this?
It looks like something getting messed up during allocation of buffers
and not DMA operation related at all.  I don't see the problem here...

Digging deeper I think the problem is due to this:
  ------- Comment #6 From Olaf Hering 2008-11-18 01:32:52 MST  -------

Also, the patches do not seem to apply and compile with our kernel-source.
bcom_task->bd changed to void*, this breaks sound/ppc/mpc52xx_ac97.c. It looks
like an unneeded cosmetic change.
The Kconfig changes are appearently also not required for an existing .config.

I removed the bcom_task->bd  change.
The patch does depend on bcom_task->bd being (void *).  Without it the
calculation of BD offsets is entirely wrong in bcom_get_bd().

That being said, the next version of the patch will leave the pointer
as (bcom_bd *) and handle the offset calculation in a different way.

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help