From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chin Liang See Date: Fri, 18 Dec 2015 15:48:32 +0800 Subject: [U-Boot] [PATCH] arm: socfpga: Fix emac1 doesn't work on socdk board In-Reply-To: References: Message-ID: <1450424912.2152.11.camel@altera.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Shengjiangm On Fri, 2015-12-18 at 07:39 +0000, ?? ? wrote: > Hi Chin, > > I will check it. > > Best Regards > ShengjiangWu > To ease reading, we will put comment below previous comments. In the mean time, opensource will always use plaintext email. FYI, I use evoluation to handle email communication instead of company default outlook. Or you can use gmail if you email software don't support plaintext. Thanks Chin Liang > On Dec 17, 2015, at 11:28 PM, Chin Liang See > wrote: > > > Hi Shengjiang, > > > > On Fri, 2015-12-18 at 15:13 +0800, shengjiangwu wrote: > > > Updated pinmux group MIXED1IO[0-13] for RGMII1. > > > Updated EMAC1 clock. > > > Signed-off-by: shengjiangwu > > > Cc: Chin Liang See > > > Cc: Dinh Nguyen > > > Cc: Dinh Nguyen > > > Cc: Pavel Machek > > > Cc: Marek Vasut > > > Cc: Stefan Roese > > Thanks for the patch. > > > > > --- > > > board/altera/cyclone5-socdk/qts/pinmux_config.h | 28 +++++++++++- > > > - > > > ---------- > > > board/altera/cyclone5-socdk/qts/pll_config.h | 4 ++-- > > > 2 files changed, 16 insertions(+), 16 deletions(-) > > > diff --git a/board/altera/cyclone5-socdk/qts/pinmux_config.h > > > b/board/altera/cyclone5-socdk/qts/pinmux_config.h > > > index 33cf1fd..442b1e0 100644 > > > --- a/board/altera/cyclone5-socdk/qts/pinmux_config.h > > > +++ b/board/altera/cyclone5-socdk/qts/pinmux_config.h > > > @@ -72,20 +72,20 @@ const u8 sys_mgr_init_table[] = { > > > 0, /* GENERALIO29 */ > > > 0, /* GENERALIO30 */ > > > 0, /* GENERALIO31 */ > > > - 0, /* MIXED1IO0 */ > > > - 1, /* MIXED1IO1 */ > > > - 1, /* MIXED1IO2 */ > > > - 1, /* MIXED1IO3 */ > > > - 1, /* MIXED1IO4 */ > > > - 0, /* MIXED1IO5 */ > > > - 0, /* MIXED1IO6 */ > > > - 0, /* MIXED1IO7 */ > > > - 1, /* MIXED1IO8 */ > > > - 1, /* MIXED1IO9 */ > > > - 1, /* MIXED1IO10 */ > > > - 1, /* MIXED1IO11 */ > > > - 0, /* MIXED1IO12 */ > > > - 0, /* MIXED1IO13 */ > > > + 2, /* MIXED1IO0 */ > > > + 2, /* MIXED1IO1 */ > > > + 2, /* MIXED1IO2 */ > > > + 2, /* MIXED1IO3 */ > > > + 2, /* MIXED1IO4 */ > > > + 2, /* MIXED1IO5 */ > > > + 2, /* MIXED1IO6 */ > > > + 2, /* MIXED1IO7 */ > > > + 2, /* MIXED1IO8 */ > > > + 2, /* MIXED1IO9 */ > > > + 2, /* MIXED1IO10 */ > > > + 2, /* MIXED1IO11 */ > > > + 2, /* MIXED1IO12 */ > > > + 2, /* MIXED1IO13 */ > > > 0, /* MIXED1IO14 */ > > > 1, /* MIXED1IO15 */ > > > 1, /* MIXED1IO16 */ > > > diff --git a/board/altera/cyclone5-socdk/qts/pll_config.h > > > b/board/altera/cyclone5-socdk/qts/pll_config.h > > > index 3d621ed..42905f4 100644 > > > --- a/board/altera/cyclone5-socdk/qts/pll_config.h > > > +++ b/board/altera/cyclone5-socdk/qts/pll_config.h > > > @@ -31,7 +31,7 @@ > > > #define CONFIG_HPS_PERPLLGRP_VCO_NUMER 79 > > > #define CONFIG_HPS_PERPLLGRP_VCO_PSRC 0 > > > #define CONFIG_HPS_PERPLLGRP_EMAC0CLK_CNT 3 > > > -#define CONFIG_HPS_PERPLLGRP_EMAC1CLK_CNT 511 > > > +#define CONFIG_HPS_PERPLLGRP_EMAC1CLK_CNT 3 > > > #define CONFIG_HPS_PERPLLGRP_PERQSPICLK_CNT 511 > > > #define CONFIG_HPS_PERPLLGRP_PERNANDSDMMCCLK_CNT 4 > > > #define CONFIG_HPS_PERPLLGRP_PERBASECLK_CNT 4 > > > @@ -65,7 +65,7 @@ > > > #define CONFIG_HPS_CLK_PERVCO_HZ 1000000000 > > > #define CONFIG_HPS_CLK_SDRVCO_HZ 666666666 > > > #define CONFIG_HPS_CLK_EMAC0_HZ 250000000 > > > -#define CONFIG_HPS_CLK_EMAC1_HZ 250000000 > > > +#define CONFIG_HPS_CLK_EMAC1_HZ 50000000 > > I believe the EMAC1 clock is still 250MHz which result of 25MHz * > > (79+1) / (1+1) / (3+1). > > > > Thanks > > Chin Liang > > > > > #define CONFIG_HPS_CLK_USBCLK_HZ 200000000 > > > #define CONFIG_HPS_CLK_NAND_HZ 50000000 > > > #define CONFIG_HPS_CLK_SDMMC_HZ 200000000