On Tue, 2017-04-18 at 09:35 +0200, Johannes Thumshirn wrote: > On Mon, Apr 17, 2017 at 04:10:03PM -0700, Bart Van Assche wrote: > > The global variable 'rd_size' is declared as 'int' in source file > > arch/arm/kernel/atags_parse.c and as 'unsigned long' in > > drivers/block/brd.c. Fix this inconsistency. > > > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > > Cc: Russell King <linux@xxxxxxxxxxxxxxx> > > Cc: Jens Axboe <axboe@xxxxxxxxx> > > Cc: Jan Kara <jack@xxxxxxx> > > Cc: <yanaijie@xxxxxxxxxx> > > Cc: <zhaohongjiang@xxxxxxxxxx> > > Cc: <miaoxie@xxxxxxxxxx> > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > Cc: linux-block@xxxxxxxxxxxxxxx > > --- > > diff --git a/include/linux/brd.h b/include/linux/brd.h > > new file mode 100644 > > index 000000000000..dbb0f92fefc8 > > --- /dev/null > > +++ b/include/linux/brd.h > > @@ -0,0 +1 @@ > > +extern unsigned long rd_size; > > Small nit, can you add an include guard here as well? Hello Johannes, Thanks for the review. But are you aware that with the current content an include guard is overkill because it is safe to evaluate the "extern unsigned long rd_size" declaration multiple times? Bart.
