summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2017-04-24 17:08:06 -0700
committerJulius Werner <jwerner@chromium.org>2017-04-26 01:31:51 +0200
commita915cea289210b17b7df0845f8738363be659bb4 (patch)
treea51427a6fe8eeb4ad325b0ef7b81faf9cee02ee1
parentdae95f0dfe9ad94922c940d3c0522d53284b4deb (diff)
cbmem_console: Document known reimpementations of console structure/API
It turns out that there are quite a few other projects that can access the CBMEM console by now. If we ever want to make another structural or behavioral change to it, we need to know where these implementations are so we can make sure they're all getting updated. Let's try to build a comprehensive list in the file that should be the source of truth for all (coreboot's own implementation). Change-Id: Ia3d6a87230f5bfdde9d812bc7154e22880c1377a Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/19439 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/lib/cbmem_console.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/src/lib/cbmem_console.c b/src/lib/cbmem_console.c
index d570bbaa9a..eb0cc93ee4 100644
--- a/src/lib/cbmem_console.c
+++ b/src/lib/cbmem_console.c
@@ -28,6 +28,16 @@
* cursor field gets set to indicate that this happened. If the underlying
* storage allows this, the buffer will persist across multiple boots and append
* to the previous log.
+ *
+ * NOTE: These are known implementations accessing this console that need to be
+ * updated in case of structure/API changes:
+ *
+ * cbmem: [coreboot]/src/util/cbmem/cbmem.c
+ * libpayload: [coreboot]/payloads/libpayload/drivers/cbmem_console.c
+ * coreinfo: [coreboot]/payloads/coreinfo/bootlog_module.c
+ * Linux: drivers/firmware/google/memconsole-coreboot.c
+ * SeaBIOS: src/firmware/coreboot.c
+ * GRUB: grub-core/term/i386/coreboot/cbmemc.c
*/
struct cbmem_console {
u32 size;