* [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'
@ 2018-01-05 10:32 Sumit Pundir
2018-01-05 10:44 ` Shyam Saini
0 siblings, 1 reply; 3+ messages in thread
From: Sumit Pundir @ 2018-01-05 10:32 UTC (permalink / raw)
To: samuel; +Cc: gregkh, netdev, devel, linux-kernel
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@gmail.com>
---
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'
2018-01-05 10:32 [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned' Sumit Pundir
@ 2018-01-05 10:44 ` Shyam Saini
2018-01-05 11:10 ` Sumit Pundir
0 siblings, 1 reply; 3+ messages in thread
From: Shyam Saini @ 2018-01-05 10:44 UTC (permalink / raw)
To: Sumit Pundir; +Cc: samuel, Greg KH, netdev, devel, linux-kernel
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@gmail.com>
> ---
> 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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned'
2018-01-05 10:44 ` Shyam Saini
@ 2018-01-05 11:10 ` Sumit Pundir
0 siblings, 0 replies; 3+ messages in thread
From: Sumit Pundir @ 2018-01-05 11:10 UTC (permalink / raw)
To: Shyam Saini; +Cc: samuel, Greg KH, netdev, devel, linux-kernel
On Fri, Jan 5, 2018 at 4:14 PM, Shyam Saini <mayhs11saini@gmail.com> 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@gmail.com> 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@gmail.com>
>> ---
>> 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
>>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-01-05 11:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-05 10:32 [PATCH] Staging: irda: Prefer 'unsigned int' to bare use of 'unsigned' Sumit Pundir
2018-01-05 10:44 ` Shyam Saini
2018-01-05 11:10 ` Sumit Pundir
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®