Hello.
On 10/14/10 21:21, vm.rod25@xxxxxxxxx wrote:
> From: Victor Rodriguez<vm.rod25@xxxxxxxxx>
> This patch adds EMAC support for the Hawkboard-L138 system
> Signed-off-by: Victor Rodriguez<victor.rodriguez@xxxxxxxxxx>
> ---
> arch/arm/mach-davinci/board-omapl138-hawk.c | 49 +++++++++++++++++++++++++++
> 1 files changed, 49 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..3ae5178 100644
> --- a/arch/arm/mach-davinci/board-omapl138-hawk.c
> +++ b/arch/arm/mach-davinci/board-omapl138-hawk.c
> @@ -19,6 +19,53 @@
>
> #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 int __init 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 0;
> +
> + cfgchip3 = DA8XSYSCFG0_VIRT(DA8XX_CFGCHIP3_REG);
Could be initializer...
> + val = __raw_readl(cfgchip3);
> +
> + val &= ~BIT(8);
> + ret = davinci_cfg_reg_list(omapl138_hawk_mii_pins);
> + pr_info("EMAC: MII PHY configured\n");
I think this pr_info() should follow __raw_writel() call.
> + if (ret)
> + pr_warning("%s: "
> + "cpgmac/mii mux setup failed: %d\n", __func__, ret);
You should return here.
> +
> + /* 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 0;
Why this function returns anything at all if it'a always 0, and the result
is ignored?
> @@ -30,6 +77,8 @@ static __init void omapl138_hawk_init(void)
>
> davinci_serial_init(&omapl138_hawk_uart_config);
>
> + ret = omapl138_hawk_config_emac();
Why assign 'ret', if you're ignoring the result?
> +
> ret = da8xx_register_watchdog();
> if (ret)
> pr_warning("omapl138_hawk_init: "
WBR, Sergei
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
