From: Mike Frysinger <vapier@gentoo.org>
Date: Fri, 3 Feb 2012 06:21:18 -0500
Subject: [U-Boot] [PATCH] net: sh_eth: Fix build warnings and whitespace
	for sh_eth
In-Reply-To: <1328254249-2121-2-git-send-email-nobuhiro.iwamatsu.yj@renesas.com>
References: <1328254249-2121-1-git-send-email-nobuhiro.iwamatsu.yj@renesas.com>
	<1328254249-2121-2-git-send-email-nobuhiro.iwamatsu.yj@renesas.com>
Message-ID: <201202030621.19676.vapier@gentoo.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Friday 03 February 2012 02:30:48 Nobuhiro Iwamatsu wrote:
> @@ -564,8 +569,10 @@ static int sh_eth_bb_init(struct bb_miiphy_bus *bus)
> 
>  static int sh_eth_bb_mdio_active(struct bb_miiphy_bus *bus)
>  {
> +	int port;
>  	struct sh_eth_dev *eth = bus->priv;
> -	int port = eth->port;
> +
> +	port = eth->port;
> 
>  	outl(inl(PIR(port)) | PIR_MMD, PIR(port));
> 
> @@ -574,8 +581,10 @@ static int sh_eth_bb_mdio_active(struct bb_miiphy_bus
> *bus)
> 
>  static int sh_eth_bb_mdio_tristate(struct bb_miiphy_bus *bus)
>  {
> +	int port;
>  	struct sh_eth_dev *eth = bus->priv;
> -	int port = eth->port;
> +
> +	port = eth->port;
> 
>  	outl(inl(PIR(port)) & ~PIR_MMD, PIR(port));
> 
> @@ -584,8 +593,10 @@ static int sh_eth_bb_mdio_tristate(struct
> bb_miiphy_bus *bus)
> 
>  static int sh_eth_bb_set_mdio(struct bb_miiphy_bus *bus, int v)
>  {
> +	int port;
>  	struct sh_eth_dev *eth = bus->priv;
> -	int port = eth->port;
> +
> +	port = eth->port;
> 
>  	if (v)
>  		outl(inl(PIR(port)) | PIR_MDO, PIR(port));
> @@ -597,8 +608,10 @@ static int sh_eth_bb_set_mdio(struct bb_miiphy_bus
> *bus, int v)
> 
>  static int sh_eth_bb_get_mdio(struct bb_miiphy_bus *bus, int *v)
>  {
> +	int port;
>  	struct sh_eth_dev *eth = bus->priv;
> -	int port = eth->port;
> +
> +	port = eth->port;
> 
>  	*v = (inl(PIR(port)) & PIR_MDI) >> 3;
> 
> @@ -607,8 +620,10 @@ static int sh_eth_bb_get_mdio(struct bb_miiphy_bus
> *bus, int *v)
> 
>  static int sh_eth_bb_set_mdc(struct bb_miiphy_bus *bus, int v)
>  {
> +	int port;
>  	struct sh_eth_dev *eth = bus->priv;
> -	int port = eth->port;
> +
> +	port = eth->port;
> 
>  	if (v)
>  		outl(inl(PIR(port)) | PIR_MDC, PIR(port));

i don't see why any of these changes are necessary ... the old code looks fine 
to me (from a warning and style pov)
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120203/9e3c7869/attachment.pgp>