Re: [PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-16 Thread Jarod Wilson
On Fri, Jul 16, 2010 at 7:42 AM, Andy Walls wrote: > On Thu, 2010-07-15 at 18:07 -0400, Jarod Wilson wrote: >> On Thu, Jul 15, 2010 at 5:43 PM, Andy Walls wrote: >> > On Thu, 2010-07-15 at 21:08 +0200, Peter Huewe wrote: >> >> From: Peter Huewe > >> > a. PCI_ANY_ID indicates to the reader a wild

Re: [PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-16 Thread Andy Walls
On Thu, 2010-07-15 at 18:07 -0400, Jarod Wilson wrote: > On Thu, Jul 15, 2010 at 5:43 PM, Andy Walls wrote: > > On Thu, 2010-07-15 at 21:08 +0200, Peter Huewe wrote: > >> From: Peter Huewe > > a. PCI_ANY_ID indicates to the reader a wildcard match is being > > performed. The PCI_VDEVICE() macro

Re: [PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-15 Thread Jarod Wilson
On Thu, Jul 15, 2010 at 5:43 PM, Andy Walls wrote: > On Thu, 2010-07-15 at 21:08 +0200, Peter Huewe wrote: >> From: Peter Huewe >> >> This patch converts pci_table entries, where .subvendor=PCI_ANY_ID and >> .subdevice=PCI_ANY_ID, .class=0 and .class_mask=0, to use the >> PCI_VDEVICE macro, and t

Re: [PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-15 Thread Peter Hüwe
Am Donnerstag 15 Juli 2010 23:43:20 schrieb Andy Walls: > > ... use the PCI_VDEVICE macro, and thus improves readability. > I have to disagree. The patch may improve typesetting, but it degrades > clarity and maintainability from my perspective. > ... > So I'm going to NAK this for ivtv, unless so

Re: [PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-15 Thread Andy Walls
On Thu, 2010-07-15 at 21:08 +0200, Peter Huewe wrote: > From: Peter Huewe > > This patch converts pci_table entries, where .subvendor=PCI_ANY_ID and > .subdevice=PCI_ANY_ID, .class=0 and .class_mask=0, to use the > PCI_VDEVICE macro, and thus improves readability. Hi Peter, I have to disagree.

[PATCH 24/25] video/ivtv: Convert pci_table entries to PCI_VDEVICE (if PCI_ANY_ID is used)

2010-07-15 Thread Peter Huewe
From: Peter Huewe This patch converts pci_table entries, where .subvendor=PCI_ANY_ID and .subdevice=PCI_ANY_ID, .class=0 and .class_mask=0, to use the PCI_VDEVICE macro, and thus improves readability. Signed-off-by: Peter Huewe --- drivers/media/video/ivtv/ivtv-driver.c |6 ++ 1 files