From: Chen-Yu Tsai <wens@csie.org> To: Maxime Ripard <maxime.ripard@free-electrons.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Samuel Ortiz <sameo@linux.intel.com>, Lee Jones <lee.jones@linaro.org>, Rob Herring <robh+dt@kernel.org>, Mike Turquette <mturquette@linaro.org>, Emilio Lopez <emilio@elopez.com.ar>, Linus Walleij <linus.walleij@linaro.org>, linux-serial@vger.kernel.org, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, devicetree <devicetree@vger.kernel.org>, linux-kernel <linux-kernel@vger.kernel.org>, Hans de Goede <hdegoede@redhat.com>, Boris BREZILLON <boris.brezillon@free-electrons.com>, Luc Verhaegen <libv@skynet.be> Subject: Re: [PATCH 17/22] mfd: sun6i-prcm: Add support for Allwinner A23 PRCM Date: Thu, 29 May 2014 12:23:58 +0800 [thread overview] Message-ID: <CAGb2v65v7rqBwFbn1JtQv44nQxepAq-ciwsBdHaELbzWXQNU9g@mail.gmail.com> (raw) In-Reply-To: <20140527083021.GE4730@lukather> On Tue, May 27, 2014 at 4:30 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Mon, May 26, 2014 at 12:36:38PM +0800, Chen-Yu Tsai wrote: >> On Mon, May 26, 2014 at 3:14 AM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> > On Fri, May 23, 2014 at 03:51:20PM +0800, Chen-Yu Tsai wrote: >> >> The Allwinner A23 SoC has a PRCM unit like the previous A31 SoC. >> >> The differences are the AR100 clock can no longer be modified, >> >> and the APB0 clock has different divisors. >> >> >> >> This patch adds a compatible with a modified subdevice list for >> >> the A23. >> >> >> >> Signed-off-by: Chen-Yu Tsai <wens@csie.org> >> >> --- >> >> .../devicetree/bindings/mfd/sun6i-prcm.txt | 2 +- >> >> drivers/mfd/sun6i-prcm.c | 30 ++++++++++++++++++++++ >> >> 2 files changed, 31 insertions(+), 1 deletion(-) >> >> >> >> diff --git a/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt b/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> index 1f5a31f..03c5a55 100644 >> >> --- a/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> +++ b/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> @@ -4,7 +4,7 @@ PRCM is an MFD device exposing several Power Management related devices >> >> (like clks and reset controllers). >> >> >> >> Required properties: >> >> - - compatible: "allwinner,sun6i-a31-prcm" >> >> + - compatible: "allwinner,sun6i-a31-prcm" or "allwinner,sun8i-a23-prcm" >> >> - reg: The PRCM registers range >> >> >> >> The prcm node may contain several subdevices definitions: >> >> diff --git a/drivers/mfd/sun6i-prcm.c b/drivers/mfd/sun6i-prcm.c >> >> index 718fc4d..c96bb6c 100644 >> >> --- a/drivers/mfd/sun6i-prcm.c >> >> +++ b/drivers/mfd/sun6i-prcm.c >> >> @@ -76,16 +76,46 @@ static const struct mfd_cell sun6i_a31_prcm_subdevs[] = { >> >> }, >> >> }; >> >> >> >> +static const struct mfd_cell sun8i_a23_prcm_subdevs[] = { >> >> + { >> >> + .name = "sun6i-a31-apb0-clk", >> >> + .of_compatible = "allwinner,sun8i-a23-apb0-clk", >> >> + .num_resources = ARRAY_SIZE(sun6i_a31_apb0_clk_res), >> >> + .resources = sun6i_a31_apb0_clk_res, >> >> + }, >> >> + { >> >> + .name = "sun6i-a31-apb0-gates-clk", >> >> + .of_compatible = "allwinner,sun6i-a31-apb0-gates-clk", >> >> + .num_resources = ARRAY_SIZE(sun6i_a31_apb0_gates_clk_res), >> >> + .resources = sun6i_a31_apb0_gates_clk_res, >> >> + }, >> > >> > Did I just have a brainfart, or didn't you just introduced an >> > sun8i-a23-apb0-gates-clk driver? >> >> My bad, I missed this one. The one in the DT is correct. >> It actually works, so I'm a bit confused here. >> Which one does the driver core actually use? > > Actually, it will use both, and will match the of_compatible string > try to match it against the mfd-sub-nodes compatibles. If nothing > matches, it won't fill the dev.of_node field. So actually it was supposed to use "allwinner,sun6i-a31-apb0-gates-clk", as that clk driver takes clock-indices, so no need to define gate masks and tie them to some new compatible. There was no sun8i-a23-apb0-gates-clk driver. I must not be getting enough sleep. >> > And there's no ar100 and ahb0 clocks ? >> >> The ahb0 is a fixed-factor (1:1) clock. There are no controls for it. >> The ar100 clock on the A23 has no controls either, so fixed-factor >> clock again. > > Ack. > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com
WARNING: multiple messages have this Message-ID (diff)
From: wens@csie.org (Chen-Yu Tsai) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 17/22] mfd: sun6i-prcm: Add support for Allwinner A23 PRCM Date: Thu, 29 May 2014 12:23:58 +0800 [thread overview] Message-ID: <CAGb2v65v7rqBwFbn1JtQv44nQxepAq-ciwsBdHaELbzWXQNU9g@mail.gmail.com> (raw) In-Reply-To: <20140527083021.GE4730@lukather> On Tue, May 27, 2014 at 4:30 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Mon, May 26, 2014 at 12:36:38PM +0800, Chen-Yu Tsai wrote: >> On Mon, May 26, 2014 at 3:14 AM, Maxime Ripard >> <maxime.ripard@free-electrons.com> wrote: >> > On Fri, May 23, 2014 at 03:51:20PM +0800, Chen-Yu Tsai wrote: >> >> The Allwinner A23 SoC has a PRCM unit like the previous A31 SoC. >> >> The differences are the AR100 clock can no longer be modified, >> >> and the APB0 clock has different divisors. >> >> >> >> This patch adds a compatible with a modified subdevice list for >> >> the A23. >> >> >> >> Signed-off-by: Chen-Yu Tsai <wens@csie.org> >> >> --- >> >> .../devicetree/bindings/mfd/sun6i-prcm.txt | 2 +- >> >> drivers/mfd/sun6i-prcm.c | 30 ++++++++++++++++++++++ >> >> 2 files changed, 31 insertions(+), 1 deletion(-) >> >> >> >> diff --git a/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt b/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> index 1f5a31f..03c5a55 100644 >> >> --- a/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> +++ b/Documentation/devicetree/bindings/mfd/sun6i-prcm.txt >> >> @@ -4,7 +4,7 @@ PRCM is an MFD device exposing several Power Management related devices >> >> (like clks and reset controllers). >> >> >> >> Required properties: >> >> - - compatible: "allwinner,sun6i-a31-prcm" >> >> + - compatible: "allwinner,sun6i-a31-prcm" or "allwinner,sun8i-a23-prcm" >> >> - reg: The PRCM registers range >> >> >> >> The prcm node may contain several subdevices definitions: >> >> diff --git a/drivers/mfd/sun6i-prcm.c b/drivers/mfd/sun6i-prcm.c >> >> index 718fc4d..c96bb6c 100644 >> >> --- a/drivers/mfd/sun6i-prcm.c >> >> +++ b/drivers/mfd/sun6i-prcm.c >> >> @@ -76,16 +76,46 @@ static const struct mfd_cell sun6i_a31_prcm_subdevs[] = { >> >> }, >> >> }; >> >> >> >> +static const struct mfd_cell sun8i_a23_prcm_subdevs[] = { >> >> + { >> >> + .name = "sun6i-a31-apb0-clk", >> >> + .of_compatible = "allwinner,sun8i-a23-apb0-clk", >> >> + .num_resources = ARRAY_SIZE(sun6i_a31_apb0_clk_res), >> >> + .resources = sun6i_a31_apb0_clk_res, >> >> + }, >> >> + { >> >> + .name = "sun6i-a31-apb0-gates-clk", >> >> + .of_compatible = "allwinner,sun6i-a31-apb0-gates-clk", >> >> + .num_resources = ARRAY_SIZE(sun6i_a31_apb0_gates_clk_res), >> >> + .resources = sun6i_a31_apb0_gates_clk_res, >> >> + }, >> > >> > Did I just have a brainfart, or didn't you just introduced an >> > sun8i-a23-apb0-gates-clk driver? >> >> My bad, I missed this one. The one in the DT is correct. >> It actually works, so I'm a bit confused here. >> Which one does the driver core actually use? > > Actually, it will use both, and will match the of_compatible string > try to match it against the mfd-sub-nodes compatibles. If nothing > matches, it won't fill the dev.of_node field. So actually it was supposed to use "allwinner,sun6i-a31-apb0-gates-clk", as that clk driver takes clock-indices, so no need to define gate masks and tie them to some new compatible. There was no sun8i-a23-apb0-gates-clk driver. I must not be getting enough sleep. >> > And there's no ar100 and ahb0 clocks ? >> >> The ahb0 is a fixed-factor (1:1) clock. There are no controls for it. >> The ar100 clock on the A23 has no controls either, so fixed-factor >> clock again. > > Ack. > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com
next prev parent reply other threads:[~2014-05-29 4:24 UTC|newest] Thread overview: 153+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-05-23 7:51 [PATCH 00/21] ARM: sunxi: Introduce Allwinner A23 (sun8i) support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 01/22] serial: 8250_dw: Add optional reset control support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 8:19 ` Arnd Bergmann 2014-05-23 8:19 ` Arnd Bergmann 2014-05-23 7:51 ` [PATCH 02/22] clk: sunxi: register clock gates with clkdev Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:47 ` Maxime Ripard 2014-05-25 18:47 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 03/22] clk: sunxi: add "pll6" to sun6i protected clock list Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:48 ` Maxime Ripard 2014-05-25 18:48 ` Maxime Ripard 2014-05-26 4:47 ` Chen-Yu Tsai 2014-05-26 4:47 ` Chen-Yu Tsai 2014-05-26 4:47 ` Chen-Yu Tsai 2014-05-27 8:32 ` Maxime Ripard 2014-05-27 8:32 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 04/22] clk: sunxi: move "ahb_sdram" to " Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:51 ` Maxime Ripard 2014-05-25 18:51 ` Maxime Ripard 2014-05-26 9:43 ` Chen-Yu Tsai 2014-05-26 9:43 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 05/22] clk: sunxi: Fix gate indexing for sun6i-a31-apb0-gates Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 06/22] clk: sunxi: Support factor clocks with N multiplier factor starting from 1 Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:43 ` Maxime Ripard 2014-05-25 18:43 ` Maxime Ripard 2014-05-25 18:43 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 07/22] clk: sunxi: Fix PLL6 calculation on sun6i Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 13:09 ` Emilio López 2014-05-23 13:09 ` Emilio López 2014-05-23 13:09 ` Emilio López 2014-05-23 14:43 ` Chen-Yu Tsai 2014-05-23 14:43 ` Chen-Yu Tsai 2014-05-23 14:43 ` Chen-Yu Tsai 2014-05-25 18:43 ` Maxime Ripard 2014-05-25 18:43 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 08/22] clk: sunxi: Specify number of child clocks for divs clocks Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 09/22] clk: sunxi: Implement A31 PLL6 as a divs clock for 2x output Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:56 ` Maxime Ripard 2014-05-25 18:56 ` Maxime Ripard 2014-05-25 18:56 ` Maxime Ripard 2014-05-26 3:47 ` Chen-Yu Tsai 2014-05-26 3:47 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 10/22] clk: sunxi: Add support for PLL6 pre-divider on AHB1 clock Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:02 ` Maxime Ripard 2014-05-25 19:02 ` Maxime Ripard 2014-05-25 19:02 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 11/22] ARM: sun6i: DT: Add PLL6 multiple outputs Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:59 ` Maxime Ripard 2014-05-25 18:59 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 12/22] ARM: sun6i: DT: Add PLL6 pre-divider clock for AHB1 mux input Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 13/22] clk: sunxi: Add A23 clocks support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:05 ` Maxime Ripard 2014-05-25 19:05 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 14/22] clk: sunxi: Add A23 APB0 support to sun6i-a31-apb0-clk Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 15/22] pinctrl: sunxi: Add A23 PIO controller support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:08 ` Maxime Ripard 2014-05-25 19:08 ` Maxime Ripard 2014-06-17 10:25 ` Chen-Yu Tsai 2014-06-17 10:25 ` Chen-Yu Tsai 2014-06-17 10:25 ` Chen-Yu Tsai 2014-06-17 14:18 ` Maxime Ripard 2014-06-17 14:18 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 16/22] pinctrl: sunxi: Add A23 R_PIO " Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:11 ` Maxime Ripard 2014-05-25 19:11 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 17/22] mfd: sun6i-prcm: Add support for Allwinner A23 PRCM Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:14 ` Maxime Ripard 2014-05-25 19:14 ` Maxime Ripard 2014-05-26 4:36 ` Chen-Yu Tsai 2014-05-26 4:36 ` Chen-Yu Tsai 2014-05-27 8:30 ` Maxime Ripard 2014-05-27 8:30 ` Maxime Ripard 2014-05-29 4:23 ` Chen-Yu Tsai [this message] 2014-05-29 4:23 ` Chen-Yu Tsai 2014-05-29 19:31 ` Maxime Ripard 2014-05-29 19:31 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 18/22] ARM: sunxi: Introduce Allwinner A23 support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:22 ` Maxime Ripard 2014-05-25 19:22 ` Maxime Ripard 2014-05-25 19:22 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 19/22] ARM: sunxi: Add earlyprintk support using R_UART (sun6i/sun8i) Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 18:46 ` Maxime Ripard 2014-05-25 18:46 ` Maxime Ripard 2014-05-26 9:25 ` Chen-Yu Tsai 2014-05-26 9:25 ` Chen-Yu Tsai 2014-05-27 8:34 ` Maxime Ripard 2014-05-27 8:34 ` Maxime Ripard 2014-05-23 7:51 ` [PATCH 20/22] ARM: sun8i: Add SMP support for the Allwinner A23 Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:26 ` Maxime Ripard 2014-05-25 19:26 ` Maxime Ripard 2014-05-26 3:57 ` Chen-Yu Tsai 2014-05-26 3:57 ` Chen-Yu Tsai 2014-05-27 8:09 ` Marc Zyngier 2014-05-27 8:09 ` Marc Zyngier 2014-05-27 8:09 ` Marc Zyngier 2014-05-23 7:51 ` [PATCH 21/22] ARM: sunxi: Add Allwinner A23 dtsi Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:38 ` Maxime Ripard 2014-05-25 19:38 ` Maxime Ripard 2014-05-26 4:02 ` Chen-Yu Tsai 2014-05-26 4:02 ` Chen-Yu Tsai 2014-05-23 7:51 ` [PATCH 22/22] ARM: sun8i: dt: Add Ippo-q8h v5 support Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-23 7:51 ` Chen-Yu Tsai 2014-05-25 19:39 ` Maxime Ripard 2014-05-25 19:39 ` Maxime Ripard 2014-05-26 4:23 ` Chen-Yu Tsai 2014-05-26 4:23 ` Chen-Yu Tsai 2014-05-26 4:23 ` Chen-Yu Tsai 2014-05-27 8:22 ` Maxime Ripard 2014-05-27 8:22 ` Maxime Ripard
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAGb2v65v7rqBwFbn1JtQv44nQxepAq-ciwsBdHaELbzWXQNU9g@mail.gmail.com \ --to=wens@csie.org \ --cc=boris.brezillon@free-electrons.com \ --cc=devicetree@vger.kernel.org \ --cc=emilio@elopez.com.ar \ --cc=gregkh@linuxfoundation.org \ --cc=hdegoede@redhat.com \ --cc=lee.jones@linaro.org \ --cc=libv@skynet.be \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-serial@vger.kernel.org \ --cc=maxime.ripard@free-electrons.com \ --cc=mturquette@linaro.org \ --cc=robh+dt@kernel.org \ --cc=sameo@linux.intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.