This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: New port for ARM Industrial Module AIM 711
On Mittwoch, 3. März 2004 17:48, John Dallaway wrote:
> 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.
>
Yes, i will do that.
Thank you,
Roland
--
___________________________________________________
VS Vision Systems GmbH, Industrial Image Processing
Dipl.-Ing. Roland Caßebohm
Aspelohe 27A, D-22848 Norderstedt, Germany
Mail: roland.cassebohm@visionsystems.de
http://www.visionsystems.de
___________________________________________________