* [PATCH 0/3] clk: meson: drop unnecessary AO clock flags
@ 2018-03-26 8:18 Yixun Lan
2018-03-26 8:18 ` [PATCH 1/3] clk: meson: drop CLK_SET_RATE_PARENT flag Yixun Lan
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Yixun Lan @ 2018-03-26 8:18 UTC (permalink / raw)
To: linus-amlogic
This first two patches will drop unnecessary flags for
the AO clock driver.
The third DTS patch is a follow-up fix for the second patch,
and I've tested them on P200, P212, Q200, S400 boards.
Note: this patch series was made on top of patch [0]
[0] https://lkml.kernel.org/r/20180323143816.200573-1-yixun.lan at amlogic.com
Yixun Lan (3):
clk: meson: drop CLK_SET_RATE_PARENT flag
clk: meson: drop CLK_IGNORE_UNUSED flag
ARM64: dts: meson: fix clock source of the pclk for UART_AO
arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++--
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++--
arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++--
drivers/clk/meson/axg-aoclk.c | 1 -
drivers/clk/meson/gxbb-aoclk.c | 1 -
5 files changed, 6 insertions(+), 8 deletions(-)
--
2.15.1
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH 1/3] clk: meson: drop CLK_SET_RATE_PARENT flag 2018-03-26 8:18 [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Yixun Lan @ 2018-03-26 8:18 ` Yixun Lan 2018-03-26 8:18 ` [PATCH 2/3] clk: meson: drop CLK_IGNORE_UNUSED flag Yixun Lan ` (2 subsequent siblings) 3 siblings, 0 replies; 8+ messages in thread From: Yixun Lan @ 2018-03-26 8:18 UTC (permalink / raw) To: linus-amlogic The clk81 is not expected to be changed, so drop this flag. Signed-off-by: Yixun Lan <yixun.lan@amlogic.com> --- drivers/clk/meson/gxbb-aoclk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c index 0f089cbce594..e7bc04d3e24f 100644 --- a/drivers/clk/meson/gxbb-aoclk.c +++ b/drivers/clk/meson/gxbb-aoclk.c @@ -70,7 +70,7 @@ static struct clk_regmap _name##_ao = { \ .ops = &clk_regmap_gate_ops, \ .parent_names = (const char *[]){ "clk81" }, \ .num_parents = 1, \ - .flags = (CLK_SET_RATE_PARENT | CLK_IGNORE_UNUSED), \ + .flags = CLK_IGNORE_UNUSED, \ }, \ } -- 2.15.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/3] clk: meson: drop CLK_IGNORE_UNUSED flag 2018-03-26 8:18 [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Yixun Lan 2018-03-26 8:18 ` [PATCH 1/3] clk: meson: drop CLK_SET_RATE_PARENT flag Yixun Lan @ 2018-03-26 8:18 ` Yixun Lan 2018-03-26 8:18 ` [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO Yixun Lan 2018-03-27 9:21 ` [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Jerome Brunet 3 siblings, 0 replies; 8+ messages in thread From: Yixun Lan @ 2018-03-26 8:18 UTC (permalink / raw) To: linus-amlogic Rely on drivers to request the clock explicitly. Previous the kernel will leave the clock on while bootloader adready initilized the clock, this wasn't optimal way, so fix it here. Signed-off-by: Yixun Lan <yixun.lan@amlogic.com> --- drivers/clk/meson/axg-aoclk.c | 1 - drivers/clk/meson/gxbb-aoclk.c | 1 - 2 files changed, 2 deletions(-) diff --git a/drivers/clk/meson/axg-aoclk.c b/drivers/clk/meson/axg-aoclk.c index 121ecb45264f..026b2fe56a29 100644 --- a/drivers/clk/meson/axg-aoclk.c +++ b/drivers/clk/meson/axg-aoclk.c @@ -27,7 +27,6 @@ static struct clk_regmap _name##_ao = { \ .ops = &clk_regmap_gate_ops, \ .parent_names = (const char *[]){ "clk81" }, \ .num_parents = 1, \ - .flags = CLK_IGNORE_UNUSED, \ }, \ } diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c index e7bc04d3e24f..005d84ff590d 100644 --- a/drivers/clk/meson/gxbb-aoclk.c +++ b/drivers/clk/meson/gxbb-aoclk.c @@ -70,7 +70,6 @@ static struct clk_regmap _name##_ao = { \ .ops = &clk_regmap_gate_ops, \ .parent_names = (const char *[]){ "clk81" }, \ .num_parents = 1, \ - .flags = CLK_IGNORE_UNUSED, \ }, \ } -- 2.15.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO 2018-03-26 8:18 [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Yixun Lan 2018-03-26 8:18 ` [PATCH 1/3] clk: meson: drop CLK_SET_RATE_PARENT flag Yixun Lan 2018-03-26 8:18 ` [PATCH 2/3] clk: meson: drop CLK_IGNORE_UNUSED flag Yixun Lan @ 2018-03-26 8:18 ` Yixun Lan 2018-03-27 9:20 ` Jerome Brunet 2018-03-27 9:21 ` [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Jerome Brunet 3 siblings, 1 reply; 8+ messages in thread From: Yixun Lan @ 2018-03-26 8:18 UTC (permalink / raw) To: linus-amlogic >From the hardware perspective, the clk81 is the parent of the UART_AO clocks. So it make more sense to mark the CLKID_AO_UART1/2 as the pclk. Previous we had flag=CLK_IGNORE_UNUSED in AO clock driver. This will make kernel leave clock as it is while bootloader set the clock on. Signed-off-by: Yixun Lan <yixun.lan@amlogic.com> --- arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++-- arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++-- arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi index 48584d5a329b..e538e2448bb5 100644 --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi @@ -533,7 +533,7 @@ compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; reg = <0x0 0x3000 0x0 0x18>; interrupts = <GIC_SPI 193 IRQ_TYPE_EDGE_RISING>; - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; clock-names = "xtal", "pclk", "baud"; status = "disabled"; }; @@ -542,7 +542,7 @@ compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; reg = <0x0 0x4000 0x0 0x18>; interrupts = <GIC_SPI 197 IRQ_TYPE_EDGE_RISING>; - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; clock-names = "xtal", "pclk", "baud"; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi index 3290a4dc3522..a458ea50c2ef 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi @@ -783,12 +783,12 @@ }; &uart_AO { - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; clock-names = "xtal", "pclk", "baud"; }; &uart_AO_B { - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; clock-names = "xtal", "pclk", "baud"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi index 4f355f17eed6..474f4f9630e4 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi @@ -733,12 +733,12 @@ }; &uart_AO { - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; clock-names = "xtal", "pclk", "baud"; }; &uart_AO_B { - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; clock-names = "xtal", "pclk", "baud"; }; -- 2.15.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO 2018-03-26 8:18 ` [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO Yixun Lan @ 2018-03-27 9:20 ` Jerome Brunet 2018-03-27 12:35 ` Yixun Lan 0 siblings, 1 reply; 8+ messages in thread From: Jerome Brunet @ 2018-03-27 9:20 UTC (permalink / raw) To: linus-amlogic On Mon, 2018-03-26 at 16:18 +0800, Yixun Lan wrote: > From the hardware perspective, the clk81 is the parent of > the UART_AO clocks. So it make more sense to mark > the CLKID_AO_UART1/2 as the pclk. > > Previous we had flag=CLK_IGNORE_UNUSED in AO clock driver. > This will make kernel leave clock as it is while bootloader > set the clock on. > Hi Yixun, I'm not sure I understand your description correctly so I'll phrase. Please tell me if I misunderstood something The actual pclk of the AO uarts is the corresponding clkc_ao uart gate, not the main clock controller clk81. This was not problem so far, because the uart_gate had the CLK_IGNORE_UNUSED flag, which kept the gate open. In another patch, you removed this flag, so you need to give the correct clock in DT to make things work again ? right ? If I got it correctly, please rephrase the description a bit. Also you need to be careful in which order patches get merged. 1. You need the clkc_ao in place, exporting CLKID_AO_UARTx with CLK_IGNORE_UNUSED. 2. This patch can be merged (still with CLK_IGNORE_UNUSED) 3. Once the DT patch have reached mainline, you should a patch to remove to CLK_IGNORE_UNUSED flag This way, we avoid breaking bisect, with revision where CLKID_AO_UARTx would get disabled by the CCF clk_disable_unused() mechanism while the UART driver does not claim the appropriate clock. > Signed-off-by: Yixun Lan <yixun.lan@amlogic.com> > --- > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++-- > arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++-- > arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++-- > 3 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi > index 48584d5a329b..e538e2448bb5 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi > @@ -533,7 +533,7 @@ > compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; > reg = <0x0 0x3000 0x0 0x18>; > interrupts = <GIC_SPI 193 IRQ_TYPE_EDGE_RISING>; > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > status = "disabled"; > }; > @@ -542,7 +542,7 @@ > compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; > reg = <0x0 0x4000 0x0 0x18>; > interrupts = <GIC_SPI 197 IRQ_TYPE_EDGE_RISING>; > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > status = "disabled"; > }; > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > index 3290a4dc3522..a458ea50c2ef 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi > @@ -783,12 +783,12 @@ > }; > > &uart_AO { > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > }; > > &uart_AO_B { > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi > index 4f355f17eed6..474f4f9630e4 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi > @@ -733,12 +733,12 @@ > }; > > &uart_AO { > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > }; > > &uart_AO_B { > - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; > + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; > clock-names = "xtal", "pclk", "baud"; > }; > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO 2018-03-27 9:20 ` Jerome Brunet @ 2018-03-27 12:35 ` Yixun Lan 0 siblings, 0 replies; 8+ messages in thread From: Yixun Lan @ 2018-03-27 12:35 UTC (permalink / raw) To: linus-amlogic On 03/27/2018 05:20 PM, Jerome Brunet wrote: > On Mon, 2018-03-26 at 16:18 +0800, Yixun Lan wrote: >> From the hardware perspective, the clk81 is the parent of >> the UART_AO clocks. So it make more sense to mark >> the CLKID_AO_UART1/2 as the pclk. >> >> Previous we had flag=CLK_IGNORE_UNUSED in AO clock driver. >> This will make kernel leave clock as it is while bootloader >> set the clock on. >> > > Hi Yixun, > > I'm not sure I understand your description correctly so I'll phrase. Please tell > me if I misunderstood something > > The actual pclk of the AO uarts is the corresponding clkc_ao uart gate, not the > main clock controller clk81. This was not problem so far, because the uart_gate > had the CLK_IGNORE_UNUSED flag, which kept the gate open. > Yes, exactly > In another patch, you removed this flag, so you need to give the correct clock > in DT to make things work again ? right ? > Yes, right. > If I got it correctly, please rephrase the description a bit. > Also you need to be careful in which order patches get merged. > > 1. You need the clkc_ao in place, exporting CLKID_AO_UARTx with > CLK_IGNORE_UNUSED. > > 2. This patch can be merged (still with CLK_IGNORE_UNUSED) > > 3. Once the DT patch have reached mainline, you should a patch to remove to > CLK_IGNORE_UNUSED flag > > This way, we avoid breaking bisect, with revision where CLKID_AO_UARTx would get > disabled by the CCF clk_disable_unused() mechanism while the UART driver does > not claim the appropriate clock. > I wasn't sure about what's the best approach I just gave a warning about that patch 2 (removing CLK_IGNORE_UNUSED) and patch 3 (fix the DTS) should be merged together and your suggestion sounds good, I will adjust > >> Signed-off-by: Yixun Lan <yixun.lan@amlogic.com> >> --- >> arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++-- >> arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++-- >> arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++-- >> 3 files changed, 6 insertions(+), 6 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi >> index 48584d5a329b..e538e2448bb5 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-axg.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-axg.dtsi >> @@ -533,7 +533,7 @@ >> compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; >> reg = <0x0 0x3000 0x0 0x18>; >> interrupts = <GIC_SPI 193 IRQ_TYPE_EDGE_RISING>; >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> status = "disabled"; >> }; >> @@ -542,7 +542,7 @@ >> compatible = "amlogic,meson-gx-uart", "amlogic,meson-ao-uart"; >> reg = <0x0 0x4000 0x0 0x18>; >> interrupts = <GIC_SPI 197 IRQ_TYPE_EDGE_RISING>; >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> status = "disabled"; >> }; >> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi >> index 3290a4dc3522..a458ea50c2ef 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi >> @@ -783,12 +783,12 @@ >> }; >> >> &uart_AO { >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> }; >> >> &uart_AO_B { >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> }; >> >> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi >> index 4f355f17eed6..474f4f9630e4 100644 >> --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi >> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi >> @@ -733,12 +733,12 @@ >> }; >> >> &uart_AO { >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART1>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> }; >> >> &uart_AO_B { >> - clocks = <&xtal>, <&clkc CLKID_CLK81>, <&xtal>; >> + clocks = <&xtal>, <&clkc_AO CLKID_AO_UART2>, <&xtal>; >> clock-names = "xtal", "pclk", "baud"; >> }; >> > > . > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 0/3] clk: meson: drop unnecessary AO clock flags 2018-03-26 8:18 [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Yixun Lan ` (2 preceding siblings ...) 2018-03-26 8:18 ` [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO Yixun Lan @ 2018-03-27 9:21 ` Jerome Brunet 2018-03-27 12:42 ` Yixun Lan 3 siblings, 1 reply; 8+ messages in thread From: Jerome Brunet @ 2018-03-27 9:21 UTC (permalink / raw) To: linus-amlogic On Mon, 2018-03-26 at 16:18 +0800, Yixun Lan wrote: > This first two patches will drop unnecessary flags for > the AO clock driver. > The third DTS patch is a follow-up fix for the second patch, > and I've tested them on P200, P212, Q200, S400 boards. > > Note: this patch series was made on top of patch [0] > > [0] https://lkml.kernel.org/r/20180323143816.200573-1-yixun.lan at amlogic.com > > Yixun Lan (3): > clk: meson: drop CLK_SET_RATE_PARENT flag > clk: meson: drop CLK_IGNORE_UNUSED flag > ARM64: dts: meson: fix clock source of the pclk for UART_AO Please add patch 1 and 2 to the next version of your clkc_ao series Patch 3 should sent independently to Kevin, with note warning about the different dependencies (see comment in patch 3) > > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++-- > arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++-- > arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++-- > drivers/clk/meson/axg-aoclk.c | 1 - > drivers/clk/meson/gxbb-aoclk.c | 1 - > 5 files changed, 6 insertions(+), 8 deletions(-) > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 0/3] clk: meson: drop unnecessary AO clock flags 2018-03-27 9:21 ` [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Jerome Brunet @ 2018-03-27 12:42 ` Yixun Lan 0 siblings, 0 replies; 8+ messages in thread From: Yixun Lan @ 2018-03-27 12:42 UTC (permalink / raw) To: linus-amlogic HI Jerome On 03/27/2018 05:21 PM, Jerome Brunet wrote: > On Mon, 2018-03-26 at 16:18 +0800, Yixun Lan wrote: >> This first two patches will drop unnecessary flags for >> the AO clock driver. >> The third DTS patch is a follow-up fix for the second patch, >> and I've tested them on P200, P212, Q200, S400 boards. >> >> Note: this patch series was made on top of patch [0] >> >> [0] https://lkml.kernel.org/r/20180323143816.200573-1-yixun.lan at amlogic.com >> >> Yixun Lan (3): >> clk: meson: drop CLK_SET_RATE_PARENT flag >> clk: meson: drop CLK_IGNORE_UNUSED flag >> ARM64: dts: meson: fix clock source of the pclk for UART_AO > > Please add patch 1 and 2 to the next version of your clkc_ao series > I was thinking about patch 1 & 2 as independent patch of the clkc_ao series.. anyway, as your suggestion in another thread, I could include the patch 1 into next clkc_ao series send patch 3 (DTS part) independently to Kevin, once patch 3 accepted into mainline, I will then send patch 2 as a fix (this will avoid breaking bisect) > Patch 3 should sent independently to Kevin, with note warning about the > different dependencies (see comment in patch 3) > >> >> arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 4 ++-- >> arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 4 ++-- >> arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 4 ++-- >> drivers/clk/meson/axg-aoclk.c | 1 - >> drivers/clk/meson/gxbb-aoclk.c | 1 - >> 5 files changed, 6 insertions(+), 8 deletions(-) >> > > . > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-03-27 12:42 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-03-26 8:18 [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Yixun Lan 2018-03-26 8:18 ` [PATCH 1/3] clk: meson: drop CLK_SET_RATE_PARENT flag Yixun Lan 2018-03-26 8:18 ` [PATCH 2/3] clk: meson: drop CLK_IGNORE_UNUSED flag Yixun Lan 2018-03-26 8:18 ` [PATCH 3/3] ARM64: dts: meson: fix clock source of the pclk for UART_AO Yixun Lan 2018-03-27 9:20 ` Jerome Brunet 2018-03-27 12:35 ` Yixun Lan 2018-03-27 9:21 ` [PATCH 0/3] clk: meson: drop unnecessary AO clock flags Jerome Brunet 2018-03-27 12:42 ` Yixun Lan
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).