On Mon, Sep 21, 2020 at 6:11 PM Andrew Lunn <and...@lunn.ch> wrote:
>
> > +struct bnxt_pcie_stat {
> > + const char *name;
> > + u16 offset;
> > + u8 size;
> > + const char *format;
> > +};
> > +
> > +static const struct bnxt_pcie_stat bnxt_pcie_stats[] = {
> > + { .name = "PL Signal integrity errors ", .offset = 0, .size = 4,
> > .format = "%lld" },
> > + { .name = "DL Signal integrity errors ", .offset = 4, .size = 4,
> > .format = "%lld" },
> > + { .name = "TLP Signal integrity errors ", .offset = 8, .size = 4,
> > .format = "%lld" },
>
> These look like statistics. Could they be part of ethtool -S
Only a couple of PCIe statistics look like counters. But to have all
PCIe statistics (most of them are not plain counters) at one place,
dumping them in the register dump. Thanks.
>
> Andrew