2020-07-21 05:41:10

by Hanks Chen

Subject: [PATCH 1/2] dt-bindings: clock: remove UART3 clock support

remove the redundant clk interface of uart.

Fixes: 710774e04861 ("clk: mediatek: Add MT6779 clock support")
Signed-off-by: Hanks Chen <[email protected]>
---
include/dt-bindings/clock/mt6779-clk.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/include/dt-bindings/clock/mt6779-clk.h b/include/dt-bindings/clock/mt6779-clk.h
index b083139afbd2..2b5f2354d7eb 100644
--- a/include/dt-bindings/clock/mt6779-clk.h
+++ b/include/dt-bindings/clock/mt6779-clk.h
@@ -229,7 +229,6 @@
#define CLK_INFRA_UART0 21
#define CLK_INFRA_UART1 22
#define CLK_INFRA_UART2 23
-#define CLK_INFRA_UART3 24
#define CLK_INFRA_GCE_26M 25
#define CLK_INFRA_CQ_DMA_FPC 26
#define CLK_INFRA_BTIF 27
--
2.18.0

2020-07-21 22:11:10

by Matthias Brugger

Subject: Re: [PATCH 1/2] dt-bindings: clock: remove UART3 clock support

On 21/07/2020 07:40, Hanks Chen wrote:
> remove the redundant clk interface of uart.
>
> Fixes: 710774e04861 ("clk: mediatek: Add MT6779 clock support")

These are cosmetic changes. They don't fix any bug, so we won't need a Fixes tag
here. Neither on 2/2.

Regards,
Matthias

> Signed-off-by: Hanks Chen <[email protected]>
> ---
> include/dt-bindings/clock/mt6779-clk.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/include/dt-bindings/clock/mt6779-clk.h b/include/dt-bindings/clock/mt6779-clk.h
> index b083139afbd2..2b5f2354d7eb 100644
> --- a/include/dt-bindings/clock/mt6779-clk.h
> +++ b/include/dt-bindings/clock/mt6779-clk.h
> @@ -229,7 +229,6 @@
> #define CLK_INFRA_UART0 21
> #define CLK_INFRA_UART1 22
> #define CLK_INFRA_UART2 23
> -#define CLK_INFRA_UART3 24
> #define CLK_INFRA_GCE_26M 25
> #define CLK_INFRA_CQ_DMA_FPC 26
> #define CLK_INFRA_BTIF 27
>

2020-07-22 03:12:10

by Hanks Chen

Subject: Re: [PATCH 1/2] dt-bindings: clock: remove UART3 clock support

On Wed, 2020-07-22 at 00:10 +0200, Matthias Brugger wrote:
>
> On 21/07/2020 07:40, Hanks Chen wrote:
> > remove the redundant clk interface of uart.
> >
> > Fixes: 710774e04861 ("clk: mediatek: Add MT6779 clock support")
>
> These are cosmetic changes. They don't fix any bug, so we won't need a Fixes tag
> here. Neither on 2/2.
>
> Regards,
> Matthias
>
Got it, I'll remove it in next version.

Thanks!

Hanks

> > Signed-off-by: Hanks Chen <[email protected]>
> > ---
> > include/dt-bindings/clock/mt6779-clk.h | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/include/dt-bindings/clock/mt6779-clk.h b/include/dt-bindings/clock/mt6779-clk.h
> > index b083139afbd2..2b5f2354d7eb 100644
> > --- a/include/dt-bindings/clock/mt6779-clk.h
> > +++ b/include/dt-bindings/clock/mt6779-clk.h
> > @@ -229,7 +229,6 @@
> > #define CLK_INFRA_UART0 21
> > #define CLK_INFRA_UART1 22
> > #define CLK_INFRA_UART2 23
> > -#define CLK_INFRA_UART3 24
> > #define CLK_INFRA_GCE_26M 25
> > #define CLK_INFRA_CQ_DMA_FPC 26
> > #define CLK_INFRA_BTIF 27
> >