diff --git a/harness/mame/NETWORK_REPORT.md b/harness/mame/NETWORK_REPORT.md index 24fa043..38a527b 100644 --- a/harness/mame/NETWORK_REPORT.md +++ b/harness/mame/NETWORK_REPORT.md @@ -4340,3 +4340,12 @@ It is answered: | does the firmware's init/ring/descriptor/TDMD sequence work on real silicon | **yes** (sections 61, 64.1) | | was the old failure our payload or the chip | **our payload**: descriptor length must be `-len` (`0xFFE8`/`0xFA0A`) | | can the chip be handed back to the OS from a payload | **yes at the chip level** (`REST = 0x73`), not at the driver level | + +### 64.5 The stray white box in the photo + +The photo shows one white box half over the `REST` row. That is the **old `ok` +indicator** (`rect(120, 850, 24, 24, 0xFF)`), left over from the white-box +layout: the text row `REST` sits at y 846-867 and the box at y 850-874, so they +overlap. The text panel already reports `OK YES/NO`, so both indicator boxes +(`ok` at x 120 and TX-done at x 160) have been removed. Cosmetic only — the +values in 64.1/64.2 are unaffected. diff --git a/harness/udpstream/build/lance.coff b/harness/udpstream/build/lance.coff index 5421f5c..5c2949c 100644 Binary files a/harness/udpstream/build/lance.coff and b/harness/udpstream/build/lance.coff differ diff --git a/harness/udpstream/build/lance_fixlen.coff b/harness/udpstream/build/lance_fixlen.coff index e7bb6c2..b0bf7fb 100644 Binary files a/harness/udpstream/build/lance_fixlen.coff and b/harness/udpstream/build/lance_fixlen.coff differ diff --git a/harness/udpstream/lance.c b/harness/udpstream/lance.c index 50b4d66..ba10db9 100644 --- a/harness/udpstream/lance.c +++ b/harness/udpstream/lance.c @@ -364,12 +364,8 @@ int main() } else { draw_text(312, 804, lab_no); } - if (ok) { - rect(120, 850, 24, 24, 0xFF); - } - if (!(txmd1 & TMD1_OWN)) { - rect(160, 850, 24, 24, 0xFF); - } + /* the old white-box ok / tx-done indicators were here; the text panel + * shows OK as YES/NO and they overlapped the REST row, so they are gone */ /* Give the chip back to the OS. * diff --git a/harness/udpstream/lance_fixlen.c b/harness/udpstream/lance_fixlen.c index 00514b5..58d38e4 100644 --- a/harness/udpstream/lance_fixlen.c +++ b/harness/udpstream/lance_fixlen.c @@ -364,12 +364,8 @@ int main() } else { draw_text(312, 804, lab_no); } - if (ok) { - rect(120, 850, 24, 24, 0xFF); - } - if (!(txmd1 & TMD1_OWN)) { - rect(160, 850, 24, 24, 0xFF); - } + /* the old white-box ok / tx-done indicators were here; the text panel + * shows OK as YES/NO and they overlapped the REST row, so they are gone */ /* Give the chip back to the OS. *