On Mon, Jan 11, 2021 at 11:15:06PM -0800, Randy Dunlap wrote: > On 1/8/21 1:25 PM, mgr...@linux.intel.com wrote: > > diff --git a/drivers/misc/intel_tsens/Kconfig > > b/drivers/misc/intel_tsens/Kconfig > > index 8b263fdd80c3..c2138339bd89 100644 > > --- a/drivers/misc/intel_tsens/Kconfig > > +++ b/drivers/misc/intel_tsens/Kconfig > > @@ -14,6 +14,20 @@ config INTEL_TSENS_LOCAL_HOST > > Say Y if using a processor that includes the Intel VPU such as > > Keem Bay. If unsure, say N. > > > > +config INTEL_TSENS_I2C_SLAVE > > + bool "I2C slave driver for intel tsens" > > Why bool instead of tristate? Becuase the I2C driver depends on a file scoped global i2c_plat_data instanciated in the INTELL_TSENS_LOCAL_HOST DRIVER (intel_tsens_thermal.[ch])
Udhaya, would you care to comment further? --mark > > > + depends on INTEL_TSENS_LOCAL_HOST > > + select I2C > > + select I2C_SLAVE > > + help > > + This option enables tsens i2c slave driver. > > I2C > > > + > > + This driver is used for reporting thermal data via I2C > > + SMBUS to remote host. > > + Enable this option if you want to have support for thermal > > + management controller > > controller. > > > + Say Y if using a processor that includes the Intel VPU such as > > + Keem Bay. If unsure, say N. > > > -- > ~Randy >