Commit 0efcc3f2 authored by Heiner Kallweit's avatar Heiner Kallweit Committed by Jakub Kicinski
Browse files

sky2: Stop printing VPD info to debugfs 



Sky2 is parsing the VPD and adds the parsed information to its debugfs
file. This isn't needed in kernel, userspace tools like lspci can be
used to display such information nicely. Therefore remove this from
the driver.

lspci -vv:

Capabilities: [50] Vital Product Data
	Product Name: Marvell Yukon 88E8070 Gigabit Ethernet Controller
	Read-only fields:
		[PN] Part number: Yukon 88E8070
		[EC] Engineering changes: Rev. 1.0
		[MN] Manufacture ID: Marvell
		[SN] Serial number: AbCdEfG970FD4
		[CP] Extended capability: 01 10 cc 03
		[RV] Reserved: checksum good, 9 byte(s) reserved
	Read/write fields:
		[RW] Read-write area: 1 byte(s) free
	End

Relevant part in debugfs file:

0000:01:00.0 Product Data
Marvell Yukon 88E8070 Gigabit Ethernet Controller
 Part Number: Yukon 88E8070
 Engineering Level: Rev. 1.0
 Manufacturer: Marvell
 Serial Number: AbCdEfG970FD4

Signed-off-by: default avatarHeiner Kallweit <hkallweit1@gmail.com>
Acked-by: default avatarStephen Hemminger <stephen@networkplumber.org>
Link: https://lore.kernel.org/r/bbaee8ab-9b2e-de04-ee7b-571e094cc5fe@gmail.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent af54faab

drivers/net/ethernet/marvell/sky2.c

+1 −83
Original line number Diff line number Diff line
@@ -4440,86 +4440,6 @@ static const struct ethtool_ops sky2_ethtool_ops = { 


static struct dentry *sky2_debug;





/*

 * Read and parse the first part of Vital Product Data

 */

#define VPD_SIZE	128

#define VPD_MAGIC	0x82



static const struct vpd_tag {

	char tag[2];

	char *label;

} vpd_tags[] = {

	{ "PN",	"Part Number" },

	{ "EC", "Engineering Level" },

	{ "MN", "Manufacturer" },

	{ "SN", "Serial Number" },

	{ "YA", "Asset Tag" },

	{ "VL", "First Error Log Message" },

	{ "VF", "Second Error Log Message" },

	{ "VB", "Boot Agent ROM Configuration" },

	{ "VE", "EFI UNDI Configuration" },

};



static void sky2_show_vpd(struct seq_file *seq, struct sky2_hw *hw)

{

	size_t vpd_size;

	loff_t offs;

	u8 len;

	unsigned char *buf;

	u16 reg2;



	reg2 = sky2_pci_read16(hw, PCI_DEV_REG2);

	vpd_size = 1 << ( ((reg2 & PCI_VPD_ROM_SZ) >> 14) + 8);



	seq_printf(seq, "%s Product Data\n", pci_name(hw->pdev));

	buf = kmalloc(vpd_size, GFP_KERNEL);

	if (!buf) {

		seq_puts(seq, "no memory!\n");

		return;

	}



	if (pci_read_vpd(hw->pdev, 0, vpd_size, buf) < 0) {

		seq_puts(seq, "VPD read failed\n");

		goto out;

	}



	if (buf[0] != VPD_MAGIC) {

		seq_printf(seq, "VPD tag mismatch: %#x\n", buf[0]);

		goto out;

	}

	len = buf[1];

	if (len == 0 || len > vpd_size - 4) {

		seq_printf(seq, "Invalid id length: %d\n", len);

		goto out;

	}



	seq_printf(seq, "%.*s\n", len, buf + 3);

	offs = len + 3;



	while (offs < vpd_size - 4) {

		int i;



		if (!memcmp("RW", buf + offs, 2))	/* end marker */

			break;

		len = buf[offs + 2];

		if (offs + len + 3 >= vpd_size)

			break;



		for (i = 0; i < ARRAY_SIZE(vpd_tags); i++) {

			if (!memcmp(vpd_tags[i].tag, buf + offs, 2)) {

				seq_printf(seq, " %s: %.*s\n",

					   vpd_tags[i].label, len, buf + offs + 3);

				break;

			}

		}

		offs += len + 3;

	}

out:

	kfree(buf);

}



static int sky2_debug_show(struct seq_file *seq, void *v)

{

	struct net_device *dev = seq->private;
@@ -4529,9 +4449,7 @@ static int sky2_debug_show(struct seq_file *seq, void *v) 
	unsigned idx, last;

	int sop;



	sky2_show_vpd(seq, hw);



	seq_printf(seq, "\nIRQ src=%x mask=%x control=%x\n",

	seq_printf(seq, "IRQ src=%x mask=%x control=%x\n",

		   sky2_read32(hw, B0_ISRC),

		   sky2_read32(hw, B0_IMSK),

		   sky2_read32(hw, B0_Y2_SP_ICR));