2019-10-18 23:38:14

by Chandra Annamaneni

Subject: [PATCH 2/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

Resolved: ERROR: else should follow close brace '}'

Signed-off-by: Chandra Annamaneni <[email protected]>
---
Previous versions of these patches were not split into different
patches, did not have different patch numbers and did not have the
keyword staging. The previous version of this patch had the wrong
description and subject.
drivers/staging/kpc2000/kpc2000_spi.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c
index 5712a88c8788..929136cdc3e1 100644
--- a/drivers/staging/kpc2000/kpc2000_spi.c
+++ b/drivers/staging/kpc2000/kpc2000_spi.c
@@ -226,8 +226,7 @@ kp_spi_txrx_pio(struct spi_device *spidev, struct spi_transfer *transfer)
kp_spi_write_reg(cs, KP_SPI_REG_TXDATA, val);
processed++;
}
- }
- else if (rx) {
+ } else if (rx) {
for (i = 0 ; i < c ; i++) {
char test = 0;

--
2.20.1

2019-10-25 19:20:14

by Greg Kroah-Hartman

Subject: Re: [PATCH 2/4] staging: KPC2000: kpc2000_spi.c: Fix style issues (misaligned brace)

On Fri, Oct 18, 2019 at 12:09:48AM -0700, Chandra Annamaneni wrote:
> Resolved: ERROR: else should follow close brace '}'
>
> Signed-off-by: Chandra Annamaneni <[email protected]>
> ---
> Previous versions of these patches were not split into different
> patches, did not have different patch numbers and did not have the
> keyword staging. The previous version of this patch had the wrong
> description and subject.

Ok, but I only see 1 patch here.

Can you resend the whole series properly, with a "v2" showing what
changed?

thanks,

greg k-h