This is the mail archive of the
ecos-discuss@sourceware.org
mailing list for the eCos project.
Re: AT91SAM7S platform, io_plf.h
On Wed, Sep 17, 2008 at 06:11:56PM +0200, G?bor T?r?k wrote:
> I was wondering, why there are defines related to AT91SAM7X
> peripherals in this file belonging to the AT91SAM7S platform. Any
> idea?
> I would like to expand the list of the defines (add reset controller,
> ADC and maybe others) and if the AT91SAM7X aren't needed here, I can
> remove them.
The AT91SAM7X is a AT91SAM7S plus an ethernet controller. The easiest
way to support the 7X was to extend the 7S. So these defines are
needed.
The reset control is already defined. It is in the var_io.h because
all AT91's have the same reset controller.
When adding the ADC defines you first need to determine what level
they go. Is the ADC controller specific to AT91SAM7 or is it generic
to all/many of the AT91 devices? It does look like many different
AT91SAM devices have ADC controllers. I took a quick look at the
AT91SAM9XE128 and it seems to have the same ADC controller. Maybe also
the AT91SAM7L devices use the same controller?
So to me it looks like the register definition belong in var_io.h.
Andrew
--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss