linux-kernel - Re: [PATCH] ACPI / LPSS: Add BYT ACPI mode PWM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<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/