Re: SL811 problem on mach-pxa

[David Brownell <david-b@pacbell.net>]

On Thursday 17 February 2005 11:11 am, Frank Buss wrote:
> > Some of that looks reasonable, not all.  In particular, don't
> > change the convention on resources (memory to i/o), or expect
> > that the two regions involve more than one byte each ... the
> > hardware only has two single-byte registers!
> 
> ok, perhaps I've misunderstood the meaning of IORESOURCE_IO and
> IORESOURCE_MEM. Is IORESOURCE_IO for "outb" and "inb" (Intel assembler,
> don't know the Arm aquivalent)?

Yes, and there _is_ no ARM equivalent.  Modern CPUs generaly won't
bother with a separate physical I/O space, and special instructions
to access it.  So Linux drivers use macros that always boil down to
normal memory-mapped I/O accessors ... that's what ARM does, and
oddly enough all current users of this driver are on PXA hardware.

I'm not sure what the CFU1U(*) CF/PCMCIA support will need; if that
card maps those registers to I/O space, this driver will need some
minor tweaks (in addition to the PCMCIA framework glue that registers
a new platform_device).  That is, the registers would normally be
memory-mapped, but on some boards they might need to be in I/O space.

All your platform should need to do is initialize a platform_device
with the three resources (two memory, one IRQ), and platform_data
initialized to match the structure defined for that purpose.


> > That looks like the sort of thing that
> > should be done in the reset() routine rather than start(); 
> > and it should
> > certainly use a symbolic constant not 0x08.
> 
> do you mean sl811->board->reset? 

If your board is wired to support a separate chip reset (maybe
through a GPIO or something), yes; otherwise, I suppose in this
case I meant the port_power() routine should be handling that.

This driver doesn't have a separate hc_driver.reset() entry, it
does the analagous stuff as part of powering up the chip ... but
as you've noted, that approach is a bit troublesome on boards
that don't have that level of control over the hardware.

It's best to ask questions about USB drivers on linux-usb-devel,
not all USB developers make time to swim through LKML.

- Dave

(*) http://www.ratocsystems.com/english/products/subpages/cfu1u.html
    CompactFlash single-port USB host adapter, for PDAs, using
    the SL811HS chip.