From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 1/6] arm, am33xx: add defines for gmii_sel_register bits
Date: Mon, 19 Aug 2013 11:34:08 -0400 [thread overview]
Message-ID: <20130819153408.GE1311@bill-the-cat> (raw)
In-Reply-To: <1376923141-18002-2-git-send-email-hs@denx.de>
On Mon, Aug 19, 2013 at 04:38:56PM +0200, Heiko Schocher wrote:
> Signed-off-by: Heiko Schocher <hs@denx.de>
> Acked-by: Mugunthan V N <mugunthanvnm@ti.com>
Looks fine, but can we get this tested on the isee board too? It's a
functional change there (since it was setting the NOTUSED bit that HW
folks say really should not be set and will be marked as reserved in the
next TRM respin). Thanks!
>
> ---
> - changes for v2:
> defined all bits used in the gmii_sel register as
> Tom Rini suggested
> - changes for v3:
> rebased against u-boot-ti commit bb2a5d8f87fffb4fadfb205837decbd1b3e75f88
> - changes for v4:
> - rebased against u-boot-ti commit 425faf74cd8189c87919f7e72a0101c684ee3b9f
> - add changes requested from Tom Rini:
> - use <space> after "#define" instead <tab>
> - rename struct name "reserved" to "resv1"
> - remove GMII1_SEL_NOTUSED and GMII2_SEL_NOTUSED defines, also
> the GMII2_SEL_NOTUSED usage on the igep0033 board
> - add "Acked-by: Mugunthan V N <mugunthanvnm@ti.com>"
> ---
> arch/arm/include/asm/arch-am33xx/cpu.h | 19 +++++++++++++++++++
> board/isee/igep0033/board.c | 6 ++----
> board/phytec/pcm051/board.c | 2 --
> board/ti/am335x/board.c | 6 +-----
> 4 Dateien ge??ndert, 22 Zeilen hinzugef??gt(+), 11 Zeilen entfernt(-)
>
> diff --git a/arch/arm/include/asm/arch-am33xx/cpu.h b/arch/arm/include/asm/arch-am33xx/cpu.h
> index 10b56e0..f77ac1e 100644
> --- a/arch/arm/include/asm/arch-am33xx/cpu.h
> +++ b/arch/arm/include/asm/arch-am33xx/cpu.h
> @@ -486,6 +486,25 @@ struct ctrl_dev {
> unsigned int resv4[4];
> unsigned int miisel; /* offset 0x50 */
> };
> +
> +/* gmii_sel register defines */
> +#define GMII1_SEL_MII 0x0
> +#define GMII1_SEL_RMII 0x1
> +#define GMII1_SEL_RGMII 0x2
> +#define GMII2_SEL_MII 0x0
> +#define GMII2_SEL_RMII 0x4
> +#define GMII2_SEL_RGMII 0x8
> +#define RGMII1_IDMODE BIT(4)
> +#define RGMII2_IDMODE BIT(5)
> +#define RMII1_IO_CLK_EN BIT(6)
> +#define RMII2_IO_CLK_EN BIT(7)
> +
> +#define MII_MODE_ENABLE (GMII1_SEL_MII | GMII2_SEL_MII)
> +#define RMII_MODE_ENABLE (GMII1_SEL_RMII | GMII2_SEL_RMII)
> +#define RGMII_MODE_ENABLE (GMII1_SEL_RGMII | GMII2_SEL_RGMII)
> +#define RGMII_INT_DELAY (RGMII1_IDMODE | RGMII2_IDMODE)
> +#define RMII_CHIPCKL_ENABLE (RMII1_IO_CLK_EN | RMII2_IO_CLK_EN)
> +
> #endif /* __ASSEMBLY__ */
> #endif /* __KERNEL_STRICT_NAMES */
>
> diff --git a/board/isee/igep0033/board.c b/board/isee/igep0033/board.c
> index a24c22b..9e91f68 100644
> --- a/board/isee/igep0033/board.c
> +++ b/board/isee/igep0033/board.c
> @@ -27,9 +27,6 @@
>
> DECLARE_GLOBAL_DATA_PTR;
>
> -/* MII mode defines */
> -#define RMII_MODE_ENABLE 0x4D
> -
> static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;
>
> #ifdef CONFIG_SPL_BUILD
> @@ -158,7 +155,8 @@ int board_eth_init(bd_t *bis)
> eth_setenv_enetaddr("ethaddr", mac_addr);
> }
>
> - writel(RMII_MODE_ENABLE, &cdev->miisel);
> + writel((GMII1_SEL_RMII | RMII1_IO_CLK_EN),
> + &cdev->miisel);
>
> rv = cpsw_register(&cpsw_data);
> if (rv < 0)
> diff --git a/board/phytec/pcm051/board.c b/board/phytec/pcm051/board.c
> index f53c5bb..e40b0bd 100644
> --- a/board/phytec/pcm051/board.c
> +++ b/board/phytec/pcm051/board.c
> @@ -31,8 +31,6 @@
> DECLARE_GLOBAL_DATA_PTR;
>
> /* MII mode defines */
> -#define MII_MODE_ENABLE 0x0
> -#define RGMII_MODE_ENABLE 0xA
> #define RMII_RGMII2_MODE_ENABLE 0x49
>
> static struct ctrl_dev *cdev = (struct ctrl_dev *)CTRL_DEVICE_BASE;
> diff --git a/board/ti/am335x/board.c b/board/ti/am335x/board.c
> index 04c37e2..cc04426 100644
> --- a/board/ti/am335x/board.c
> +++ b/board/ti/am335x/board.c
> @@ -30,10 +30,6 @@
>
> DECLARE_GLOBAL_DATA_PTR;
>
> -/* MII mode defines */
> -#define MII_MODE_ENABLE 0x0
> -#define RGMII_MODE_ENABLE 0x3A
> -
> /* GPIO that controls power to DDR on EVM-SK */
> #define GPIO_DDR_VTT_EN 7
>
> @@ -460,7 +456,7 @@ int board_eth_init(bd_t *bis)
> cpsw_slaves[0].phy_if = cpsw_slaves[1].phy_if =
> PHY_INTERFACE_MODE_MII;
> } else {
> - writel(RGMII_MODE_ENABLE, &cdev->miisel);
> + writel((RGMII_MODE_ENABLE | RGMII_INT_DELAY), &cdev->miisel);
> cpsw_slaves[0].phy_if = cpsw_slaves[1].phy_if =
> PHY_INTERFACE_MODE_RGMII;
> }
> --
> 1.7.11.7
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130819/ed92ac39/attachment.pgp>
next prev parent reply other threads:[~2013-08-19 15:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-19 14:38 [U-Boot] [PATCH v4 0/6] arm, am335x: add support for siemens boards Heiko Schocher
2013-08-19 14:38 ` [U-Boot] [PATCH v4 1/6] arm, am33xx: add defines for gmii_sel_register bits Heiko Schocher
2013-08-19 15:34 ` Tom Rini [this message]
2013-08-19 14:38 ` [U-Boot] [PATCH v4 2/6] arm, am335x: add some missing registers and defines for lcd and epwm support Heiko Schocher
2013-08-19 14:38 ` [U-Boot] [PATCH v4 3/6] arm, spl: add watchdog library to SPL Heiko Schocher
2013-08-19 14:38 ` [U-Boot] [PATCH v4 4/6] arm, am335x: add watchdog support Heiko Schocher
2013-08-19 14:39 ` [U-Boot] [PATCH v4 5/6] video: add formike lcd panel init Heiko Schocher
2013-08-19 17:54 ` Anatolij Gustschin
2013-08-19 14:39 ` [U-Boot] [PATCH v4 6/6] arm, am335x: add support for 3 siemens boards Heiko Schocher
2013-08-19 15:35 ` [U-Boot] [PATCH v4 0/6] arm, am335x: add support for " Tom Rini
2013-08-19 17:58 ` Anatolij Gustschin
2013-08-19 18:05 ` Tom Rini
2013-08-28 16:01 ` [U-Boot] [PATCH] siemens-am33x-common.h: Always build CONFIG_OMAP_GPIO support Tom Rini
2013-08-28 18:25 ` [U-Boot] [PATCH v4 0/6] arm, am335x: add support for siemens boards Tom Rini
-- strict thread matches above, loose matches on Subject: below --
2013-08-28 9:00 [U-Boot] [PATCH v4 1/6] arm, am33xx: add defines for gmii_sel_register bits Enric Balletbo Serra
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=20130819153408.GE1311@bill-the-cat \
--to=trini@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox