Hi Alison,
On Tuesday, May 21, 2013 11:02:58 AM, Alison Wang wrote:
> This patch adds FEC support for Vybrid MVF600 platform.
>
> In function fec_open(), RCR register is only set as RGMII mode. But RCR
> register should be set as RMII mode for MVF600 platform.
> This configuration is already done in fec_reg_setup(), so this piece of
> code could just leave untouched the FEC_RCNTRL_RGMII / FEC_RCNTRL_RMII /
> FEC_RCNTRL_MII_MODE bits.
>
> Signed-off-by: Alison Wang <b18...@freescale.com>
> ---
> Changes in v3:
> - Remove the changes for FEC_RCNTRL_RGMII / FEC_RCNTRL_RMII /
> FEC_RCNTRL_MII_MODE bits, as they are already set in fec_reg_setup()
>
> Changes in v2:
> - Use common FEC driver fec_mxc.c
>
> drivers/net/fec_mxc.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c
> index 4dbcdca..da95e28 100644
> --- a/drivers/net/fec_mxc.c
> +++ b/drivers/net/fec_mxc.c
> @@ -516,9 +516,7 @@ static int fec_open(struct eth_device *edev)
> #ifdef FEC_QUIRK_ENET_MAC
> {
> u32 ecr = readl(&fec->eth->ecntrl) & ~FEC_ECNTRL_SPEED;
> - u32 rcr = (readl(&fec->eth->r_cntrl) &
> - ~(FEC_RCNTRL_RMII | FEC_RCNTRL_RMII_10T)) |
> - FEC_RCNTRL_RGMII | FEC_RCNTRL_MII_MODE;
> + u32 rcr = readl(&fec->eth->r_cntrl) & ~FEC_RCNTRL_RMII_10T;
> if (speed == _1000BASET)
> ecr |= FEC_ECNTRL_SPEED;
> else if (speed != _100BASET)
Reviewed-by: Benoît Thébaudeau <benoit.thebaud...@advansee.com>
Best regards,
Benoît
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
