diff mbox

[01/14] ieee802154: adf7242: use unsigned int over only unsigned

Message ID 20170922121405.31789-2-stefan@osg.samsung.com
State Accepted
Headers show

Commit Message

Stefan Schmidt Sept. 22, 2017, 12:13 p.m. UTC
Bring it in line with the rest of the ieee802154 drivers.
Found by checkpatch.

Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
---
 drivers/net/ieee802154/adf7242.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Hennerich, Michael Nov. 6, 2017, 3:37 p.m. UTC | #1
On 22.09.2017 14:13, Stefan Schmidt wrote:
> Bring it in line with the rest of the ieee802154 drivers.
> Found by checkpatch.
> 
> Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>

Acked-by: Michael Hennerich <michael.hennerich@analog.com>

> ---
>   drivers/net/ieee802154/adf7242.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/ieee802154/adf7242.c b/drivers/net/ieee802154/adf7242.c
> index 3e4c8b21403c..400fdbd3a120 100644
> --- a/drivers/net/ieee802154/adf7242.c
> +++ b/drivers/net/ieee802154/adf7242.c
> @@ -311,8 +311,8 @@ static int adf7242_status(struct adf7242_local *lp, u8 *stat)
>   	return status;
>   }
>   
> -static int adf7242_wait_status(struct adf7242_local *lp, unsigned status,
> -			       unsigned mask, int line)
> +static int adf7242_wait_status(struct adf7242_local *lp, unsigned int status,
> +			       unsigned int mask, int line)
>   {
>   	int cnt = 0, ret = 0;
>   	u8 stat;
> @@ -477,7 +477,7 @@ static int adf7242_write_reg(struct adf7242_local *lp, u16 addr, u8 data)
>   	return status;
>   }
>   
> -static int adf7242_cmd(struct adf7242_local *lp, unsigned cmd)
> +static int adf7242_cmd(struct adf7242_local *lp, unsigned int cmd)
>   {
>   	int status;
>   
> @@ -920,7 +920,7 @@ static void adf7242_debug(u8 irq1)
>   static irqreturn_t adf7242_isr(int irq, void *data)
>   {
>   	struct adf7242_local *lp = data;
> -	unsigned xmit;
> +	unsigned int xmit;
>   	u8 irq1;
>   
>   	adf7242_wait_status(lp, RC_STATUS_PHY_RDY, RC_STATUS_MASK, __LINE__);
>
Stefan Schmidt Nov. 6, 2017, 3:39 p.m. UTC | #2
Hello.

On 06.11.2017 16:37, Michael Hennerich wrote:
> On 22.09.2017 14:13, Stefan Schmidt wrote:
>> Bring it in line with the rest of the ieee802154 drivers.
>> Found by checkpatch.
>>
>> Signed-off-by: Stefan Schmidt <stefan@osg.samsung.com>
> 
> Acked-by: Michael Hennerich <michael.hennerich@analog.com>
> 
>> ---
>>   drivers/net/ieee802154/adf7242.c | 8 ++++----
>>   1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/net/ieee802154/adf7242.c
>> b/drivers/net/ieee802154/adf7242.c
>> index 3e4c8b21403c..400fdbd3a120 100644
>> --- a/drivers/net/ieee802154/adf7242.c
>> +++ b/drivers/net/ieee802154/adf7242.c
>> @@ -311,8 +311,8 @@ static int adf7242_status(struct adf7242_local
>> *lp, u8 *stat)
>>       return status;
>>   }
>>   -static int adf7242_wait_status(struct adf7242_local *lp, unsigned
>> status,
>> -                   unsigned mask, int line)
>> +static int adf7242_wait_status(struct adf7242_local *lp, unsigned int
>> status,
>> +                   unsigned int mask, int line)
>>   {
>>       int cnt = 0, ret = 0;
>>       u8 stat;
>> @@ -477,7 +477,7 @@ static int adf7242_write_reg(struct adf7242_local
>> *lp, u16 addr, u8 data)
>>       return status;
>>   }
>>   -static int adf7242_cmd(struct adf7242_local *lp, unsigned cmd)
>> +static int adf7242_cmd(struct adf7242_local *lp, unsigned int cmd)
>>   {
>>       int status;
>>   @@ -920,7 +920,7 @@ static void adf7242_debug(u8 irq1)
>>   static irqreturn_t adf7242_isr(int irq, void *data)
>>   {
>>       struct adf7242_local *lp = data;
>> -    unsigned xmit;
>> +    unsigned int xmit;
>>       u8 irq1;
>>         adf7242_wait_status(lp, RC_STATUS_PHY_RDY, RC_STATUS_MASK,
>> __LINE__);
>>
> 
> 

This patch has been applied to the wpan-next tree and will be
part of the next pull request to net-next. Thanks!

regards
Stefan Schmidt
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/ieee802154/adf7242.c b/drivers/net/ieee802154/adf7242.c
index 3e4c8b21403c..400fdbd3a120 100644
--- a/drivers/net/ieee802154/adf7242.c
+++ b/drivers/net/ieee802154/adf7242.c
@@ -311,8 +311,8 @@  static int adf7242_status(struct adf7242_local *lp, u8 *stat)
 	return status;
 }
 
-static int adf7242_wait_status(struct adf7242_local *lp, unsigned status,
-			       unsigned mask, int line)
+static int adf7242_wait_status(struct adf7242_local *lp, unsigned int status,
+			       unsigned int mask, int line)
 {
 	int cnt = 0, ret = 0;
 	u8 stat;
@@ -477,7 +477,7 @@  static int adf7242_write_reg(struct adf7242_local *lp, u16 addr, u8 data)
 	return status;
 }
 
-static int adf7242_cmd(struct adf7242_local *lp, unsigned cmd)
+static int adf7242_cmd(struct adf7242_local *lp, unsigned int cmd)
 {
 	int status;
 
@@ -920,7 +920,7 @@  static void adf7242_debug(u8 irq1)
 static irqreturn_t adf7242_isr(int irq, void *data)
 {
 	struct adf7242_local *lp = data;
-	unsigned xmit;
+	unsigned int xmit;
 	u8 irq1;
 
 	adf7242_wait_status(lp, RC_STATUS_PHY_RDY, RC_STATUS_MASK, __LINE__);