linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: "David H. Lynch Jr." <dhlii@dlasys.net>
To: Andrei Konovalov <akonovalov@ru.mvista.com>
Cc: linuxppc-dev@ozlabs.org, linuxppc-embedded@ozlabs.org
Subject: Re: Xilinx git tree at source.mvista.com
Date: Thu, 24 May 2007 20:47:34 -0400	[thread overview]
Message-ID: <46563226.8000000@dlasys.net> (raw)
In-Reply-To: <46558E54.80909@ru.mvista.com>

Andrei Konovalov wrote:
> Hi David,
>
> David H. Lynch Jr. wrote:
>>>   
>> I have an almost working FIFO TEMAC driver. It is similarly based.
>> it started out based on the Trek webserver sample
>
> Is this a reference design by Xilinx? Linux based or standalone?
    I did a Local Link hard TEMAC driver - that I eventually got
working. However, I could not find anyway to
    enable interrupts in the Local Link TEMAC so the driver was strictly
polled and between that and other
    issues, perfomances was abysmal almost 50% of all inbound packets
were dropped, we switched to the
    PLB FIFO TEMAC I stripped out all the SG and DMA stuff (as it is not
in our hardware) and converted it to
    a fairly normal Linux ethernet driver. It sends, it receives, but I
beleive it is not properly confirming to Linux
    that packets were successfully sent.

    In the interim I have been using the posted driver that uses the EDK.
    Until more recently that has lacked features like
    autonegotiation.

    I beleive its current flaw is primarily massive violations of kernel
code style guidelines.


>
>> I spent probably 3-4 days de-EDKing it into something that fit into a
>> single source and was closer to ko norms.
>> It is based on approximately the EDK 8.1 stuff. You are welcome to it,
>> if it could be helpful in anyway.
>> I am all for getting an acceptable driver into the ko tree.
>
> You could post your driver to the list when you think it is in good
> enough shape.
> If your driver is based on the linux TEMAC driver from EDK, it shouldn't
> be very different from my version (my added value is mostly replacing the
> custom PHY code with the PHY lib stuff). Then we could merge our
> drivers (or
> whatever would make sense).
>
> I would be interested to have a look at your current code just to see
> how much has it cost to "de-EDK" the FIFO part. You could email me
> your (even not quite working) driver privately if you want.

    I will email you a copy separately. I do not care what you choose to
do with it.
    At the moment I have no time to take it further - something about
asses, alligators
    and clearing swamps.

    Mostly I would just like to see a Linux friendly driver make its way
into the kernel.

    I have almost exactly the same code working as a GHS integrity driver.
    In fact I sort of ported a mini shim layer to GHS to implement Linux
SKB's under GHS.
    I did trip over something with GHS. If I was able to 64bit align the
data part of the skb
    the send/receive code code be vastly simplified.
    I have not but I have not done that yet.






>
>
> Thanks,
> Andrei
>


-- 
Dave Lynch 					  	    DLA Systems
Software Development:  				         Embedded Linux
717.627.3770 	       dhlii@dlasys.net 	  http://www.dlasys.net
fax: 1.253.369.9244 			           Cell: 1.717.587.7774
Over 25 years' experience in platforms, languages, and technologies too numerous to list.

"Any intelligent fool can make things bigger and more complex... It takes a touch of genius - and a lot of courage to move in the opposite direction."
Albert Einstein

  parent reply	other threads:[~2007-05-25  0:47 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-24  8:53 Xilinx git tree at source.mvista.com Andrei Konovalov
2007-05-24  9:57 ` David H. Lynch Jr.
2007-05-24 13:08   ` Andrei Konovalov
2007-05-24 15:42     ` Wolfgang Reissnegger
2007-05-25  1:49       ` David H. Lynch Jr.
2007-05-25  4:26         ` Wolfgang Reissnegger
2007-05-28 23:30           ` John Williams
2007-05-29  8:07             ` Andrei Konovalov
2007-05-25  0:47     ` David H. Lynch Jr. [this message]
2007-06-12 10:29 ` Esben Nielsen
2007-06-12 17:09   ` Dale Farnsworth

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=46563226.8000000@dlasys.net \
    --to=dhlii@dlasys.net \
    --cc=akonovalov@ru.mvista.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=linuxppc-embedded@ozlabs.org \
    /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;
as well as URLs for NNTP newsgroup(s).