All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom
@ 2021-07-22 19:34 Andy Shevchenko
  2021-07-23 13:40 ` Rafael J. Wysocki
  0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2021-07-22 19:34 UTC (permalink / raw)
  To: Rafael J. Wysocki, Andy Shevchenko, linux-acpi, linux-kernel, linux-clk
  Cc: Rafael J. Wysocki, Len Brown, Michael Turquette, Stephen Boyd

The LPT stands for Lynxpoint PCH. However the driver is used on a few
Intel Atom SoCs. Rename it to reflect this in a way how another clock
driver, i.e. clk-pmc-atom, is called.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---

Good to go either via ACPI or CCF tree.

 drivers/acpi/acpi_lpss.c                       |  6 ++++--
 drivers/clk/x86/Makefile                       |  2 +-
 drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} | 12 ++++++------
 include/linux/platform_data/x86/clk-lpss.h     |  2 +-
 4 files changed, 12 insertions(+), 10 deletions(-)
 rename drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} (76%)

diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
index 894b7e6ae144..7f163074e4e4 100644
--- a/drivers/acpi/acpi_lpss.c
+++ b/drivers/acpi/acpi_lpss.c
@@ -385,7 +385,9 @@ static struct platform_device *lpss_clk_dev;
 
 static inline void lpt_register_clock_device(void)
 {
-	lpss_clk_dev = platform_device_register_simple("clk-lpt", -1, NULL, 0);
+	lpss_clk_dev = platform_device_register_simple("clk-lpss-atom",
+						       PLATFORM_DEVID_NONE,
+						       NULL, 0);
 }
 
 static int register_device_clock(struct acpi_device *adev,
@@ -1337,7 +1339,7 @@ void __init acpi_lpss_init(void)
 	const struct x86_cpu_id *id;
 	int ret;
 
-	ret = lpt_clk_init();
+	ret = lpss_atom_clk_init();
 	if (ret)
 		return;
 
diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
index 18564efdc651..1244c4e568ff 100644
--- a/drivers/clk/x86/Makefile
+++ b/drivers/clk/x86/Makefile
@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 obj-$(CONFIG_PMC_ATOM)		+= clk-pmc-atom.o
 obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE)	+= clk-fch.o
-clk-x86-lpss-objs		:= clk-lpt.o
+clk-x86-lpss-y			:= clk-lpss-atom.o
 obj-$(CONFIG_X86_INTEL_LPSS)	+= clk-x86-lpss.o
 obj-$(CONFIG_CLK_LGM_CGU)	+= clk-cgu.o clk-cgu-pll.o clk-lgm.o
diff --git a/drivers/clk/x86/clk-lpt.c b/drivers/clk/x86/clk-lpss-atom.c
similarity index 76%
rename from drivers/clk/x86/clk-lpt.c
rename to drivers/clk/x86/clk-lpss-atom.c
index fbe9fd3ed948..aa9d0bb98f8b 100644
--- a/drivers/clk/x86/clk-lpt.c
+++ b/drivers/clk/x86/clk-lpss-atom.c
@@ -13,7 +13,7 @@
 #include <linux/platform_data/x86/clk-lpss.h>
 #include <linux/platform_device.h>
 
-static int lpt_clk_probe(struct platform_device *pdev)
+static int lpss_atom_clk_probe(struct platform_device *pdev)
 {
 	struct lpss_clk_data *drvdata;
 	struct clk *clk;
@@ -34,14 +34,14 @@ static int lpt_clk_probe(struct platform_device *pdev)
 	return 0;
 }
 
-static struct platform_driver lpt_clk_driver = {
+static struct platform_driver lpss_atom_clk_driver = {
 	.driver = {
-		.name = "clk-lpt",
+		.name = "clk-lpss-atom",
 	},
-	.probe = lpt_clk_probe,
+	.probe = lpss_atom_clk_probe,
 };
 
-int __init lpt_clk_init(void)
+int __init lpss_atom_clk_init(void)
 {
-	return platform_driver_register(&lpt_clk_driver);
+	return platform_driver_register(&lpss_atom_clk_driver);
 }
diff --git a/include/linux/platform_data/x86/clk-lpss.h b/include/linux/platform_data/x86/clk-lpss.h
index 207e1a317800..41df326583f9 100644
--- a/include/linux/platform_data/x86/clk-lpss.h
+++ b/include/linux/platform_data/x86/clk-lpss.h
@@ -15,6 +15,6 @@ struct lpss_clk_data {
 	struct clk *clk;
 };
 
-extern int lpt_clk_init(void);
+extern int lpss_atom_clk_init(void);
 
 #endif /* __CLK_LPSS_H */
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom
  2021-07-22 19:34 [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom Andy Shevchenko
@ 2021-07-23 13:40 ` Rafael J. Wysocki
  2021-08-04 16:03   ` Rafael J. Wysocki
  0 siblings, 1 reply; 4+ messages in thread
From: Rafael J. Wysocki @ 2021-07-23 13:40 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Rafael J. Wysocki, ACPI Devel Maling List,
	Linux Kernel Mailing List, linux-clk, Rafael J. Wysocki,
	Len Brown, Michael Turquette, Stephen Boyd

On Thu, Jul 22, 2021 at 9:34 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> The LPT stands for Lynxpoint PCH. However the driver is used on a few
> Intel Atom SoCs. Rename it to reflect this in a way how another clock
> driver, i.e. clk-pmc-atom, is called.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>
> Good to go either via ACPI or CCF tree.

In case you want the latter:

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

>  drivers/acpi/acpi_lpss.c                       |  6 ++++--
>  drivers/clk/x86/Makefile                       |  2 +-
>  drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} | 12 ++++++------
>  include/linux/platform_data/x86/clk-lpss.h     |  2 +-
>  4 files changed, 12 insertions(+), 10 deletions(-)
>  rename drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} (76%)
>
> diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> index 894b7e6ae144..7f163074e4e4 100644
> --- a/drivers/acpi/acpi_lpss.c
> +++ b/drivers/acpi/acpi_lpss.c
> @@ -385,7 +385,9 @@ static struct platform_device *lpss_clk_dev;
>
>  static inline void lpt_register_clock_device(void)
>  {
> -       lpss_clk_dev = platform_device_register_simple("clk-lpt", -1, NULL, 0);
> +       lpss_clk_dev = platform_device_register_simple("clk-lpss-atom",
> +                                                      PLATFORM_DEVID_NONE,
> +                                                      NULL, 0);
>  }
>
>  static int register_device_clock(struct acpi_device *adev,
> @@ -1337,7 +1339,7 @@ void __init acpi_lpss_init(void)
>         const struct x86_cpu_id *id;
>         int ret;
>
> -       ret = lpt_clk_init();
> +       ret = lpss_atom_clk_init();
>         if (ret)
>                 return;
>
> diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
> index 18564efdc651..1244c4e568ff 100644
> --- a/drivers/clk/x86/Makefile
> +++ b/drivers/clk/x86/Makefile
> @@ -1,6 +1,6 @@
>  # SPDX-License-Identifier: GPL-2.0-only
>  obj-$(CONFIG_PMC_ATOM)         += clk-pmc-atom.o
>  obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE)  += clk-fch.o
> -clk-x86-lpss-objs              := clk-lpt.o
> +clk-x86-lpss-y                 := clk-lpss-atom.o
>  obj-$(CONFIG_X86_INTEL_LPSS)   += clk-x86-lpss.o
>  obj-$(CONFIG_CLK_LGM_CGU)      += clk-cgu.o clk-cgu-pll.o clk-lgm.o
> diff --git a/drivers/clk/x86/clk-lpt.c b/drivers/clk/x86/clk-lpss-atom.c
> similarity index 76%
> rename from drivers/clk/x86/clk-lpt.c
> rename to drivers/clk/x86/clk-lpss-atom.c
> index fbe9fd3ed948..aa9d0bb98f8b 100644
> --- a/drivers/clk/x86/clk-lpt.c
> +++ b/drivers/clk/x86/clk-lpss-atom.c
> @@ -13,7 +13,7 @@
>  #include <linux/platform_data/x86/clk-lpss.h>
>  #include <linux/platform_device.h>
>
> -static int lpt_clk_probe(struct platform_device *pdev)
> +static int lpss_atom_clk_probe(struct platform_device *pdev)
>  {
>         struct lpss_clk_data *drvdata;
>         struct clk *clk;
> @@ -34,14 +34,14 @@ static int lpt_clk_probe(struct platform_device *pdev)
>         return 0;
>  }
>
> -static struct platform_driver lpt_clk_driver = {
> +static struct platform_driver lpss_atom_clk_driver = {
>         .driver = {
> -               .name = "clk-lpt",
> +               .name = "clk-lpss-atom",
>         },
> -       .probe = lpt_clk_probe,
> +       .probe = lpss_atom_clk_probe,
>  };
>
> -int __init lpt_clk_init(void)
> +int __init lpss_atom_clk_init(void)
>  {
> -       return platform_driver_register(&lpt_clk_driver);
> +       return platform_driver_register(&lpss_atom_clk_driver);
>  }
> diff --git a/include/linux/platform_data/x86/clk-lpss.h b/include/linux/platform_data/x86/clk-lpss.h
> index 207e1a317800..41df326583f9 100644
> --- a/include/linux/platform_data/x86/clk-lpss.h
> +++ b/include/linux/platform_data/x86/clk-lpss.h
> @@ -15,6 +15,6 @@ struct lpss_clk_data {
>         struct clk *clk;
>  };
>
> -extern int lpt_clk_init(void);
> +extern int lpss_atom_clk_init(void);
>
>  #endif /* __CLK_LPSS_H */
> --
> 2.30.2
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom
  2021-07-23 13:40 ` Rafael J. Wysocki
@ 2021-08-04 16:03   ` Rafael J. Wysocki
       [not found]     ` <162809409848.19113.15878488082512415331@swboyd.mtv.corp.google.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Rafael J. Wysocki @ 2021-08-04 16:03 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: Andy Shevchenko, Rafael J. Wysocki, ACPI Devel Maling List,
	Linux Kernel Mailing List, linux-clk, Rafael J. Wysocki,
	Len Brown, Michael Turquette, Stephen Boyd

On Fri, Jul 23, 2021 at 3:40 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
>
> On Thu, Jul 22, 2021 at 9:34 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> >
> > The LPT stands for Lynxpoint PCH. However the driver is used on a few
> > Intel Atom SoCs. Rename it to reflect this in a way how another clock
> > driver, i.e. clk-pmc-atom, is called.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > ---
> >
> > Good to go either via ACPI or CCF tree.
>
> In case you want the latter:
>
> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

Assuming that this hasn't been picked up, I'm going to apply it as
5.15 material.

Thanks!

> >  drivers/acpi/acpi_lpss.c                       |  6 ++++--
> >  drivers/clk/x86/Makefile                       |  2 +-
> >  drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} | 12 ++++++------
> >  include/linux/platform_data/x86/clk-lpss.h     |  2 +-
> >  4 files changed, 12 insertions(+), 10 deletions(-)
> >  rename drivers/clk/x86/{clk-lpt.c => clk-lpss-atom.c} (76%)
> >
> > diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> > index 894b7e6ae144..7f163074e4e4 100644
> > --- a/drivers/acpi/acpi_lpss.c
> > +++ b/drivers/acpi/acpi_lpss.c
> > @@ -385,7 +385,9 @@ static struct platform_device *lpss_clk_dev;
> >
> >  static inline void lpt_register_clock_device(void)
> >  {
> > -       lpss_clk_dev = platform_device_register_simple("clk-lpt", -1, NULL, 0);
> > +       lpss_clk_dev = platform_device_register_simple("clk-lpss-atom",
> > +                                                      PLATFORM_DEVID_NONE,
> > +                                                      NULL, 0);
> >  }
> >
> >  static int register_device_clock(struct acpi_device *adev,
> > @@ -1337,7 +1339,7 @@ void __init acpi_lpss_init(void)
> >         const struct x86_cpu_id *id;
> >         int ret;
> >
> > -       ret = lpt_clk_init();
> > +       ret = lpss_atom_clk_init();
> >         if (ret)
> >                 return;
> >
> > diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
> > index 18564efdc651..1244c4e568ff 100644
> > --- a/drivers/clk/x86/Makefile
> > +++ b/drivers/clk/x86/Makefile
> > @@ -1,6 +1,6 @@
> >  # SPDX-License-Identifier: GPL-2.0-only
> >  obj-$(CONFIG_PMC_ATOM)         += clk-pmc-atom.o
> >  obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE)  += clk-fch.o
> > -clk-x86-lpss-objs              := clk-lpt.o
> > +clk-x86-lpss-y                 := clk-lpss-atom.o
> >  obj-$(CONFIG_X86_INTEL_LPSS)   += clk-x86-lpss.o
> >  obj-$(CONFIG_CLK_LGM_CGU)      += clk-cgu.o clk-cgu-pll.o clk-lgm.o
> > diff --git a/drivers/clk/x86/clk-lpt.c b/drivers/clk/x86/clk-lpss-atom.c
> > similarity index 76%
> > rename from drivers/clk/x86/clk-lpt.c
> > rename to drivers/clk/x86/clk-lpss-atom.c
> > index fbe9fd3ed948..aa9d0bb98f8b 100644
> > --- a/drivers/clk/x86/clk-lpt.c
> > +++ b/drivers/clk/x86/clk-lpss-atom.c
> > @@ -13,7 +13,7 @@
> >  #include <linux/platform_data/x86/clk-lpss.h>
> >  #include <linux/platform_device.h>
> >
> > -static int lpt_clk_probe(struct platform_device *pdev)
> > +static int lpss_atom_clk_probe(struct platform_device *pdev)
> >  {
> >         struct lpss_clk_data *drvdata;
> >         struct clk *clk;
> > @@ -34,14 +34,14 @@ static int lpt_clk_probe(struct platform_device *pdev)
> >         return 0;
> >  }
> >
> > -static struct platform_driver lpt_clk_driver = {
> > +static struct platform_driver lpss_atom_clk_driver = {
> >         .driver = {
> > -               .name = "clk-lpt",
> > +               .name = "clk-lpss-atom",
> >         },
> > -       .probe = lpt_clk_probe,
> > +       .probe = lpss_atom_clk_probe,
> >  };
> >
> > -int __init lpt_clk_init(void)
> > +int __init lpss_atom_clk_init(void)
> >  {
> > -       return platform_driver_register(&lpt_clk_driver);
> > +       return platform_driver_register(&lpss_atom_clk_driver);
> >  }
> > diff --git a/include/linux/platform_data/x86/clk-lpss.h b/include/linux/platform_data/x86/clk-lpss.h
> > index 207e1a317800..41df326583f9 100644
> > --- a/include/linux/platform_data/x86/clk-lpss.h
> > +++ b/include/linux/platform_data/x86/clk-lpss.h
> > @@ -15,6 +15,6 @@ struct lpss_clk_data {
> >         struct clk *clk;
> >  };
> >
> > -extern int lpt_clk_init(void);
> > +extern int lpss_atom_clk_init(void);
> >
> >  #endif /* __CLK_LPSS_H */
> > --
> > 2.30.2
> >

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom
       [not found]     ` <162809409848.19113.15878488082512415331@swboyd.mtv.corp.google.com>
@ 2021-08-04 16:28       ` Rafael J. Wysocki
  0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2021-08-04 16:28 UTC (permalink / raw)
  To: Stephen Boyd
  Cc: Rafael J. Wysocki, Andy Shevchenko, Rafael J. Wysocki,
	ACPI Devel Maling List, Linux Kernel Mailing List, linux-clk,
	Rafael J. Wysocki, Len Brown, Michael Turquette

On Wed, Aug 4, 2021 at 6:21 PM Stephen Boyd <sboyd@kernel.org> wrote:
>
> Quoting Rafael J. Wysocki (2021-08-04 09:03:31)
> > On Fri, Jul 23, 2021 at 3:40 PM Rafael J. Wysocki <rafael@kernel.org> wrote:
> > >
> > > On Thu, Jul 22, 2021 at 9:34 PM Andy Shevchenko
> > > <andriy.shevchenko@linux.intel.com> wrote:
> > > >
> > > > The LPT stands for Lynxpoint PCH. However the driver is used on a few
> > > > Intel Atom SoCs. Rename it to reflect this in a way how another clock
> > > > driver, i.e. clk-pmc-atom, is called.
> > > >
> > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > > > ---
> > > >
> > > > Good to go either via ACPI or CCF tree.
> > >
> > > In case you want the latter:
> > >
> > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> >
> > Assuming that this hasn't been picked up, I'm going to apply it as
> > 5.15 material.
> >
>
> I applied it already to clk tree. Maybe I forgot to send the email,
> sorry.

No prob, dropping it now.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-08-04 16:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-22 19:34 [PATCH v1 1/1] clk: x86: Rename clk-lpt to more specific clk-lpss-atom Andy Shevchenko
2021-07-23 13:40 ` Rafael J. Wysocki
2021-08-04 16:03   ` Rafael J. Wysocki
     [not found]     ` <162809409848.19113.15878488082512415331@swboyd.mtv.corp.google.com>
2021-08-04 16:28       ` Rafael J. Wysocki

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.