From: Zong Li <zong.li@sifive.com> To: Palmer Dabbelt <palmer@dabbelt.com> Cc: Paul Walmsley <paul.walmsley@sifive.com>, Stephen Boyd <sboyd@kernel.org>, Andreas Schwab <schwab@linux-m68k.org>, Pragnesh Patel <pragnesh.patel@openfive.com>, Albert Ou <aou@eecs.berkeley.edu>, Michael Turquette <mturquette@baylibre.com>, Yash Shah <yash.shah@sifive.com>, "linux-kernel@vger.kernel.org List" <linux-kernel@vger.kernel.org>, linux-clk@vger.kernel.org, linux-riscv <linux-riscv@lists.infradead.org>, Pragnesh Patel <Pragnesh.patel@sifive.com> Subject: Re: [PATCH v4 2/4] clk: sifive: Use common name for prci configuration Date: Mon, 23 Nov 2020 15:16:39 +0800 [thread overview] Message-ID: <CANXhq0r+TOu_=c+KpNYFu3xzFtfmORoyDR+8mRrXwRCgpGWe8A@mail.gmail.com> (raw) In-Reply-To: <mhng-ca3ea720-b6b1-4b6e-a58f-43fadd7f1c18@palmerdabbelt-glaptop1> On Sat, Nov 21, 2020 at 9:29 AM Palmer Dabbelt <palmer@dabbelt.com> wrote: > > On Wed, 11 Nov 2020 02:06:06 PST (-0800), zong.li@sifive.com wrote: > > Use generic name CLK_SIFIVE_PRCI instead of CLK_SIFIVE_FU540_PRCI. This > > patch is prepared for fu740 support. > > > > Signed-off-by: Zong Li <zong.li@sifive.com> > > Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Reviewed-by: Pragnesh Patel <Pragnesh.patel@sifive.com> > > --- > > arch/riscv/Kconfig.socs | 2 +- > > drivers/clk/sifive/Kconfig | 6 +++--- > > drivers/clk/sifive/Makefile | 2 +- > > 3 files changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/arch/riscv/Kconfig.socs b/arch/riscv/Kconfig.socs > > index 8a55f6156661..3284d5c291be 100644 > > --- a/arch/riscv/Kconfig.socs > > +++ b/arch/riscv/Kconfig.socs > > @@ -5,7 +5,7 @@ config SOC_SIFIVE > > select SERIAL_SIFIVE if TTY > > select SERIAL_SIFIVE_CONSOLE if TTY > > select CLK_SIFIVE > > - select CLK_SIFIVE_FU540_PRCI > > + select CLK_SIFIVE_PRCI > > select SIFIVE_PLIC > > help > > This enables support for SiFive SoC platform hardware. > > diff --git a/drivers/clk/sifive/Kconfig b/drivers/clk/sifive/Kconfig > > index f3b4eb9cb0f5..ab48cf7e0105 100644 > > --- a/drivers/clk/sifive/Kconfig > > +++ b/drivers/clk/sifive/Kconfig > > @@ -8,12 +8,12 @@ menuconfig CLK_SIFIVE > > > > if CLK_SIFIVE > > > > -config CLK_SIFIVE_FU540_PRCI > > - bool "PRCI driver for SiFive FU540 SoCs" > > +config CLK_SIFIVE_PRCI > > + bool "PRCI driver for SiFive SoCs" > > select CLK_ANALOGBITS_WRPLL_CLN28HPC > > help > > Supports the Power Reset Clock interface (PRCI) IP block found in > > - FU540 SoCs. If this kernel is meant to run on a SiFive FU540 SoC, > > + FU540 SoCs. If this kernel is meant to run on a SiFive FU540 SoC, > > This just removes the double-space. Presumably in should also remove the > "FU540", as this clock driver will now function for multiple SiFive SOCs? > I'd like to list the support SoCs here, so in the third patch, I list the FU740 in the description as well. I would remove the SoC names if it is better by using a generic term. What do you think about that? > > enable this driver. > > > > endif > > diff --git a/drivers/clk/sifive/Makefile b/drivers/clk/sifive/Makefile > > index 627effe2ece1..fe3e2cb4c4d8 100644 > > --- a/drivers/clk/sifive/Makefile > > +++ b/drivers/clk/sifive/Makefile > > @@ -1,4 +1,4 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > obj-y += sifive-prci.o > > > > -obj-$(CONFIG_CLK_SIFIVE_FU540_PRCI) += fu540-prci.o > > +obj-$(CONFIG_CLK_SIFIVE_PRCI) += fu540-prci.o > > Probably best to rename the source file as well. I added fu740-prci.c in the third patch, these two files fu740-prci.c and fu540-prci.c hold the soc-dependent code, and sifive-prci.c is the core of this driver.
WARNING: multiple messages have this Message-ID (diff)
From: Zong Li <zong.li@sifive.com> To: Palmer Dabbelt <palmer@dabbelt.com> Cc: Albert Ou <aou@eecs.berkeley.edu>, Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, "linux-kernel@vger.kernel.org List" <linux-kernel@vger.kernel.org>, Pragnesh Patel <pragnesh.patel@openfive.com>, Yash Shah <yash.shah@sifive.com>, Andreas Schwab <schwab@linux-m68k.org>, Paul Walmsley <paul.walmsley@sifive.com>, linux-riscv <linux-riscv@lists.infradead.org>, linux-clk@vger.kernel.org, Pragnesh Patel <Pragnesh.patel@sifive.com> Subject: Re: [PATCH v4 2/4] clk: sifive: Use common name for prci configuration Date: Mon, 23 Nov 2020 15:16:39 +0800 [thread overview] Message-ID: <CANXhq0r+TOu_=c+KpNYFu3xzFtfmORoyDR+8mRrXwRCgpGWe8A@mail.gmail.com> (raw) In-Reply-To: <mhng-ca3ea720-b6b1-4b6e-a58f-43fadd7f1c18@palmerdabbelt-glaptop1> On Sat, Nov 21, 2020 at 9:29 AM Palmer Dabbelt <palmer@dabbelt.com> wrote: > > On Wed, 11 Nov 2020 02:06:06 PST (-0800), zong.li@sifive.com wrote: > > Use generic name CLK_SIFIVE_PRCI instead of CLK_SIFIVE_FU540_PRCI. This > > patch is prepared for fu740 support. > > > > Signed-off-by: Zong Li <zong.li@sifive.com> > > Reviewed-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Acked-by: Palmer Dabbelt <palmerdabbelt@google.com> > > Reviewed-by: Pragnesh Patel <Pragnesh.patel@sifive.com> > > --- > > arch/riscv/Kconfig.socs | 2 +- > > drivers/clk/sifive/Kconfig | 6 +++--- > > drivers/clk/sifive/Makefile | 2 +- > > 3 files changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/arch/riscv/Kconfig.socs b/arch/riscv/Kconfig.socs > > index 8a55f6156661..3284d5c291be 100644 > > --- a/arch/riscv/Kconfig.socs > > +++ b/arch/riscv/Kconfig.socs > > @@ -5,7 +5,7 @@ config SOC_SIFIVE > > select SERIAL_SIFIVE if TTY > > select SERIAL_SIFIVE_CONSOLE if TTY > > select CLK_SIFIVE > > - select CLK_SIFIVE_FU540_PRCI > > + select CLK_SIFIVE_PRCI > > select SIFIVE_PLIC > > help > > This enables support for SiFive SoC platform hardware. > > diff --git a/drivers/clk/sifive/Kconfig b/drivers/clk/sifive/Kconfig > > index f3b4eb9cb0f5..ab48cf7e0105 100644 > > --- a/drivers/clk/sifive/Kconfig > > +++ b/drivers/clk/sifive/Kconfig > > @@ -8,12 +8,12 @@ menuconfig CLK_SIFIVE > > > > if CLK_SIFIVE > > > > -config CLK_SIFIVE_FU540_PRCI > > - bool "PRCI driver for SiFive FU540 SoCs" > > +config CLK_SIFIVE_PRCI > > + bool "PRCI driver for SiFive SoCs" > > select CLK_ANALOGBITS_WRPLL_CLN28HPC > > help > > Supports the Power Reset Clock interface (PRCI) IP block found in > > - FU540 SoCs. If this kernel is meant to run on a SiFive FU540 SoC, > > + FU540 SoCs. If this kernel is meant to run on a SiFive FU540 SoC, > > This just removes the double-space. Presumably in should also remove the > "FU540", as this clock driver will now function for multiple SiFive SOCs? > I'd like to list the support SoCs here, so in the third patch, I list the FU740 in the description as well. I would remove the SoC names if it is better by using a generic term. What do you think about that? > > enable this driver. > > > > endif > > diff --git a/drivers/clk/sifive/Makefile b/drivers/clk/sifive/Makefile > > index 627effe2ece1..fe3e2cb4c4d8 100644 > > --- a/drivers/clk/sifive/Makefile > > +++ b/drivers/clk/sifive/Makefile > > @@ -1,4 +1,4 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > obj-y += sifive-prci.o > > > > -obj-$(CONFIG_CLK_SIFIVE_FU540_PRCI) += fu540-prci.o > > +obj-$(CONFIG_CLK_SIFIVE_PRCI) += fu540-prci.o > > Probably best to rename the source file as well. I added fu740-prci.c in the third patch, these two files fu740-prci.c and fu540-prci.c hold the soc-dependent code, and sifive-prci.c is the core of this driver. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2020-11-23 7:17 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-11 10:06 [PATCH v4 0/4] clk: add driver for the SiFive FU740 Zong Li 2020-11-11 10:06 ` Zong Li 2020-11-11 10:06 ` [PATCH v4 1/4] clk: sifive: Extract prci core to common base Zong Li 2020-11-11 10:06 ` Zong Li 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-24 18:42 ` kernel test robot 2020-11-24 18:42 ` kernel test robot 2020-11-26 3:04 ` Zong Li 2020-11-26 3:04 ` Zong Li 2020-11-11 10:06 ` [PATCH v4 2/4] clk: sifive: Use common name for prci configuration Zong Li 2020-11-11 10:06 ` Zong Li 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-23 7:16 ` Zong Li [this message] 2020-11-23 7:16 ` Zong Li 2020-11-11 10:06 ` [PATCH v4 3/4] clk: sifive: Add a driver for the SiFive FU740 PRCI IP block Zong Li 2020-11-11 10:06 ` Zong Li 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-23 7:21 ` Zong Li 2020-11-23 7:21 ` Zong Li 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-23 7:30 ` Zong Li 2020-11-23 7:30 ` Zong Li 2020-11-11 10:06 ` [PATCH v4 4/4] clk: sifive: Fix the wrong bit field shift Zong Li 2020-11-11 10:06 ` Zong Li 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-21 1:29 ` Palmer Dabbelt 2020-11-23 7:18 ` Zong Li 2020-11-23 7:18 ` Zong Li
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='CANXhq0r+TOu_=c+KpNYFu3xzFtfmORoyDR+8mRrXwRCgpGWe8A@mail.gmail.com' \ --to=zong.li@sifive.com \ --cc=Pragnesh.patel@sifive.com \ --cc=aou@eecs.berkeley.edu \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-riscv@lists.infradead.org \ --cc=mturquette@baylibre.com \ --cc=palmer@dabbelt.com \ --cc=paul.walmsley@sifive.com \ --cc=pragnesh.patel@openfive.com \ --cc=sboyd@kernel.org \ --cc=schwab@linux-m68k.org \ --cc=yash.shah@sifive.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.