From: Wei Xu <xuwei5@hisilicon•com>
To: Jiri Pirko <jiri@resnulli•us>, kbuild test robot <lkp@intel•com>,
huangdaode <huangdaode@hisilicon•com>,
"Liguozhu (Kenneth)" <liguozhu@hisilicon•com>,
yankejian 00180145 <y00180145@notesmail•huawei.com.cn>,
Salil Mehta <salil.mehta@huawei•com>, <yisenzhuang@huawei•com>
Cc: Vivien Didelot <vivien.didelot@savoirfairelinux•com>,
<kbuild-all@01•org>, <netdev@vger•kernel.org>,
<linux-kernel@vger•kernel.org>, <kernel@savoirfairelinux•com>,
"David S. Miller" <davem@davemloft•net>,
Scott Feldman <sfeldma@gmail•com>,
Florian Fainelli <f.fainelli@gmail•com>,
Andrew Lunn <andrew@lunn•ch>,
Neil Armstrong <narmstrong@baylibre•com>,
Sergei Shtylyov <sergei.shtylyov@cogentembedded•com>,
Linuxarm <linuxarm@huawei•com>
Subject: Re: [PATCH net-next 3/6] net: remove dsa.h include from linux/netdevice.h
Date: Thu, 8 Oct 2015 11:37:22 +0100 [thread overview]
Message-ID: <56164762.1000302@hisilicon.com> (raw)
In-Reply-To: <20151008091825.GG2186@nanopsycho.orion>
On 10/8/2015 10:18 AM, Jiri Pirko wrote:
> Thu, Oct 08, 2015 at 11:04:48AM CEST, lkp@intel•com wrote:
>> Hi Vivien,
>>
>> [auto build test ERROR on net-next/master -- if it's inappropriate base, please ignore]
>>
>> config: arm64-allyesconfig (attached as .config)
>> 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=arm64
>>
>> All errors (new ones prefixed by >>):
>>
>> In file included from drivers/net/ethernet/hisilicon/hns/hnae.c:15:0:
>>>> drivers/net/ethernet/hisilicon/hns/hnae.h:465:2: error: unknown type name 'phy_interface_t'
>> phy_interface_t phy_if;
>> ^
>>
>> vim +/phy_interface_t +465 drivers/net/ethernet/hisilicon/hns/hnae.h
>
>
Hi Jiri,
> Looks like hnae.c needs to do "#include <linux/phy.h>" directly.
> Cc'ing maintainer.
>
Thanks!
We will send the fix patch soon.
Best Regards,
Wei
>
>>
>> 6fe6611f huangdaode 2015-09-17 449 struct hnae_ae_dev {
>> 6fe6611f huangdaode 2015-09-17 450 struct device cls_dev; /* the class dev */
>> 6fe6611f huangdaode 2015-09-17 451 struct device *dev; /* the presented dev */
>> 6fe6611f huangdaode 2015-09-17 452 struct hnae_ae_ops *ops;
>> 6fe6611f huangdaode 2015-09-17 453 struct list_head node;
>> 6fe6611f huangdaode 2015-09-17 454 struct module *owner; /* the module who provides this dev */
>> 6fe6611f huangdaode 2015-09-17 455 int id;
>> 6fe6611f huangdaode 2015-09-17 456 char name[AE_NAME_SIZE];
>> 6fe6611f huangdaode 2015-09-17 457 struct list_head handle_list;
>> 6fe6611f huangdaode 2015-09-17 458 spinlock_t lock; /* lock to protect the handle_list */
>> 6fe6611f huangdaode 2015-09-17 459 };
>> 6fe6611f huangdaode 2015-09-17 460
>> 6fe6611f huangdaode 2015-09-17 461 struct hnae_handle {
>> 6fe6611f huangdaode 2015-09-17 462 struct device *owner_dev; /* the device which make use of this handle */
>> 6fe6611f huangdaode 2015-09-17 463 struct hnae_ae_dev *dev; /* the device who provides this handle */
>> 6fe6611f huangdaode 2015-09-17 464 struct device_node *phy_node;
>> 6fe6611f huangdaode 2015-09-17 @465 phy_interface_t phy_if;
>> 6fe6611f huangdaode 2015-09-17 466 u32 if_support;
>> 6fe6611f huangdaode 2015-09-17 467 int q_num;
>> 6fe6611f huangdaode 2015-09-17 468 int vf_id;
>> 6fe6611f huangdaode 2015-09-17 469 u32 eport_id;
>> 6fe6611f huangdaode 2015-09-17 470 enum hnae_port_type port_type;
>> 6fe6611f huangdaode 2015-09-17 471 struct list_head node; /* list to hnae_ae_dev->handle_list */
>> 6fe6611f huangdaode 2015-09-17 472 struct hnae_buf_ops *bops; /* operation for the buffer */
>> 6fe6611f huangdaode 2015-09-17 473 struct hnae_queue **qs; /* array base of all queues */
>>
>> :::::: The code at line 465 was first introduced by commit
>> :::::: 6fe6611ff275522a4e4c0359e2f46cdd07780d2f net: add Hisilicon Network Subsystem hnae framework support
>>
>> :::::: TO: huangdaode <huangdaode@hisilicon•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
>
>
>
> .
>
next prev parent reply other threads:[~2015-10-08 10:37 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-07 23:48 [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops Vivien Didelot
2015-10-07 23:48 ` [PATCH net-next 1/6] net: dsa: add uses_hw_tag Vivien Didelot
2015-10-07 23:48 ` [PATCH net-next 2/6] net: dsa: include dsa.h in dsa_priv.h Vivien Didelot
2015-10-07 23:48 ` [PATCH net-next 3/6] net: remove dsa.h include from linux/netdevice.h Vivien Didelot
2015-10-08 9:04 ` kbuild test robot
2015-10-08 9:18 ` Jiri Pirko
2015-10-08 10:37 ` Wei Xu [this message]
[not found] ` <56165D5F.7050300@hisilicon.com>
2015-10-08 12:18 ` Jiri Pirko
2015-10-08 12:36 ` huangdaode
2015-10-10 23:35 ` kbuild test robot
2015-10-07 23:48 ` [PATCH net-next 4/6] net: dsa: add port_fdb_prepare Vivien Didelot
2015-10-08 0:25 ` Andrew Lunn
2015-10-08 12:55 ` Vivien Didelot
2015-10-08 15:07 ` Andrew Lunn
2015-10-08 6:19 ` Scott Feldman
2015-10-07 23:48 ` [PATCH net-next 5/6] net: dsa: push prepare phase in port_fdb_add Vivien Didelot
2015-10-08 6:19 ` Scott Feldman
2015-10-07 23:48 ` [PATCH net-next 6/6] net: dsa: use switchdev obj in port_fdb_del Vivien Didelot
2015-10-08 6:20 ` Scott Feldman
2015-10-08 12:28 ` [PATCH net-next 0/6] net: dsa: push switchdev prepare phase in FDB ops David Miller
2015-10-08 13:32 ` Vivien Didelot
2015-10-08 13:47 ` Jiri Pirko
2015-10-08 14:11 ` Vivien Didelot
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=56164762.1000302@hisilicon.com \
--to=xuwei5@hisilicon$(echo .)com \
--cc=andrew@lunn$(echo .)ch \
--cc=davem@davemloft$(echo .)net \
--cc=f.fainelli@gmail$(echo .)com \
--cc=huangdaode@hisilicon$(echo .)com \
--cc=jiri@resnulli$(echo .)us \
--cc=kbuild-all@01$(echo .)org \
--cc=kernel@savoirfairelinux$(echo .)com \
--cc=liguozhu@hisilicon$(echo .)com \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linuxarm@huawei$(echo .)com \
--cc=lkp@intel$(echo .)com \
--cc=narmstrong@baylibre$(echo .)com \
--cc=netdev@vger$(echo .)kernel.org \
--cc=salil.mehta@huawei$(echo .)com \
--cc=sergei.shtylyov@cogentembedded$(echo .)com \
--cc=sfeldma@gmail$(echo .)com \
--cc=vivien.didelot@savoirfairelinux$(echo .)com \
--cc=y00180145@notesmail$(echo .)huawei.com.cn \
--cc=yisenzhuang@huawei$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox