Index: /trunk/src/VBox/Devices/Audio/DevCodec.cpp
===================================================================
--- /trunk/src/VBox/Devices/Audio/DevCodec.cpp	(revision 31042)
+++ /trunk/src/VBox/Devices/Audio/DevCodec.cpp	(revision 31043)
@@ -59,4 +59,5 @@
 #define CODEC_B_INDEX(cmd)      ((cmd) & CODEC_VERB_B_INDEX)
 
+//** @todo r=michaln: Please document why this is bit 36 (it's not the same as the link format)
 #define CODEC_RESPONSE_UNSOLICITED RT_BIT_64(36)
 
@@ -89,5 +90,8 @@
     Log(("codecUnimplemented: cmd(raw:%x: cad:%x, d:%c, nid:%x, verb:%x)\n", cmd,
         CODEC_CAD(cmd), CODEC_DIRECT(cmd) ? 'N' : 'Y', CODEC_NID(cmd), CODEC_VERBDATA(cmd)));
+    //** @todo r=michaln: The zero used as a codec ID shouldn't be hardcoded!
     if (CODEC_CAD(cmd) != 0)
+        //** @todo r=michaln: Are we really supposed to respond to commands sent to nonexistent codecs??
+        //** @todo r=michaln: Where in the specs is this response format defined?
         *pResp = ((uint64_t)CODEC_CAD(cmd) << 4)| 0xFF;
     else
@@ -485,4 +489,6 @@
     {
         *pfn = CODEC_CAD(cmd) != 0 ? codecUnimplemented : codecBreak;
+        //** @todo r=michaln: Why "intelHD" and not e.g. "HDAcodec"?
+        //** @todo r=michaln: There needs to be a counter to avoid log flooding (see e.g. DevRTC.cpp)
         LogRel(("intelHD: cmd %x was ignored\n", cmd));
         return VINF_SUCCESS;