On 11/18/2013 03:16 PM, Mark Rutland wrote: > >> +static void bcm2835_i2s_stop_clock(struct bcm2835_i2s_dev *dev) >> +{ >> + uint32_t clkreg; > > s/uint32_t/u32/ (and elsewhere...) Please don't do that.