From: Andrei Konovalov <akonovalov@ru•mvista.com>
To: Grant Likely <grant.likely@secretlab•ca>
Cc: Linux PPC <linuxppc-embedded@ozlabs•org>,
Mohammad Sadegh Sadri <mamsadegh@hotmail•com>
Subject: Re: Mr Grant Likely's New Patches for 2.6 Kernel and Xilinx FPGA ( Virtex-4 & Virtex-2
Date: Mon, 16 Apr 2007 20:51:57 +0400 [thread overview]
Message-ID: <4623A9AD.4020100@ru.mvista.com> (raw)
In-Reply-To: <528646bc0704160926i73c7976ree34564a113b3caf@mail.gmail.com>
Grant Likely wrote:
> On 4/16/07, Andrei Konovalov <akonovalov@ru•mvista.com> wrote:
>> Grant Likely wrote:
>> > No, I did not include TEMAC support. However, I will publish my git
>> > tree tomorrow which will include the TEMAC driver.
>>
>>
>> Seems the TEMAC driver is not on your git server yet. Hence the
>> questions.
>> Guess this is PLB TEMAC version 3.00a as you wrote auto negotiation is
>> supported.
>
> yup
>
>> Is your driver based on the one by Xilinx (the one from EDK 8.2.02 and
>> the update posted by Rick Moleres)?
>
> Yup, it's just the adapter.c file that Rick posted plus some fixups to
> get it to compile in my tree. Nothing really exciting. I'm very
> interested in the phylib work that you've done. I hope to take a look
> at that in the next day or so.
The feedback is very welcome.
Mostly was trying to use what is already in the kernel, and not to reinvent
the wheel.
I wouldn't say I've completely done with it.
Few SGDMA related ioctls to add (should be fairly straightforward).
And to check the advertised capabilities - as I wrote in the previous
posting, it looks like the driver stops advertising 1000Mbits at some
point (after turning auto negotiation off, switching to the lower speeds,
and trying to go back to 1000Mbits).
> I'm only publishing my tree as a
> convienient starting point for those who need it.
>
> Ideally, I'd like to look at doing a full rewrite so it's in a
> condition acceptable for mainline, but I don't have time for that at
> the moment.
I am not certain if the "linux adapter plus the OS independent code"
could easily be accepted (provided that the OS independent code is
reformatted as much as possible to fit into the community).
Afraid complete rewrite is fairly large effort, and, moreover, so far PLB TEMAC
has been significantly changing from one EDK release to the next. So
one could have to rewrite considerable parts of the driver every EDK release.
Would be nice to reuse the man-hours Xilinx put in their OS indpendent code.
Thanks,
Andrei
> Cheers,
> g.
>
prev parent reply other threads:[~2007-04-16 16:47 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-04-15 10:42 Mr Grant Likely's New Patches for 2.6 Kernel and Xilinx FPGA ( Virtex-4 & Virtex-2 ) Mohammad Sadegh Sadri
2007-04-15 14:27 ` Mr Grant Likely's New Patches for 2.6 Kernel and Xilinx FPGA ( Virtex-4 & Virtex-2 Grant Likely
2007-04-16 16:10 ` Andrei Konovalov
2007-04-16 16:26 ` Grant Likely
2007-04-16 16:51 ` Andrei Konovalov [this message]
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=4623A9AD.4020100@ru.mvista.com \
--to=akonovalov@ru$(echo .)mvista.com \
--cc=grant.likely@secretlab$(echo .)ca \
--cc=linuxppc-embedded@ozlabs$(echo .)org \
--cc=mamsadegh@hotmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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