Hello.
Stanislaw Gruszka wrote:
+void __init at91_add_device_ide(struct at91_ide_data *data)
+{
This platform initialization code is virtually identical to the
at91_cf version submitted by Uros Bizjak, albeit for the SAM9260.
Similarly the "at91_cf_data" and "at91_ide_data" structs.
Wouldn't it be better to have a single CF initialization function, and
you specify whether you want "plain CF" mode or True-IDE mode?
You could even pass the same platform-resources structure (ie,
base-address of CF region), and have at91_ide setup the location of
the task-file and control registers from that base address.
Yes, this can be merged together. Ronetix already did a SAM9263 board
and linux patch with "full" CF device.
http://download.ronetix.info/sk-eb926x/linux/kernel/2.6.28/003_linux-2.6.28-at91-ronetix-12012009.patch
This patch is clearly incomplete because drivers/pcmcia/at91_cf.c
does accesses AT91RM9200 specific SMC register, but the patch does
nothing about making this work on AT91SAM9. Though it indeed seems that
AT91SAM9 can be supported by that driver with minimum change -- perhaps
by using cpi_is_*()...
And WTF is that: sound/arm/at91-ac97.c.rej? :-/
There are some difference,
The CF memory mappings seem compatible b/w AT91SAM9 and AT91RM9200,
except the latter doesn't support True IDE mode.
but I think CF stuff can have one initialization function. Could this be a separate patch in later date? I would like marge
at91_ide into mainline without changes if there will be no objections.
I would still like 2 resources merged into 1 -- however, possibly
from the offset 0x00c00000, i.e. not including PCMCIA compatible memory
space.
Cheers
Stanislaw Gruszka
MBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html