On Fri, Jan 22, 2021 at 4:08 AM Heiner Kallweit <hkallwe...@gmail.com> wrote: > > When working on the PCI VPD code I also tested with a Broadcom BCM95719 > card. tg3 uses internal NVRAM access with this card, so I forced it to > PCI VPD mode for testing. PCI VPD access fails > (i + PCI_VPD_LRDT_TAG_SIZE + j > len) because only TG3_NVM_VPD_LEN (256) > bytes are read, but PCI VPD has 400 bytes on this card. > > So add a constant TG3_NVM_PCI_VPD_MAX_LEN that defines the maximum > PCI VPD size. The actual VPD size is returned by pci_read_vpd(). > In addition it's not worth looping over pci_read_vpd(). If we miss the > 125ms timeout per VPD dword read then definitely something is wrong, > and if the tg3 module loading is killed then there's also not much > benefit in retrying the VPD read. > > Signed-off-by: Heiner Kallweit <hkallwe...@gmail.com>
Reviewed-by: Michael Chan <michael.c...@broadcom.com> Thanks.
smime.p7s
Description: S/MIME Cryptographic Signature
