linux-kernel - [PATCH] scsi: mpt3sas: Again output non-intel branding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <0d7a6df3210f114db1caba414b2fb9c864ee8612.1610475094.git.joe@perches.com>
Date:   Tue, 12 Jan 2021 10:14:07 -0800
From:   Joe Perches <joe@...ches.com>
To:     Sathya Prakash <sathya.prakash@...adcom.com>,
        Sreekanth Reddy <sreekanth.reddy@...adcom.com>,
        Suganath Prabu Subramani 
        <suganath-prabu.subramani@...adcom.com>
Cc:     "James E.J. Bottomley" <jejb@...ux.ibm.com>,
        "Martin K. Petersen" <martin.petersen@...cle.com>,
        MPT-FusionLinux.pdl@...adcom.com, linux-scsi@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: [PATCH] scsi: mpt3sas: Again output non-intel branding

commit 989e43c7ec34 ("mpt3sas: Added OEM Gen2 PnP ID branding names")
consolidated individual functions for OEM branding into a single function
but left a test for intel only at the top so no other vendor branding
was output.  Remove the test.

Signed-off-by: Joe Perches <joe@...ches.com>
---

If no one noticed in the last 5 years, maybe the whole branding output
should be deleted instead...

 drivers/scsi/mpt3sas/mpt3sas_base.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
index 1ffed396d91f..2faea35806b2 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_base.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
@@ -4195,9 +4195,6 @@ _base_display_OEMs_branding(struct MPT3SAS_ADAPTER *ioc)
 	const char *b = NULL;	/* brand */
 	const char *v = NULL;	/* vendor */
 
-	if (ioc->pdev->subsystem_vendor != PCI_VENDOR_ID_INTEL)
-		return;
-
 	switch (ioc->pdev->subsystem_vendor) {
 	case PCI_VENDOR_ID_INTEL:
 		switch (ioc->pdev->device) {
-- 
2.30.0