All of lore.kernel.org
 help / color / mirror / Atom feed
From: Troy Kisky <troy.kisky@boundarydevices.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/5] i.mx6q: mx6qsabrelite: Update the network configuration
Date: Mon, 16 Jan 2012 12:51:15 -0700	[thread overview]
Message-ID: <4F147FB3.3050003@boundarydevices.com> (raw)
In-Reply-To: <CAFQ4atRcWZ1boVOhtY=Ge+5c6komVomOKeH5GBUZyoU2MRe-ow@mail.gmail.com>

On 1/16/2012 6:48 AM, Jason Hui wrote:
> On Fri, Jan 13, 2012 at 8:10 PM, Dirk Behme<dirk.behme@de.bosch.com>  wrote:
>> From: Troy Kisky<troy.kisky@boundarydevices.com>
>>
>> Enable the usage of PHY_MICREL_KSZ9021, force the master mode and
>> minimize the tx clock delay.
>>
>> There is an issue with the gigabit ethernet mode at the SabreLite boards:
>> When operating on a Gb LAN, the FEC occasionally receives packets larger
>> than the MTU that appear to be caused by concatenation of multiple
>> received packets into one.
No, the extra length are usually zero bytes.

>> The problem was identified on the phy side. The current schematic has the
>> center tap pin 9 of the MAG/JAC USB combo to 3.3 filtered supply. Letting
>> this pin float solves the problem.
>>
>> Signed-off-by: Troy Kisky<troy.kisky@boundarydevices.com>
>> CC: Troy Kisky<troy.kisky@boundarydevices.com>
>> CC: Stefano Babic<sbabic@denx.de>
>> CC: Jason Liu<jason.hui@linaro.org>
Dirk should either signoff or ack
>> ---
>> Note: This patch depends on the basic SabreLite ethernet patch
>>       http://patchwork.ozlabs.org/patch/135744/
>>
>>   board/freescale/mx6qsabrelite/mx6qsabrelite.c |    8 ++++++--
>>   include/configs/mx6qsabrelite.h               |    1 +
>>   2 files changed, 7 insertions(+), 2 deletions(-)
>>
>> diff --git a/board/freescale/mx6qsabrelite/mx6qsabrelite.c b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
>> index d80165e..7879049 100644
>> --- a/board/freescale/mx6qsabrelite/mx6qsabrelite.c
>> +++ b/board/freescale/mx6qsabrelite/mx6qsabrelite.c
>> @@ -193,13 +193,17 @@ int board_mmc_init(bd_t *bis)
>>
>>   int fecmxc_mii_postcall(int phy)
>>   {
>> -       /* prefer master mode */
>> -       miiphy_write("FEC", phy, MII_1000BASET_CTRL, 0x0f00);
>> +       /* force master mode */
>> +       miiphy_write("FEC", phy, MII_1000BASET_CTRL, 0x1f00);
>>
>>         /* min rx data delay */
>>         miiphy_write("FEC", phy, MII_EXTENDED_CTRL, 0x8105);
>>         miiphy_write("FEC", phy, MII_EXTENDED_DATAW, 0x0000);
>>
>> +       /* min tx data delay */
>> +       miiphy_write("FEC", phy, MII_EXTENDED_CTRL, 0x8106);
>> +       miiphy_write("FEC", phy, MII_EXTENDED_DATAW, 0x0000);
>> +
>>         /* max rx/tx clock delay, min rx/tx control delay */
>>         miiphy_write("FEC", phy, MII_EXTENDED_CTRL, 0x8104);
>>         miiphy_write("FEC", phy, MII_EXTENDED_DATAW, 0xf0f0);
>> diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h
>> index 840e271..6a29142 100644
>> --- a/include/configs/mx6qsabrelite.h
>> +++ b/include/configs/mx6qsabrelite.h
>> @@ -66,6 +66,7 @@
>>   #define        CONFIG_FEC_XCV_TYPE             RGMII
>>   #define CONFIG_ETHPRIME                        "FEC"
>>   #define CONFIG_FEC_MXC_PHYADDR         6
>> +#define CONFIG_PHY_MICREL_KSZ9021
> Acked-by: Jason Liu<jason.hui@linaro.org>
>
>>   /* allow to overwrite serial and ethaddr */
>>   #define CONFIG_ENV_OVERWRITE
>> --
>> 1.7.0.4
>>

  reply	other threads:[~2012-01-16 19:51 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-13 12:10 [U-Boot] [PATCH 1/5] net: fec_mxc: Add 1000 Mbps selection Dirk Behme
2012-01-13 12:10 ` [U-Boot] [PATCH 2/5] net: fec_mxc: Increase autonegotiation timeout Dirk Behme
2012-01-13 12:10 ` [U-Boot] [PATCH 3/5] miiphyutil: Add Micrel KSZ9021 support to miiphy_speed Dirk Behme
2012-01-13 12:44   ` Marek Vasut
2012-01-15  0:56   ` Mike Frysinger
2012-01-19 12:38     ` Dirk Behme
2012-01-19 16:19       ` Dirk Behme
2012-01-19 18:24         ` Mike Frysinger
2012-01-19 18:24       ` Mike Frysinger
2012-01-19 12:27   ` Dirk Behme
2012-01-13 12:10 ` [U-Boot] [PATCH 4/5] net: fec_mxc: Nove autonegoatiate restart after mii_postcall Dirk Behme
2012-01-13 12:44   ` Marek Vasut
2012-01-13 21:42     ` Troy Kisky
2012-01-13 13:54   ` Stefano Babic
2012-01-13 21:22     ` Troy Kisky
2012-01-13 12:10 ` [U-Boot] [PATCH 5/5] i.mx6q: mx6qsabrelite: Update the network configuration Dirk Behme
2012-01-16 13:48   ` Jason Hui
2012-01-16 19:51     ` Troy Kisky [this message]
2012-01-13 12:42 ` [U-Boot] [PATCH 1/5] net: fec_mxc: Add 1000 Mbps selection Marek Vasut

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=4F147FB3.3050003@boundarydevices.com \
    --to=troy.kisky@boundarydevices.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.