pcie_check: fix some pcie_check compilation warnings

1. Fixed warning: suggest parentheses around ‘-’ in operand of ‘&’ [-Wparentheses]: if (width > 0 && width < 17 && (width & width - 1) == 0) 2. Removed unused result, nextpoint. Signed-off-by: Pengfei Xu <[email protected]>
intel · Jul 4, 2024 · fb3ce8a · fb3ce8a
1 parent 4c982cd
commit fb3ce8a
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/BM/tools/pcie/pcie_check.c b/BM/tools/pcie/pcie_check.c
@@ -261,7 +261,7 @@ void speed_show(u8 speed)
 void linkwidth(u8 width)
 {
 	printf("\tLink Capabilities Register(0x0c bit9:4) width:%02x - ", width);
-	if (width > 0 && width < 17 && (width & width - 1) == 0)
+	if (width > 0 && width < 17 && (width & (width - 1)) == 0)
 		printf("x%d\n", width);
 	else
 		printf("reserved\n");
@@ -511,8 +511,7 @@ int scan_pci(void)
 	u32 bus, dev, fun;
 	// Must 32bit for data check!
 	u32 *ptrdata = malloc(sizeof(unsigned long) * 4096);
-	u8 nextpoint;
-	int fd, result;
+	int fd;
 
 	fd = open("/dev/mem", O_RDWR);