perf/x86/uncore: Remove unnecessary ?: operator around pcibios_err_to_errno() call

ij-intel · Ingo Molnar · commit 2c65477f14a3 · 2023-08-24T21:25:24.000+02:00
If err == 0, pcibios_err_to_errno(err) returns 0 so the ?: construct can be removed. Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Signed-off-by: Ingo Molnar <mingo@kernel.org> Link: https://lore.kernel.org/r/20230824132832.78705-15-ilpo.jarvinen@linux.intel.com
diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/intel/uncore_snbep.c
@@ -1502,7 +1502,7 @@ static int snbep_pci2phy_map_init(int devid, int nodeid_loc, int idmap_loc, bool
 
 	pci_dev_put(ubox_dev);
 
-	return err ? pcibios_err_to_errno(err) : 0;
+	return pcibios_err_to_errno(err);
 }
 
 int snbep_uncore_pci_init(void)

Original file line number	Diff line number	Diff line change
`@@ -1502,7 +1502,7 @@ static int snbep_pci2phy_map_init(int devid, int nodeid_loc, int idmap_loc, bool`
`1502`	`1502`
`1503`	`1503`	`pci_dev_put(ubox_dev);`
`1504`	`1504`
`1505`		`- return err ? pcibios_err_to_errno(err) : 0;`
	`1505`	`+ return pcibios_err_to_errno(err);`
`1506`	`1506`	`}`
`1507`	`1507`
`1508`	`1508`	`int snbep_uncore_pci_init(void)`