On Fri, Jan 5, 2018 at 4:14 PM, Shyam Saini <mayhs11saini@xxxxxxxxx> wrote: > Hi, > > Did you read this? > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/staging/irda/TODO > > > Thanks, > Shyam Hi Shyam, Thanks for the update. Totally forgot to read it. Thanks, Sumit On Fri, Jan 5, 2018 at 4:14 PM, Shyam Saini <mayhs11saini@xxxxxxxxx> wrote: > 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
