This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: io-pcmcia patch
- From: Gary Thomas <gary at mlbassoc dot com>
- To: Shaun Louie <sal at microplex dot com>
- Cc: eCos Patches <ecos-patches at sources dot redhat dot com>
- Date: Wed, 19 Mar 2008 12:10:04 -0600
- Subject: Re: io-pcmcia patch
- References: <47DF0763.4010109@microplex.com>
Shaun Louie wrote:
Hi,
This patch adds the cf_remove_handler() function (counterpart to
cf_register_handler). It also adds calls to cf_hwr_interrupt_enable()
and cf_hwr_interrupt_disable(). Since these are new functions to be
provided by the driver, some empty functions will need to be added in
any existing drivers (e.g. devs/pcmcia/arm/cerf). I can provide a patch
if you like, but I won't be able to test. I'll be providing a PCMCIA
driver for the PowerPC MPC8xx shortly.
This (and the MPC8xx driver) overall look OK, but we'll need
a few things before much more can be done with them.
* You need a copyright assignment (unless I missed it)
Please see http://ecos.sourceware.org/assign.html
* The MPC8xx driver is too platform specific. We try to
split drivers into architecture (or CPU family) + target.
In this case, I think that all of the details like where the
window is mapped, what I/O pins are used for VSn, etc,
should be in a target [include] file, not coded directly
within the MPC8xx driver.
Once these items are taken care of, we can consider this
contribution further.
Thanks & keep up the good work
--
------------------------------------------------------------
Gary Thomas | Consulting for the
MLB Associates | Embedded world
------------------------------------------------------------