[Intel-wired-lan] [PATCH v4] igc: add support to interrupt, eeprom, registers and link self-tests

[Kirsher, Jeffrey T]

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of
> Brown, Aaron F
> Sent: Friday, May 1, 2020 14:42
> To: Lifshits, Vitaly <vitaly.lifshits at intel.com>; intel-wired-lan at osuosl.org
> Subject: Re: [Intel-wired-lan] [PATCH v4] igc: add support to interrupt, eeprom,
> registers and link self-tests
> 
> > From: Brown, Aaron F
> > Sent: Friday, May 1, 2020 2:26 PM
> > To: Vitaly Lifshits <vitaly.lifshits at intel.com>;
> > intel-wired-lan at osuosl.org
> > Subject: RE: [Intel-wired-lan] [PATCH v4] igc: add support to
> > interrupt, eeprom, registers and link self-tests
> >
> > > From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf
> > > Of Vitaly Lifshits
> > > Sent: Sunday, April 5, 2020 5:16 AM
> > > To: intel-wired-lan at osuosl.org
> > > Subject: [Intel-wired-lan] [PATCH v4] igc: add support to interrupt,
> > > eeprom, registers and link self-tests
> > >
> > > Introduced igc_diag.c and igc_diag.h, these files have the
> > > diagnostics functionality of igc driver. For the time being these
> > > files are being used by ethtool self-test callbacks.
> > > Which mean that interrupt, eeprom, registers and link self-tests for
> > > ethtool were implemented.
> > >
> > > Signed-off-by: Vitaly Lifshits <vitaly.lifshits at intel.com>
> > > Reported-by: kbuild test robot <lkp at intel.com>
> > > Reported-by: Dan Carpenter <dan.carpenter at oracle.com>
> > > ---
> > > v2: Fix return 0/1 to boolean value in igc_reg_test function
> > > v3: Address community comments
> > > v4: Fix interrupt test and address community comments
> > > ---
> > >  drivers/net/ethernet/intel/igc/Makefile      |   2 +-
> > >  drivers/net/ethernet/intel/igc/igc.h         |   4 +
> > >  drivers/net/ethernet/intel/igc/igc_diag.c    | 336
> > > +++++++++++++++++++++++++++
> > >  drivers/net/ethernet/intel/igc/igc_diag.h    |  37 +++
> > >  drivers/net/ethernet/intel/igc/igc_ethtool.c |  63 +++++
> > >  drivers/net/ethernet/intel/igc/igc_main.c    |   4 +-
> > >  drivers/net/ethernet/intel/igc/igc_regs.h    |   2 +
> > >  7 files changed, 445 insertions(+), 3 deletions(-)  create mode
> > > 100644 drivers/net/ethernet/intel/igc/igc_diag.c
> > >  create mode 100644 drivers/net/ethernet/intel/igc/igc_diag.h
> > >
> >
> > Tested-by: Aaron Brown <aaron.f.brown at intel.com>
> Sorry, I was intending the apply this to a different patch.  This is the patch that
> enables ethtool diags, which causes the call trace I sent out earlier today.

[Kirsher, Jeffrey T] 
I was just about to ask why the mixed messages...

> _______________________________________________
> Intel-wired-lan mailing list
> Intel-wired-lan at osuosl.org
> https://lists.osuosl.org/mailman/listinfo/intel-wired-lan