[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJZ5v0jXnwC-C8mAWtQDtoPko9ALAYhpm3X-TZ5L83ROEUJWmA@mail.gmail.com>
Date: Fri, 5 Nov 2021 16:02:26 +0100
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Ajit Kumar Pandey <AjitKumar.Pandey@....com>
Cc: Stephen Boyd <sboyd@...nel.org>,
linux-clk <linux-clk@...r.kernel.org>, Vijendar.Mukunda@....com,
Alex Deucher <Alexander.Deucher@....com>,
Basavaraj.Hiregoudar@....com, Sunil-kumar.Dommati@....com,
"Rafael J. Wysocki" <rafael@...nel.org>,
Len Brown <lenb@...nel.org>,
"open list:ACPI" <linux-acpi@...r.kernel.org>,
open list <linux-kernel@...r.kernel.org>,
Mario Limonciello <mario.limonciello@....com>
Subject: Re: [PATCH v2 RESEND 2/5] drivers: acpi: acpi_apd: Remove unused
device property "is-rv"
On Fri, Oct 29, 2021 at 8:33 AM Ajit Kumar Pandey
<AjitKumar.Pandey@....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 clock config based selection to extend such support on others
> architecture. This change removed unused code from acpi driver.
>
> Signed-off-by: Ajit Kumar Pandey <AjitKumar.Pandey@....com>
Is this and the [3/5] applicable without the [1/5]?
If so, I can pick them up once they are approved by some other AMD
folks, preferably Mario.
> ---
> 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
>
Powered by blists - more mailing lists