From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH] sh_eth: coding style fixes Date: Fri, 03 Jan 2014 13:52:45 +0400 Message-ID: <52C6886D.2000801@cogentembedded.com> References: <201312310325.21951.sergei.shtylyov@cogentembedded.com> <20140102.193407.23280653408847634.davem@davemloft.net> <1388713347.2259.86.camel@joe-AO722> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: netdev@vger.kernel.org, linux-sh@vger.kernel.org To: Joe Perches , David Miller Return-path: In-Reply-To: <1388713347.2259.86.camel@joe-AO722> Sender: linux-sh-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hello. On 03-01-2014 5:42, Joe Perches wrote: >>> @@ -1952,8 +1949,8 @@ static void sh_eth_tx_timeout(struct net >>> netif_stop_queue(ndev); >>> >>> if (netif_msg_timer(mdp)) >>> - dev_err(&ndev->dev, "%s: transmit timed out, status %8.8x," >>> - " resetting...\n", ndev->name, (int)sh_eth_read(ndev, EESR= )); >>> + dev_err(&ndev->dev, "%s: transmit timed out, status %8.8x, reset= ting...\n" >>> + ndev->name, (int)sh_eth_read(ndev, EESR)); >>> >>> /* tx_errors count up */ >>> ndev->stats.tx_errors++; >> You did not build test this. >> drivers/net/ethernet/renesas/sh_eth.c: In function =E2=80=98sh_eth_t= x_timeout=E2=80=99: >> drivers/net/ethernet/renesas/sh_eth.c:1953:4: error: expected =E2=80= =98)=E2=80=99 before =E2=80=98ndev=E2=80=99 > Also, this would likely be better as: > > netif_err(mdp, timer, ndev, "transmit timed out, status %#08lx, rese= tting...\n", > sh_eth_read(ndev, EESR)); If checkpoint.pl did point this out, I would have fixed it. Since i= t=20 doesn't, it's probably material of another patch. WBR, Sergei