[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 12 Dec 2011 16:42:07 +0200
From: Veli-Pekka Peltola <veli-pekka.peltola@...egiga.com>
To: Richard Zhao <richard.zhao@...escale.com>
CC: Fabio Estevam <festevam@...il.com>, patches@...aro.org,
netdev@...r.kernel.org, jgq516@...il.com, eric.miao@...aro.org,
davem@...emloft.net, shawn.guo@...aro.org,
Richard Zhao <richard.zhao@...aro.org>,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 1/1] net/fec: add KSZ9021RN phy fixup
On 12/12/2011 03:51 PM, Richard Zhao wrote:
> On Mon, Dec 12, 2011 at 10:56:48AM -0200, Fabio Estevam wrote:
>> On Mon, Dec 12, 2011 at 8:09 AM, Richard Zhao<richard.zhao@...aro.org> wrote:
>>> For imx6q sabrelite board, set phy RGMII pad skew.
>>>
>>> Signed-off-by: Richard Zhao<richard.zhao@...aro.org>
>>> ---
>>> drivers/net/ethernet/freescale/fec.c | 19 +++++++++++++++++++
>>> 1 files changed, 19 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/drivers/net/ethernet/freescale/fec.c b/drivers/net/ethernet/freescale/fec.c
>>> index 112af9b..d3b4463 100644
>>> --- a/drivers/net/ethernet/freescale/fec.c
>>> +++ b/drivers/net/ethernet/freescale/fec.c
>>> @@ -1513,6 +1513,21 @@ static inline void fec_reset_phy(struct platform_device *pdev)
>>> }
>>> #endif /* CONFIG_OF */
>>>
>>> +/* For imx6q sabrelite board: set KSZ9021RN RGMII pad skew */
>>> +static int ksz9021rn_phy_fixup(struct phy_device *phydev)
>>> +{
>>> + /* min rx data delay */
>>> + phy_write(phydev, 0x0b, 0x8105);
>>> + phy_write(phydev, 0x0c, 0x0000);
>>> +
>>> + /* max rx/tx clock delay, min rx/tx control delay */
>>> + phy_write(phydev, 0x0b, 0x8104);
>>> + phy_write(phydev, 0x0c, 0xf0f0);
>>> + phy_write(phydev, 0x0b, 0x104);
>>> +
>>> + return 0;
>>
>> This should go to drivers/net/phy/micrel.c.
> Why? It's specific to fec.
For me it seems to be a board specific and should be placed to the
machine file. Please include linux/micrel_phy.h and use proper defines
(PHY_ID_KSZ9021, MICREL_PHY_ID_MASK).
--
Veli-Pekka Peltola
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists