linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] QD65xx I/O ports
@ 2004-04-06 11:31 Geert Uytterhoeven
  2004-04-08 22:50 ` Bartlomiej Zolnierkiewicz
  0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2004-04-06 11:31 UTC (permalink / raw)
  To: Andrew Morton, Bartlomiej Zolnierkiewicz
  Cc: Linux Kernel Development, linux-ide


I/O port numbers can be larger than 8-bit on many platforms (this caused a
warning when {out,in}b() cast reg to a pointer on platforms with memory mapped
I/O)

--- linux-2.6.5/drivers/ide/legacy/qd65xx.c.orig	2003-10-09 10:02:47.000000000 +0200
+++ linux-2.6.5/drivers/ide/legacy/qd65xx.c	2004-04-01 13:31:54.000000000 +0200
@@ -92,7 +92,7 @@

 static int timings[4]={-1,-1,-1,-1}; /* stores current timing for each timer */

-static void qd_write_reg (u8 content, u8 reg)
+static void qd_write_reg (u8 content, unsigned long reg)
 {
 	unsigned long flags;

@@ -101,7 +101,7 @@
 	spin_unlock_irqrestore(&ide_lock, flags);
 }

-u8 __init qd_read_reg (u8 reg)
+u8 __init qd_read_reg (unsigned long reg)
 {
 	unsigned long flags;
 	u8 read;

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] QD65xx I/O ports
  2004-04-06 11:31 [PATCH] QD65xx I/O ports Geert Uytterhoeven
@ 2004-04-08 22:50 ` Bartlomiej Zolnierkiewicz
  2004-04-13 13:47   ` Erik Mouw
  0 siblings, 1 reply; 5+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2004-04-08 22:50 UTC (permalink / raw)
  To: Geert Uytterhoeven, Andrew Morton; +Cc: Linux Kernel Development, linux-ide

On Tuesday 06 of April 2004 13:31, Geert Uytterhoeven wrote:
> I/O port numbers can be larger than 8-bit on many platforms (this caused a
> warning when {out,in}b() cast reg to a pointer on platforms with memory
> mapped I/O)

Was VESA Local Bus ever used on something else than 486?
I think it's better to just add "depends on X86" to drivers/ide/Kconfig.

Cheers,
Bartlomiej


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] QD65xx I/O ports
  2004-04-08 22:50 ` Bartlomiej Zolnierkiewicz
@ 2004-04-13 13:47   ` Erik Mouw
  2004-04-14  1:40     ` Cef (LKML)
  0 siblings, 1 reply; 5+ messages in thread
From: Erik Mouw @ 2004-04-13 13:47 UTC (permalink / raw)
  To: Bartlomiej Zolnierkiewicz
  Cc: Geert Uytterhoeven, Andrew Morton, Linux Kernel Development, linux-ide

On Fri, Apr 09, 2004 at 12:50:24AM +0200, Bartlomiej Zolnierkiewicz wrote:
> On Tuesday 06 of April 2004 13:31, Geert Uytterhoeven wrote:
> > I/O port numbers can be larger than 8-bit on many platforms (this caused a
> > warning when {out,in}b() cast reg to a pointer on platforms with memory
> > mapped I/O)
> 
> Was VESA Local Bus ever used on something else than 486?

IIRC there were early Pentium boards with VESA Local Bus (VLB), but my
memory is vague about that.

> I think it's better to just add "depends on X86" to drivers/ide/Kconfig.

VLB is nothing more than ISA with a 32 bit address and data bus. That
is exactly how many busses on embedded CPUs work, but with the
difference that they map the ISA 16bit memory and IO space somewhere in
their 4GB memory space.

For example: the Digital/Intel StrongARM SA-11x0 has a 32 bit bus,
which (with the help of some glue logic) works like a 32 bit ISA bus.
Quite some embedded hardware emulates a 16 bit ISA bus by mapping the
ISA memory and IO space in a special memory area which can live
anywhere in the 4GB space, and that's why Geert wants an unsigned long
address in {out,in}b().


Erik

-- 
+-- Erik Mouw -- www.harddisk-recovery.com -- +31 70 370 12 90 --
| Lab address: Delftechpark 26, 2628 XH, Delft, The Netherlands

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] QD65xx I/O ports
  2004-04-13 13:47   ` Erik Mouw
@ 2004-04-14  1:40     ` Cef (LKML)
  2004-04-14  1:52       ` Bartlomiej Zolnierkiewicz
  0 siblings, 1 reply; 5+ messages in thread
From: Cef (LKML) @ 2004-04-14  1:40 UTC (permalink / raw)
  To: linux-kernel

On Tue, 13 Apr 2004 11:47 pm, Erik Mouw wrote:
> On Fri, Apr 09, 2004 at 12:50:24AM +0200, Bartlomiej Zolnierkiewicz wrote:
> > On Tuesday 06 of April 2004 13:31, Geert Uytterhoeven wrote:
> > > I/O port numbers can be larger than 8-bit on many platforms (this
> > > caused a warning when {out,in}b() cast reg to a pointer on platforms
> > > with memory mapped I/O)
> >
> > Was VESA Local Bus ever used on something else than 486?
>
> IIRC there were early Pentium boards with VESA Local Bus (VLB), but my
> memory is vague about that.

I can confirm that. I used to own one of those beasts. PCI, VLB & ISA all on 
the one board. First generation Pentium's (+5v core) only as far as I 
remember.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] QD65xx I/O ports
  2004-04-14  1:40     ` Cef (LKML)
@ 2004-04-14  1:52       ` Bartlomiej Zolnierkiewicz
  0 siblings, 0 replies; 5+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2004-04-14  1:52 UTC (permalink / raw)
  To: Cef (LKML); +Cc: linux-kernel

On Wednesday 14 of April 2004 03:40, Cef (LKML) wrote:
> On Tue, 13 Apr 2004 11:47 pm, Erik Mouw wrote:
> > On Fri, Apr 09, 2004 at 12:50:24AM +0200, Bartlomiej Zolnierkiewicz wrote:
> > > On Tuesday 06 of April 2004 13:31, Geert Uytterhoeven wrote:
> > > > I/O port numbers can be larger than 8-bit on many platforms (this
> > > > caused a warning when {out,in}b() cast reg to a pointer on platforms
> > > > with memory mapped I/O)
> > >
> > > Was VESA Local Bus ever used on something else than 486?
> >
> > IIRC there were early Pentium boards with VESA Local Bus (VLB), but my
> > memory is vague about that.
>
> I can confirm that. I used to own one of those beasts. PCI, VLB & ISA all
> on the one board. First generation Pentium's (+5v core) only as far as I
> remember.

Yes, you are right.  I also remember them. :-)


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2004-04-14  1:52 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-04-06 11:31 [PATCH] QD65xx I/O ports Geert Uytterhoeven
2004-04-08 22:50 ` Bartlomiej Zolnierkiewicz
2004-04-13 13:47   ` Erik Mouw
2004-04-14  1:40     ` Cef (LKML)
2004-04-14  1:52       ` Bartlomiej Zolnierkiewicz

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).