* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-17 19:00 [PATCH 1/6] IP100A, add end of pci id table Jesse Huang
@ 2006-08-17 14:45 ` Jeff Garzik
2006-08-18 5:52 ` Jesse Huang
2006-08-17 15:19 ` Alexey Dobriyan
1 sibling, 1 reply; 7+ messages in thread
From: Jeff Garzik @ 2006-08-17 14:45 UTC (permalink / raw)
To: Jesse Huang; +Cc: linux-kernel, netdev, akpm
Jesse Huang wrote:
> @@ -212,7 +212,7 @@ static const struct pci_device_id sundan
> { 0x1186, 0x1002, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 4 },
> { 0x13F0, 0x0201, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 5 },
> { 0x13F0, 0x0200, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 6 },
> - { }
> + { 0,}
> };
> MODULE_DEVICE_TABLE(pci, sundance_pci_tbl);
>
> @@ -231,7 +231,7 @@ static const struct pci_id_info pci_id_t
> {"D-Link DL10050-based FAST Ethernet Adapter"},
> {"Sundance Technology Alta"},
> {"IC Plus Corporation IP100A FAST Ethernet Adapter"},
> - { } /* terminate list. */
> + { NULL,} /* terminate list. */
NAK.
An empty array element "{ }" implies NULL. It is the kernel standard to
prefer "{ }" over an explicit initialization. Looks cleaner.
Jeff
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-17 19:00 [PATCH 1/6] IP100A, add end of pci id table Jesse Huang
2006-08-17 14:45 ` Jeff Garzik
@ 2006-08-17 15:19 ` Alexey Dobriyan
2006-08-18 6:15 ` Jesse Huang
1 sibling, 1 reply; 7+ messages in thread
From: Alexey Dobriyan @ 2006-08-17 15:19 UTC (permalink / raw)
To: Jesse Huang; +Cc: linux-kernel, netdev, akpm, jgarzik
On Thu, Aug 17, 2006 at 03:00:47PM -0400, Jesse Huang wrote:
> Add "0," and "NULL," to sundance_pci_tbl and pci_id_table.
> @@ -212,7 +212,7 @@ static const struct pci_device_id sundan
> { 0x1186, 0x1002, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 4 },
> { 0x13F0, 0x0201, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 5 },
> { 0x13F0, 0x0200, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 6 },
> - { }
> + { 0,}
> };
> MODULE_DEVICE_TABLE(pci, sundance_pci_tbl);
>
> @@ -231,7 +231,7 @@ static const struct pci_id_info pci_id_t
> {"D-Link DL10050-based FAST Ethernet Adapter"},
> {"Sundance Technology Alta"},
> {"IC Plus Corporation IP100A FAST Ethernet Adapter"},
> - { } /* terminate list. */
> + { NULL,} /* terminate list. */
They are already properly terminated. You don't have to do anything.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/6] IP100A, add end of pci id table
@ 2006-08-17 19:00 Jesse Huang
2006-08-17 14:45 ` Jeff Garzik
2006-08-17 15:19 ` Alexey Dobriyan
0 siblings, 2 replies; 7+ messages in thread
From: Jesse Huang @ 2006-08-17 19:00 UTC (permalink / raw)
To: linux-kernel, netdev, akpm, jgarzik, jesse
From: Jesse Huang <jesse@icplus.com.tw>
Add "0," and "NULL," to sundance_pci_tbl and pci_id_table.
Change Logs:
Add "0," and "NULL," to sundance_pci_tbl and pci_id_table.
---
drivers/net/sundance.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
9ef94f8b85a0070e49bb1883a9f124be1711761d
diff --git a/drivers/net/sundance.c b/drivers/net/sundance.c
index ac17377..eb81d91 100755
--- a/drivers/net/sundance.c
+++ b/drivers/net/sundance.c
@@ -21,8 +21,8 @@
*/
#define DRV_NAME "sundance"
-#define DRV_VERSION "1.1"
-#define DRV_RELDATE "27-Jun-2006"
+#define DRV_VERSION "1.2"
+#define DRV_RELDATE "03-Aug-2006"
/* The user-configurable values.
@@ -212,7 +212,7 @@ static const struct pci_device_id sundan
{ 0x1186, 0x1002, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 4 },
{ 0x13F0, 0x0201, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 5 },
{ 0x13F0, 0x0200, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 6 },
- { }
+ { 0,}
};
MODULE_DEVICE_TABLE(pci, sundance_pci_tbl);
@@ -231,7 +231,7 @@ static const struct pci_id_info pci_id_t
{"D-Link DL10050-based FAST Ethernet Adapter"},
{"Sundance Technology Alta"},
{"IC Plus Corporation IP100A FAST Ethernet Adapter"},
- { } /* terminate list. */
+ { NULL,} /* terminate list. */
};
/* This driver was written to use PCI memory space, however x86-oriented
--
1.3.GIT
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-17 14:45 ` Jeff Garzik
@ 2006-08-18 5:52 ` Jesse Huang
2006-08-18 11:09 ` Jeff Garzik
0 siblings, 1 reply; 7+ messages in thread
From: Jesse Huang @ 2006-08-18 5:52 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-kernel, netdev, akpm
Hi Jeff:
Sorry for that. I will remove those. Am I need to resent all of those
patch or send all in one patch?
Jesse Huang.
----- Original Message -----
From: "Jeff Garzik" <jgarzik@pobox.com>
To: "Jesse Huang" <jesse@icplus.com.tw>
Cc: <linux-kernel@vger.kernel.org>; <netdev@vger.kernel.org>;
<akpm@osdl.org>
Sent: Thursday, August 17, 2006 10:45 PM
Subject: Re: [PATCH 1/6] IP100A, add end of pci id table
Jesse Huang wrote:
> @@ -212,7 +212,7 @@ static const struct pci_device_id sundan
> { 0x1186, 0x1002, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 4 },
> { 0x13F0, 0x0201, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 5 },
> { 0x13F0, 0x0200, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 6 },
> - { }
> + { 0,}
> };
> MODULE_DEVICE_TABLE(pci, sundance_pci_tbl);
>
> @@ -231,7 +231,7 @@ static const struct pci_id_info pci_id_t
> {"D-Link DL10050-based FAST Ethernet Adapter"},
> {"Sundance Technology Alta"},
> {"IC Plus Corporation IP100A FAST Ethernet Adapter"},
> - { } /* terminate list. */
> + { NULL,} /* terminate list. */
NAK.
An empty array element "{ }" implies NULL. It is the kernel standard to
prefer "{ }" over an explicit initialization. Looks cleaner.
Jeff
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-17 15:19 ` Alexey Dobriyan
@ 2006-08-18 6:15 ` Jesse Huang
0 siblings, 0 replies; 7+ messages in thread
From: Jesse Huang @ 2006-08-18 6:15 UTC (permalink / raw)
To: Alexey Dobriyan; +Cc: linux-kernel, netdev, akpm, jgarzik
Hi Alexey:
I will remove that. Thanks for that.
Jesse Huang
----- Original Message -----
From: "Alexey Dobriyan" <adobriyan@gmail.com>
To: "Jesse Huang" <jesse@icplus.com.tw>
Cc: <linux-kernel@vger.kernel.org>; <netdev@vger.kernel.org>;
<akpm@osdl.org>; <jgarzik@pobox.com>
Sent: Thursday, August 17, 2006 11:19 PM
Subject: Re: [PATCH 1/6] IP100A, add end of pci id table
On Thu, Aug 17, 2006 at 03:00:47PM -0400, Jesse Huang wrote:
> Add "0," and "NULL," to sundance_pci_tbl and pci_id_table.
> @@ -212,7 +212,7 @@ static const struct pci_device_id sundan
> { 0x1186, 0x1002, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 4 },
> { 0x13F0, 0x0201, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 5 },
> { 0x13F0, 0x0200, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 6 },
> - { }
> + { 0,}
> };
> MODULE_DEVICE_TABLE(pci, sundance_pci_tbl);
>
> @@ -231,7 +231,7 @@ static const struct pci_id_info pci_id_t
> {"D-Link DL10050-based FAST Ethernet Adapter"},
> {"Sundance Technology Alta"},
> {"IC Plus Corporation IP100A FAST Ethernet Adapter"},
> - { } /* terminate list. */
> + { NULL,} /* terminate list. */
They are already properly terminated. You don't have to do anything.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-18 5:52 ` Jesse Huang
@ 2006-08-18 11:09 ` Jeff Garzik
2006-08-21 3:28 ` Jesse Huang
0 siblings, 1 reply; 7+ messages in thread
From: Jeff Garzik @ 2006-08-18 11:09 UTC (permalink / raw)
To: Jesse Huang; +Cc: linux-kernel, netdev, akpm
Jesse Huang wrote:
> Hi Jeff:
>
> Sorry for that. I will remove those. Am I need to resent all of those
> patch or send all in one patch?
Normally, when I merged a patch, you will receive a one-word reply
"applied" or "merged". Or for a series of patches, "applies patches 1-6".
For this series of patches, you will need to resend the entire patchset,
modified with the comments so far.
It is quite normal for some patchsets to undergo several iterations
before being merged, when engineering review highlights issues.
Thanks and regards,
Jeff
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/6] IP100A, add end of pci id table
2006-08-18 11:09 ` Jeff Garzik
@ 2006-08-21 3:28 ` Jesse Huang
0 siblings, 0 replies; 7+ messages in thread
From: Jesse Huang @ 2006-08-21 3:28 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-kernel, netdev, akpm
Hi Jeff:
Ok, I see. I will resend all of the patches after I finish modify. Thanks.
Jesse
----- Original Message -----
From: "Jeff Garzik" <jgarzik@pobox.com>
To: "Jesse Huang" <jesse@icplus.com.tw>
Cc: <linux-kernel@vger.kernel.org>; <netdev@vger.kernel.org>;
<akpm@osdl.org>
Sent: Friday, August 18, 2006 7:09 PM
Subject: Re: [PATCH 1/6] IP100A, add end of pci id table
Jesse Huang wrote:
> Hi Jeff:
>
> Sorry for that. I will remove those. Am I need to resent all of those
> patch or send all in one patch?
Normally, when I merged a patch, you will receive a one-word reply
"applied" or "merged". Or for a series of patches, "applies patches 1-6".
For this series of patches, you will need to resend the entire patchset,
modified with the comments so far.
It is quite normal for some patchsets to undergo several iterations
before being merged, when engineering review highlights issues.
Thanks and regards,
Jeff
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2006-08-21 3:29 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-08-17 19:00 [PATCH 1/6] IP100A, add end of pci id table Jesse Huang
2006-08-17 14:45 ` Jeff Garzik
2006-08-18 5:52 ` Jesse Huang
2006-08-18 11:09 ` Jeff Garzik
2006-08-21 3:28 ` Jesse Huang
2006-08-17 15:19 ` Alexey Dobriyan
2006-08-18 6:15 ` Jesse Huang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).