linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] net: mvneta: Fix a typo
@ 2020-04-12 21:20 Christophe JAILLET
  2020-04-12 21:35 ` Joe Perches
  2020-04-13  4:21 ` David Miller
  0 siblings, 2 replies; 7+ messages in thread
From: Christophe JAILLET @ 2020-04-12 21:20 UTC (permalink / raw)
  To: thomas.petazzoni, davem
  Cc: netdev, linux-kernel, kernel-janitors, Christophe JAILLET

s/mvmeta/mvneta/

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
 drivers/net/ethernet/marvell/mvneta.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c
index 5be61f73b6ab..51889770958d 100644
--- a/drivers/net/ethernet/marvell/mvneta.c
+++ b/drivers/net/ethernet/marvell/mvneta.c
@@ -5383,7 +5383,7 @@ static int __init mvneta_driver_init(void)
 {
 	int ret;
 
-	ret = cpuhp_setup_state_multi(CPUHP_AP_ONLINE_DYN, "net/mvmeta:online",
+	ret = cpuhp_setup_state_multi(CPUHP_AP_ONLINE_DYN, "net/mvneta:online",
 				      mvneta_cpu_online,
 				      mvneta_cpu_down_prepare);
 	if (ret < 0)
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-12 21:20 [PATCH] net: mvneta: Fix a typo Christophe JAILLET
@ 2020-04-12 21:35 ` Joe Perches
  2020-04-13  6:56   ` Christophe JAILLET
  2020-04-13  4:21 ` David Miller
  1 sibling, 1 reply; 7+ messages in thread
From: Joe Perches @ 2020-04-12 21:35 UTC (permalink / raw)
  To: Christophe JAILLET, thomas.petazzoni, davem
  Cc: netdev, linux-kernel, kernel-janitors

On Sun, 2020-04-12 at 23:20 +0200, Christophe JAILLET wrote:
> s/mvmeta/mvneta/

nice. how did you find this?

> diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c
[]
> @@ -5383,7 +5383,7 @@ static int __init mvneta_driver_init(void)
>  {
>  	int ret;
>  
> -	ret = cpuhp_setup_state_multi(CPUHP_AP_ONLINE_DYN, "net/mvmeta:online",
> +	ret = cpuhp_setup_state_multi(CPUHP_AP_ONLINE_DYN, "net/mvneta:online",



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-12 21:20 [PATCH] net: mvneta: Fix a typo Christophe JAILLET
  2020-04-12 21:35 ` Joe Perches
@ 2020-04-13  4:21 ` David Miller
  2020-04-13  4:22   ` David Miller
  1 sibling, 1 reply; 7+ messages in thread
From: David Miller @ 2020-04-13  4:21 UTC (permalink / raw)
  To: christophe.jaillet
  Cc: thomas.petazzoni, netdev, linux-kernel, kernel-janitors

From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Date: Sun, 12 Apr 2020 23:20:34 +0200

> s/mvmeta/mvneta/
> 
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>

Applied, but please reply to Jakub's feedback explaining how you
found this.

Thank you.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-13  4:21 ` David Miller
@ 2020-04-13  4:22   ` David Miller
  0 siblings, 0 replies; 7+ messages in thread
From: David Miller @ 2020-04-13  4:22 UTC (permalink / raw)
  To: christophe.jaillet
  Cc: thomas.petazzoni, netdev, linux-kernel, kernel-janitors

From: David Miller <davem@davemloft.net>
Date: Sun, 12 Apr 2020 21:21:01 -0700 (PDT)

> From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> Date: Sun, 12 Apr 2020 23:20:34 +0200
> 
>> s/mvmeta/mvneta/
>> 
>> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> 
> Applied, but please reply to Jakub's feedback explaining how you
> found this.

I meant "Joe's feedback" of course, my bad.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-12 21:35 ` Joe Perches
@ 2020-04-13  6:56   ` Christophe JAILLET
  2020-04-13  7:15     ` Joe Perches
  0 siblings, 1 reply; 7+ messages in thread
From: Christophe JAILLET @ 2020-04-13  6:56 UTC (permalink / raw)
  To: Joe Perches, thomas.petazzoni, davem
  Cc: netdev, linux-kernel, kernel-janitors

Le 12/04/2020 à 23:35, Joe Perches a écrit :
> On Sun, 2020-04-12 at 23:20 +0200, Christophe JAILLET wrote:
>> s/mvmeta/mvneta/
> nice. how did you find this?


Hi,

This is based on a bash script I've made a while ago (see [1])
I've slightly updated it, but the idea is still the same. I search 
strings in a file with some variation on the file name (2 inverted 
chars, 1 missing char or 1 modified char).

The output is horrible, and a lot of filtering should be done.
It is much like noise, with MANY false positives. But I manage to dig 
some interesting stuff out of it.

If interested in the updated script, just ask, but except the concept 
itself, I'm not sure than anything else worth anything and is should be 
rewritten from scratch.

The update includes some tweaks in order to search into Kconfig files 
instead.

CJ

[1]: https://marc.info/?l=kernel-janitors&m=156382201306781&w=4


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-13  6:56   ` Christophe JAILLET
@ 2020-04-13  7:15     ` Joe Perches
  2020-04-13  7:53       ` Christophe JAILLET
  0 siblings, 1 reply; 7+ messages in thread
From: Joe Perches @ 2020-04-13  7:15 UTC (permalink / raw)
  To: Christophe JAILLET, thomas.petazzoni, davem
  Cc: netdev, linux-kernel, kernel-janitors

On Mon, 2020-04-13 at 08:56 +0200, Christophe JAILLET wrote:
> Le 12/04/2020 à 23:35, Joe Perches a écrit :
> > On Sun, 2020-04-12 at 23:20 +0200, Christophe JAILLET wrote:
> > > s/mvmeta/mvneta/
> > nice. how did you find this?
> 
> Hi,
> 
> This is based on a bash script I've made a while ago (see [1])
> I've slightly updated it, but the idea is still the same. I search 
> strings in a file with some variation on the file name (2 inverted 
> chars, 1 missing char or 1 modified char).
> 
> The output is horrible, and a lot of filtering should be done.
> It is much like noise, with MANY false positives. But I manage to dig 
> some interesting stuff out of it.
> 
> If interested in the updated script, just ask, but except the concept 
> itself, I'm not sure than anything else worth anything and is should be 
> rewritten from scratch.
> 
> The update includes some tweaks in order to search into Kconfig files 
> instead.
> 
> CJ
> 
> [1]: https://marc.info/?l=kernel-janitors&m=156382201306781&w=4

Nice.

I was wondering if you used levenshtein distance or something else.

https://en.wikipedia.org/wiki/Levenshtein_distance



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] net: mvneta: Fix a typo
  2020-04-13  7:15     ` Joe Perches
@ 2020-04-13  7:53       ` Christophe JAILLET
  0 siblings, 0 replies; 7+ messages in thread
From: Christophe JAILLET @ 2020-04-13  7:53 UTC (permalink / raw)
  To: Joe Perches, thomas.petazzoni, davem
  Cc: netdev, linux-kernel, kernel-janitors

Le 13/04/2020 à 09:15, Joe Perches a écrit :
> On Mon, 2020-04-13 at 08:56 +0200, Christophe JAILLET wrote:
>> Le 12/04/2020 à 23:35, Joe Perches a écrit :
>>> On Sun, 2020-04-12 at 23:20 +0200, Christophe JAILLET wrote:
>>>> s/mvmeta/mvneta/
>>> nice. how did you find this?
>> Hi,
>>
>> This is based on a bash script I've made a while ago (see [1])
>> I've slightly updated it, but the idea is still the same. I search
>> strings in a file with some variation on the file name (2 inverted
>> chars, 1 missing char or 1 modified char).
>>
>> The output is horrible, and a lot of filtering should be done.
>> It is much like noise, with MANY false positives. But I manage to dig
>> some interesting stuff out of it.
>>
>> If interested in the updated script, just ask, but except the concept
>> itself, I'm not sure than anything else worth anything and is should be
>> rewritten from scratch.
>>
>> The update includes some tweaks in order to search into Kconfig files
>> instead.
>>
>> CJ
>>
>> [1]: https://marc.info/?l=kernel-janitors&m=156382201306781&w=4
> Nice.
>
> I was wondering if you used levenshtein distance or something else.
>
> https://en.wikipedia.org/wiki/Levenshtein_distance
>
>
Well, kind of hand-written version :)

If of any interest:
https://marc.info/?l=linux-driver-devel&m=141798041130581&w=4

I don't remember having played with it myself, but it looks interesting.

CJ


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-04-13  8:43 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-12 21:20 [PATCH] net: mvneta: Fix a typo Christophe JAILLET
2020-04-12 21:35 ` Joe Perches
2020-04-13  6:56   ` Christophe JAILLET
2020-04-13  7:15     ` Joe Perches
2020-04-13  7:53       ` Christophe JAILLET
2020-04-13  4:21 ` David Miller
2020-04-13  4:22   ` David Miller

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).