From:  Pablo Menichini <pablo@menichini.com.ar>

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  A global variable should not be named 'objects' like in
  irias_object.c. This patch puts it into the right
  namespace.
  Also, strndup() is made static.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The maxiradio driver defines a few variables that should be
  made static.

From:  Pavel Machek <pavel@ucw.cz>

  With local pages present, swsusp's accounting goes wrong and you get
  nice BUG(). This fixes it, please apply.

From:  Pablo Menichini <pablo@menichini.com.ar>

From:  Adrian Bunk <bunk@fs.tum.de>

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  de_get and de_put are used only in the file they are
  defined in, so make them static

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The opti92x driver has global variables named 'rates' and 'bits'
  that pollute the namespace. This makes them static.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The variable rif_table should be neither global nor initialized to 0.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  net/irda/qos.c exports some variables that are not used
  in any other file. This makes them static to avoid
  namespace conflicts

From:  Bob Miller <rem@osdl.org>

  Added and #if/#endif pair to "remove" the un-used unknown_chipset declaration.

  Comments in generic.c indicate that unknow_chipset this will be used again
  in the future.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  A global variable should not be called 'log', this makes it static

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The function init_irq is only used in ide-probe.c, so it should
  better be static.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  balloc[] and tails[] are used only in the file they are defined
  in, so they should be made static

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The strdup function in sunrpc should not be global

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  Variables named 'io' and 'dma' should not be global

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  A global variable should not be called 'verbose'. It can
  be made static since it is only used in one file.

From:  Bob Miller <rem@osdl.org>

  Deleted an unused stack variable from sc1200_suspend.

From:  Pablo Menichini <pablo@menichini.com.ar>

From:  Pablo Menichini <pablo@menichini.com.ar>

From:  william stinson <wstinson@wanadoo.fr>

  this proposed patch for drivers/scsi/cpqfcTSinit.c scsi driver for
  Linux removes two calls to check_region using request_region instead

  This is patch number 28 in a series of check_region patches I am doing as
  part of the kernel janitors project.

From: Rusty Lynch <rusty@stinkycat.com>

  VT support requires drivers/char/keyboard.c which makes function
  calls implemented in drivers/input/, so that attempting to set
  CONFIG_INPUT=m or just not setting CONFIG_INPUT will result in a
  compile error if CONFIG_VT is on.

From:  william stinson <wstinson@wanadoo.fr>

  this patch for drivers/ide/legacy/umc8672.c ide driver for Linux
  removes one call to check_region using request_region instead

  I don't have this hardware so patch is not tested.  This patch removes
  all references to check_region in this driver.

  This is patch number 31 in a series of check_region patches I am doing as
   part of the kernel janitors project.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The pas2 driver has a global variable named 'lock', which
  pollutes the global namespace. This patch adds a pas_
  prefix. The same is done for 'translate_code'.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The i8k driver has global variables named 'force' and 'power_status'
  that pollute the name space. This makes them static.

From:  Geert Uytterhoeven <geert@linux-m68k.org>

  Kill duplicate extern char _stext (already declared globally 14 lines before)

From:  Chris Wilson <chris@qwirx.com>

  As part of the Linux Kernel Janitors project, I would like to submit my
  patch for bpqether.c.

  The document Documentation/cli-sti-removal.txt says that cli() should no
  longer be used to disable interrupts. This patch removes all references to
  cli() and {save,restore}_flags.

  - added a static spinlock to protect bpq_devices
  - changed cli/sti and {save,restore}_flags to taking the spinlock and
    disabling interrupts with spin_lock_irqsave
  - included my previous patch for proc_net_create, but as a separate hunk,
    so if you've already applied then just ignore the rejected hunk.

  I have verified that the patched driver compiles without warnings, but
  since I don't have the hardware I can't test it. Please treat with
  caution.

From:  william stinson <wstinson@wanadoo.fr>

  this proposed patch for drivers/atm/ambassador.c atm driver for Linux
  removes one call to check_region using request_region instead

  I don't have this hardware so patch is not tested.  This patch removes
  all references to check_region in this driver.

  This is patch number 30 in a series of check_region patches I am doing as
   part of the kernel janitors project.

From:  Bob Miller <rem@osdl.org>

  The function, cs5520_tune_chipset() is declared to return an int.
  Added a return statement instead of just falling of off the bottom.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  A global variable should not be called 'debuglevel'. It can
  be made static since it is only used in one file.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  A global variable should not be called 'ioctl_mutex',
  this makes it static.

The address allows oprofile and ksymoops to work again.  The state is
simply informative.

From:  Marcus Alanen <maalanen@ra.abo.fi>

  Moves the snd_legacy_find_free_ioport definition to opti92x-ad1848.c,
  since it is the only user.

From:  Arnd Bergmann <arnd@bergmann-dalldorf.de>

  The skfp driver has a global function named 'set_int'. This makes
  it static to avoid namespace pollution.

into penguin.transmeta.com:/home/penguin/torvalds/repositories/kernel/linux

into raven.il.steeleye.com:/home/jejb/BK/scsi-for-linus-2.5

The below does two things:
(i) do not try to spin up a CF reader without media
(ii) be careful when asking for the cache parameters mode page

[do not ask for this page when no media are present:
it is meaningless, and some devices react badly if we do;
check the reply so that we do not read past the end of the reply;
first give a small transport length - some USB devices are unhappy
if we ask for more than they provide]

Andries

This cleans up the delay code by moving the timer-specific
implementations into the timer_ops struct.  Thus, rather then doing:

	if(x86_delay_tsc)
		__rdtsc_delay(loops);
	else if(x86_delay_cyclone)
		__cyclone_delay(loops);
	else if(whatever....

we just simply do:

	timer->delay(loops);

Making it much easier to accommodate alternate time sources.

This creates an empty timer_opt structure (timer_none) which is then
used as a default initializer to the timer pointer.  This lets us avoid
having to check before dereferencing the timer in future code.

This patch moves BUG() and PAGE_BUG() from asm/page.h into asm/bug.h.

We also fix up linux/dcache.h, which included asm/page.h for the sole
purpose of getting the BUG() definition.

Since linux/kernel.h and linux/smp.h make use of BUG(), asm/bug.h is
included there as well.

In addition, linux/jbd.h did not contain a clear path with which to
obtain the archtecture BUG() definition, but did contain its own
definition.