2023-07-31 14:35:20

by Yue Haibing

[permalink] [raw]
Subject: [PATCH] mac80211: mesh: Remove unused function declaration mesh_ids_set_default()

Commit ccf80ddfe492 ("mac80211: mesh function and data structures definitions")
introducted this but never implemented.

Signed-off-by: Yue Haibing <[email protected]>
---
net/mac80211/mesh.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/net/mac80211/mesh.h b/net/mac80211/mesh.h
index 6c94222a9df5..ad8469293d71 100644
--- a/net/mac80211/mesh.h
+++ b/net/mac80211/mesh.h
@@ -212,7 +212,6 @@ int mesh_rmc_check(struct ieee80211_sub_if_data *sdata,
const u8 *addr, struct ieee80211s_hdr *mesh_hdr);
bool mesh_matches_local(struct ieee80211_sub_if_data *sdata,
struct ieee802_11_elems *ie);
-void mesh_ids_set_default(struct ieee80211_if_mesh *mesh);
int mesh_add_meshconf_ie(struct ieee80211_sub_if_data *sdata,
struct sk_buff *skb);
int mesh_add_meshid_ie(struct ieee80211_sub_if_data *sdata,
--
2.34.1



2023-08-01 10:51:44

by Simon Horman

[permalink] [raw]
Subject: Re: [PATCH] mac80211: mesh: Remove unused function declaration mesh_ids_set_default()

On Mon, Jul 31, 2023 at 10:07:12PM +0800, Yue Haibing wrote:
> Commit ccf80ddfe492 ("mac80211: mesh function and data structures definitions")
> introducted this but never implemented.
>
> Signed-off-by: Yue Haibing <[email protected]>

Reviewed-by: Simon Horman <[email protected]>


2023-08-08 19:10:03

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH] mac80211: mesh: Remove unused function declaration mesh_ids_set_default()

On Mon, 2023-07-31 at 22:07 +0800, Yue Haibing wrote:
> Commit ccf80ddfe492 ("mac80211: mesh function and data structures definitions")
> introducted this but never implemented.
>

Btw, are you detecting these with some kind of tool? Having the tool
would probably be more useful than you sending all these patches all the
time ...

johannes

2023-08-08 19:26:27

by Yue Haibing

[permalink] [raw]
Subject: Re: [PATCH] mac80211: mesh: Remove unused function declaration mesh_ids_set_default()

On 2023/8/8 16:24, Johannes Berg wrote:
> On Mon, 2023-07-31 at 22:07 +0800, Yue Haibing wrote:
>> Commit ccf80ddfe492 ("mac80211: mesh function and data structures definitions")
>> introducted this but never implemented.
>>
>
> Btw, are you detecting these with some kind of tool? Having the tool
> would probably be more useful than you sending all these patches all the
> time ...
>
Just use grep to check like below

grep -rP --include=*.c '^(?:\w+\s+){1,3}\*{0,3}\s{0,3}\w+[\t]*\(' * |grep -P -oh '\w+\s*\('|sort| uniq -c| sort -n| grep -P '^\s+1\b'|sed -r -e 's/^\s+1\s+//' -e 's/\(//'|while read line ;
do
IS=`git grep -w $line |wc -l` ;
if [ $IS -eq 1 ];then
git grep -wn $line
fi
done


> johannes
> .
>