Re: [PATCH v4] iio: pressure: mpl115: Implementing low power mode by shutdown gpio

From: Jonathan Cameron
Date: Sun Oct 09 2022 - 13:12:25 EST


On Mon, 3 Oct 2022 18:17:57 +0800
kernel test robot <lkp@xxxxxxxxx> wrote:

> Hi Rajat,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on jic23-iio/togreg]
> [also build test ERROR on linus/master v6.0 next-20220930]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Rajat-Khandelwal/iio-pressure-mpl115-Implementing-low-power-mode-by-shutdown-gpio/20221003-133904
> base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
> config: x86_64-randconfig-a005-20221003
> compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://github.com/intel-lab-lkp/linux/commit/284b52f6eb0616aeb44203d7114c4fb1e80b785d
> git remote add linux-review https://github.com/intel-lab-lkp/linux
> git fetch --no-tags linux-review Rajat-Khandelwal/iio-pressure-mpl115-Implementing-low-power-mode-by-shutdown-gpio/20221003-133904
> git checkout 284b52f6eb0616aeb44203d7114c4fb1e80b785d
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag where applicable
> | Reported-by: kernel test robot <lkp@xxxxxxxxx>
>
> All errors (new ones prefixed by >>, old ones prefixed by <<):
>
> >> ERROR: modpost: "mpl115_dev_pm_ops" [drivers/iio/pressure/mpl115_spi.ko] undefined!
>

You have probably figured this out, but you need the
EXPORT_NS_RUNTIME_DEV_PM_OPS()

with appropriate namespace set.

Thanks,
Jonathan