diff mbox series

[1/2] clk: stm32: END_PRIMARY_CLK should be declare after CLK_SYSCLK

Message ID 1516286980-8603-2-git-send-email-gabriel.fernandez@st.com
State Not Applicable, archived
Headers show
Series [1/2] clk: stm32: END_PRIMARY_CLK should be declare after CLK_SYSCLK | expand

Commit Message

Gabriel Fernandez Jan. 18, 2018, 2:49 p.m. UTC 
From: Gabriel Fernandez <gabriel.fernandez@st.com>

Update of END_PRIMARY_CLK was missed, it should be after CLK_SYSCLK
hsi and sysclk are overwritten by gpioa and gpiob.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@st.com>
---
 include/dt-bindings/clock/stm32fx-clock.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Philippe CORNU Jan. 18, 2018, 4:37 p.m. UTC | #1 
Hi Gabriel,

Tested successfully on f469 disco board.

Tested-by: Philippe Cornu <philippe.cornu@st.com>


Many thanks,
Philippe :-)

On 01/18/2018 03:49 PM, gabriel.fernandez@st.com wrote:
> From: Gabriel Fernandez <gabriel.fernandez@st.com>

> 

> Update of END_PRIMARY_CLK was missed, it should be after CLK_SYSCLK

> hsi and sysclk are overwritten by gpioa and gpiob.

> 

> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@st.com>

> ---

>   include/dt-bindings/clock/stm32fx-clock.h | 6 +++---

>   1 file changed, 3 insertions(+), 3 deletions(-)

> 

> diff --git a/include/dt-bindings/clock/stm32fx-clock.h b/include/dt-bindings/clock/stm32fx-clock.h

> index 49bb3c2..4d523b0 100644

> --- a/include/dt-bindings/clock/stm32fx-clock.h

> +++ b/include/dt-bindings/clock/stm32fx-clock.h

> @@ -33,11 +33,11 @@

>   #define CLK_SAI2		11

>   #define CLK_I2SQ_PDIV		12

>   #define CLK_SAIQ_PDIV		13

> -

> -#define END_PRIMARY_CLK		14

> -

>   #define CLK_HSI			14

>   #define CLK_SYSCLK		15

> +

> +#define END_PRIMARY_CLK		16

> +

>   #define CLK_HDMI_CEC		16

>   #define CLK_SPDIF		17

>   #define CLK_USART1		18

>
Rob Herring Jan. 29, 2018, 6:52 p.m. UTC | #2 
On Thu, Jan 18, 2018 at 03:49:39PM +0100, gabriel.fernandez@st.com wrote:
> From: Gabriel Fernandez <gabriel.fernandez@st.com>
> 
> Update of END_PRIMARY_CLK was missed, it should be after CLK_SYSCLK
> hsi and sysclk are overwritten by gpioa and gpiob.
> 
> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@st.com>
> ---
>  include/dt-bindings/clock/stm32fx-clock.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/include/dt-bindings/clock/stm32fx-clock.h b/include/dt-bindings/clock/stm32fx-clock.h
index 49bb3c2..4d523b0 100644
--- a/include/dt-bindings/clock/stm32fx-clock.h
+++ b/include/dt-bindings/clock/stm32fx-clock.h
@@ -33,11 +33,11 @@ 
 #define CLK_SAI2		11
 #define CLK_I2SQ_PDIV		12
 #define CLK_SAIQ_PDIV		13
-
-#define END_PRIMARY_CLK		14
-
 #define CLK_HSI			14
 #define CLK_SYSCLK		15
+
+#define END_PRIMARY_CLK		16
+
 #define CLK_HDMI_CEC		16
 #define CLK_SPDIF		17
 #define CLK_USART1		18