From: Florian Fainelli <f.fainelli@gmail.com>
To: kbuild test robot <lkp@intel.com>
Cc: kbuild-all@01.org, netdev@vger.kernel.org, davem@davemloft.net,
mw@semihalf.com, arnd@arndb.de,
gregory.clement@free-electrons.com, Shaohui.Xie@freescale.com
Subject: Re: [PATCH net-next v3 2/3] net: fsl: Allow most drivers to be built with COMPILE_TEST
Date: Wed, 16 Nov 2016 11:52:45 -0800 [thread overview]
Message-ID: <0216ebc6-1889-6ca7-974c-90524ed61651@gmail.com> (raw)
In-Reply-To: <201611161135.ksuIHp17%fengguang.wu@intel.com>
On 11/15/2016 07:23 PM, kbuild test robot wrote:
> Hi Florian,
>
> [auto build test WARNING on net-next/master]
>
> url: https://github.com/0day-ci/linux/commits/Florian-Fainelli/net-gianfar_ptp-Rename-FS-bit-to-FIPERST/20161116-095805
> config: sh-allmodconfig (attached as .config)
> compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> reproduce:
> wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # save the attached .config to linux build tree
> make.cross ARCH=sh
>
> All warnings (new ones prefixed by >>):
>
> drivers/net/ethernet/freescale/fsl_pq_mdio.c: In function 'fsl_pq_mdio_remove':
>>> drivers/net/ethernet/freescale/fsl_pq_mdio.c:498:27: warning: unused variable 'priv' [-Wunused-variable]
> struct fsl_pq_mdio_priv *priv = bus->priv;
Humm, this looks bogus, the variable is used see below:
> ^~~~
>
> vim +/priv +498 drivers/net/ethernet/freescale/fsl_pq_mdio.c
>
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 482 return 0;
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 483
> dd3b8a32 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 484 error:
> dd3b8a32 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 485 if (priv->map)
> b3319b10 drivers/net/fsl_pq_mdio.c Anton Vorontsov 2009-12-30 486 iounmap(priv->map);
> dd3b8a32 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 487
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 488 kfree(new_bus);
> dd3b8a32 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 489
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 490 return err;
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 491 }
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 492
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 493
> 5078ac79 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 494 static int fsl_pq_mdio_remove(struct platform_device *pdev)
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 495 {
> 5078ac79 drivers/net/ethernet/freescale/fsl_pq_mdio.c Timur Tabi 2012-08-29 496 struct device *device = &pdev->dev;
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 497 struct mii_bus *bus = dev_get_drvdata(device);
> b3319b10 drivers/net/fsl_pq_mdio.c Anton Vorontsov 2009-12-30 @498 struct fsl_pq_mdio_priv *priv = bus->priv;
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 499
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 500 mdiobus_unregister(bus);
> 1577ecef drivers/net/fsl_pq_mdio.c Andy Fleming 2009-02-04 501
> b3319b10 drivers/net/fsl_pq_mdio.c Anton Vorontsov 2009-12-30 502 iounmap(priv->map);
Right here.
What compiler version is this?
--
Florian
next prev parent reply other threads:[~2016-11-16 19:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-16 0:40 [PATCH net-next v3 0/3] net: ethernet: Allow Marvell & Freescale to COMPILE_TEST Florian Fainelli
2016-11-16 0:40 ` [PATCH net-next v2 1/3] net: fsl: Allow most drivers to be built with COMPILE_TEST Florian Fainelli
2016-11-16 0:40 ` [PATCH net-next v3 1/3] net: gianfar_ptp: Rename FS bit to FIPERST Florian Fainelli
2016-11-16 0:40 ` [PATCH net-next v3 2/3] net: fsl: Allow most drivers to be built with COMPILE_TEST Florian Fainelli
2016-11-16 3:23 ` kbuild test robot
2016-11-16 19:52 ` Florian Fainelli [this message]
2016-11-17 1:38 ` Fengguang Wu
2016-11-17 10:47 ` Arnd Bergmann
2016-11-17 16:27 ` David Miller
2016-11-16 0:40 ` [PATCH net-next v2 2/3] net: gianfar_ptp: Rename FS bit to FIPERST Florian Fainelli
2016-11-16 0:40 ` [PATCH net-next v3 3/3] net: marvell: Allow drivers to be built with COMPILE_TEST Florian Fainelli
2016-11-16 19:04 ` kbuild test robot
2016-11-16 20:06 ` Florian Fainelli
2016-11-16 19:02 ` [PATCH net-next v3 0/3] net: ethernet: Allow Marvell & Freescale to COMPILE_TEST David Miller
2016-11-16 19:07 ` Florian Fainelli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=0216ebc6-1889-6ca7-974c-90524ed61651@gmail.com \
--to=f.fainelli@gmail.com \
--cc=Shaohui.Xie@freescale.com \
--cc=arnd@arndb.de \
--cc=davem@davemloft.net \
--cc=gregory.clement@free-electrons.com \
--cc=kbuild-all@01.org \
--cc=lkp@intel.com \
--cc=mw@semihalf.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.