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
- From: Roland Caßebohm <roland dot cassebohm at VisionSystems dot de>
- To: "John Dallaway" <jld at ecoscentric dot com>
- Cc: <ecos-patches at ecos dot sourceware dot org>
- Date: Wed, 31 Mar 2004 15:03:13 +0200
- Subject: Re: New port for ARM Industrial Module AIM 711
- References: <200403031753.44381.roland.cassebohm@visionsystems.de>
Hello,
does anybody had time to look at the code or tested the
run-time code?
Or could somebody tell me how could I help to get the
port integrated in the public cvs tree?
Thank you,
Roland
On Mittwoch, 3. März 2004 17:53, Roland Caßebohm wrote:
> 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
http://www.visionsystems.de
___________________________________________________