small drivers/char/rio/ cleanups (fwd)
PATCH 1/1 small drivers/char/rio/ cleanups (fwd)
The atch below still compiles and applies against 2.6.10-rc2-mm1.
Please apply.
----- Forwarded message from Adrian Bunk <[email protected]> -----
Date: Sat, 6 Nov 2004 23:28:46 +0100
From: Adrian Bunk <[email protected]>
To: [email protected]
Cc: [email protected]
Subject: [2.6 patch] small drivers/char/rio/ cleanups
The patch below does cleanups under drivers/char/rio/ including the following:
- remove some completely unused code
- make some needlessly global code static
- remove #ifndef linux code
- remove never enabled #ifdef XPRINT_SUPPORT code
- RIOStrlen -> string.h strlen
- RIOStrCmp -> string.h strcmp
diffstat output:
drivers/char/rio/func.h | 21 ---- drivers/char/rio/rio_linux.c | 25 +---- drivers/char/rio/rioboot.c | 4 drivers/char/rio/riocmd.c | 58 ------------ drivers/char/rio/rioctrl.c | 33 ------ drivers/char/rio/rioinit.c | 57 +++--------- drivers/char/rio/riointr.c | 166 ----------------------------------- drivers/char/rio/rioroute.c | 18 ++- drivers/char/rio/riotable.c | 19 ---- drivers/char/rio/riotty.c | 17 ++- 10 files changed, 57 insertions(+), 361 deletions(-)
drivers/char/rio/func.h renamed from drivers/char/rio/func.h.old +0 -21
@@ -47,7 +47,6 @@int RIOBootCodeUNKNOWN (struct rio_info *,struct DownLoad *);void msec_timeout (struct Host *);int RIOBootRup (struct rio_info *,uint ,struct Host *,struct PKT *); -int RIOBootComplete (struct rio_info *,struct Host *,uint ,struct PktCmd *);int RIOBootOk (struct rio_info *,struct Host *,ulong );int RIORtaBound (struct rio_info *,uint );void FillSlot (int ,int ,uint ,struct Host *);
@@ -61,15 +60,10 @@int RIOKillNeighbour (struct rio_info *,caddr_t );int RIOSuspendBootRta (struct Host *,int ,int );int RIOFoadWakeup (struct rio_info *); -int RIOCommandRup (struct rio_info *,uint ,struct Host *,struct PKT *);struct CmdBlk *RIOGetCmdBlk (void );void RIOFreeCmdBlk (struct CmdBlk *);int RIOQueueCmdBlk (struct Host *,uint ,struct CmdBlk *);void RIOPollHostCommands (struct rio_info *,struct Host *); -int RIOStrlen (registerchar *); -int RIOStrCmp (registerchar *, registerchar *); -int RIOStrnCmp (registerchar *, registerchar *,int ); -void RIOStrNCpy (char *,char *,int );int RIOWFlushMark (int ,struct CmdBlk *);int RIORFlushEnable (int ,struct CmdBlk *);int RIOUnUse (int ,struct CmdBlk *);
@@ -77,7 +71,6 @@/* rioctrl.c */ int copyin (int ,caddr_t ,int ); -int copyout (caddr_t ,int ,int );int riocontrol (struct rio_info *,dev_t ,int ,caddr_t ,int );int RIOPreemptiveCmd (struct rio_info *,struct Port *,uchar );
@@ -89,24 +82,17 @@caddr_t RIOCheckForATCard (int );int RIOAssignAT (struct rio_info *,int ,caddr_t ,int );int RIOBoardTest (paddr_t ,caddr_t ,uchar ,int ); -int RIOScrub (int ,BYTE *,int ); -void RIOAllocateInterrupts (struct rio_info *); -void RIOStopInterrupts (struct rio_info *,int ,int );void RIOAllocDataStructs (struct rio_info *);void RIOSetupDataStructs (struct rio_info *);int RIODefaultName (struct rio_info *,struct Host *,uint ); -int RIOReport (struct rio_info *);struct rioVersion *RIOVersid (void );int RIOMapin (paddr_t ,int ,caddr_t *);void RIOMapout (paddr_t ,long ,caddr_t );void RIOHostReset (uint ,volatile struct DpRam *,uint );/* riointr.c */ -void riopoll (struct rio_info *); -void riointr (struct rio_info *);void RIOTxEnable (char *);void RIOServiceHost (struct rio_info *,struct Host *,int ); -void RIOReceive (struct rio_info *,struct Port *);int riotproc (struct rio_info *, registerstruct ttystatics *,int ,int );/* rioparam.c */
@@ -123,22 +109,15 @@/* rioroute.c */ int RIORouteRup (struct rio_info *,uint ,struct Host *,struct PKT *);void RIOFixPhbs (struct rio_info *,struct Host *,uint ); -int RIOCheckIsolated (struct rio_info *,struct Host *,uint ); -int RIOIsolate (struct rio_info *,struct Host *,uint ); -int RIOCheck (struct Host *,uint );uint GetUnitType (uint );int RIOSetChange (struct rio_info *); -void RIOConCon (struct rio_info *,struct Host *,uint ,uint ,uint ,uint ,int );int RIOFindFreeID (struct rio_info *,struct Host *,uint *,uint *); -int RIOFreeDisconnected (struct rio_info *,struct Host *,int ); -int RIORemoveFromSavedTable (struct rio_info *,struct Map *);/* riotty.c */ int riotopen (struct tty_struct * tty,struct file * filp);int riotclose (void *ptr); -int RIOCookMode (struct ttystatics *);int riotioctl (struct rio_info *,struct tty_struct *, registerint , registercaddr_t );void ttyseth (struct Port *,struct ttystatics *,struct old_sgttyb *sg);
drivers/char/rio/rio_linux.c renamed from drivers/char/rio/rio_linux.c.old +8 -17
@@ -33,10 +33,6 @@* */ -#define RCS_ID "$Id: rio.c,v 1.1 1999/07/11 10:13:54 wolff Exp wolff $" -#define RCS_REV "$Revision: 1.1 $" - -#include <linux/module.h> #include <linux/config.h> #include <linux/kdev_t.h>
@@ -198,7 +194,7 @@ unsigned int cmd, unsigned long arg);static int rio_init_drivers (void ); -void my_hd (void *addr,int len); +static void my_hd (void *addr,int len);static struct tty_driver *rio_driver, *rio_driver2;
@@ -206,11 +202,6 @@sources use all over the place. */ struct rio_info *p; -/* struct rio_board boards[RIO_HOSTS]; */ -struct rio_port *rio_ports; - -int rio_initialized; -int rio_nports;int rio_debug;
@@ -218,12 +209,12 @@- Set rio_poll to 1 to poll every timer tick (10ms on Intel). This is used when the card cannot use an interrupt for some reason. */ -int rio_poll =1 ; +static int rio_poll =1 ;/* These are the only open spaces in my computer. Yours may have more or less.... */ -int rio_probe_addrs[]= {0xc0000 ,0xd0000 ,0xe0000 }; +static int rio_probe_addrs[]= {0xc0000 ,0xd0000 ,0xe0000 };#define NR_RIO_ADDRS (sizeof(rio_probe_addrs)/sizeof (int))
@@ -264,7 +255,7 @@ .ioctl = rio_fw_ioctl, }; -struct miscdevice rio_fw_device = { +static struct miscdevice rio_fw_device = {RIOCTL_MISC_MINOR ,"rioctl" , &rio_fw_fops };
@@ -302,7 +293,7 @@#ifdef DEBUG -void my_hd (void *ad,int len) +static void my_hd (void *ad,int len) {int i, j, ch;unsigned char *addr = ad;
@@ -387,7 +378,7 @@ } -void rio_reset_interrupt (struct Host *HostP) +static void rio_reset_interrupt (struct Host *HostP) {func_enter ();
@@ -824,7 +815,7 @@* ********************************************************************** */ -struct vpd_prom *get_VPD_PROM (struct Host *hp) +static struct vpd_prom *get_VPD_PROM (struct Host *hp) {static struct vpd_prom vpdp;char *p;
@@ -1045,7 +1036,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 (struct pci_dev *pdev) +static void fix_rio_pci (struct pci_dev *pdev) {unsigned int hwbase;unsigned long rebase;
drivers/char/rio/rioboot.c renamed from drivers/char/rio/rioboot.c.old +3 -1
@@ -80,6 +80,8 @@#include "cmdblk.h" #include "route.h" +static int RIOBootComplete (struct rio_info *p,struct Host *HostP,uint Rup,struct PktCmd *PktCmdP ); +static uchar RIOAtVec2Ctrl[] = {
@@ -802,7 +804,7 @@** If booted by an RTA, HostP->Mapping[Rup].RtaUniqueNum is the booting RTA. ** RtaUniq is the booted RTA. */ -int RIOBootComplete (struct rio_info *p,struct Host *HostP,uint Rup,struct PktCmd *PktCmdP ) +static int RIOBootComplete (struct rio_info *p,struct Host *HostP,uint Rup,struct PktCmd *PktCmdP ) {struct Map *MapP =NULL ;struct Map *MapP2 =NULL ;
drivers/char/rio/riocmd.c renamed from drivers/char/rio/riocmd.c.old +1 -57
70 lines not on this page · raw message
drivers/char/rio/riotable.c renamed from drivers/char/rio/riotable.c.old +3 -16
@@ -37,6 +37,7 @@#include <linux/slab.h> #include <linux/errno.h> #include <linux/interrupt.h> +#include <linux/string.h> #include <asm/io.h> #include <asm/system.h>
@@ -276,7 +277,7 @@return -ENXIO ; }rio_dprintk (RIO_DEBUG_TABLE ,"RIONewTable: entering(9)\n" ); -if (RIOStrCmp (MapP->Name, +if (strcmp (MapP->Name, p->RIOConnectTable[SubEnt].Name)==0 && !(MapP->Flags &RTA16_SECOND_SLOT )) {/* (9) */ rio_dprintk (RIO_DEBUG_TABLE ,"RTA name %s used twice\n" , MapP->Name); p->RIOError.Error =NAME_USED_TWICE ;
@@ -405,7 +406,7 @@for ( Host2=0 ; Host2<p->RIONumHosts; Host2++ ) {if (Host2 == Host)continue ; -if (RIOStrCmp (p->RIOHosts[Host].Name, p->RIOHosts[Host2].Name) +if (strcmp (p->RIOHosts[Host].Name, p->RIOHosts[Host2].Name) ==0 ) { NameIsUnique =0 ; Host1++;
@@ -940,20 +941,6 @@ PortP->FirstOpen =1 ;/* -** handle the xprint issues -*/ -#ifdef XPRINT_SUPPORT - PortP->Xprint.XpActive =0 ; - PortP->Xprint.XttyP = &riox_tty[SysPort]; -/* TO FROM MAXLEN */ -RIOStrNCpy ( PortP->Xprint.XpOn, RIOConf.XpOn,MAX_XP_CTRL_LEN ); -RIOStrNCpy ( PortP->Xprint.XpOff, RIOConf.XpOff,MAX_XP_CTRL_LEN ); - PortP->Xprint.XpCps = RIOConf.XpCps; - PortP->Xprint.XpLen =RIOStrlen (PortP->Xprint.XpOn)+ -RIOStrlen (PortP->Xprint.XpOff); -#endif - -/* ** Buffers 'n things */ PortP->RxDataStart =0 ;
drivers/char/rio/riotty.c renamed from drivers/char/rio/riotty.c.old +11 -6
@@ -40,6 +40,7 @@#include <linux/slab.h> #include <linux/errno.h> #include <linux/tty.h> +#include <linux/string.h> #include <asm/io.h> #include <asm/system.h> #include <asm/string.h>
@@ -97,6 +98,9 @@int RIOShortCommand (struct rio_info *p,struct Port *PortP,int command,int len,int arg); +#if 0 +static int RIOCookMode (struct ttystatics *); +#endif extern int conv_vb[];/* now defined in ttymgr.c */ extern int conv_bv[];/* now defined in ttymgr.c */
@@ -726,7 +730,8 @@** COOK_WELL if the line discipline must be used to do the processing ** COOK_MEDIUM if the card can do all the processing necessary. */ -int +#if 0 +static int RIOCookMode (struct ttystatics *tp) {/*
@@ -757,7 +762,7 @@*/ return COOK_MEDIUM; } - +#endif static void RIOClearUp (PortP )
@@ -1011,8 +1016,8 @@pseterr (EFAULT ); } PortP->Xprint.XpOn[MAX_XP_CTRL_LEN -1 ] ='\0' ; - PortP->Xprint.XpLen =RIOStrlen (PortP->Xprint.XpOn)+ -RIOStrlen (PortP->Xprint.XpOff); + PortP->Xprint.XpLen =strlen (PortP->Xprint.XpOn)+ +strlen (PortP->Xprint.XpOff);rio_spin_unlock_irqrestore (&PortP->portSem, flags);return 0 ;
@@ -1026,8 +1031,8 @@pseterr (EFAULT ); } PortP->Xprint.XpOff[MAX_XP_CTRL_LEN -1 ] ='\0' ; - PortP->Xprint.XpLen =RIOStrlen (PortP->Xprint.XpOn)+ -RIOStrlen (PortP->Xprint.XpOff); + PortP->Xprint.XpLen =strlen (PortP->Xprint.XpOn)+ +strlen (PortP->Xprint.XpOff);rio_spin_unlock_irqrestore (&PortP->portSem, flags);return 0 ;
drivers/char/rio/rioctrl.c renamed from drivers/char/rio/rioctrl.c.old +1 -32
@@ -130,7 +130,6 @@#define drv_makedev (maj, min) ((((uint) maj & 0xff) << 8) | ((uint) min & 0xff)) -#ifdef linuxint copyin (int arg,caddr_t dp,int siz) {int rv;
@@ -141,8 +140,7 @@else return rv; } - -int copyout (caddr_t dp,int arg,int siz) +static int copyout (caddr_t dp,int arg,int siz) {int rv;
@@ -152,35 +150,6 @@else return rv; } -#else - -int -copyin (arg, dp, siz) -int arg; -caddr_t dp; -int siz; -{ -if (rbounds ((unsigned long ) arg) >= siz) { -bcopy ( arg, dp, siz ); -return OK ; - }else -return (COPYFAIL ); -} - -int -copyout (dp, arg, siz) -caddr_t dp; -int arg; -int siz; -{ -if (wbounds ((unsigned long ) arg) >= siz ) { -bcopy ( dp, arg, siz ); -return OK ; - }else -return (COPYFAIL ); -} -#endif - int riocontrol(p, dev, cmd, arg, su) struct rio_info * p;
drivers/char/rio/rioinit.c renamed from drivers/char/rio/rioinit.c.old +15 -42
100 lines not on this page · raw message
drivers/char/rio/riointr.c renamed from drivers/char/rio/riointr.c.old +3 -163
186 lines not on this page · raw message
drivers/char/rio/rioroute.c renamed from drivers/char/rio/rioroute.c.old +12 -6
@@ -83,6 +83,12 @@#include "list.h" #include "sam.h" +static int RIOCheckIsolated (struct rio_info *,struct Host *,uint ); +static int RIOIsolate (struct rio_info *,struct Host *,uint ); +static int RIOCheck (struct Host *,uint ); +static void RIOConCon (struct rio_info *,struct Host *,uint ,uint ,uint ,uint ,int ); + +/* ** Incoming on the ROUTE_RUP ** I wrote this while I was tired. Forgive me.
@@ -717,7 +723,7 @@** the world about it. This is done to ensure that the configurator ** only gets up-to-date information about what is going on. */ -int +static int RIOCheckIsolated(p, HostP, UnitId) struct rio_info * p; struct Host *HostP;
@@ -747,7 +753,7 @@** all the units attached to it. This will mean that the entire ** subnet will re-introduce itself. */ -int +static int RIOIsolate(p, HostP, UnitId) struct rio_info * p; struct Host * HostP;
@@ -782,7 +788,7 @@return 1 ; } -int +static int RIOCheck(HostP, UnitId) struct Host *HostP; uint UnitId;
@@ -883,7 +889,7 @@return (0 ); } -void +static void RIOConCon(p, HostP, FromId, FromLink, ToId, ToLink, Change) struct rio_info * p; struct Host *HostP;
@@ -966,7 +972,7 @@** Delete and RTA entry from the saved table given to us ** by the configuration program. */ -int +static int RIORemoveFromSavedTable (struct rio_info *p,struct Map *pMap) {int entry;
@@ -993,7 +999,7 @@** Scan the unit links to and return zero if the unit is completely ** disconnected. */ -int +static int RIOFreeDisconnected (struct rio_info *p,struct Host *HostP,int unit) {int link;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
----- End forwarded message -----