[PATCH] PCI/AER: fix boolreturn.cocci warnings

[PATCH] PCI/AER: fix boolreturn.cocci warnings

[kbuild test robot]

From: kbuild test robot <fengguang.wu@intel.com>

drivers/pci/pcie/aer.c:349:9-10: WARNING: return of 0/1 in function 'aer_acpi_firmware_first' with return type bool

 Return statements in functions returning bool should use
 true/false instead of 1/0.
Generated by: scripts/coccinelle/misc/boolreturn.cocci

Fixes: 69c7f453d545 ("PCI/AER: Honor "pcie_ports=native" even if HEST sets FIRMWARE_FIRST")
CC: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Signed-off-by: kbuild test robot <fengguang.wu@intel.com>
---

 aer.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/pci/pcie/aer.c
+++ b/drivers/pci/pcie/aer.c
@@ -346,7 +346,7 @@ bool aer_acpi_firmware_first(void)
 	};
 
 	if (pcie_ports_native)
-		return 0;
+		return false;
 
 	if (!parsed) {
 		apei_hest_parse(aer_hest_parse, &info);

[pci:pci/aer 7/14] drivers/pci/pcie/aer.c:349:9-10: WARNING: return of 0/1 in function 'aer_acpi_firmware_first' with return type bool

[kbuild test robot]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git pci/aer
head:   8138a743c2b056d86d0643670765da11fa9bed17
commit: 69c7f453d545930c8cc60478c27be5e06581ba4b [7/14] PCI/AER: Honor "pcie_ports=native" even if HEST sets FIRMWARE_FIRST


coccinelle warnings: (new ones prefixed by >>)

>> drivers/pci/pcie/aer.c:349:9-10: WARNING: return of 0/1 in function 'aer_acpi_firmware_first' with return type bool

Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Re: [PATCH] PCI/AER: fix boolreturn.cocci warnings

[Bjorn Helgaas]

On Fri, Jul 20, 2018 at 07:54:17AM +0800, kbuild test robot wrote:
> From: kbuild test robot <fengguang.wu@intel.com>
> 
> drivers/pci/pcie/aer.c:349:9-10: WARNING: return of 0/1 in function 'aer_acpi_firmware_first' with return type bool
> 
>  Return statements in functions returning bool should use
>  true/false instead of 1/0.
> Generated by: scripts/coccinelle/misc/boolreturn.cocci
> 
> Fixes: 69c7f453d545 ("PCI/AER: Honor "pcie_ports=native" even if HEST sets FIRMWARE_FIRST")
> CC: Alexandru Gagniuc <mr.nuke.me@gmail.com>
> Signed-off-by: kbuild test robot <fengguang.wu@intel.com>

I folded this into the original patch.

> ---
> 
>  aer.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- a/drivers/pci/pcie/aer.c
> +++ b/drivers/pci/pcie/aer.c
> @@ -346,7 +346,7 @@ bool aer_acpi_firmware_first(void)
>  	};
>  
>  	if (pcie_ports_native)
> -		return 0;
> +		return false;
>  
>  	if (!parsed) {
>  		apei_hest_parse(aer_hest_parse, &info);