Messages in this thread | | | From | Alexander Kapshuk <> | Date | Tue, 14 Mar 2017 13:32:46 +0200 | Subject | Re: [PATCH] rtl8188eu: Correcting macro ROUND macro defination |
| |
On Tue, Mar 14, 2017 at 1:26 PM, Pushkar Jambhlekar <pushkar.iit@gmail.com> wrote: > Description: > There should not be ';' after do ... while(0) in macro defination > > Signed-off-by: Pushkar Jambhlekar <pushkar.iit@gmail.com> > --- > drivers/staging/rtl8188eu/core/rtw_security.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/rtl8188eu/core/rtw_security.c b/drivers/staging/rtl8188eu/core/rtw_security.c > index b283a490..5b1ef22 100644 > --- a/drivers/staging/rtl8188eu/core/rtw_security.c > +++ b/drivers/staging/rtl8188eu/core/rtw_security.c > @@ -1690,4 +1690,4 @@ do { \ > d##1 = TE0(s##1) ^ TE1(s##2) ^ TE2(s##3) ^ TE3(s##0) ^ rk[4 * i + 1]; \ > d##2 = TE0(s##2) ^ TE1(s##3) ^ TE2(s##0) ^ TE3(s##1) ^ rk[4 * i + 2]; \ > d##3 = TE0(s##3) ^ TE1(s##0) ^ TE2(s##1) ^ TE3(s##2) ^ rk[4 * i + 3]; \ > -} while (0); > +} while (0) > -- > 2.7.4 >
A semicolon is required after 'while(bool);' in 'do..while'. Without it, you get a compile time error.
| |