From e1f9d641fb9c14331ff62b4db6fe0844beb529ec Mon Sep 17 00:00:00 2001 From: Takashi Sakamoto Date: Tue, 8 Sep 2026 21:29:29 +0900 Subject: [PATCH] firewire: core: fix type of transaction data in config ROM parser test The type of transaction data should be __be32 in the stub function for the asynchronous transactions. Fixes: bae885b9bee5 ("firewire: core: add test to detect irm-is-1394-1995-only quirk in config ROM parser") Fixes: 1c4326e7d830 ("firewire: core: add test to avoid excessive configuration ROM length") Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202609060257.8wQo9w9N-lkp@intel.com/ Link: https://lore.kernel.org/r/20260908122929.2886698-1-o-takashi@sakamocchi.jp Signed-off-by: Takashi Sakamoto --- drivers/firewire/config-rom-parser-test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/firewire/config-rom-parser-test.c b/drivers/firewire/config-rom-parser-test.c index e5c173353057..45f1ead326c5 100644 --- a/drivers/firewire/config-rom-parser-test.c +++ b/drivers/firewire/config-rom-parser-test.c @@ -206,7 +206,7 @@ static int stub_run_transaction_regular(struct fw_card *card, int tcode, int des KUNIT_ASSERT_EQ(test, length, 4); unsigned int index = (offset - (CSR_REGISTER_BASE | CSR_CONFIG_ROM)) / sizeof(u32); - u32 *quadlet = payload; + __be32 *quadlet = payload; KUNIT_EXPECT_LE(test, speed, param->expected_speed); KUNIT_EXPECT_LT(test, index, param->quadlet_length); @@ -264,7 +264,7 @@ static int stub_run_transaction_malformed(struct fw_card *card, int tcode, int d KUNIT_ASSERT_EQ(test, length, 4); unsigned int index = (offset - (CSR_REGISTER_BASE | CSR_CONFIG_ROM)) / sizeof(u32); - u32 *quadlet = payload; + __be32 *quadlet = payload; if (index < ARRAY_SIZE(config_rom_first_part)) *quadlet = cpu_to_be32(config_rom_first_part[index]);