Re: [PATCH] [TRIVIAL] Remove braces in drivers/scsi/in2000.h to fix build for clang. No functional change otherwise.

2013-08-29 Thread Jan-Simon Möller
Ping. @Christoph do you take care of removing then ? Or can we merge this patch as-is now. Best, JS On Wednesday 21 August 2013 15:11:33 Jan-Simon Möller wrote: > On Wednesday 21 August 2013 09:26:14 Jan-Simon Möller wrote: > > > > Description: > > > > Clang c

Re: [PATCH] [TRIVIAL] Remove braces in drivers/scsi/in2000.h to fix build for clang. No functional change otherwise.

2013-08-21 Thread Jan-Simon Möller
On Wednesday 21 August 2013 09:26:14 Jan-Simon Möller wrote: > > > Description: > > > Clang chokes on the notation "insw (%%dx)" but works for "insw %%dx" > > > (outsw likewise); GNU as accepts both forms. > > > > > > Signe

Re: [PATCH] [TRIVIAL] Remove braces in drivers/scsi/in2000.h to fix build for clang. No functional change otherwise.

2013-08-21 Thread Jan-Simon Möller
> > > > Description: > > Clang chokes on the notation "insw (%%dx)" but works for "insw %%dx" > > (outsw likewise); GNU as accepts both forms. > > > > Signed-off-by: Jan-Simon M??ller > > I think the right fix is to simply kill the FAST_READ_IO/FAST_WRITE_IO > defines and their associated i