netdev - [PATCH v1 net 2/3] stmmac: dwmac-loongson: drop useless check for compatible fallback

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <a182921b6d637fc16f2bcc8b8032c99e6a4cbf28.1702289232.git.siyanteng@loongson.cn>
Date: Mon, 11 Dec 2023 18:33:53 +0800
From: Yanteng Si <siyanteng@...ngson.cn>
To: andrew@...n.ch,
	tsbogend@...ha.franken.de
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
	robh+dt@...nel.org,
	krzysztof.kozlowski+dt@...aro.org,
	conor+dt@...nel.org,
	peppe.cavallaro@...com,
	alexandre.torgue@...s.st.com,
	joabreu@...opsys.com,
	davem@...emloft.net,
	edumazet@...gle.com,
	kuba@...nel.org,
	pabeni@...hat.com,
	mcoquelin.stm32@...il.com,
	devicetree@...r.kernel.org,
	linux-mips@...r.kernel.org,
	chenhuacai@...ngson.cn,
	netdev@...r.kernel.org,
	loongarch@...ts.linux.dev,
	chris.chenfeiyang@...il.com,
	Yanteng Si <siyanteng@...ngson.cn>,
	Conor Dooley <conor.dooley@...rochip.com>,
	Jiaxun Yang <jiaxun.yang@...goat.com>
Subject: [PATCH v1 net 2/3] stmmac: dwmac-loongson: drop useless check for compatible fallback

From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>

Device binds to proper PCI ID (LOONGSON, 0x7a03), already listed in DTS,
so checking for some other compatible does not make sense.  It cannot be
bound to unsupported platform.

Drop useless, incorrect (space in between) and undocumented compatible.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Signed-off-by: Yanteng Si <siyanteng@...ngson.cn>
Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
Acked-by: Jiaxun Yang <jiaxun.yang@...goat.com>
---
 drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
index e7701326adc6..9e40c28d453a 100644
--- a/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
+++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-loongson.c
@@ -59,11 +59,6 @@ static int loongson_dwmac_probe(struct pci_dev *pdev, const struct pci_device_id
 		return -ENODEV;
 	}
 
-	if (!of_device_is_compatible(np, "loongson, pci-gmac")) {
-		pr_info("dwmac_loongson_pci: Incompatible OF node\n");
-		return -ENODEV;
-	}
-
 	plat = devm_kzalloc(&pdev->dev, sizeof(*plat), GFP_KERNEL);
 	if (!plat)
 		return -ENOMEM;
-- 
2.31.4