Re: Linux 2.4.15-pre4 - merge with Alan

Neil Brown (neilb@cse.unsw.edu.au)
Tue, 13 Nov 2001 16:54:00 +1100 (EST)


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/