Hello.

On 10/15/10 07:08, [email protected] wrote:

From: Victor Rodriguez<[email protected]>

This patch adds EMAC support for the Hawkboard-L138 system

Signed-off-by: Victor Rodriguez<[email protected]>
---
  arch/arm/mach-davinci/board-omapl138-hawk.c |   50 +++++++++++++++++++++++++++
  1 files changed, 50 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-davinci/board-omapl138-hawk.c 
b/arch/arm/mach-davinci/board-omapl138-hawk.c
index c472dd8..2220105 100644
--- a/arch/arm/mach-davinci/board-omapl138-hawk.c
+++ b/arch/arm/mach-davinci/board-omapl138-hawk.c
@@ -19,6 +19,54 @@

  #include<mach/cp_intc.h>
  #include<mach/da8xx.h>
+#include<mach/mux.h>
+
+#define HAWKBOARD_PHY_ID               "0:07"
+
+static short omapl138_hawk_mii_pins[] __initdata = {
+       DA850_MII_TXEN, DA850_MII_TXCLK, DA850_MII_COL, DA850_MII_TXD_3,
+       DA850_MII_TXD_2, DA850_MII_TXD_1, DA850_MII_TXD_0, DA850_MII_RXER,
+       DA850_MII_CRS, DA850_MII_RXCLK, DA850_MII_RXDV, DA850_MII_RXD_3,
+       DA850_MII_RXD_2, DA850_MII_RXD_1, DA850_MII_RXD_0, DA850_MDIO_CLK,
+       DA850_MDIO_D,
+       -1
+};
+
+static __init void omapl138_hawk_config_emac(void)
+{
+       void __iomem *cfgchip3;
+       int ret;
+       u32 val;
+       struct davinci_soc_info *soc_info =&davinci_soc_info;
+
+       if (!machine_is_omapl138_hawkboard())
+               return;
+
+       cfgchip3 = DA8XX_SYSCFG0_VIRT(DA8XX_CFGCHIP3_REG);
+
+       val = __raw_readl(cfgchip3);
+
+       val&= ~BIT(8);
+       ret = davinci_cfg_reg_list(omapl138_hawk_mii_pins);
+       if (ret) {
+               pr_warning("%s: "
+                       "cpgmac/mii mux setup failed: %d\n", __func__, ret);
+               return;
+       }
+
+       pr_info("EMAC: MII PHY configured\n");

   I said this should *follow* __raw_writel(), not precede it...

+       /* configure the CFGCHIP3 register for MII */
+       __raw_writel(val, cfgchip3);
+
+       soc_info->emac_pdata->phy_id = HAWKBOARD_PHY_ID;
+
+       ret = da8xx_register_emac();
+       if (ret)
+               pr_warning("%s: "
+                       "emac registration failed: %d\n", __func__, ret);
+               return;

   'return' not needed here. And you've forgotten to enclose it in {} anyway. 
:-)

WBR. Sergei
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to