Hi, Did you read this? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/staging/irda/TODO Thanks, Shyam > This patch fixes the following checkpatch.pl issue at multiple lines: > > WARNING: Prefer 'unsigned int' to bare use of 'unsigned' > > Signed-off-by: Sumit Pundir <pundirsumit11@xxxxxxxxx> > --- > drivers/staging/irda/drivers/act200l-sir.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/irda/drivers/act200l-sir.c b/drivers/staging/irda/drivers/act200l-sir.c > index e891751..62a02e5 100644 > --- a/drivers/staging/irda/drivers/act200l-sir.c > +++ b/drivers/staging/irda/drivers/act200l-sir.c > @@ -29,7 +29,7 @@ > static int act200l_reset(struct sir_dev *dev); > static int act200l_open(struct sir_dev *dev); > static int act200l_close(struct sir_dev *dev); > -static int act200l_change_speed(struct sir_dev *dev, unsigned speed); > +static int act200l_change_speed(struct sir_dev *dev, unsigned int speed); > > /* Regsiter 0: Control register #1 */ > #define ACT200L_REG0 0x00 > @@ -134,7 +134,7 @@ static int act200l_close(struct sir_dev *dev) > * Set the speed for the ACTiSYS ACT-IR200L type dongle. > * > */ > -static int act200l_change_speed(struct sir_dev *dev, unsigned speed) > +static int act200l_change_speed(struct sir_dev *dev, unsigned int speed) > { > u8 control[3]; > int ret = 0; > @@ -191,8 +191,8 @@ static int act200l_change_speed(struct sir_dev *dev, unsigned speed) > > static int act200l_reset(struct sir_dev *dev) > { > - unsigned state = dev->fsm.substate; > - unsigned delay = 0; > + unsigned int state = dev->fsm.substate; > + unsigned int delay = 0; > static const u8 control[9] = { > ACT200L_REG15, > ACT200L_REG13 | ACT200L_SHDW, > -- > 2.7.4 > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel