|
From: | Thomas Huth |
Subject: | Re: [PATCH] tests/qtest: npcm7xx-emc-test: Skip checking MAC |
Date: | Mon, 19 Sep 2022 14:44:36 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0 |
On 06/09/2022 18.31, Patrick Venture wrote:
The register tests walks all the registers to verify they are initially 0 when appropriate. However, if the MAC address is set in the register space, this should not be checked against 0. Reviewed-by: Hao Wu <wuhaotsh@google.com> Change-Id: I02426e39bdab33ceedd42c49d233e8680d4ec058
What's that change-id good for?
Signed-off-by: Patrick Venture <venture@google.com> --- tests/qtest/npcm7xx_emc-test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/qtest/npcm7xx_emc-test.c b/tests/qtest/npcm7xx_emc-test.c index 7c435ac915..207d8515b7 100644 --- a/tests/qtest/npcm7xx_emc-test.c +++ b/tests/qtest/npcm7xx_emc-test.c @@ -378,7 +378,8 @@ static void test_init(gconstpointer test_data)#undef CHECK_REG - for (i = 0; i < NUM_CAMML_REGS; ++i) {+ /* Skip over the MAC address registers, which is BASE+0 */ + for (i = 1; i < NUM_CAMML_REGS; ++i) { g_assert_cmpuint(emc_read(qts, mod, REG_CAMM_BASE + i * 2), ==, 0); g_assert_cmpuint(emc_read(qts, mod, REG_CAML_BASE + i * 2), ==,
Basically ack, but one question: Where should that non-zero MAC address come from / when did you hit a problem here? If QEMU is started without any mac settings at all (like it is done here), the register never contains a non-zero value, does it?
Thomas
[Prev in Thread] | Current Thread | [Next in Thread] |