2021-03-05 02:06:51

by Bhaskar Chowdhury

[permalink] [raw]
Subject: [PATCH] arch: mips: bcm63xx: Spello fix in the file clk.c



s/revelant/relevant/

Signed-off-by: Bhaskar Chowdhury <[email protected]>
---
arch/mips/bcm63xx/clk.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/bcm63xx/clk.c b/arch/mips/bcm63xx/clk.c
index 164115944a7f..5a3e325275d0 100644
--- a/arch/mips/bcm63xx/clk.c
+++ b/arch/mips/bcm63xx/clk.c
@@ -76,7 +76,7 @@ static struct clk clk_enet_misc = {
};

/*
- * Ethernet MAC clocks: only revelant on 6358, silently enable misc
+ * Ethernet MAC clocks: only relevant on 6358, silently enable misc
* clocks
*/
static void enetx_set(struct clk *clk, int enable)
--
2.30.1


2021-03-06 05:30:52

by Florian Fainelli

[permalink] [raw]
Subject: Re: [PATCH] arch: mips: bcm63xx: Spello fix in the file clk.c



On 3/4/2021 6:05 PM, Bhaskar Chowdhury wrote:
>
>
> s/revelant/relevant/
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>

Acked-by: Florian Fainelli <[email protected]>

Previous changes to this file/directory usually used MIPS: BCM63XX: as a
subject prefix, maybe Thomas can fix it up while applying for consistency.

Thanks
--
Florian

2021-03-08 09:36:23

by Philippe Mathieu-Daudé

[permalink] [raw]
Subject: Re: [PATCH] arch: mips: bcm63xx: Spello fix in the file clk.c

Hi Bhaskar,

On Fri, Mar 5, 2021 at 3:06 AM Bhaskar Chowdhury <[email protected]> wrote:
>
>
>
> s/revelant/relevant/

Let me recommend you this great post explaining why and how you could
improve your patch subject and description:
https://www.freecodecamp.org/news/writing-good-commit-messages-a-practical-guide/#how-to-write-good-commit-messages

Regards,

Phil.

>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> arch/mips/bcm63xx/clk.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/mips/bcm63xx/clk.c b/arch/mips/bcm63xx/clk.c
> index 164115944a7f..5a3e325275d0 100644
> --- a/arch/mips/bcm63xx/clk.c
> +++ b/arch/mips/bcm63xx/clk.c
> @@ -76,7 +76,7 @@ static struct clk clk_enet_misc = {
> };
>
> /*
> - * Ethernet MAC clocks: only revelant on 6358, silently enable misc
> + * Ethernet MAC clocks: only relevant on 6358, silently enable misc
> * clocks
> */
> static void enetx_set(struct clk *clk, int enable)
> --
> 2.30.1
>

2021-03-10 15:20:34

by Thomas Bogendoerfer

[permalink] [raw]
Subject: Re: [PATCH] arch: mips: bcm63xx: Spello fix in the file clk.c

On Fri, Mar 05, 2021 at 07:35:35AM +0530, Bhaskar Chowdhury wrote:
>
>
> s/revelant/relevant/
>
> Signed-off-by: Bhaskar Chowdhury <[email protected]>
> ---
> arch/mips/bcm63xx/clk.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

applied to mips-next with fixed subject.

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]