Re: [PATCH] Make ide-cs work for hardware with 8-bit CF-Interface

From: David Hinds
Date: Tue Sep 06 2005 - 17:21:32 EST


On Tue, Sep 06, 2005 at 06:47:08PM +0200, Thomas Kleffel (LKML) wrote:
>
> The following patch is against vanilla 2.6.13.
>
> ldiff -uprN a/drivers/ide/legacy/ide-cs.c b/drivers/ide/legacy/ide-cs.c
> --- a/drivers/ide/legacy/ide-cs.c 2005-08-08 15:30:35.000000000 +0200
> +++ b/drivers/ide/legacy/ide-cs.c 2005-09-05 02:09:47.000000000 +0200
> @@ -186,7 +186,8 @@ static int idecs_register(unsigned long
> {
> hw_regs_t hw;
> memset(&hw, 0, sizeof(hw));
> - ide_init_hwif_ports(&hw, io, ctl, NULL);
> + ide_std_init_ports(&hw, io, ctl);
> + hw.io_ports[IDE_DATA_OFFSET] = io + 0x08;
> hw.irq = irq;
> hw.chipset = ide_pci;
> return ide_register_hw_with_fixup(&hw, NULL, ide_undecoded_slave);

You can't do this, at least not exactly this way. io + 0x08 may not
be a mapped IO address; it is only valid when a card is mapped with
one contiguous 16-bit IO window. PCMCIA IDE cards are not necessarily
mapped that way: they may be mapped with one 8-port window and one
1-port window, to match standard IBM PC IDE port locations. In that
case, the registers at 0x08 and 0x09 are not available. The CF spec
may impose more uniformity here than the PCMCIA spec does. I do know
that some IDE cards do end up configured with discontiguous register
allocations when used with ide-cs.c.

-- Dave
-
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/