public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Vladimir Zapolskiy <vz@mleia.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/2] net: lpc32xx: add support to RMII phy
Date: Mon, 06 Jul 2015 10:55:25 +0300	[thread overview]
Message-ID: <559A346D.80907@mleia.com> (raw)
In-Reply-To: <20150706084621.2bfccdaa@lilith>

Hello Albert,

On 06.07.2015 09:46, Albert ARIBAUD (3ADEV) wrote:
> Hello Vladimir,
> 
> On Mon,  6 Jul 2015 07:22:09 +0300, Vladimir Zapolskiy <vz@mleia.com>
> wrote:
>> This is an update to LPC32xx MAC driver, which needs to have small
>> quirks in order to properly work with an external phy connected by
>> RMII interface.
>>
>> RMII mode of MAC operation is selected, if CONFIG_RMII is defined,
>> this option is aligned with a number of boards, which already have
>> the same config value.
>>
>> Also hopefully the first change from the series speeds up auto
>> negotiation due to corrected configuration of MAC on intialization.
>>
>> Tested with SMSC LAN8700 phy.
>>
>> Vladimir Zapolskiy (2):
>>   net: lpc32xx: improve MAC configuration on reset and initialization
>>   net: lpc32xx: add RMII phy mode support
> 
> This series actually requires two other lpc32xx patches that you posted
> earlier but which are not yet in u-boot/master or u-boot-arm/master:
> 
> 	http://patchwork.ozlabs.org/patch/489100/
> 	http://patchwork.ozlabs.org/patch/489190/
> 
> In the future, please indicate when such a dependency is required, so
> that testers know how to apply your patchs beforehand.

my fault, certainly will add such a note in future...

> This point aside, I've tested the four patches above u-boot-arm/master
> on a work_92105 target which has CONFIG_MII, and found no regression...
> and a dramatic increase in TFTP throughput, so thanks. :)

Thank you so much for testing MII, this is something I'm unable to do on
my end.

> (not adding my Tested-by because I don't want to mislead anyone: I
> haven't tested RMII)
> 
> Amicalement,
> 

--
With best wishes,
Vladimir

  reply	other threads:[~2015-07-06  7:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-06  4:22 [U-Boot] [PATCH 0/2] net: lpc32xx: add support to RMII phy Vladimir Zapolskiy
2015-07-06  4:22 ` [U-Boot] [PATCH 1/2] net: lpc32xx: improve MAC configuration on reset and initialization Vladimir Zapolskiy
2015-08-12 19:34   ` Joe Hershberger
2015-07-06  4:22 ` [U-Boot] [PATCH 2/2] net: lpc32xx: add RMII phy mode support Vladimir Zapolskiy
2015-07-06 18:50   ` LEMIEUX, SYLVAIN
2015-07-06 20:45     ` Vladimir Zapolskiy
2015-07-06 21:48       ` LEMIEUX, SYLVAIN
2015-07-15 18:56   ` LEMIEUX, SYLVAIN
2015-07-29 18:53   ` LEMIEUX, SYLVAIN
2015-08-12 19:34   ` Joe Hershberger
2015-07-06  6:46 ` [U-Boot] [PATCH 0/2] net: lpc32xx: add support to RMII phy Albert ARIBAUD
2015-07-06  7:55   ` Vladimir Zapolskiy [this message]
2015-08-05 22:16     ` Vladimir Zapolskiy

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=559A346D.80907@mleia.com \
    --to=vz@mleia.com \
    --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