-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix doxygen comments for hwtimer.h, queue.h and io.h #1041
Conversation
void hwtimer_init(void); | ||
void hwtimer_init_comp(uint32_t fcpu); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function exists and is not static
.
Sorry, you only moved the prototype.
addresses: RIOT-OS#1041 (comment)
addresses: RIOT-OS#1041 (diff)
addresses: RIOT-OS#1041 (comment)
uint32_t priority; | ||
struct queue_node_t *next; /**< next queue node */ | ||
unsigned int data; /**< queue node data */ | ||
uint32_t priority; /**< queue node priority */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you align above comments?
addresses RIOT-OS#1041 (comment)
addresses RIOT-OS#1041 (comment)
addresses RIOT-OS#1041 (comment) etc
*/ | ||
void hwtimer_spin(unsigned long ticks); | ||
|
||
int hwtimer_active(void); | ||
|
||
/** @} */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this file includes 2 closing @}
but only one opening @{
void queue_add_head(queue_node_t *root, queue_node_t *new_obj); | ||
|
||
/** | ||
* @brief remove the queues head |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
queue's head
addresses RIOT-OS#1041 (comment)
Anything else or shall I squash? |
@@ -17,7 +17,7 @@ | |||
* interrupt context and must use the shortest possible execution time (e.g. | |||
* set a flag and trigger a worker thread). | |||
* | |||
* <b>The hardware timer must not be used within applications</b>, use \ref vtimer | |||
* <b>The hardware timer must not be used within applications</b>, use \ref sys_vtimer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think actually should not be used (until you know what you're doing)
would be more accurate.
addresses RIOT-OS#1041 (comment)
Everything addressed again, squash now? |
@@ -22,6 +22,12 @@ | |||
#ifndef IO_H | |||
#define IO_H | |||
|
|||
/** | |||
* @brief Feuerwhere putstring implementation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feuerware
(Feuerwhere was only the name of the project) - or call it firmware_puts. ;)
One last comment from my side, otherwise I'm fine with squashing and merging. |
rebased, squashed |
has got one explicit and one implicit ACKs -> go |
Fix doxygen comments for hwtimer.h, queue.h and io.h
Some things struck me as strange and I added "@todo" comments to them. I'd propose to add tickets in their regard unless someone says here and now that everything is in order with the respective bits.Fixes #951