Skip to content

Commit 602ef9c

Browse files
committed
TODO & Doc update
1 parent dee59af commit 602ef9c

File tree

2 files changed

+14
-3
lines changed

2 files changed

+14
-3
lines changed

docs/notes/Scratch.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ run and produce output both for non-graphical & graphical builds.
5454
|-------|------------|-----------|---------------|---------------|---------------|---------|
5555
| Debug | E9_ENABLED | Graphical | kearly_printf | kdebug_printf | kdisp_putc | e9_putc |
5656
|-------|------------|-----------|---------------|---------------|---------------|---------|
57-
| No | No | No | No | No | No | No |
57+
| No | No | No | invalid | invalid | invalid | invalid |
5858
| Yes | No | No | kdisp_putc | No | Yes | No |
5959
| No | Yes | No | invalid | invalid | invalid | invalid |
6060
| Yes | Yes | No | kdisp_putc | Yes | Yes | Yes |
@@ -81,6 +81,10 @@ As with `kdisp_importantPrint`, its of no use. `kpanic` will
8181
* DEBUG & Graphical -> + `kdebug_printf`
8282
* DEBUG & Text -> Print panic text to `kdebug_printf` + `kearly_printf`.
8383

84+
Also note that these above rules & build options are not for Unittests. It is required to test even
85+
the debug components, so the `ifdef` guards for debug specific components must allow when building
86+
unittests.
87+
8488
------------------------------------
8589

8690
## Logging & Debugging a graphical OS

docs/notes/TODO.md

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,16 @@
106106
|-------------------------------------------------------------------------|-----------|------------|
107107
| GRAPHICS | | |
108108
|-------------------------------------------------------------------------|-----------|------------|
109-
| [ ] Logs in graphics mode for kearly_printf & kdebug_printf | 13 Nov 24 | |
109+
| [X] kearly_printf, kdebug_printf should be debug specific funcitons. | 16 Nov 24 | 17 Nov 24 |
110+
| Implementation: | | |
111+
| * `kearly_printf` is now a DEBUG specific function. | | |
112+
| * `kearly_printf` would print to VGA Text buffer in text mode. In | | |
113+
| graphics mode it does nothing. | | |
114+
| * `kdebug_printf` is removed. | | |
110115
|-------------------------------------------------------------------------|-----------|------------|
111-
| [ ] k_panic in graphics mode - Need to have very low dependency. May be | 13 Nov 24 | |
116+
| [X] Logs in graphics mode should not print to VGA text buffer | 13 Nov 24 | 17 Nov 24 |
117+
|-------------------------------------------------------------------------|-----------|------------|
118+
| [X] k_panic in graphics mode - Need to have very low dependency. May be | 13 Nov 24 | 17 Nov 24 |
112119
| have to draw directly to video frame buffer | | |
113120
|-------------------------------------------------------------------------|-----------|------------|
114121
| APP LIB | | |

0 commit comments

Comments
 (0)