On 10/05/10 09:32, Hennerich, Michael wrote: > Jonathan Cameron wrote on 2010-10-04: >> ... >>>>> Hi Jonathan, >>>>> >>>>> We have a few more iio drivers in our repository. Over the next >>>>> couple of weeks I'm trying to get them out. Excellent. >>>>> >>>>>> because I would like it to go in before the event clean up series >>>>>> currently sat in my tree. That way I can update this one at the >>>>>> same time. >>>>> >>>>> Oops - this patch was already against your tree: >>>>> http://git.kernel.org/?p=linux/kernel/git/jic23/iio_temp.git >>>>> >>>>> Need to check if it works on Greg's staging. >>>> Ah. Shouldn't be too bad. That's what I get for putting temporary >>>> trees up - that one only exists for testing the adis16260 and >>>> adis16350 driver changes. >>> >>> Well - it looks bad. It basically errors on everything I changed in >>> order to make it work on your temp tree. I'll send Greg my original >>> version... >> Are you sure this was against staging-next branch of the staging-next >> tree? (which is where Greg will merge this) Far as I can see all the >> changes since that in my temp tree are to do with the event code >> cleanups and I don't think that would generate most of these. The >> index an type changes + the rearrangement of the buffer code has been >> merged for a while. >> >> >> It applies and builds as is. git's am checking throws up >> /home/jic23/src/kernel/staging-next-2.6/.git/rebase-apply/patch:610: >> trailing whitespace. >> i & 0x1 ? >> and sparse gives drivers/staging/iio/adc/ad799x_core.c: In function >> 'ad799x_interrupt_bh': drivers/staging/iio/adc/ad799x_core.c:391: >> warning: suggest parentheses around + or - inside shift >> drivers/staging/iio/adc/ad799x_core.c:392: warning: right shift count >>> = width of type >> drivers/staging/iio/adc/ad799x_core.c:392: warning: suggest >> parentheses around + or - inside shift >> >> both of which are probably worth fixing. > > Hmm - I don't see this warning. > What version of sparse are you using - and do you pass any extra arguments to sparse? > > CHECK drivers/staging/iio/adc/ad799x_core.c > CC drivers/staging/iio/adc/ad799x_core.o > LD drivers/staging/iio/adc/ad799x.o Hi Michael, Whilst testing the light sensor I discovered that this warning isn't form sparse, its from gcc itself. Sorry for the false info. I'm using a 4.3.3 arm cross compiler. Jonathan -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html