All of lore.kernel.org
 help / color / mirror / Atom feed
From: Giuseppe Pagano <giuseppe.pagano@seco.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 1/4] udoo: Add ethernet support (FEC + Micrel KSZ9031).
Date: Wed, 20 Nov 2013 08:55:01 +0100	[thread overview]
Message-ID: <1384934101.3300.20.camel@localhost> (raw)
In-Reply-To: <1384794334.4799.53.camel@localhost>

On Mon, 2013-11-18 at 18:05 +0100, Giuseppe Pagano wrote:
> 
> > 
> > > +	ksz9031_phy_extended_write(phydev, 0x02,
> > > +				   MII_KSZ9031_EXT_RGMII_CLOCK_SKEW,
> > > +				   MII_KSZ9031_MOD_DATA_NO_POST_INC, 0x03FF);
> > 
> I have september 2012 revision of KSZ9031's manual, and I think it is
> complete enough about skew parameter (preliminary version was not). 
> Default value for GTX_CLK and RX_CLK swek register is 0x001ef (0 ns
> delay for both). I use maximum positive delay (value 0x03ff) as safe
> value, but it doesn't work on every board.

To be more clear (for people who do not have KSZ9031 manual). 
GTX and RX skew delay have 5 bit resolution, so
MII_KSZ9031_EXT_RGMII_CLOCK_SKEW register have this default value bit:

reserved   GTX skew   RX skew
 0000_00----01_111-----0_1111   (0x01EF -> 0ns delay)
 0000_00----00_000-----0_0000   (0x0000 -> -0.96ns delay both)
 0000_00----11_111-----1_1111   (0x03FF -> +0.96ns delay both)
 0000_00----11_111-----0_1111   (0x03EF -> +0.96ns delay only for GTX)
etc.


> 
> Best regards
> Giuseppe Pagano

  reply	other threads:[~2013-11-20  7:55 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-15 16:42 [U-Boot] [PATCH v2 0/4] udoo: Move and optimize platform register setting Giuseppe Pagano
2013-11-15 16:42 ` [U-Boot] [PATCH v2 1/4] udoo: Add ethernet support (FEC + Micrel KSZ9031) Giuseppe Pagano
2013-11-18 15:51   ` Stefano Babic
2013-11-18 17:05     ` Giuseppe Pagano
2013-11-20  7:55       ` Giuseppe Pagano [this message]
2013-11-19  4:34     ` Fabio Estevam
2013-11-25 15:06     ` Fabio Estevam
2013-11-28  8:26   ` Stefano Babic
2013-11-28  9:23     ` Giuseppe Pagano
2013-11-15 16:42 ` [U-Boot] [PATCH v2 2/4] nitrogen6x: Move setup_sata to common part Giuseppe Pagano
2013-11-28  8:07   ` Stefano Babic
2013-11-15 16:42 ` [U-Boot] [PATCH v2 3/4] udoo: Add SATA support on uDoo Board Giuseppe Pagano
2013-11-15 16:42 ` [U-Boot] [PATCH v2 4/4] udoo: Fix watchdog during kernel boot Giuseppe Pagano
2013-11-28  8:26   ` Stefano Babic
2013-11-28  8:23 ` [U-Boot] [PATCH v2 0/4] udoo: Move and optimize platform register setting Stefano Babic

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=1384934101.3300.20.camel@localhost \
    --to=giuseppe.pagano@seco.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.