[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210118122950.GE4077@smile.fi.intel.com>
Date: Mon, 18 Jan 2021 14:29:50 +0200
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Laurent Pinchart <laurent.pinchart@...asonboard.com>
Cc: Daniel Scally <djrscally@...il.com>, linux-kernel@...r.kernel.org,
linux-acpi@...r.kernel.org, linux-gpio@...r.kernel.org,
linux-i2c@...r.kernel.org, platform-driver-x86@...r.kernel.org,
devel@...ica.org, rjw@...ysocki.net, lenb@...nel.org,
andy@...nel.org, mika.westerberg@...ux.intel.com,
linus.walleij@...aro.org, bgolaszewski@...libre.com,
wsa@...nel.org, lee.jones@...aro.org, hdegoede@...hat.com,
mgross@...ux.intel.com, robert.moore@...el.com,
erik.kaneda@...el.com, sakari.ailus@...ux.intel.com,
kieran.bingham@...asonboard.com
Subject: Re: [PATCH v2 1/7] acpi: utils: move acpi_lpss_dep() to utils
On Mon, Jan 18, 2021 at 09:24:10AM +0200, Laurent Pinchart wrote:
> On Mon, Jan 18, 2021 at 12:34:22AM +0000, Daniel Scally wrote:
...
> > +bool acpi_lpss_dep(struct acpi_device *adev, acpi_handle handle);
>
> "lpss" stands for low power subsystem, an Intel device within the PCH
> that handles I2C, SPI, UART, ... I think the function should be renamed,
> as it's now generic. acpi_dev_has_dep() is a potential candidate, I'm
> sure better ones exist. A bit of kerneldoc would also not hurt.
Actually a good suggestions. Please apply my tag after addressing above.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists