Page MenuHomeFreeBSD

sfxge(4): fix diagnostics support build without Siena
ClosedPublic

Authored by arybchik on Nov 21 2018, 11:25 AM.
Tags
None
Referenced Files
F81971313: D18117.diff
Sun, Dec 15, 1:44 AM
Unknown Object (File)
Sun, Nov 17, 10:00 PM
Unknown Object (File)
Sun, Nov 17, 3:35 PM
Unknown Object (File)
Mar 16 2024, 4:01 AM
Unknown Object (File)
Jan 11 2024, 9:56 PM
Unknown Object (File)
Dec 20 2023, 1:16 AM
Unknown Object (File)
Nov 15 2023, 5:08 PM
Unknown Object (File)
Nov 7 2023, 2:55 AM
Subscribers

Details

Summary

The compilation failed because __efx_sram_pattern_fns was used in
efx_nic.c, but defined in efx_sram.c which is only needed when
supporting Siena.

To fix it move all the code using __efx_sram_pattern_fns into
Siena-specific files (except for the definition in efx_sram.c itself,
as that file only needs to be included in Siena-supporting builds
anyway).

The functions to test registers and tables are unlikely to apply to any
new hardware and so can be moved into Siena files. Since Huntington
such tests have been implemented in firmware.

Submitted by: Mark Spender <mspender at solarflare.com>
Sponsored by: Solarflare Communications, Inc.
MFC after: 1 week

Test Plan

amd64 tinderbox kernel builds tested
driver loaded/unloaded, interfaces up/down, traffic works fine

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Nov 24 2018, 4:32 PM
This revision was automatically updated to reflect the committed changes.