Re: New port for ARM Industrial Module AIM 711

[John Dallaway <jld at ecoscentric dot com>]

Hi Roland

Roland Caßebohm wrote:

> > > > The "enable", "disable" and "set_value" commands should be implemented
> > > > as "requires" contraints in CYGPKG_HAL_ARM_AIM711 rather than in
> > > > ecos.db. It is best to avoid these commands wherever possible since
> > > > their implementation is not elegant. For example, the corresponding CDL
> > > > options will be given a "user value" rather than an "inferred value".
> > > >
> > > > I will leave comments on the run-time aspects to other maintainers.
> > >
> > > How is the best way to change this? Maybe write them as "required" in
> > > the cdl file of the AIM 711 hal?
> >
> > Exactly. The correct values for these CDL options will them be inferred
> > when the target is selected.
>
> OK, I will make an additional patch which correct this. Or is it better
> to send the whole package new while it is not in CVS?

A whole new package, while inefficient, would probably be easier to 
handle. I suggest you wait for any feedback on the run-time code first 
though.

John Dallaway
eCosCentric Limited
--
Visit us at Embedded Systems Conference :: San Francisco Mar 30-Apr 1