Messages in this thread | | | From | Vadim Pasternak <> | Subject | RE: [patch v1] drivers/platform/x86: select hwmon for Mellanox hotplug driver in Kconfig | Date | Mon, 24 Oct 2016 06:01:36 +0000 |
| |
> -----Original Message----- > From: Randy Dunlap [mailto:rdunlap@infradead.org] > Sent: Sunday, October 23, 2016 7:57 PM > To: Vadim Pasternak <vadimp@mellanox.com>; dvhart@infradead.org; > fengguang.wu@intel.com > Cc: davem@davemloft.net; geert@linux-m68k.org; akpm@linux- > foundation.org; kvalo@codeaurora.org; gregkh@linuxfoundation.org; > mchehab@kernel.org; linux@roeck-us.net; linux-kernel@vger.kernel.org; > platform-driver-x86@vger.kernel.org; jiri@resnulli.us; > andriy.shevchenko@linux.intel.com > Subject: Re: [patch v1] drivers/platform/x86: select hwmon for Mellanox > hotplug driver in Kconfig > > On 10/23/16 05:45, vadimp@mellanox.com wrote: > > From: Vadim Pasternak <vadimp@mellanox.com> > > > > Add select HWMON for mlxcpld-hotplug driver config, since it requires > > hwmon APIs. > > > > Signed-off-by: Vadim Pasternak <vadimp@mellanox.com> > > --- > > drivers/platform/x86/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/platform/x86/Kconfig > > b/drivers/platform/x86/Kconfig index 65d9b04..1853769 100644 > > --- a/drivers/platform/x86/Kconfig > > +++ b/drivers/platform/x86/Kconfig > > @@ -1032,6 +1032,7 @@ config MLX_CPLD_PLATFORM > > tristate "Mellanox platform hotplug driver support" > > default n > > depends on MLX_PLATFORM > > + select HWMON > > select I2C > > ---help--- > > This driver handles hot-plug events for the power suppliers, power > > > > Hi, > did you test this? I would expect it to cause a kconfig warning, something about > HWMON having a dependency on HAS_IOMEM.
I re-tested it again. I don't see any warning from kconfig. It just set " CONFIG_HWMON=m" with no warnings. Could I miss something?
Thanks, Vadim. > > thanks. > -- > ~Randy
| |