To LUGNET HomepageTo LUGNET News HomepageTo LUGNET Guide Homepage
 Help on Searching
 
Post new message to lugnet.robotics.rcx.legosOpen lugnet.robotics.rcx.legos in your NNTP NewsreaderTo LUGNET News Traffic PageSign In (Members)
 Robotics / RCX / legOS / 1366
1365  |  1367
Subject: 
Re: questions, comments, and suggestions
Newsgroups: 
lugnet.robotics.rcx.legos
Date: 
Fri, 11 Aug 2000 23:27:13 GMT
Viewed: 
4051 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:
  Re: questions, comments, and suggestions
 
(...) If you don't find it defined in gcc-2.95.2/config/h8300/*, then yes. Eddie C. Dost ecd@skynet.be (26 years ago, 12-Aug-00, to lugnet.robotics.rcx.legos)

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. (...) (...) (26 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
    

©2005 LUGNET. All rights reserved. - hosted by steinbruch.info GbR