* [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-09 9:59 ` Akshu Agrawal
0 siblings, 0 replies; 22+ messages in thread
From: Akshu Agrawal @ 2018-05-09 9:59 UTC (permalink / raw)
Cc: djkurtz, akshu.agrawal, Alexander.Deucher, mturquette, sboyd,
christian.koenig, airlied, Shaoyun.Liu, linux-kernel, linux-clk,
rjw, lenb, linux-acpi
Stoney SoC provides oscout clock. This clock can support 25Mhz and
48Mhz of frequency.
The clock is available for general system use.
Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
---
v2: config change, added SPDX tag and used clk_hw_register_.
v3: Fix kbuild warning for checking of NULL pointer
v4: unregister clk_hw in driver remove, add .suppress_bind_attrs
v5: Fix license, used static array
drivers/clk/x86/Makefile | 3 +-
drivers/clk/x86/clk-st.c | 77 ++++++++++++++++++++++++++++++++++++
include/linux/platform_data/clk-st.h | 17 ++++++++
3 files changed, 96 insertions(+), 1 deletion(-)
create mode 100644 drivers/clk/x86/clk-st.c
create mode 100644 include/linux/platform_data/clk-st.h
diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
index 1367afb..00303bc 100644
--- a/drivers/clk/x86/Makefile
+++ b/drivers/clk/x86/Makefile
@@ -1,3 +1,4 @@
+obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
+obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE) += clk-st.o
clk-x86-lpss-objs := clk-lpt.o
obj-$(CONFIG_X86_INTEL_LPSS) += clk-x86-lpss.o
-obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c
new file mode 100644
index 0000000..fb62f39
--- /dev/null
+++ b/drivers/clk/x86/clk-st.c
@@ -0,0 +1,77 @@
+// SPDX-License-Identifier: MIT
+/*
+ * clock framework for AMD Stoney based clocks
+ *
+ * Copyright 2018 Advanced Micro Devices, Inc.
+ */
+
+#include <linux/clk.h>
+#include <linux/clkdev.h>
+#include <linux/clk-provider.h>
+#include <linux/platform_data/clk-st.h>
+#include <linux/platform_device.h>
+
+/* Clock Driving Strength 2 register */
+#define CLKDRVSTR2 0x28
+/* Clock Control 1 register */
+#define MISCCLKCNTL1 0x40
+/* Auxiliary clock1 enable bit */
+#define OSCCLKENB 2
+/* 25Mhz auxiliary output clock freq bit */
+#define OSCOUT1CLK25MHZ 16
+
+#define ST_CLK_48M 0
+#define ST_CLK_25M 1
+#define ST_CLK_MUX 2
+#define ST_CLK_GATE 3
+#define ST_MAX_CLKS 4
+
+static const char * const clk_oscout1_parents[] = { "clk48MHz", "clk25MHz" };
+static struct clk_hw *hws[ST_MAX_CLKS];
+
+static int st_clk_probe(struct platform_device *pdev)
+{
+ struct st_clk_data *st_data;
+
+ st_data = dev_get_platdata(&pdev->dev);
+ if (!st_data || !st_data->base)
+ return -EINVAL;
+
+ hws[ST_CLK_48M] = clk_hw_register_fixed_rate(NULL, "clk48MHz", NULL, 0,
+ 48000000);
+ hws[ST_CLK_25M] = clk_hw_register_fixed_rate(NULL, "clk25MHz", NULL, 0,
+ 25000000);
+
+ hws[ST_CLK_MUX] = clk_hw_register_mux(NULL, "oscout1_mux",
+ clk_oscout1_parents, ARRAY_SIZE(clk_oscout1_parents),
+ 0, st_data->base + CLKDRVSTR2, OSCOUT1CLK25MHZ, 3, 0, NULL);
+
+ clk_set_parent(hws[ST_CLK_MUX]->clk, hws[ST_CLK_25M]->clk);
+
+ hws[ST_CLK_GATE] = clk_hw_register_gate(NULL, "oscout1", "oscout1_mux",
+ 0, st_data->base + MISCCLKCNTL1, OSCCLKENB,
+ CLK_GATE_SET_TO_DISABLE, NULL);
+
+ clk_hw_register_clkdev(hws[ST_CLK_GATE], "oscout1", NULL);
+
+ return 0;
+}
+
+static int st_clk_remove(struct platform_device *pdev)
+{
+ int i;
+
+ for (i = 0; i < ST_MAX_CLKS; i++)
+ clk_hw_unregister(hws[i]);
+ return 0;
+}
+
+static struct platform_driver st_clk_driver = {
+ .driver = {
+ .name = "clk-st",
+ .suppress_bind_attrs = true,
+ },
+ .probe = st_clk_probe,
+ .remove = st_clk_remove,
+};
+builtin_platform_driver(st_clk_driver);
diff --git a/include/linux/platform_data/clk-st.h b/include/linux/platform_data/clk-st.h
new file mode 100644
index 0000000..7cdb6a4
--- /dev/null
+++ b/include/linux/platform_data/clk-st.h
@@ -0,0 +1,17 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * clock framework for AMD Stoney based clock
+ *
+ * Copyright 2018 Advanced Micro Devices, Inc.
+ */
+
+#ifndef __CLK_ST_H
+#define __CLK_ST_H
+
+#include <linux/compiler.h>
+
+struct st_clk_data {
+ void __iomem *base;
+};
+
+#endif /* __CLK_ST_H */
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-09 9:59 ` Akshu Agrawal
0 siblings, 0 replies; 22+ messages in thread
From: Akshu Agrawal @ 2018-05-09 9:59 UTC (permalink / raw)
Cc: djkurtz, akshu.agrawal, Alexander.Deucher, mturquette, sboyd,
christian.koenig, airlied, Shaoyun.Liu, linux-kernel, linux-clk,
rjw, lenb, linux-acpi
Stoney SoC provides oscout clock. This clock can support 25Mhz and
48Mhz of frequency.
The clock is available for general system use.
Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
---
v2: config change, added SPDX tag and used clk_hw_register_.
v3: Fix kbuild warning for checking of NULL pointer
v4: unregister clk_hw in driver remove, add .suppress_bind_attrs
v5: Fix license, used static array
drivers/clk/x86/Makefile | 3 +-
drivers/clk/x86/clk-st.c | 77 ++++++++++++++++++++++++++++++++++++
include/linux/platform_data/clk-st.h | 17 ++++++++
3 files changed, 96 insertions(+), 1 deletion(-)
create mode 100644 drivers/clk/x86/clk-st.c
create mode 100644 include/linux/platform_data/clk-st.h
diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
index 1367afb..00303bc 100644
--- a/drivers/clk/x86/Makefile
+++ b/drivers/clk/x86/Makefile
@@ -1,3 +1,4 @@
+obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
+obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE) += clk-st.o
clk-x86-lpss-objs := clk-lpt.o
obj-$(CONFIG_X86_INTEL_LPSS) += clk-x86-lpss.o
-obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c
new file mode 100644
index 0000000..fb62f39
--- /dev/null
+++ b/drivers/clk/x86/clk-st.c
@@ -0,0 +1,77 @@
+// SPDX-License-Identifier: MIT
+/*
+ * clock framework for AMD Stoney based clocks
+ *
+ * Copyright 2018 Advanced Micro Devices, Inc.
+ */
+
+#include <linux/clk.h>
+#include <linux/clkdev.h>
+#include <linux/clk-provider.h>
+#include <linux/platform_data/clk-st.h>
+#include <linux/platform_device.h>
+
+/* Clock Driving Strength 2 register */
+#define CLKDRVSTR2 0x28
+/* Clock Control 1 register */
+#define MISCCLKCNTL1 0x40
+/* Auxiliary clock1 enable bit */
+#define OSCCLKENB 2
+/* 25Mhz auxiliary output clock freq bit */
+#define OSCOUT1CLK25MHZ 16
+
+#define ST_CLK_48M 0
+#define ST_CLK_25M 1
+#define ST_CLK_MUX 2
+#define ST_CLK_GATE 3
+#define ST_MAX_CLKS 4
+
+static const char * const clk_oscout1_parents[] = { "clk48MHz", "clk25MHz" };
+static struct clk_hw *hws[ST_MAX_CLKS];
+
+static int st_clk_probe(struct platform_device *pdev)
+{
+ struct st_clk_data *st_data;
+
+ st_data = dev_get_platdata(&pdev->dev);
+ if (!st_data || !st_data->base)
+ return -EINVAL;
+
+ hws[ST_CLK_48M] = clk_hw_register_fixed_rate(NULL, "clk48MHz", NULL, 0,
+ 48000000);
+ hws[ST_CLK_25M] = clk_hw_register_fixed_rate(NULL, "clk25MHz", NULL, 0,
+ 25000000);
+
+ hws[ST_CLK_MUX] = clk_hw_register_mux(NULL, "oscout1_mux",
+ clk_oscout1_parents, ARRAY_SIZE(clk_oscout1_parents),
+ 0, st_data->base + CLKDRVSTR2, OSCOUT1CLK25MHZ, 3, 0, NULL);
+
+ clk_set_parent(hws[ST_CLK_MUX]->clk, hws[ST_CLK_25M]->clk);
+
+ hws[ST_CLK_GATE] = clk_hw_register_gate(NULL, "oscout1", "oscout1_mux",
+ 0, st_data->base + MISCCLKCNTL1, OSCCLKENB,
+ CLK_GATE_SET_TO_DISABLE, NULL);
+
+ clk_hw_register_clkdev(hws[ST_CLK_GATE], "oscout1", NULL);
+
+ return 0;
+}
+
+static int st_clk_remove(struct platform_device *pdev)
+{
+ int i;
+
+ for (i = 0; i < ST_MAX_CLKS; i++)
+ clk_hw_unregister(hws[i]);
+ return 0;
+}
+
+static struct platform_driver st_clk_driver = {
+ .driver = {
+ .name = "clk-st",
+ .suppress_bind_attrs = true,
+ },
+ .probe = st_clk_probe,
+ .remove = st_clk_remove,
+};
+builtin_platform_driver(st_clk_driver);
diff --git a/include/linux/platform_data/clk-st.h b/include/linux/platform_data/clk-st.h
new file mode 100644
index 0000000..7cdb6a4
--- /dev/null
+++ b/include/linux/platform_data/clk-st.h
@@ -0,0 +1,17 @@
+/* SPDX-License-Identifier: MIT */
+/*
+ * clock framework for AMD Stoney based clock
+ *
+ * Copyright 2018 Advanced Micro Devices, Inc.
+ */
+
+#ifndef __CLK_ST_H
+#define __CLK_ST_H
+
+#include <linux/compiler.h>
+
+struct st_clk_data {
+ void __iomem *base;
+};
+
+#endif /* __CLK_ST_H */
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-09 9:59 ` Akshu Agrawal
(?)
(?)
@ 2018-05-09 15:58 ` Daniel Kurtz
-1 siblings, 0 replies; 22+ messages in thread
From: Daniel Kurtz @ 2018-05-09 15:58 UTC (permalink / raw)
To: Akshu Agrawal
Cc: Alexander.Deucher, Mike Turquette, sboyd, christian.koenig,
Dave Airlie, Shaoyun.Liu, linux-kernel, linux-clk,
Rafael J. Wysocki, Len Brown, linux-acpi
On Wed, May 9, 2018 at 4:01 AM Akshu Agrawal <akshu.agrawal@amd.com> wrote:
> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> 48Mhz of frequency.
> The clock is available for general system use.
> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>
> ---
> v2: config change, added SPDX tag and used clk_hw_register_.
> v3: Fix kbuild warning for checking of NULL pointer
> v4: unregister clk_hw in driver remove, add .suppress_bind_attrs
> v5: Fix license, used static array
> drivers/clk/x86/Makefile | 3 +-
> drivers/clk/x86/clk-st.c | 77
++++++++++++++++++++++++++++++++++++
> include/linux/platform_data/clk-st.h | 17 ++++++++
> 3 files changed, 96 insertions(+), 1 deletion(-)
> create mode 100644 drivers/clk/x86/clk-st.c
> create mode 100644 include/linux/platform_data/clk-st.h
> diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
> index 1367afb..00303bc 100644
> --- a/drivers/clk/x86/Makefile
> +++ b/drivers/clk/x86/Makefile
> @@ -1,3 +1,4 @@
> +obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
> +obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE) += clk-st.o
> clk-x86-lpss-objs := clk-lpt.o
> obj-$(CONFIG_X86_INTEL_LPSS) += clk-x86-lpss.o
> -obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
> diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c
> new file mode 100644
> index 0000000..fb62f39
> --- /dev/null
> +++ b/drivers/clk/x86/clk-st.c
> @@ -0,0 +1,77 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * clock framework for AMD Stoney based clocks
> + *
> + * Copyright 2018 Advanced Micro Devices, Inc.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/clkdev.h>
> +#include <linux/clk-provider.h>
> +#include <linux/platform_data/clk-st.h>
> +#include <linux/platform_device.h>
> +
> +/* Clock Driving Strength 2 register */
> +#define CLKDRVSTR2 0x28
> +/* Clock Control 1 register */
> +#define MISCCLKCNTL1 0x40
> +/* Auxiliary clock1 enable bit */
> +#define OSCCLKENB 2
> +/* 25Mhz auxiliary output clock freq bit */
> +#define OSCOUT1CLK25MHZ 16
> +
> +#define ST_CLK_48M 0
> +#define ST_CLK_25M 1
> +#define ST_CLK_MUX 2
> +#define ST_CLK_GATE 3
> +#define ST_MAX_CLKS 4
> +
> +static const char * const clk_oscout1_parents[] = { "clk48MHz",
"clk25MHz" };
> +static struct clk_hw *hws[ST_MAX_CLKS];
> +
> +static int st_clk_probe(struct platform_device *pdev)
> +{
> + struct st_clk_data *st_data;
> +
> + st_data = dev_get_platdata(&pdev->dev);
> + if (!st_data || !st_data->base)
> + return -EINVAL;
> +
> + hws[ST_CLK_48M] = clk_hw_register_fixed_rate(NULL, "clk48MHz",
NULL, 0,
> + 48000000);
> + hws[ST_CLK_25M] = clk_hw_register_fixed_rate(NULL, "clk25MHz",
NULL, 0,
> + 25000000);
> +
> + hws[ST_CLK_MUX] = clk_hw_register_mux(NULL, "oscout1_mux",
> + clk_oscout1_parents, ARRAY_SIZE(clk_oscout1_parents),
> + 0, st_data->base + CLKDRVSTR2, OSCOUT1CLK25MHZ, 3, 0,
NULL);
> +
> + clk_set_parent(hws[ST_CLK_MUX]->clk, hws[ST_CLK_25M]->clk);
> +
> + hws[ST_CLK_GATE] = clk_hw_register_gate(NULL, "oscout1",
"oscout1_mux",
> + 0, st_data->base + MISCCLKCNTL1, OSCCLKENB,
> + CLK_GATE_SET_TO_DISABLE, NULL);
> +
> + clk_hw_register_clkdev(hws[ST_CLK_GATE], "oscout1", NULL);
> +
> + return 0;
> +}
> +
> +static int st_clk_remove(struct platform_device *pdev)
> +{
> + int i;
> +
> + for (i = 0; i < ST_MAX_CLKS; i++)
> + clk_hw_unregister(hws[i]);
> + return 0;
> +}
> +
> +static struct platform_driver st_clk_driver = {
> + .driver = {
> + .name = "clk-st",
> + .suppress_bind_attrs = true,
> + },
> + .probe = st_clk_probe,
> + .remove = st_clk_remove,
> +};
> +builtin_platform_driver(st_clk_driver);
> diff --git a/include/linux/platform_data/clk-st.h
b/include/linux/platform_data/clk-st.h
> new file mode 100644
> index 0000000..7cdb6a4
> --- /dev/null
> +++ b/include/linux/platform_data/clk-st.h
> @@ -0,0 +1,17 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * clock framework for AMD Stoney based clock
> + *
> + * Copyright 2018 Advanced Micro Devices, Inc.
> + */
> +
> +#ifndef __CLK_ST_H
> +#define __CLK_ST_H
> +
> +#include <linux/compiler.h>
> +
> +struct st_clk_data {
> + void __iomem *base;
> +};
> +
> +#endif /* __CLK_ST_H */
> --
> 1.9.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-09 9:59 ` Akshu Agrawal
` (2 preceding siblings ...)
(?)
@ 2018-05-15 9:32 ` Rafael J. Wysocki
2018-05-15 9:39 ` Agrawal, Akshu
-1 siblings, 1 reply; 22+ messages in thread
From: Rafael J. Wysocki @ 2018-05-15 9:32 UTC (permalink / raw)
To: Akshu Agrawal
Cc: djkurtz, Alexander.Deucher, mturquette, sboyd, christian.koenig,
airlied, Shaoyun.Liu, linux-kernel, linux-clk, lenb, linux-acpi
On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> 48Mhz of frequency.
> The clock is available for general system use.
>
> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
I'm not sure if the Stephen Boyd's comments on one of the previous
versions of this patch have been addressed. Have they?
In any case, if I'm expected to take this, I need an ACK from Stephen on it.
Thanks,
Rafael
> ---
> v2: config change, added SPDX tag and used clk_hw_register_.
> v3: Fix kbuild warning for checking of NULL pointer
> v4: unregister clk_hw in driver remove, add .suppress_bind_attrs
> v5: Fix license, used static array
> drivers/clk/x86/Makefile | 3 +-
> drivers/clk/x86/clk-st.c | 77 ++++++++++++++++++++++++++++++++++++
> include/linux/platform_data/clk-st.h | 17 ++++++++
> 3 files changed, 96 insertions(+), 1 deletion(-)
> create mode 100644 drivers/clk/x86/clk-st.c
> create mode 100644 include/linux/platform_data/clk-st.h
>
> diff --git a/drivers/clk/x86/Makefile b/drivers/clk/x86/Makefile
> index 1367afb..00303bc 100644
> --- a/drivers/clk/x86/Makefile
> +++ b/drivers/clk/x86/Makefile
> @@ -1,3 +1,4 @@
> +obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
> +obj-$(CONFIG_X86_AMD_PLATFORM_DEVICE) += clk-st.o
> clk-x86-lpss-objs := clk-lpt.o
> obj-$(CONFIG_X86_INTEL_LPSS) += clk-x86-lpss.o
> -obj-$(CONFIG_PMC_ATOM) += clk-pmc-atom.o
> diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c
> new file mode 100644
> index 0000000..fb62f39
> --- /dev/null
> +++ b/drivers/clk/x86/clk-st.c
> @@ -0,0 +1,77 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * clock framework for AMD Stoney based clocks
> + *
> + * Copyright 2018 Advanced Micro Devices, Inc.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/clkdev.h>
> +#include <linux/clk-provider.h>
> +#include <linux/platform_data/clk-st.h>
> +#include <linux/platform_device.h>
> +
> +/* Clock Driving Strength 2 register */
> +#define CLKDRVSTR2 0x28
> +/* Clock Control 1 register */
> +#define MISCCLKCNTL1 0x40
> +/* Auxiliary clock1 enable bit */
> +#define OSCCLKENB 2
> +/* 25Mhz auxiliary output clock freq bit */
> +#define OSCOUT1CLK25MHZ 16
> +
> +#define ST_CLK_48M 0
> +#define ST_CLK_25M 1
> +#define ST_CLK_MUX 2
> +#define ST_CLK_GATE 3
> +#define ST_MAX_CLKS 4
> +
> +static const char * const clk_oscout1_parents[] = { "clk48MHz", "clk25MHz" };
> +static struct clk_hw *hws[ST_MAX_CLKS];
> +
> +static int st_clk_probe(struct platform_device *pdev)
> +{
> + struct st_clk_data *st_data;
> +
> + st_data = dev_get_platdata(&pdev->dev);
> + if (!st_data || !st_data->base)
> + return -EINVAL;
> +
> + hws[ST_CLK_48M] = clk_hw_register_fixed_rate(NULL, "clk48MHz", NULL, 0,
> + 48000000);
> + hws[ST_CLK_25M] = clk_hw_register_fixed_rate(NULL, "clk25MHz", NULL, 0,
> + 25000000);
> +
> + hws[ST_CLK_MUX] = clk_hw_register_mux(NULL, "oscout1_mux",
> + clk_oscout1_parents, ARRAY_SIZE(clk_oscout1_parents),
> + 0, st_data->base + CLKDRVSTR2, OSCOUT1CLK25MHZ, 3, 0, NULL);
> +
> + clk_set_parent(hws[ST_CLK_MUX]->clk, hws[ST_CLK_25M]->clk);
> +
> + hws[ST_CLK_GATE] = clk_hw_register_gate(NULL, "oscout1", "oscout1_mux",
> + 0, st_data->base + MISCCLKCNTL1, OSCCLKENB,
> + CLK_GATE_SET_TO_DISABLE, NULL);
> +
> + clk_hw_register_clkdev(hws[ST_CLK_GATE], "oscout1", NULL);
> +
> + return 0;
> +}
> +
> +static int st_clk_remove(struct platform_device *pdev)
> +{
> + int i;
> +
> + for (i = 0; i < ST_MAX_CLKS; i++)
> + clk_hw_unregister(hws[i]);
> + return 0;
> +}
> +
> +static struct platform_driver st_clk_driver = {
> + .driver = {
> + .name = "clk-st",
> + .suppress_bind_attrs = true,
> + },
> + .probe = st_clk_probe,
> + .remove = st_clk_remove,
> +};
> +builtin_platform_driver(st_clk_driver);
> diff --git a/include/linux/platform_data/clk-st.h b/include/linux/platform_data/clk-st.h
> new file mode 100644
> index 0000000..7cdb6a4
> --- /dev/null
> +++ b/include/linux/platform_data/clk-st.h
> @@ -0,0 +1,17 @@
> +/* SPDX-License-Identifier: MIT */
> +/*
> + * clock framework for AMD Stoney based clock
> + *
> + * Copyright 2018 Advanced Micro Devices, Inc.
> + */
> +
> +#ifndef __CLK_ST_H
> +#define __CLK_ST_H
> +
> +#include <linux/compiler.h>
> +
> +struct st_clk_data {
> + void __iomem *base;
> +};
> +
> +#endif /* __CLK_ST_H */
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-15 9:32 ` Rafael J. Wysocki
@ 2018-05-15 9:39 ` Agrawal, Akshu
2018-05-15 21:14 ` Stephen Boyd
0 siblings, 1 reply; 22+ messages in thread
From: Agrawal, Akshu @ 2018-05-15 9:39 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: djkurtz, Alexander.Deucher, mturquette, sboyd, christian.koenig,
airlied, Shaoyun.Liu, linux-kernel, linux-clk, lenb, linux-acpi
On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
> On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
>> Stoney SoC provides oscout clock. This clock can support 25Mhz and
>> 48Mhz of frequency.
>> The clock is available for general system use.
>>
>> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
>
> I'm not sure if the Stephen Boyd's comments on one of the previous
> versions of this patch have been addressed. Have they?
>
> In any case, if I'm expected to take this, I need an ACK from Stephen on it.
>
> Thanks,
> Rafael
>
All comments of Stephen Boyd and Daniel Kurtz have been addressed.
Stephen, if you are Ok with this can you please ACK it.
Thanks,
Akshu
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-15 9:39 ` Agrawal, Akshu
@ 2018-05-15 21:14 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-15 21:14 UTC (permalink / raw)
To: Agrawal, Akshu, Rafael J. Wysocki
Cc: djkurtz, Alexander.Deucher, mturquette, christian.koenig,
airlied, Shaoyun.Liu, linux-kernel, linux-clk, lenb, linux-acpi
Quoting Agrawal, Akshu (2018-05-15 02:39:08)
>
>
> On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
> > On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
> >> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> >> 48Mhz of frequency.
> >> The clock is available for general system use.
> >>
> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> >
> > I'm not sure if the Stephen Boyd's comments on one of the previous
> > versions of this patch have been addressed. Have they?
> >
> > In any case, if I'm expected to take this, I need an ACK from Stephen on it.
> >
> > Thanks,
> > Rafael
> >
>
> All comments of Stephen Boyd and Daniel Kurtz have been addressed.
>
> Stephen, if you are Ok with this can you please ACK it.
>
It could go via clk tree and meet up with the acpi patch in -next right?
I'm fine with it going through acpi path though so whichever way works.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-15 21:14 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-15 21:14 UTC (permalink / raw)
To: Agrawal, Akshu, Rafael J. Wysocki
Cc: djkurtz, Alexander.Deucher, mturquette, christian.koenig,
airlied, Shaoyun.Liu, linux-kernel, linux-clk, lenb, linux-acpi
Quoting Agrawal, Akshu (2018-05-15 02:39:08)
> =
> =
> On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
> > On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
> >> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> >> 48Mhz of frequency.
> >> The clock is available for general system use.
> >>
> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> > =
> > I'm not sure if the Stephen Boyd's comments on one of the previous
> > versions of this patch have been addressed. Have they?
> > =
> > In any case, if I'm expected to take this, I need an ACK from Stephen o=
n it.
> > =
> > Thanks,
> > Rafael
> > =
> =
> All comments of Stephen Boyd and Daniel Kurtz have been addressed.
> =
> Stephen, if you are Ok with this can you please ACK it.
> =
It could go via clk tree and meet up with the acpi patch in -next right?
I'm fine with it going through acpi path though so whichever way works.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-15 21:14 ` Stephen Boyd
(?)
@ 2018-05-16 8:23 ` Rafael J. Wysocki
2018-05-16 21:18 ` Stephen Boyd
-1 siblings, 1 reply; 22+ messages in thread
From: Rafael J. Wysocki @ 2018-05-16 8:23 UTC (permalink / raw)
To: Stephen Boyd
Cc: Agrawal, Akshu, Rafael J. Wysocki, Daniel Kurtz, Alex Deucher,
Michael Turquette, Christian König, Dave Airlie,
Shaoyun.Liu, Linux Kernel Mailing List, linux-clk, Len Brown,
ACPI Devel Maling List
On Tue, May 15, 2018 at 11:14 PM, Stephen Boyd <sboyd@kernel.org> wrote:
> Quoting Agrawal, Akshu (2018-05-15 02:39:08)
>>
>>
>> On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
>> > On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
>> >> Stoney SoC provides oscout clock. This clock can support 25Mhz and
>> >> 48Mhz of frequency.
>> >> The clock is available for general system use.
>> >>
>> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
>> >
>> > I'm not sure if the Stephen Boyd's comments on one of the previous
>> > versions of this patch have been addressed. Have they?
>> >
>> > In any case, if I'm expected to take this, I need an ACK from Stephen on it.
>> >
>> > Thanks,
>> > Rafael
>> >
>>
>> All comments of Stephen Boyd and Daniel Kurtz have been addressed.
>>
>> Stephen, if you are Ok with this can you please ACK it.
>>
>
> It could go via clk tree and meet up with the acpi patch in -next right?
> I'm fine with it going through acpi path though so whichever way works.
It's better if it goes in as a series IMO and then if it goes via
ACPI, the clock dependency will be clear from the git history.
So if you don't mind, I'll queue this series up for 4.18.
Thanks,
Rafael
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-16 8:23 ` Rafael J. Wysocki
@ 2018-05-16 21:18 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-16 21:18 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Agrawal, Akshu, Rafael J. Wysocki, Daniel Kurtz, Alex Deucher,
Michael Turquette, Christian König, Dave Airlie,
Shaoyun.Liu, Linux Kernel Mailing List, linux-clk, Len Brown,
ACPI Devel Maling List
Quoting Rafael J. Wysocki (2018-05-16 01:23:27)
> On Tue, May 15, 2018 at 11:14 PM, Stephen Boyd <sboyd@kernel.org> wrote:
> > Quoting Agrawal, Akshu (2018-05-15 02:39:08)
> >>
> >>
> >> On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
> >> > On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
> >> >> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> >> >> 48Mhz of frequency.
> >> >> The clock is available for general system use.
> >> >>
> >> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> >> >
> >> > I'm not sure if the Stephen Boyd's comments on one of the previous
> >> > versions of this patch have been addressed. Have they?
> >> >
> >> > In any case, if I'm expected to take this, I need an ACK from Stephen on it.
> >> >
> >> > Thanks,
> >> > Rafael
> >> >
> >>
> >> All comments of Stephen Boyd and Daniel Kurtz have been addressed.
> >>
> >> Stephen, if you are Ok with this can you please ACK it.
> >>
> >
> > It could go via clk tree and meet up with the acpi patch in -next right?
> > I'm fine with it going through acpi path though so whichever way works.
>
> It's better if it goes in as a series IMO and then if it goes via
> ACPI, the clock dependency will be clear from the git history.
>
> So if you don't mind, I'll queue this series up for 4.18.
>
Ok
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-16 21:18 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-16 21:18 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Agrawal, Akshu, Rafael J. Wysocki, Daniel Kurtz, Alex Deucher,
Michael Turquette, Christian König, Dave Airlie,
Shaoyun.Liu, Linux Kernel Mailing List, linux-clk, Len Brown,
ACPI Devel Maling List
Quoting Rafael J. Wysocki (2018-05-16 01:23:27)
> On Tue, May 15, 2018 at 11:14 PM, Stephen Boyd <sboyd@kernel.org> wrote:
> > Quoting Agrawal, Akshu (2018-05-15 02:39:08)
> >>
> >>
> >> On 5/15/2018 3:02 PM, Rafael J. Wysocki wrote:
> >> > On Wednesday, May 9, 2018 11:59:00 AM CEST Akshu Agrawal wrote:
> >> >> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> >> >> 48Mhz of frequency.
> >> >> The clock is available for general system use.
> >> >>
> >> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> >> >
> >> > I'm not sure if the Stephen Boyd's comments on one of the previous
> >> > versions of this patch have been addressed. Have they?
> >> >
> >> > In any case, if I'm expected to take this, I need an ACK from Stephe=
n on it.
> >> >
> >> > Thanks,
> >> > Rafael
> >> >
> >>
> >> All comments of Stephen Boyd and Daniel Kurtz have been addressed.
> >>
> >> Stephen, if you are Ok with this can you please ACK it.
> >>
> >
> > It could go via clk tree and meet up with the acpi patch in -next right?
> > I'm fine with it going through acpi path though so whichever way works.
> =
> It's better if it goes in as a series IMO and then if it goes via
> ACPI, the clock dependency will be clear from the git history.
> =
> So if you don't mind, I'll queue this series up for 4.18.
> =
Ok
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
2018-05-09 9:59 ` Akshu Agrawal
(?)
@ 2018-05-15 21:11 ` Stephen Boyd
-1 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-15 21:11 UTC (permalink / raw)
Cc: djkurtz, akshu.agrawal, Alexander.Deucher, mturquette,
christian.koenig, airlied, Shaoyun.Liu, linux-kernel, linux-clk,
rjw, lenb, linux-acpi
Quoting Akshu Agrawal (2018-05-09 02:59:00)
> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> 48Mhz of frequency.
> The clock is available for general system use.
>
> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> ---
Reviewed-by: Stephen Boyd <sboyd@kernel.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-15 21:11 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-15 21:11 UTC (permalink / raw)
To: , Akshu Agrawal
Cc: djkurtz, akshu.agrawal, Alexander.Deucher, mturquette,
christian.koenig, airlied, Shaoyun.Liu, linux-kernel, linux-clk,
rjw, lenb, linux-acpi
Quoting Akshu Agrawal (2018-05-09 02:59:00)
> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> 48Mhz of frequency.
> The clock is available for general system use.
> =
> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> ---
Reviewed-by: Stephen Boyd <sboyd@kernel.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v5 1/2] clk: x86: Add ST oscout platform clock
@ 2018-05-15 21:11 ` Stephen Boyd
0 siblings, 0 replies; 22+ messages in thread
From: Stephen Boyd @ 2018-05-15 21:11 UTC (permalink / raw)
To: Akshu Agrawal
Cc: djkurtz, akshu.agrawal, Alexander.Deucher, mturquette,
christian.koenig, airlied, Shaoyun.Liu, linux-kernel, linux-clk,
rjw, lenb, linux-acpi
Quoting Akshu Agrawal (2018-05-09 02:59:00)
> Stoney SoC provides oscout clock. This clock can support 25Mhz and
> 48Mhz of frequency.
> The clock is available for general system use.
>
> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com>
> ---
Reviewed-by: Stephen Boyd <sboyd@kernel.org>
^ permalink raw reply [flat|nested] 22+ messages in thread