From: Yijing Wang <wangyijing@huawei.com>
To: "Maciej W. Rozycki" <macro@linux-mips.org>,
"David S. Miller" <davem@davemloft.net>
Cc: <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Hanjun Guo <guohanjun@huawei.com>
Subject: Re: [PATCH 2/2] net/fddi: Replace local marco with PCI standard macro
Date: Fri, 6 Dec 2013 11:06:28 +0800 [thread overview]
Message-ID: <52A13F34.2010403@huawei.com> (raw)
In-Reply-To: <alpine.LFD.2.10.1312052154190.1599@eddie.linux-mips.org>
On 2013/12/6 6:06, Maciej W. Rozycki wrote:
> On Thu, 5 Dec 2013, Yijing Wang wrote:
>
>> Replace local marco DFX_BUS_PCI with PCI standard marco
>> dev_is_pci().
>
> Typos above: marco -> macro
Sorry for the mistake, David, should i need to resend this patch to fix this typo error?
>
>> Signed-off-by: Yijing Wang <wangyijing@huawei.com>
>> ---
>> drivers/net/fddi/defxx.c | 20 +++++++-------------
>> 1 files changed, 7 insertions(+), 13 deletions(-)
>>
>> diff --git a/drivers/net/fddi/defxx.c b/drivers/net/fddi/defxx.c
>> index 0b40e1c..0344f71 100644
>> --- a/drivers/net/fddi/defxx.c
>> +++ b/drivers/net/fddi/defxx.c
>> @@ -241,12 +241,6 @@ static char version[] =
>> */
>> #define NEW_SKB_SIZE (PI_RCV_DATA_K_SIZE_MAX+128)
>>
>> -#ifdef CONFIG_PCI
>> -#define DFX_BUS_PCI(dev) (dev->bus == &pci_bus_type)
>> -#else
>> -#define DFX_BUS_PCI(dev) 0
>> -#endif
>> -
>> #ifdef CONFIG_EISA
>> #define DFX_BUS_EISA(dev) (dev->bus == &eisa_bus_type)
>> #else
>> @@ -436,7 +430,7 @@ static void dfx_port_read_long(DFX_board_t *bp, int offset, u32 *data)
>> static void dfx_get_bars(struct device *bdev,
>> resource_size_t *bar_start, resource_size_t *bar_len)
>> {
>> - int dfx_bus_pci = DFX_BUS_PCI(bdev);
>> + int dfx_bus_pci = dev_is_pci(bdev);
>> int dfx_bus_eisa = DFX_BUS_EISA(bdev);
>> int dfx_bus_tc = DFX_BUS_TC(bdev);
>> int dfx_use_mmio = DFX_MMIO || dfx_bus_tc;
>
> Acked-by: Maciej W. Rozycki <macro@linux-mips.org>
>
> Maciej
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
>
--
Thanks!
Yijing
next prev parent reply other threads:[~2013-12-06 3:07 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-05 11:38 [PATCH 2/2] net/fddi: Replace local marco with PCI standard macro Yijing Wang
2013-12-05 22:06 ` Maciej W. Rozycki
2013-12-06 3:06 ` Yijing Wang [this message]
2013-12-06 4:44 ` David Miller
2013-12-06 7:11 ` Yijing Wang
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=52A13F34.2010403@huawei.com \
--to=wangyijing@huawei.com \
--cc=davem@davemloft.net \
--cc=guohanjun@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=macro@linux-mips.org \
--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.