All of lore.kernel.org
 help / color / mirror / Atom feed
From: agust@denx.de (Anatolij Gustschin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: i.MX6: update KSZ9031 phy fixup
Date: Thu, 3 Apr 2014 16:56:57 +0200	[thread overview]
Message-ID: <20140403165657.510b6493@crub> (raw)
In-Reply-To: <1396533947.25686.22.camel@EMBLYSD005.adetelgroup.com>

On Thu, 03 Apr 2014 16:05:47 +0200
Hubert Chaumette <hchaumette@adeneo-embedded.com> wrote:
...
> > It would be better to configure the pad skews in the board specific
> > way in the device tree. There is a binding for ksz9021 PHY in
> > Documentation/devicetree/bindings/net/micrel-ksz9021.txt.
> 
> I wonder why it it's not used in arch/arm/boot/dts/imx6q-sabrelite.dts
> instead of ksz9021rn_phy_fixup().

Probably because nobody cared to use it.

> > I have a patch for setting ksz9031rn GTX/RX CLK pad skew in a similar
> > way over device tree and plan to submit it when the net-next merge
> > window for v3.16 opens.
> 
> So, you have already implemented the ksz9031 binding ?

Partially (for GTX/RX CLK). I'm attaching a patch that I currently have.
For ksz9031rn in my DTS fec node I used:

	/* ksz9031rn: min 0, max 1860, step 60 */
        rxc-skew-ps = <1620>;
        txc-skew-ps = <1620>;

Unfortunately I'm short on time and won't be able to implement other
pad skew settings in the near future, so please fill free to extend
and push the patch.

Thanks!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ARM-imx6-configure-ksz90x1rn-PHY-pad-skew-settings-o.patch
Type: text/x-patch
Size: 6002 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140403/f898a40d/attachment.bin>

  reply	other threads:[~2014-04-03 14:56 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-21 17:08 [PATCH] ARM: i.MX6: update KSZ9031 phy fixup Hubert Chaumette
2014-03-21 17:08 ` Hubert Chaumette
2014-03-21 21:23 ` Eric Bénard
2014-03-21 21:23   ` Eric Bénard
2014-03-24 15:30   ` CHAUMETTE Hubert
2014-03-24 15:30     ` CHAUMETTE Hubert
2014-03-24 15:41     ` Eric Bénard
2014-03-24 15:41       ` Eric Bénard
2014-04-02 13:14       ` Shawn Guo
2014-04-02 13:14         ` Shawn Guo
2014-04-02 13:38         ` Eric Bénard
2014-04-02 13:38           ` Eric Bénard
2014-04-02 14:14           ` Ben Dooks
2014-04-02 14:14             ` Ben Dooks
2014-04-02 20:01 ` Anatolij Gustschin
2014-04-03 14:05   ` Hubert Chaumette
2014-04-03 14:56     ` Anatolij Gustschin [this message]
2014-04-06 11:06     ` Shawn Guo
2014-04-06 11:26       ` Russell King - ARM Linux
2014-04-06 11:26         ` Russell King - ARM Linux
2014-04-06 12:04         ` Shawn Guo
2014-04-06 12:04           ` Shawn Guo

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=20140403165657.510b6493@crub \
    --to=agust@denx.de \
    --cc=linux-arm-kernel@lists.infradead.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 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.