* [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv"
[not found] <20211212180527.1641362-1-AjitKumar.Pandey@amd.com>
@ 2021-12-12 18:05 ` Ajit Kumar Pandey
2021-12-17 18:06 ` Rafael J. Wysocki
2021-12-12 18:05 ` [PATCH v5 3/5] ACPI: APD: Add a fmw property clk-name Ajit Kumar Pandey
1 sibling, 1 reply; 5+ messages in thread
From: Ajit Kumar Pandey @ 2021-12-12 18:05 UTC (permalink / raw)
To: sboyd, linux-clk
Cc: Vijendar.Mukunda, Alexander.Deucher, Basavaraj.Hiregoudar,
Sunil-kumar.Dommati, Mario.Limonciello, Ajit Kumar Pandey,
Rafael J. Wysocki, Len Brown, open list:ACPI, open list
Initially "is-rv" device property is added for 48MHz fixed clock
support on Raven or RV architecture. It's unused now as we moved
to pci device_id based selection to extend such support on other
architectures. This change removed unused code from acpi driver.
Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>
Reviewed-by: Mario Limonciello <Mario.Limonciello@amd.com>
---
drivers/acpi/acpi_apd.c | 3 ---
include/linux/platform_data/clk-fch.h | 1 -
2 files changed, 4 deletions(-)
diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
index 6e02448d15d9..6913e9712852 100644
--- a/drivers/acpi/acpi_apd.c
+++ b/drivers/acpi/acpi_apd.c
@@ -87,9 +87,6 @@ static int fch_misc_setup(struct apd_private_data *pdata)
if (ret < 0)
return -ENOENT;
- if (!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 b9f682459f08..850ca776156d 100644
--- a/include/linux/platform_data/clk-fch.h
+++ b/include/linux/platform_data/clk-fch.h
@@ -12,7 +12,6 @@
struct fch_clk_data {
void __iomem *base;
- u32 is_rv;
};
#endif /* __CLK_FCH_H */
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v5 3/5] ACPI: APD: Add a fmw property clk-name
[not found] <20211212180527.1641362-1-AjitKumar.Pandey@amd.com>
2021-12-12 18:05 ` [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv" Ajit Kumar Pandey
@ 2021-12-12 18:05 ` Ajit Kumar Pandey
1 sibling, 0 replies; 5+ messages in thread
From: Ajit Kumar Pandey @ 2021-12-12 18:05 UTC (permalink / raw)
To: sboyd, linux-clk
Cc: Vijendar.Mukunda, Alexander.Deucher, Basavaraj.Hiregoudar,
Sunil-kumar.Dommati, Mario.Limonciello, Ajit Kumar Pandey,
Rafael J. Wysocki, Len Brown, open list:ACPI, open list
Add a new device property to fetch clk-name from firmware.
Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>
Reviewed-by: Mario Limonciello <Mario.Limonciello@amd.com>
---
drivers/acpi/acpi_apd.c | 10 ++++++++++
include/linux/platform_data/clk-fch.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
index 6913e9712852..2b958b426b03 100644
--- a/drivers/acpi/acpi_apd.c
+++ b/drivers/acpi/acpi_apd.c
@@ -87,6 +87,16 @@ static int fch_misc_setup(struct apd_private_data *pdata)
if (ret < 0)
return -ENOENT;
+ if (!acpi_dev_get_property(adev, "clk-name", ACPI_TYPE_STRING, &obj)) {
+ clk_data->name = devm_kzalloc(&adev->dev, obj->string.length,
+ GFP_KERNEL);
+
+ strcpy(clk_data->name, obj->string.pointer);
+ } else {
+ /* Set default name to mclk if entry missing in firmware */
+ clk_data->name = "mclk";
+ }
+
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..11a2a23fd9b2 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;
+ char *name;
};
#endif /* __CLK_FCH_H */
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv"
2021-12-12 18:05 ` [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv" Ajit Kumar Pandey
@ 2021-12-17 18:06 ` Rafael J. Wysocki
2021-12-17 18:19 ` Limonciello, Mario
0 siblings, 1 reply; 5+ messages in thread
From: Rafael J. Wysocki @ 2021-12-17 18:06 UTC (permalink / raw)
To: Ajit Kumar Pandey
Cc: Stephen Boyd, linux-clk, Vijendar.Mukunda, Alex Deucher,
Basavaraj.Hiregoudar, Sunil-kumar.Dommati, Mario Limonciello,
Rafael J. Wysocki, Len Brown, open list:ACPI, open list
On Sun, Dec 12, 2021 at 7:06 PM Ajit Kumar Pandey
<AjitKumar.Pandey@amd.com> wrote:
>
> Initially "is-rv" device property is added for 48MHz fixed clock
> support on Raven or RV architecture. It's unused now as we moved
> to pci device_id based selection to extend such support on other
> architectures. This change removed unused code from acpi driver.
>
> Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>
> Reviewed-by: Mario Limonciello <Mario.Limonciello@amd.com>
Does this or the next patch depend on the rest of the series, or can I
simply apply them both?
> ---
> drivers/acpi/acpi_apd.c | 3 ---
> include/linux/platform_data/clk-fch.h | 1 -
> 2 files changed, 4 deletions(-)
>
> diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
> index 6e02448d15d9..6913e9712852 100644
> --- a/drivers/acpi/acpi_apd.c
> +++ b/drivers/acpi/acpi_apd.c
> @@ -87,9 +87,6 @@ static int fch_misc_setup(struct apd_private_data *pdata)
> if (ret < 0)
> return -ENOENT;
>
> - if (!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 b9f682459f08..850ca776156d 100644
> --- a/include/linux/platform_data/clk-fch.h
> +++ b/include/linux/platform_data/clk-fch.h
> @@ -12,7 +12,6 @@
>
> struct fch_clk_data {
> void __iomem *base;
> - u32 is_rv;
> };
>
> #endif /* __CLK_FCH_H */
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv"
2021-12-17 18:06 ` Rafael J. Wysocki
@ 2021-12-17 18:19 ` Limonciello, Mario
2021-12-17 18:23 ` Rafael J. Wysocki
0 siblings, 1 reply; 5+ messages in thread
From: Limonciello, Mario @ 2021-12-17 18:19 UTC (permalink / raw)
To: Rafael J. Wysocki, Pandey, Ajit Kumar
Cc: Stephen Boyd, linux-clk, Mukunda, Vijendar, Deucher, Alexander,
Hiregoudar, Basavaraj, Dommati, Sunil-kumar, Len Brown,
open list:ACPI, open list
[Public]
> -----Original Message-----
> From: Rafael J. Wysocki <rafael@kernel.org>
> Sent: Friday, December 17, 2021 12:07
> To: Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>
> Cc: Stephen Boyd <sboyd@kernel.org>; linux-clk <linux-clk@vger.kernel.org>;
> Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Deucher, Alexander
> <Alexander.Deucher@amd.com>; Hiregoudar, Basavaraj
> <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-
> kumar.Dommati@amd.com>; Limonciello, Mario
> <Mario.Limonciello@amd.com>; Rafael J. Wysocki <rafael@kernel.org>; Len
> Brown <lenb@kernel.org>; open list:ACPI <linux-acpi@vger.kernel.org>; open
> list <linux-kernel@vger.kernel.org>
> Subject: Re: [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device
> property "is-rv"
>
> On Sun, Dec 12, 2021 at 7:06 PM Ajit Kumar Pandey
> <AjitKumar.Pandey@amd.com> wrote:
> >
> > Initially "is-rv" device property is added for 48MHz fixed clock
> > support on Raven or RV architecture. It's unused now as we moved
> > to pci device_id based selection to extend such support on other
> > architectures. This change removed unused code from acpi driver.
> >
> > Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>
> > Reviewed-by: Mario Limonciello <Mario.Limonciello@amd.com>
>
> Does this or the next patch depend on the rest of the series, or can I
> simply apply them both?
There are (intentional) dependencies within this series on the order. It
would be better for the whole series to come through one tree.
>
> > ---
> > drivers/acpi/acpi_apd.c | 3 ---
> > include/linux/platform_data/clk-fch.h | 1 -
> > 2 files changed, 4 deletions(-)
> >
> > diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
> > index 6e02448d15d9..6913e9712852 100644
> > --- a/drivers/acpi/acpi_apd.c
> > +++ b/drivers/acpi/acpi_apd.c
> > @@ -87,9 +87,6 @@ static int fch_misc_setup(struct apd_private_data
> *pdata)
> > if (ret < 0)
> > return -ENOENT;
> >
> > - if (!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 b9f682459f08..850ca776156d 100644
> > --- a/include/linux/platform_data/clk-fch.h
> > +++ b/include/linux/platform_data/clk-fch.h
> > @@ -12,7 +12,6 @@
> >
> > struct fch_clk_data {
> > void __iomem *base;
> > - u32 is_rv;
> > };
> >
> > #endif /* __CLK_FCH_H */
> > --
> > 2.25.1
> >
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv"
2021-12-17 18:19 ` Limonciello, Mario
@ 2021-12-17 18:23 ` Rafael J. Wysocki
0 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2021-12-17 18:23 UTC (permalink / raw)
To: Limonciello, Mario
Cc: Rafael J. Wysocki, Pandey, Ajit Kumar, Stephen Boyd, linux-clk,
Mukunda, Vijendar, Deucher, Alexander, Hiregoudar, Basavaraj,
Dommati, Sunil-kumar, Len Brown, open list:ACPI, open list
On Fri, Dec 17, 2021 at 7:19 PM Limonciello, Mario
<Mario.Limonciello@amd.com> wrote:
>
> [Public]
>
> > -----Original Message-----
> > From: Rafael J. Wysocki <rafael@kernel.org>
> > Sent: Friday, December 17, 2021 12:07
> > To: Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>
> > Cc: Stephen Boyd <sboyd@kernel.org>; linux-clk <linux-clk@vger.kernel.org>;
> > Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Deucher, Alexander
> > <Alexander.Deucher@amd.com>; Hiregoudar, Basavaraj
> > <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-
> > kumar.Dommati@amd.com>; Limonciello, Mario
> > <Mario.Limonciello@amd.com>; Rafael J. Wysocki <rafael@kernel.org>; Len
> > Brown <lenb@kernel.org>; open list:ACPI <linux-acpi@vger.kernel.org>; open
> > list <linux-kernel@vger.kernel.org>
> > Subject: Re: [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device
> > property "is-rv"
> >
> > On Sun, Dec 12, 2021 at 7:06 PM Ajit Kumar Pandey
> > <AjitKumar.Pandey@amd.com> wrote:
> > >
> > > Initially "is-rv" device property is added for 48MHz fixed clock
> > > support on Raven or RV architecture. It's unused now as we moved
> > > to pci device_id based selection to extend such support on other
> > > architectures. This change removed unused code from acpi driver.
> > >
> > > Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>
> > > Reviewed-by: Mario Limonciello <Mario.Limonciello@amd.com>
> >
> > Does this or the next patch depend on the rest of the series, or can I
> > simply apply them both?
>
> There are (intentional) dependencies within this series on the order. It
> would be better for the whole series to come through one tree.
So please route these two along with the rest of the series and I
don't think you need my ACKs for this.
>
> >
> > > ---
> > > drivers/acpi/acpi_apd.c | 3 ---
> > > include/linux/platform_data/clk-fch.h | 1 -
> > > 2 files changed, 4 deletions(-)
> > >
> > > diff --git a/drivers/acpi/acpi_apd.c b/drivers/acpi/acpi_apd.c
> > > index 6e02448d15d9..6913e9712852 100644
> > > --- a/drivers/acpi/acpi_apd.c
> > > +++ b/drivers/acpi/acpi_apd.c
> > > @@ -87,9 +87,6 @@ static int fch_misc_setup(struct apd_private_data
> > *pdata)
> > > if (ret < 0)
> > > return -ENOENT;
> > >
> > > - if (!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 b9f682459f08..850ca776156d 100644
> > > --- a/include/linux/platform_data/clk-fch.h
> > > +++ b/include/linux/platform_data/clk-fch.h
> > > @@ -12,7 +12,6 @@
> > >
> > > struct fch_clk_data {
> > > void __iomem *base;
> > > - u32 is_rv;
> > > };
> > >
> > > #endif /* __CLK_FCH_H */
> > > --
> > > 2.25.1
> > >
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-12-17 18:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20211212180527.1641362-1-AjitKumar.Pandey@amd.com>
2021-12-12 18:05 ` [PATCH v5 2/5] drivers: acpi: acpi_apd: Remove unused device property "is-rv" Ajit Kumar Pandey
2021-12-17 18:06 ` Rafael J. Wysocki
2021-12-17 18:19 ` Limonciello, Mario
2021-12-17 18:23 ` Rafael J. Wysocki
2021-12-12 18:05 ` [PATCH v5 3/5] ACPI: APD: Add a fmw property clk-name Ajit Kumar Pandey
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).