Attention is currently required from: Martin L Roth.
Julius Werner has posted comments on this change by Martin L Roth. ( https://review.coreboot.org/c/coreboot/+/87186?usp=email )
Change subject: Documentation/lib: Update Timestamp documentation ......................................................................
Patch Set 4:
(1 comment)
File Documentation/lib/timestamp.md:
https://review.coreboot.org/c/coreboot/+/87186/comment/7e1d65c9_8d0d97ce?usp... : PS4, Line 192: ### Core Functions
Not all our docstrings are formatted for Doxygen anyway.
Right, but we could slowly change that file-by-file. I'm thinking that whenever someone wants to take the time to write documentation that goes into this detail, they might as well spend that same time to update the respective header docstrings into doxygen format and set up an import into their documentation page, and then the next time someone updates that code they'll hopefully also update the docstring comments and stick to the comment style already used in that header file.
So I think this Breathe thing sounds like a cool idea and if someone wants to set that up it might really help us keep our documentation more up-to-date in the future. I think this might succeed where the previous doxygen Makefile target failed because nobody ever bothers to run make just to look at some documentation, but if this is automatically tied in to `doc.coreboot.org` that should make it easy enough to access that people will actually see and use it. But I understand that's going to take effort (and maybe some wider discussion first), so not trying to hold up the rest of this CL with that.