* [v2 1/4] ACPI: APD: Change name from ST to FCH [not found] <20200728082857.10829-1-akshu.agrawal@amd.com> @ 2020-07-28 8:28 ` Akshu Agrawal [not found] ` <159598596077.1360974.483730969007254506@swboyd.mtv.corp.google.com> 2020-07-28 8:28 ` [3/4] ACPI: APD: Add a fmw property is_raven Akshu Agrawal 1 sibling, 1 reply; 6+ messages in thread From: Akshu Agrawal @ 2020-07-28 8:28 UTC (permalink / raw) To: akshu.agrawal Cc: sboyd, rafael, Rafael J. Wysocki, Len Brown, Michael Turquette, open list:ACPI, open list, open list:COMMON CLK FRAMEWORK AMD SoC general pupose clk is present in new platforms with same MMIO mappings. We can reuse the same clk handler support for other platforms. Hence, changing name from ST(SoC) to FCH(IP) Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> --- v2: pulled in clk changes so that patch compiles individually drivers/acpi/acpi_apd.c | 14 +++++++------- drivers/clk/x86/clk-st.c | 4 ++-- .../linux/platform_data/{clk-st.h => clk-fch.h} | 10 +++++----- 3 files changed, 14 insertions(+), 14 deletions(-) rename include/linux/platform_data/{clk-st.h => clk-fch.h} (53%) diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c index ba2612e9a0eb..2d99e46add1a 100644 --- a/drivers/acpi/acpi_apd.c +++ b/drivers/acpi/acpi_apd.c @@ -8,7 +8,7 @@ */ #include <linux/clk-provider.h> -#include <linux/platform_data/clk-st.h> +#include <linux/platform_data/clk-fch.h> #include <linux/platform_device.h> #include <linux/pm_domain.h> #include <linux/clkdev.h> @@ -79,11 +79,11 @@ static int misc_check_res(struct acpi_resource *ares, void *data) return !acpi_dev_resource_memory(ares, &res); } -static int st_misc_setup(struct apd_private_data *pdata) +static int fch_misc_setup(struct apd_private_data *pdata) { struct acpi_device *adev = pdata->adev; struct platform_device *clkdev; - struct st_clk_data *clk_data; + struct fch_clk_data *clk_data; struct resource_entry *rentry; struct list_head resource_list; int ret; @@ -106,7 +106,7 @@ static int st_misc_setup(struct apd_private_data *pdata) acpi_dev_free_resource_list(&resource_list); - clkdev = platform_device_register_data(&adev->dev, "clk-st", + clkdev = platform_device_register_data(&adev->dev, "clk-fch", PLATFORM_DEVID_NONE, clk_data, sizeof(*clk_data)); return PTR_ERR_OR_ZERO(clkdev); @@ -135,8 +135,8 @@ static const struct apd_device_desc cz_uart_desc = { .properties = uart_properties, }; -static const struct apd_device_desc st_misc_desc = { - .setup = st_misc_setup, +static const struct apd_device_desc fch_misc_desc = { + .setup = fch_misc_setup, }; #endif @@ -239,7 +239,7 @@ static const struct acpi_device_id acpi_apd_device_ids[] = { { "AMD0020", APD_ADDR(cz_uart_desc) }, { "AMDI0020", APD_ADDR(cz_uart_desc) }, { "AMD0030", }, - { "AMD0040", APD_ADDR(st_misc_desc)}, + { "AMD0040", APD_ADDR(fch_misc_desc)}, #endif #ifdef CONFIG_ARM64 { "APMC0D0F", APD_ADDR(xgene_i2c_desc) }, diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c index 25d4b97aff9b..c2438874d9f2 100644 --- a/drivers/clk/x86/clk-st.c +++ b/drivers/clk/x86/clk-st.c @@ -8,7 +8,7 @@ #include <linux/clk.h> #include <linux/clkdev.h> #include <linux/clk-provider.h> -#include <linux/platform_data/clk-st.h> +#include <linux/platform_data/clk-fch.h> #include <linux/platform_device.h> /* Clock Driving Strength 2 register */ @@ -31,7 +31,7 @@ static struct clk_hw *hws[ST_MAX_CLKS]; static int st_clk_probe(struct platform_device *pdev) { - struct st_clk_data *st_data; + struct fch_clk_data *st_data; st_data = dev_get_platdata(&pdev->dev); if (!st_data || !st_data->base) diff --git a/include/linux/platform_data/clk-st.h b/include/linux/platform_data/clk-fch.h similarity index 53% rename from include/linux/platform_data/clk-st.h rename to include/linux/platform_data/clk-fch.h index 7cdb6a402b35..850ca776156d 100644 --- a/include/linux/platform_data/clk-st.h +++ b/include/linux/platform_data/clk-fch.h @@ -1,17 +1,17 @@ /* SPDX-License-Identifier: MIT */ /* - * clock framework for AMD Stoney based clock + * clock framework for AMD misc clocks * * Copyright 2018 Advanced Micro Devices, Inc. */ -#ifndef __CLK_ST_H -#define __CLK_ST_H +#ifndef __CLK_FCH_H +#define __CLK_FCH_H #include <linux/compiler.h> -struct st_clk_data { +struct fch_clk_data { void __iomem *base; }; -#endif /* __CLK_ST_H */ +#endif /* __CLK_FCH_H */ -- 2.20.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <159598596077.1360974.483730969007254506@swboyd.mtv.corp.google.com>]
* Re: [v2 1/4] ACPI: APD: Change name from ST to FCH [not found] ` <159598596077.1360974.483730969007254506@swboyd.mtv.corp.google.com> @ 2020-07-31 0:44 ` Agrawal, Akshu 2020-07-31 11:14 ` Rafael J. Wysocki 0 siblings, 1 reply; 6+ messages in thread From: Agrawal, Akshu @ 2020-07-31 0:44 UTC (permalink / raw) To: Stephen Boyd, akshu.agrawal Cc: rafael, Rafael J. Wysocki, Len Brown, Michael Turquette, linux-acpi, linux-kernel, linux-clk On 7/29/2020 6:56 AM, Stephen Boyd wrote: > Quoting Akshu Agrawal (2020-07-28 01:28:53) >> AMD SoC general pupose clk is present in new platforms with >> same MMIO mappings. We can reuse the same clk handler support >> for other platforms. Hence, changing name from ST(SoC) to FCH(IP) >> >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> >> --- > Acked-by: Stephen Boyd <sboyd@kernel.org> Hi Rafael, I see the status of these patches as Not Applicable in patchwork, is there any pending action for me? Thanks, Akshu ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [v2 1/4] ACPI: APD: Change name from ST to FCH 2020-07-31 0:44 ` Agrawal, Akshu @ 2020-07-31 11:14 ` Rafael J. Wysocki 2020-07-31 13:33 ` Agrawal, Akshu 0 siblings, 1 reply; 6+ messages in thread From: Rafael J. Wysocki @ 2020-07-31 11:14 UTC (permalink / raw) To: Agrawal, Akshu Cc: Stephen Boyd, Agrawal, Akshu, Rafael J. Wysocki, Rafael J. Wysocki, Len Brown, Michael Turquette, ACPI Devel Maling List, Linux Kernel Mailing List, linux-clk On Fri, Jul 31, 2020 at 2:44 AM Agrawal, Akshu <aagrawal2@amd.com> wrote: > > > On 7/29/2020 6:56 AM, Stephen Boyd wrote: > > Quoting Akshu Agrawal (2020-07-28 01:28:53) > >> AMD SoC general pupose clk is present in new platforms with > >> same MMIO mappings. We can reuse the same clk handler support > >> for other platforms. Hence, changing name from ST(SoC) to FCH(IP) > >> > >> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> > >> --- > > Acked-by: Stephen Boyd <sboyd@kernel.org> > > Hi Rafael, > I see the status of these patches as Not Applicable in patchwork, is > there any pending action for me? Yes, there is. You need to let me know if you want me to apply them (and I mean the whole series). :-) Besides, I only can see 3 out of 4 patches, so if you want me to apply them, can you please resend the whole series with CCs to linux-acpi? Thanks! ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [v2 1/4] ACPI: APD: Change name from ST to FCH 2020-07-31 11:14 ` Rafael J. Wysocki @ 2020-07-31 13:33 ` Agrawal, Akshu 0 siblings, 0 replies; 6+ messages in thread From: Agrawal, Akshu @ 2020-07-31 13:33 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Stephen Boyd, Agrawal, Akshu, Rafael J. Wysocki, Len Brown, Michael Turquette, ACPI Devel Maling List, Linux Kernel Mailing List, linux-clk On 7/31/2020 4:44 PM, Rafael J. Wysocki wrote: > On Fri, Jul 31, 2020 at 2:44 AM Agrawal, Akshu <aagrawal2@amd.com> wrote: >> >> On 7/29/2020 6:56 AM, Stephen Boyd wrote: >>> Quoting Akshu Agrawal (2020-07-28 01:28:53) >>>> AMD SoC general pupose clk is present in new platforms with >>>> same MMIO mappings. We can reuse the same clk handler support >>>> for other platforms. Hence, changing name from ST(SoC) to FCH(IP) >>>> >>>> Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> >>>> --- >>> Acked-by: Stephen Boyd <sboyd@kernel.org> >> Hi Rafael, >> I see the status of these patches as Not Applicable in patchwork, is >> there any pending action for me? > Yes, there is. > > You need to let me know if you want me to apply them (and I mean the > whole series). :-) Yes, please apply the whole series. > > Besides, I only can see 3 out of 4 patches, so if you want me to apply > them, can you please resend the whole series with CCs to linux-acpi? Sending them again with cc to linux-acpi. Thanks, Akshu ^ permalink raw reply [flat|nested] 6+ messages in thread
* [3/4] ACPI: APD: Add a fmw property is_raven [not found] <20200728082857.10829-1-akshu.agrawal@amd.com> 2020-07-28 8:28 ` [v2 1/4] ACPI: APD: Change name from ST to FCH Akshu Agrawal @ 2020-07-28 8:28 ` Akshu Agrawal 1 sibling, 0 replies; 6+ messages in thread From: Akshu Agrawal @ 2020-07-28 8:28 UTC (permalink / raw) To: akshu.agrawal Cc: sboyd, rafael, Rafael J. Wysocki, Len Brown, open list:ACPI, open list Since there is slight difference in AMD RV based soc in misc clk architecture. The fmw property will help in differentiating the SoCs. Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> --- drivers/acpi/acpi_apd.c | 4 ++++ include/linux/platform_data/clk-fch.h | 1 + 2 files changed, 5 insertions(+) diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c index 2d99e46add1a..d879ba28826c 100644 --- a/drivers/acpi/acpi_apd.c +++ b/drivers/acpi/acpi_apd.c @@ -82,6 +82,7 @@ static int misc_check_res(struct acpi_resource *ares, void *data) static int fch_misc_setup(struct apd_private_data *pdata) { struct acpi_device *adev = pdata->adev; + const union acpi_object *obj; struct platform_device *clkdev; struct fch_clk_data *clk_data; struct resource_entry *rentry; @@ -98,6 +99,9 @@ static int fch_misc_setup(struct apd_private_data *pdata) if (ret < 0) return -ENOENT; + acpi_dev_get_property(adev, "is-rv", ACPI_TYPE_INTEGER, &obj); + clk_data->is_rv = obj->integer.value; + list_for_each_entry(rentry, &resource_list, node) { clk_data->base = devm_ioremap(&adev->dev, rentry->res->start, resource_size(rentry->res)); diff --git a/include/linux/platform_data/clk-fch.h b/include/linux/platform_data/clk-fch.h index 850ca776156d..b9f682459f08 100644 --- a/include/linux/platform_data/clk-fch.h +++ b/include/linux/platform_data/clk-fch.h @@ -12,6 +12,7 @@ struct fch_clk_data { void __iomem *base; + u32 is_rv; }; #endif /* __CLK_FCH_H */ -- 2.20.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [0/4] Extend AMD SoC general purpose clk for all versions @ 2020-07-31 13:36 Akshu Agrawal 2020-07-31 13:36 ` [v2 1/4] ACPI: APD: Change name from ST to FCH Akshu Agrawal 0 siblings, 1 reply; 6+ messages in thread From: Akshu Agrawal @ 2020-07-31 13:36 UTC (permalink / raw) To: akshu.agrawal Cc: sboyd, rafael, rjw, lenb, mturquette, linux-acpi, linux-kernel, linux-clk, rahul.tanwar Hi, This series of patch does the following: 1. Changes the name of the AMD SoC general purpose clk from ST(a version of SoC) to FCH (name of the IP). 2. Then make the drivers support both older and newer versions of SoC. Thanks, Akshu Akshu Agrawal (4): ACPI: APD: Change name from ST to FCH clk: x86: Change name from ST to FCH ACPI: APD: Add a fmw property is_raven clk: x86: Support RV architecture drivers/acpi/acpi_apd.c | 18 ++-- drivers/clk/x86/Makefile | 2 +- drivers/clk/x86/clk-fch.c | 101 ++++++++++++++++++ drivers/clk/x86/clk-st.c | 78 -------------- .../platform_data/{clk-st.h => clk-fch.h} | 11 +- 5 files changed, 119 insertions(+), 91 deletions(-) create mode 100644 drivers/clk/x86/clk-fch.c delete mode 100644 drivers/clk/x86/clk-st.c rename include/linux/platform_data/{clk-st.h => clk-fch.h} (51%) -- 2.20.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [v2 1/4] ACPI: APD: Change name from ST to FCH 2020-07-31 13:36 [0/4] Extend AMD SoC general purpose clk for all versions Akshu Agrawal @ 2020-07-31 13:36 ` Akshu Agrawal 0 siblings, 0 replies; 6+ messages in thread From: Akshu Agrawal @ 2020-07-31 13:36 UTC (permalink / raw) To: akshu.agrawal Cc: sboyd, rafael, rjw, lenb, mturquette, linux-acpi, linux-kernel, linux-clk, rahul.tanwar AMD SoC general pupose clk is present in new platforms with same MMIO mappings. We can reuse the same clk handler support for other platforms. Hence, changing name from ST(SoC) to FCH(IP) Signed-off-by: Akshu Agrawal <akshu.agrawal@amd.com> Acked-by: Stephen Boyd <sboyd@kernel.org> --- v2: pulled in clk changes so that patch compiles individually drivers/acpi/acpi_apd.c | 14 +++++++------- drivers/clk/x86/clk-st.c | 4 ++-- .../linux/platform_data/{clk-st.h => clk-fch.h} | 10 +++++----- 3 files changed, 14 insertions(+), 14 deletions(-) rename include/linux/platform_data/{clk-st.h => clk-fch.h} (53%) diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c index ba2612e9a0eb..2d99e46add1a 100644 --- a/drivers/acpi/acpi_apd.c +++ b/drivers/acpi/acpi_apd.c @@ -8,7 +8,7 @@ */ #include <linux/clk-provider.h> -#include <linux/platform_data/clk-st.h> +#include <linux/platform_data/clk-fch.h> #include <linux/platform_device.h> #include <linux/pm_domain.h> #include <linux/clkdev.h> @@ -79,11 +79,11 @@ static int misc_check_res(struct acpi_resource *ares, void *data) return !acpi_dev_resource_memory(ares, &res); } -static int st_misc_setup(struct apd_private_data *pdata) +static int fch_misc_setup(struct apd_private_data *pdata) { struct acpi_device *adev = pdata->adev; struct platform_device *clkdev; - struct st_clk_data *clk_data; + struct fch_clk_data *clk_data; struct resource_entry *rentry; struct list_head resource_list; int ret; @@ -106,7 +106,7 @@ static int st_misc_setup(struct apd_private_data *pdata) acpi_dev_free_resource_list(&resource_list); - clkdev = platform_device_register_data(&adev->dev, "clk-st", + clkdev = platform_device_register_data(&adev->dev, "clk-fch", PLATFORM_DEVID_NONE, clk_data, sizeof(*clk_data)); return PTR_ERR_OR_ZERO(clkdev); @@ -135,8 +135,8 @@ static const struct apd_device_desc cz_uart_desc = { .properties = uart_properties, }; -static const struct apd_device_desc st_misc_desc = { - .setup = st_misc_setup, +static const struct apd_device_desc fch_misc_desc = { + .setup = fch_misc_setup, }; #endif @@ -239,7 +239,7 @@ static const struct acpi_device_id acpi_apd_device_ids[] = { { "AMD0020", APD_ADDR(cz_uart_desc) }, { "AMDI0020", APD_ADDR(cz_uart_desc) }, { "AMD0030", }, - { "AMD0040", APD_ADDR(st_misc_desc)}, + { "AMD0040", APD_ADDR(fch_misc_desc)}, #endif #ifdef CONFIG_ARM64 { "APMC0D0F", APD_ADDR(xgene_i2c_desc) }, diff --git a/drivers/clk/x86/clk-st.c b/drivers/clk/x86/clk-st.c index 25d4b97aff9b..c2438874d9f2 100644 --- a/drivers/clk/x86/clk-st.c +++ b/drivers/clk/x86/clk-st.c @@ -8,7 +8,7 @@ #include <linux/clk.h> #include <linux/clkdev.h> #include <linux/clk-provider.h> -#include <linux/platform_data/clk-st.h> +#include <linux/platform_data/clk-fch.h> #include <linux/platform_device.h> /* Clock Driving Strength 2 register */ @@ -31,7 +31,7 @@ static struct clk_hw *hws[ST_MAX_CLKS]; static int st_clk_probe(struct platform_device *pdev) { - struct st_clk_data *st_data; + struct fch_clk_data *st_data; st_data = dev_get_platdata(&pdev->dev); if (!st_data || !st_data->base) diff --git a/include/linux/platform_data/clk-st.h b/include/linux/platform_data/clk-fch.h similarity index 53% rename from include/linux/platform_data/clk-st.h rename to include/linux/platform_data/clk-fch.h index 7cdb6a402b35..850ca776156d 100644 --- a/include/linux/platform_data/clk-st.h +++ b/include/linux/platform_data/clk-fch.h @@ -1,17 +1,17 @@ /* SPDX-License-Identifier: MIT */ /* - * clock framework for AMD Stoney based clock + * clock framework for AMD misc clocks * * Copyright 2018 Advanced Micro Devices, Inc. */ -#ifndef __CLK_ST_H -#define __CLK_ST_H +#ifndef __CLK_FCH_H +#define __CLK_FCH_H #include <linux/compiler.h> -struct st_clk_data { +struct fch_clk_data { void __iomem *base; }; -#endif /* __CLK_ST_H */ +#endif /* __CLK_FCH_H */ -- 2.20.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-07-31 13:36 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20200728082857.10829-1-akshu.agrawal@amd.com> 2020-07-28 8:28 ` [v2 1/4] ACPI: APD: Change name from ST to FCH Akshu Agrawal [not found] ` <159598596077.1360974.483730969007254506@swboyd.mtv.corp.google.com> 2020-07-31 0:44 ` Agrawal, Akshu 2020-07-31 11:14 ` Rafael J. Wysocki 2020-07-31 13:33 ` Agrawal, Akshu 2020-07-28 8:28 ` [3/4] ACPI: APD: Add a fmw property is_raven Akshu Agrawal 2020-07-31 13:36 [0/4] Extend AMD SoC general purpose clk for all versions Akshu Agrawal 2020-07-31 13:36 ` [v2 1/4] ACPI: APD: Change name from ST to FCH Akshu Agrawal
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).