All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Richard Cochran <richardcochran@gmail.com>
Cc: kbuild-all@01.org, netdev@vger.kernel.org,
	David Miller <davem@davemloft.net>,
	devicetree@vger.kernel.org, Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Jacob Keller <jacob.e.keller@intel.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Miroslav Lichvar <mlichvar@redhat.com>,
	Rob Herring <robh+dt@kernel.org>,
	Willem de Bruijn <willemb@google.com>
Subject: Re: [PATCH V4 net-next 2/6] net: Introduce a new MII time stamping interface.
Date: Sat, 1 Jun 2019 01:08:46 +0800	[thread overview]
Message-ID: <201906010107.Rldph9o4%lkp@intel.com> (raw)
In-Reply-To: <cc892ec1dfe16c3e5feb26e756cb20405a386948.1559109077.git.richardcochran@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2721 bytes --]

Hi Richard,

I love your patch! Yet something to improve:

[auto build test ERROR on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Richard-Cochran/Peer-to-Peer-One-Step-time-stamping/20190531-213601
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.4.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   drivers/net/ethernet/ti/netcp_ethss.c: In function 'gbe_txtstamp_mark_pkt':
>> drivers/net/ethernet/ti/netcp_ethss.c:2535:50: error: 'struct phy_driver' has no member named 'txtstamp'
    #define HAS_PHY_TXTSTAMP(p) ((p)->drv && (p)->drv->txtstamp)
                                                     ^
   drivers/net/ethernet/ti/netcp_ethss.c:2568:16: note: in expansion of macro 'HAS_PHY_TXTSTAMP'
     if (phydev && HAS_PHY_TXTSTAMP(phydev)) {
                   ^~~~~~~~~~~~~~~~
   drivers/net/ethernet/ti/netcp_ethss.c: In function 'gbe_rxtstamp':
>> drivers/net/ethernet/ti/netcp_ethss.c:2536:50: error: 'struct phy_driver' has no member named 'rxtstamp'
    #define HAS_PHY_RXTSTAMP(p) ((p)->drv && (p)->drv->rxtstamp)
                                                     ^
   drivers/net/ethernet/ti/netcp_ethss.c:2590:16: note: in expansion of macro 'HAS_PHY_RXTSTAMP'
     if (phydev && HAS_PHY_RXTSTAMP(phydev)) {
                   ^~~~~~~~~~~~~~~~
   drivers/net/ethernet/ti/netcp_ethss.c: In function 'gbe_ioctl':
>> drivers/net/ethernet/ti/netcp_ethss.c:2832:23: error: 'struct phy_driver' has no member named 'hwtstamp'
     if (!phy || !phy->drv->hwtstamp) {
                          ^~

vim +2535 drivers/net/ethernet/ti/netcp_ethss.c

6f8d3f33 Wingman Kwok 2015-01-15  2533  
6246168b WingMan Kwok 2016-12-08  2534  #if IS_ENABLED(CONFIG_TI_CPTS)
6246168b WingMan Kwok 2016-12-08 @2535  #define HAS_PHY_TXTSTAMP(p) ((p)->drv && (p)->drv->txtstamp)
6246168b WingMan Kwok 2016-12-08 @2536  #define HAS_PHY_RXTSTAMP(p) ((p)->drv && (p)->drv->rxtstamp)
6246168b WingMan Kwok 2016-12-08  2537  

:::::: The code at line 2535 was first introduced by commit
:::::: 6246168b4a38357b135d07370464f27d4f3a68ce net: ethernet: ti: netcp: add support of cpts

:::::: TO: WingMan Kwok <w-kwok2@ti.com>
:::::: CC: David S. Miller <davem@davemloft.net>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 70940 bytes --]

  parent reply	other threads:[~2019-05-31 17:08 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-29  5:58 [PATCH V4 net-next 0/6] Peer to Peer One-Step time stamping Richard Cochran
2019-05-29  5:58 ` [PATCH V4 net-next 1/6] net: Introduce peer to peer one step PTP " Richard Cochran
2019-05-29  5:58 ` [PATCH V4 net-next 2/6] net: Introduce a new MII time stamping interface Richard Cochran
2019-05-31 15:05   ` kbuild test robot
2019-05-31 17:08   ` kbuild test robot [this message]
2019-05-31 18:23   ` kbuild test robot
2019-05-29  5:58 ` [PATCH V4 net-next 3/6] net: Add a layer for non-PHY MII time stamping drivers Richard Cochran
2019-05-29  5:58 ` [PATCH V4 net-next 4/6] dt-bindings: ptp: Introduce MII time stamping devices Richard Cochran
2019-05-29  5:58 ` [PATCH V4 net-next 5/6] net: mdio: of: Register discovered MII time stampers Richard Cochran
2019-05-29  5:58 ` [PATCH V4 net-next 6/6] ptp: Add a driver for InES time stamping IP core Richard Cochran
2019-05-31 16:03   ` kbuild test robot
2019-05-30 18:55 ` [PATCH V4 net-next 0/6] Peer to Peer One-Step time stamping David Miller
2019-05-30 19:58   ` David Miller
2019-05-31  5:51     ` Richard Cochran
2019-05-31 17:57       ` David Miller

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=201906010107.Rldph9o4%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=f.fainelli@gmail.com \
    --cc=jacob.e.keller@intel.com \
    --cc=kbuild-all@01.org \
    --cc=mark.rutland@arm.com \
    --cc=mlichvar@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=richardcochran@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=willemb@google.com \
    /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.