[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <7753658.iUv9mCkDx9@vostro.rjw.lan>
Date: Wed, 05 Feb 2014 01:10:46 +0100
From: "Rafael J. Wysocki" <rjw@...ysocki.net>
To: Chew Chiau Ee <chiau.ee.chew@...el.com>
Cc: Len Brown <lenb@...nel.org>, linux-acpi@...r.kernel.org,
linux-kernel@...r.kernel.org,
Mika Westerberg <mika.westerberg@...ux.intel.com>
Subject: Re: [PATCH] ACPI / LPSS: Add BYT ACPI mode PWM
On Tuesday, January 21, 2014 03:12:02 AM Chew Chiau Ee wrote:
> From: Chew, Chiau Ee <chiau.ee.chew@...el.com>
>
> Intel BayTrail LPSS consists of two PWM controllers which can
> be enumerated from ACPI namespace.
First, please say "Bay Trail" (or similar) instead of BYT in the subject.
Second, please say in the changelog what the patch will cause to happen.
For example, "This change will cause platform device objects to be
created for PWM controllers on Bay Trail which will allow the <driver
that's going to handle those things> driver to bind to them and handle
those devices."
Finally, the PWM maintainer has to see this.
Thanks!
> Signed-off-by: Chew, Chiau Ee <chiau.ee.chew@...el.com>
> ---
> drivers/acpi/acpi_lpss.c | 11 +++++++++++
> 1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/acpi/acpi_lpss.c b/drivers/acpi/acpi_lpss.c
> index 6745fe1..8c2bae9 100644
> --- a/drivers/acpi/acpi_lpss.c
> +++ b/drivers/acpi/acpi_lpss.c
> @@ -102,6 +102,16 @@ static struct lpss_device_desc lpt_sdio_dev_desc = {
> .ltr_required = true,
> };
>
> +static struct lpss_shared_clock pwm_clock = {
> + .name = "pwm_clk",
> + .rate = 25000000,
> +};
> +
> +static struct lpss_device_desc byt_pwm_dev_desc = {
> + .clk_required = true,
> + .shared_clock = &pwm_clock,
> +};
> +
> static struct lpss_shared_clock uart_clock = {
> .name = "uart_clk",
> .rate = 44236800,
> @@ -157,6 +167,7 @@ static const struct acpi_device_id acpi_lpss_device_ids[] = {
> { "INT33C7", },
>
> /* BayTrail LPSS devices */
> + { "80860F09", (unsigned long)&byt_pwm_dev_desc },
> { "80860F0A", (unsigned long)&byt_uart_dev_desc },
> { "80860F0E", (unsigned long)&byt_spi_dev_desc },
> { "80860F14", (unsigned long)&byt_sdio_dev_desc },
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists