* [PATCH] ieee802154: remove a redundant local variable 'i'
@ 2018-09-19 14:41 zhong jiang
2018-09-27 14:47 ` Stefan Schmidt
0 siblings, 1 reply; 3+ messages in thread
From: zhong jiang @ 2018-09-19 14:41 UTC (permalink / raw)
To: davem; +Cc: stefan, alex.aring, netdev, linux-kernel
The local variable 'i' is never used after being assigned.
hence it should be redundant adn can be removed.
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
---
net/ieee802154/nl802154.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/net/ieee802154/nl802154.c b/net/ieee802154/nl802154.c
index 99f6c25..5b90151 100644
--- a/net/ieee802154/nl802154.c
+++ b/net/ieee802154/nl802154.c
@@ -445,7 +445,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
{
struct nlattr *nl_cmds;
void *hdr;
- int i;
hdr = nl802154hdr_put(msg, portid, seq, flags, cmd);
if (!hdr)
@@ -508,7 +507,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
if (!nl_cmds)
goto nla_put_failure;
- i = 0;
#define CMD(op, n) \
do { \
if (rdev->ops->op) { \
--
1.7.12.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] ieee802154: remove a redundant local variable 'i'
2018-09-19 14:41 [PATCH] ieee802154: remove a redundant local variable 'i' zhong jiang
@ 2018-09-27 14:47 ` Stefan Schmidt
2018-09-28 6:37 ` zhong jiang
0 siblings, 1 reply; 3+ messages in thread
From: Stefan Schmidt @ 2018-09-27 14:47 UTC (permalink / raw)
To: zhong jiang, davem; +Cc: alex.aring, netdev, linux-kernel
Hello.
On 19/09/2018 16:41, zhong jiang wrote:
> The local variable 'i' is never used after being assigned.
> hence it should be redundant adn can be removed.
>
> Signed-off-by: zhong jiang <zhongjiang@huawei.com>
> ---
> net/ieee802154/nl802154.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/net/ieee802154/nl802154.c b/net/ieee802154/nl802154.c
> index 99f6c25..5b90151 100644
> --- a/net/ieee802154/nl802154.c
> +++ b/net/ieee802154/nl802154.c
> @@ -445,7 +445,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
> {
> struct nlattr *nl_cmds;
> void *hdr;
> - int i;
>
> hdr = nl802154hdr_put(msg, portid, seq, flags, cmd);
> if (!hdr)
> @@ -508,7 +507,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
> if (!nl_cmds)
> goto nla_put_failure;
>
> - i = 0;
> #define CMD(op, n) \
> do { \
> if (rdev->ops->op) { \
>
Sorry, but this patch is wrong. The variable i is used in line 513
inside the CMD() macro. The compiler clearly tells this when running
with your patch:
net/ieee802154/nl802154.c: In function ‘nl802154_send_wpan_phy’:
net/ieee802154/nl802154.c:513:4: error: ‘i’ undeclared (first use in
this function)
I would appreciate if patches sent out would at least be compile tested.
regards
Stefan Schmidt
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] ieee802154: remove a redundant local variable 'i'
2018-09-27 14:47 ` Stefan Schmidt
@ 2018-09-28 6:37 ` zhong jiang
0 siblings, 0 replies; 3+ messages in thread
From: zhong jiang @ 2018-09-28 6:37 UTC (permalink / raw)
To: Stefan Schmidt; +Cc: davem, alex.aring, netdev, linux-kernel
On 2018/9/27 22:47, Stefan Schmidt wrote:
> Hello.
>
> On 19/09/2018 16:41, zhong jiang wrote:
>> The local variable 'i' is never used after being assigned.
>> hence it should be redundant adn can be removed.
>>
>> Signed-off-by: zhong jiang <zhongjiang@huawei.com>
>> ---
>> net/ieee802154/nl802154.c | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/net/ieee802154/nl802154.c b/net/ieee802154/nl802154.c
>> index 99f6c25..5b90151 100644
>> --- a/net/ieee802154/nl802154.c
>> +++ b/net/ieee802154/nl802154.c
>> @@ -445,7 +445,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
>> {
>> struct nlattr *nl_cmds;
>> void *hdr;
>> - int i;
>>
>> hdr = nl802154hdr_put(msg, portid, seq, flags, cmd);
>> if (!hdr)
>> @@ -508,7 +507,6 @@ static int nl802154_send_wpan_phy(struct cfg802154_registered_device *rdev,
>> if (!nl_cmds)
>> goto nla_put_failure;
>>
>> - i = 0;
>> #define CMD(op, n) \
>> do { \
>> if (rdev->ops->op) { \
>>
> Sorry, but this patch is wrong. The variable i is used in line 513
> inside the CMD() macro. The compiler clearly tells this when running
> with your patch:
>
> net/ieee802154/nl802154.c: In function ‘nl802154_send_wpan_phy’:
> net/ieee802154/nl802154.c:513:4: error: ‘i’ undeclared (first use in
> this function)
>
> I would appreciate if patches sent out would at least be compile tested.
Sorry, I really double check and compile test.
Thanks,
zhong jiang
> regards
> Stefan Schmidt
>
> .
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-09-28 6:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-09-19 14:41 [PATCH] ieee802154: remove a redundant local variable 'i' zhong jiang
2018-09-27 14:47 ` Stefan Schmidt
2018-09-28 6:37 ` zhong jiang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).