[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YLU7/mJ4dFTTWN5m@smile.fi.intel.com>
Date: Mon, 31 May 2021 22:41:50 +0300
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Flavio Suligoi <f.suligoi@...m.it>,
Uwe Kleine-König
<u.kleine-koenig@...gutronix.de>,
Thierry Reding <thierry.reding@...il.com>,
linux-acpi@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-pwm@...r.kernel.org
Cc: "Rafael J. Wysocki" <rjw@...ysocki.net>,
Len Brown <lenb@...nel.org>, Lee Jones <lee.jones@...aro.org>
Subject: Re: [PATCH v1 1/5] docs: firmware-guide: ACPI: Add a PWM example
On Mon, May 31, 2021 at 07:09:13PM +0300, Andy Shevchenko wrote:
> On Mon, May 31, 2021 at 06:43:47PM +0300, Andy Shevchenko wrote:
> > When PWM support for ACPI has been added into the kernel, it missed
> > the documentation update. Hence update documentation here.
>
> I am actually in doubt that original commit message provides that short period
> intentionally. Possibly it was misinterpretation of nanoseconds (which is
> expected by PWM framework) with microseconds than might sound logical for LED.
>
> Perhaps we may add 000 to the value (1) and replace us by ms in the text (2).
Okay, I will send an extended v2 and I think I will use 600 ms in the
documentation example.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists