On Thu, 14 May 2020 14:31:11 -0700 Jeff Kirsher wrote:
> diff --git a/drivers/net/ethernet/intel/igc/igc_diag.c
> b/drivers/net/ethernet/intel/igc/igc_diag.c
> new file mode 100644
> index 000000000000..1c4536105e56
> --- /dev/null
> +++ b/drivers/net/ethernet/intel/igc/igc_diag.c
> @@ -0,0 +1,186 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/* Copyright (c) 2020 Intel Corporation */
> +
> +#include "igc.h"
> +#include "igc_diag.h"
> +
> +struct igc_reg_test reg_test[] = {
> + { IGC_FCAL, 1, PATTERN_TEST, 0xFFFFFFFF, 0xFFFFFFFF },
> + { IGC_FCAH, 1, PATTERN_TEST, 0x0000FFFF, 0xFFFFFFFF },
> + { IGC_FCT, 1, PATTERN_TEST, 0x0000FFFF, 0xFFFFFFFF },
> + { IGC_RDBAH(0), 4, PATTERN_TEST, 0xFFFFFFFF, 0xFFFFFFFF },
drivers/net/ethernet/intel/igc/igc_diag.c:7:21: warning: symbol 'reg_test' was
not declared. Should it be static?