On Monday November 12, torvalds@transmeta.com wrote:
>
> Which means that I'd also like people to double-check that there are no
> embarrassing missing pieces due to the merge (or other patches).
I'm still lamenting the loss of the "-Werror" compile switch....
I compiled a fairly vanilla config, filtered the wheat from the chaff
and got:
agpgart_be.c:1533: warning: `intel_820_cleanup' defined but not used
lvm-snap.c: In function `lvm_snapshot_release':
lvm-snap.c:545: warning: unused variable `nbhs'
uhci.c:2986: warning: initialization discards qualifiers from pointer target type
dmi_scan.c:195: warning: `disable_ide_dma' defined but not used
The following patch removes these warnings. I suspect all parts are
correct, but *please* don't apply unless you (or someone else)
corroborate.
NeilBrown
--- ./drivers/char/agp/agpgart_be.c 2001/11/13 05:38:20 1.1
+++ ./drivers/char/agp/agpgart_be.c 2001/11/13 05:38:31
@@ -1529,6 +1529,7 @@
return;
}
+#if 0
static void intel_820_cleanup(void)
{
u8 temp;
@@ -1541,7 +1542,7 @@
pci_write_config_byte(agp_bridge.dev, INTEL_APSIZE,
previous_size->size_value);
}
-
+#endif
static int intel_820_configure(void)
{
--- ./drivers/usb/uhci.c 2001/11/13 05:41:32 1.1
+++ ./drivers/usb/uhci.c 2001/11/13 05:41:36
@@ -2968,7 +2968,7 @@
}
#endif
-static const struct pci_device_id __devinitdata uhci_pci_ids[] = { {
+static const struct pci_device_id uhci_pci_ids[] = { {
/* handle any USB UHCI controller */
class: ((PCI_CLASS_SERIAL_USB << 8) | 0x00),
--- ./drivers/md/lvm-snap.c 2001/11/13 05:35:39 1.1
+++ ./drivers/md/lvm-snap.c 2001/11/13 05:35:41
@@ -542,7 +542,6 @@
void lvm_snapshot_release(lv_t * lv)
{
- int nbhs = KIO_MAX_SECTORS;
if (lv->lv_block_exception)
{
--- ./arch/i386/kernel/dmi_scan.c 2001/11/13 05:42:55 1.1
+++ ./arch/i386/kernel/dmi_scan.c 2001/11/13 05:43:01
@@ -184,6 +184,7 @@
#define NO_MATCH { NONE, NULL}
#define MATCH(a,b) { a, b }
+#if 0
/*
* We have problems with IDE DMA on some platforms. In paticular the
* KT7 series. On these it seems the newer BIOS has fixed them. The
@@ -203,6 +204,7 @@
#endif
return 0;
}
+#endif
/*
* Reboot options and system auto-detection code provided by
-
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/
This archive was generated by hypermail 2b29 : Thu Nov 15 2001 - 21:00:33 EST