Subject:
|
Re: questions, comments, and suggestions
|
Newsgroups:
|
lugnet.robotics.rcx.legos
|
Date:
|
Fri, 11 Aug 2000 23:27:13 GMT
|
Viewed:
|
1239 times
|
| |
| |
> RCX_COMPILER is an internal gcc/egcs define, when patched correctly to
> support RCX interrupts. This is still used and useful, as it saves only
> the required registers, unlike the HANDLER_WRAPPER macro, which saves
> all volatile registers.
Do you know if RCX_COMPILER patch need to be used in GCC 2.95.2 cross
compiler too?
> > *) Include files don't have the
> > #ifdef __cplusplus
> > extern "C" {
> > #endif
> > ...
> > #ifdef __cplusplus
> > }
> > #endif
> > Which makes the user do that for cpp files. That seems a bit
> > backwards to me.
>
> Should be fixed.
Fixed right now. I'm ready to commit.
> > *) battery.c: line 38-41: (comments removed)
> > #ifdef CONF_BATTERY_INDICATOR
> > unsigned int battery_refresh_counter = 0;
> > unsigned int battery_refresh_period = 2000;
> > #endif
> >
> > The 2000 is a counter, in seconds, in which the battery indicator
> > on the LCD is updated. Should not the 2000 be something that
> > can be configured by the user: be put in the config.h file?
>
> This will change into a separtate kernel thread soon, the rewriter could
> maybe make this configurable.
It will be changed in 0.2.5, but two notes:
1) The counter is in ms.
2) The counter is a variable (it can be changed by code if you need).
> > *) Is there a way we can get the auto-shut off thing? I know i've
> > forgotten to turn it off overnight! Of course, the timeout should
> > be configurable, as should its actual useage.
>
> This might be possible.
It could be in the "battery" management in 0.2.5.
> > *) include/sys/battery.h has a few references to the __template_h_ stuff
>
> Oops.
Fixed.
> > *) Is it possible to modify the kernel/program.c and/or the `dll`
> > program to allow us to upload new firmware without resetting?
>
> This should be easily done.
But do we need this?
> > *) How about including the "freshly generated API docs" in the
> > legOS/doc/ directory?
Probably it will be.
> > *) Hmm.. Doxyfile seems to be out of date (PROJECT_NUMBER = 0.2.2 ;-)
> > Also, there doesn't seem to be documenation generated for the
> > lib/* files. Although I think the info is provided in the include
> > directory. (ie, string.h).
Fixed the version number.
Bye,
Paolo.
---
One tentacle, one vote.
|
|
Message has 1 Reply:
Message is in Reply To:
| | Re: questions, comments, and suggestions
|
| (...) RCX_COMPILER is an internal gcc/egcs define, when patched correctly to support RCX interrupts. This is still used and useful, as it saves only the required registers, unlike the HANDLER_WRAPPER macro, which saves all volatile registers. (...) (...) (24 years ago, 1-Aug-00, to lugnet.robotics.rcx.legos)
|
10 Messages in This Thread:
- Entire Thread on One Page:
- Nested:
All | Brief | Compact | Dots
Linear:
All | Brief | Compact
This Message and its Replies on One Page:
- Nested:
All | Brief | Compact | Dots
Linear:
All | Brief | Compact
|
|
|
|