tree fb8e80f4fef292cd45804425b0ab2274ca9fd158
parent 301644ab480ab9787c617e1a94e19ca5c2774072
author Michael Brown <mcb30@ipxe.org> 1725276297 +0100
committer Michael Brown <mcb30@ipxe.org> 1725276297 +0100

[etherfabric] Fix use of uninitialised variable in falcon_xaui_link_ok()

The link status check in falcon_xaui_link_ok() reads from the
FCN_XX_CORE_STAT_REG_MAC register only on production hardware (where
the FPGA version reads as zero), but modifies the value and writes
back to this register unconditionally.  This triggers an uninitialised
variable warning on newer versions of gcc.

Fix by assuming that the register exists only on production hardware,
and so moving the "modify-write" portion of the "read-modify-write"
operation to also be covered by the same conditional check.

Signed-off-by: Michael Brown <mcb30@ipxe.org>