[2.5 patch] kill TWO_ZERO (fwd)

From: Adrian Bunk
Date: Sat Sep 20 2003 - 15:08:41 EST


Hi Linus,

the patch forwarded below still applies (with a few lines offset) and
compiles against -test5.

It was already approved by Rogier Wolff.

Please apply
Adrian



----- Forwarded message from Adrian Bunk <bunk@xxxxxxxxx> -----

Date: Thu, 19 Jun 2003 02:11:56 +0200
From: Adrian Bunk <bunk@xxxxxxxxx>
To: Rogier Wolff <R.E.Wolff@xxxxxxxxxxxx>,
Patrick van de Lageweg <patrick@xxxxxxxxxxxx>
Cc: linux-kernel@xxxxxxxxxxxxxxx,
trivial@xxxxxxxxxxxxxxx
Subject: [2.5 patch] kill TWO_ZERO

Three drivers under drivers/char/ in 2.5.72 use TWO_ZERO for
compatibility code with ancient 2.0 kernels. The patch below removes
this #ifdef'd code.

diffstat output:

rio/rio_linux.c | 45 ++++-----------------------------------------
specialix.c | 37 ++-----------------------------------
sx.c | 24 ++----------------------
3 files changed, 8 insertions(+), 98 deletions(-)


Please apply
Adrian

--- linux-2.5.72/drivers/char/rio/rio_linux.c.old 2003-06-18 21:57:01.000000000 +0200
+++ linux-2.5.72/drivers/char/rio/rio_linux.c 2003-06-19 02:03:07.000000000 +0200
@@ -234,14 +234,12 @@
support up to 64 bits on 64bit architectures. -- REW 20/06/99 */
long rio_irqmask = -1;

-#ifndef TWO_ZERO
MODULE_AUTHOR("Rogier Wolff <R.E.Wolff@xxxxxxxxxxxx>, Patrick van de Lageweg <patrick@xxxxxxxxxxxx>");
MODULE_DESCRIPTION("RIO driver");
MODULE_LICENSE("GPL");
MODULE_PARM(rio_poll, "i");
MODULE_PARM(rio_debug, "i");
MODULE_PARM(rio_irqmask, "i");
-#endif

static struct real_driver rio_real_driver = {
rio_disable_tx_interrupts,
@@ -1033,13 +1031,6 @@
func_exit();
}

-#ifdef TWO_ZERO
-#define PDEV unsigned char pci_bus, unsigned pci_fun
-#define pdev pci_bus, pci_fun
-#else
-#define PDEV struct pci_dev *pdev
-#endif
-

#ifdef CONFIG_PCI
/* This was written for SX, but applies to RIO too...
@@ -1061,7 +1052,7 @@
EEprom. As the bit is read/write for the CPU, we can fix it here,
if we detect that it isn't set correctly. -- REW */

-void fix_rio_pci (PDEV)
+void fix_rio_pci (struct pci_dev *pdev)
{
unsigned int hwbase;
unsigned long rebase;
@@ -1094,12 +1085,7 @@
int okboard;

#ifdef CONFIG_PCI
-#ifndef TWO_ZERO
struct pci_dev *pdev = NULL;
-#else
- unsigned char pci_bus, pci_fun;
- /* in 2.2.x pdev is a pointer defining a PCI device. In 2.0 its the bus/fn */
-#endif
unsigned int tint;
unsigned short tshort;
#endif
@@ -1127,17 +1113,11 @@

#ifdef CONFIG_PCI
/* First look for the JET devices: */
-#ifndef TWO_ZERO
while ((pdev = pci_find_device (PCI_VENDOR_ID_SPECIALIX,
PCI_DEVICE_ID_SPECIALIX_SX_XIO_IO8,
pdev))) {
if (pci_enable_device(pdev)) continue;
-#else
- for (i=0;i< RIO_NBOARDS;i++) {
- if (pcibios_find_device (PCI_VENDOR_ID_SPECIALIX,
- PCI_DEVICE_ID_SPECIALIX_SX_XIO_IO8, i,
- &pci_bus, &pci_fun)) break;
-#endif
+
/* Specialix has a whole bunch of cards with
0x2000 as the device ID. They say its because
the standard requires it. Stupid standard. */
@@ -1195,16 +1175,9 @@
} else {
iounmap((char*) (p->RIOHosts[p->RIONumHosts].Caddr));
}
-
-#ifdef TWO_ZERO
- } /* We have two variants with the opening brace, so to prevent */
-#else
- } /* Emacs from getting confused we have two closing braces too. */
-#endif
+ }

/* Then look for the older PCI card.... : */
-#ifndef TWO_ZERO
-

/* These older PCI cards have problems (only byte-mode access is
supported), which makes them a bit awkward to support.
@@ -1218,12 +1191,6 @@
PCI_DEVICE_ID_SPECIALIX_RIO,
pdev))) {
if (pci_enable_device(pdev)) continue;
-#else
- for (i=0;i< RIO_NBOARDS;i++) {
- if (pcibios_find_device (PCI_VENDOR_ID_SPECIALIX,
- PCI_DEVICE_ID_SPECIALIX_RIO, i,
- &pci_bus, &pci_fun)) break;
-#endif

#ifdef CONFIG_RIO_OLDPCI
pci_read_config_dword(pdev, PCI_BASE_ADDRESS_0, &tint);
@@ -1271,11 +1238,7 @@
printk (KERN_ERR "Found an older RIO PCI card, but the driver is not "
"compiled to support it.\n");
#endif
-#ifdef TWO_ZERO
- } /* We have two variants with the opening brace, so to prevent */
-#else
- } /* Emacs from getting confused we have two closing braces too. */
-#endif
+ }
#endif /* PCI */

/* Now probe for ISA cards... */
--- linux-2.5.72/drivers/char/sx.c.old 2003-06-19 02:03:39.000000000 +0200
+++ linux-2.5.72/drivers/char/sx.c 2003-06-19 02:04:34.000000000 +0200
@@ -2357,14 +2357,6 @@
func_exit();
}

-#ifdef TWO_ZERO
-#define PDEV unsigned char pci_bus, unsigned pci_fun
-#define pdev pci_bus, pci_fun
-#else
-#define PDEV struct pci_dev *pdev
-#endif
-
-
#ifdef CONFIG_PCI
/********************************************************
* Setting bit 17 in the CNTRL register of the PLX 9050 *
@@ -2377,7 +2369,7 @@
EEprom. As the bit is read/write for the CPU, we can fix it here,
if we detect that it isn't set correctly. -- REW */

-static void fix_sx_pci (PDEV, struct sx_board *board)
+static void fix_sx_pci (struct pci_dev *pdev, struct sx_board *board)
{
unsigned int hwbase;
unsigned long rebase;
@@ -2407,12 +2399,7 @@
struct sx_board *board;

#ifdef CONFIG_PCI
-#ifndef TWO_ZERO
struct pci_dev *pdev = NULL;
-#else
- unsigned char pci_bus, pci_fun;
- /* in 2.2.x pdev is a pointer defining a PCI device. In 2.0 its the bus/fn */
-#endif
unsigned int tint;
unsigned short tshort;
#endif
@@ -2432,19 +2419,12 @@
}

#ifdef CONFIG_PCI
-#ifndef TWO_ZERO
while ((pdev = pci_find_device (PCI_VENDOR_ID_SPECIALIX,
PCI_DEVICE_ID_SPECIALIX_SX_XIO_IO8,
pdev))) {
if (pci_enable_device(pdev))
continue;
-#else
- for (i=0;i< SX_NBOARDS;i++) {
- if (pcibios_find_device (PCI_VENDOR_ID_SPECIALIX,
- PCI_DEVICE_ID_SPECIALIX_SX_XIO_IO8, i,
- &pci_bus, &pci_fun))
- break;
-#endif
+
/* Specialix has a whole bunch of cards with
0x2000 as the device ID. They say its because
the standard requires it. Stupid standard. */
--- linux-2.5.72/drivers/char/specialix.c.old 2003-06-19 02:05:00.000000000 +0200
+++ linux-2.5.72/drivers/char/specialix.c 2003-06-19 02:06:26.000000000 +0200
@@ -92,40 +92,7 @@
#include <linux/delay.h>
#include <linux/version.h>
#include <linux/pci.h>
-
-
-/* ************************************************************** */
-/* * This section can be removed when 2.0 becomes outdated.... * */
-/* ************************************************************** */
-
-#if LINUX_VERSION_CODE < 131328 /* Less than 2.1.0 */
-#define TWO_ZERO
-#else
-#if LINUX_VERSION_CODE < 131371 /* less than 2.1.43 */
-/* This has not been extensively tested yet. Sorry. */
-#warning "You're on your own between 2.1.0 and 2.1.43.... "
-#warning "Please use a recent kernel."
-#endif
-#endif
-
-
-#ifdef TWO_ZERO
-#define Get_user(a,b) a = get_user(b)
-#define copy_from_user(a,b,c) memcpy_fromfs(a,b,c)
-#define copy_to_user(a,b,c) memcpy_tofs(a,b,c)
-#define queue_task queue_task_irq_off
-#else
-#define Get_user(a,b) get_user(a,b)
-#endif
-
-/* ************************************************************** */
-/* * End of compatibility section.. * */
-/* ************************************************************** */
-
-
-#ifndef TWO_ZERO
#include <asm/uaccess.h>
-#endif

#include "specialix_io8.h"
#include "cd1865.h"
@@ -1745,7 +1712,7 @@
if (error)
return error;

- Get_user(arg, (unsigned long *) value);
+ get_user(arg, (unsigned long *) value);
switch (cmd) {
case TIOCMBIS:
/* if (arg & TIOCM_RTS)
@@ -1937,7 +1904,7 @@
(unsigned long *) arg);
return 0;
case TIOCSSOFTCAR:
- Get_user(arg, (unsigned long *) arg);
+ get_user(arg, (unsigned long *) arg);
tty->termios->c_cflag =
((tty->termios->c_cflag & ~CLOCAL) |
(arg ? CLOCAL : 0));
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

----- End forwarded message -----

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/