[U-Boot] [PATCH v2] ftgmac100: support of gigabit eth ftgmac100

Wolfgang Denk wd at denx.de
Sun Dec 19 00:07:07 CET 2010


Dear Macpaul Lin,

In message <1292383883-24272-1-git-send-email-macpaul at andestech.com> you wrote:
> Add Faraday's ftgmac100 (gigabit ethernet)
> MAC controller's driver.
> 
> Signed-off-by: Macpaul Lin <macpaul at andestech.com>
...
> --- /dev/null
> +++ b/drivers/net/ftgmac100.c
...
> +};
> +
> +

Only one blank line, please.

> +/******************************************************************************
> + * struct mii_bus functions
> + *****************************************************************************/

Incorrect multi-line comment style.  Please fix globally.

> +int ftgmac100_phy_read(struct eth_device *dev, int addr,
> +		int reg, u16 *value)
> +{
> +	*value = ftgmac100_mdiobus_read(dev , addr, reg);
> +
> +	if (*value == -1)
> +		return -1;
> +	else
> +		return 0;
> +}

Drop the else and unindent the return 0;

> +int  ftgmac100_phy_write(struct eth_device *dev, int addr,
> +		int reg, u16 value)
> +{
> +	if (ftgmac100_mdiobus_write(dev, addr, reg, value) == -1)
> +		return -1;
> +	else
> +		return 0;
> +}

Ditto.  Please fix globally.

> +	ftgmac100_phy_write(dev, priv->phy_addr,
> +		MII_ADVERTISE, adv);

Make this a single line, please. [Same globally where applicable]

> +	if (!(status & BMSR_LSTATUS)) {
> +		printf("%s: link down\n", dev->name);
> +		return 0;
> +	} else {

Remove else and unindent the rest.


> +#ifdef CONFIG_FTGMAC100_EGIGA
> +	if (stat_ge & LPA_1000FULL)
> +		/* set gmac for 1000BaseTX and Full Duplex */
> +		maccr |= FTGMAC100_MACCR_GIGA_MODE | FTGMAC100_MACCR_FULLDUP;
> +
> +	if (stat_ge & LPA_1000HALF)
> +		/* set gmac for 1000BaseTX and Half Duplex */
> +		maccr |= FTGMAC100_MACCR_GIGA_MODE;

Braces needed for multi-line statements.  Please fix globally.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I had the rare misfortune of being one of the first people to try and
implement a PL/1 compiler.                             -- T. Cheatham


More information about the U-Boot mailing list