RE: [PATCH] PCI: altera: Adjust indentation in altera_pcie_valid_device

[Tan, Ley Foon]

> -----Original Message-----
> From: Nathan Chancellor <natechancellor@xxxxxxxxx>
> Sent: Wednesday, December 18, 2019 9:28 AM
> To: Ley Foon Tan <lftan@xxxxxxxxxx>; Lorenzo Pieralisi
> <lorenzo.pieralisi@xxxxxxx>; Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> Cc: Andrew Murray <andrew.murray@xxxxxxx>; rfi@xxxxxxxxxxxxxxxxxxxxxx;
> linux-pci@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; clang-built-
> linux@xxxxxxxxxxxxxxxx; Nathan Chancellor <natechancellor@xxxxxxxxx>
> Subject: [PATCH] PCI: altera: Adjust indentation in altera_pcie_valid_device
> 
> Clang warns:
> 
> ../drivers/pci/controller/pcie-altera.c:196:3: warning: misleading indentation;
> statement is not part of the previous 'if'
> [-Wmisleading-indentation]
>          return true;
>          ^
> ../drivers/pci/controller/pcie-altera.c:193:2: note: previous statement is here
>         if (bus->number == pcie->root_bus_nr && dev > 0)
>         ^
> 1 warning generated.
> 
> This warning occurs because there is a space after the tab on this line.
> Remove it so that the indentation is consistent with the Linux kernel coding
> style and clang no longer warns.
> 
> Fixes: eaa6111b70a7 ("PCI: altera: Add Altera PCIe host controller driver")
> Link: https://github.com/ClangBuiltLinux/linux/issues/815
> Signed-off-by: Nathan Chancellor <natechancellor@xxxxxxxxx>

Reviewed-by: Ley Foon Tan <ley.foon.tan@xxxxxxxxx>

Thanks.

Regards
Ley Foon

> ---
>  drivers/pci/controller/pcie-altera.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pci/controller/pcie-altera.c b/drivers/pci/controller/pcie-
> altera.c
> index b447c3e4abad..24cb1c331058 100644
> --- a/drivers/pci/controller/pcie-altera.c
> +++ b/drivers/pci/controller/pcie-altera.c
> @@ -193,7 +193,7 @@ static bool altera_pcie_valid_device(struct
> altera_pcie *pcie,
>         if (bus->number == pcie->root_bus_nr && dev > 0)
>                 return false;
> 
> -        return true;
> +       return true;
>  }
> 
>  static int tlp_read_packet(struct altera_pcie *pcie, u32 *value)
> --
> 2.24.1
> 
> 
> ________________________________
> 
> Confidentiality Notice.
> This message may contain information that is confidential or otherwise
> protected from disclosure. If you are not the intended recipient, you are
> hereby notified that any use, disclosure, dissemination, distribution, or
> copying of this message, or any attachments, is strictly prohibited. If you
> have received this message in error, please advise the sender by reply e-mail,
> and delete the message and any attachments. Thank you.