Re: [PATCH v2] platform/x86: support to store/show powermode value for Inspur

[kernel test robot <lkp@intel.com>]

Hi Ai,

kernel test robot noticed the following build warnings:

[auto build test WARNING on linus/master]
[also build test WARNING on v6.6-rc6 next-20231020]
[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/Ai-Chao/platform-x86-support-to-store-show-powermode-value-for-Inspur/20231017-125537
base:   linus/master
patch link:    https://lore.kernel.org/r/20231014032844.3823198-1-aichao%40kylinos.cn
patch subject: [PATCH v2] platform/x86: support to store/show powermode value for Inspur
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20231020/202310202354.NjaAKTX2-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231020/202310202354.NjaAKTX2-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202310202354.NjaAKTX2-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/platform/x86/inspur-wmi.c:70: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
    * Set Power Mode to EC RAM. If Power Mode value greater than 0x3,
   drivers/platform/x86/inspur-wmi.c:111: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
    * Get Power Mode from EC RAM, If Power Mode value greater than 0x3,


vim +70 drivers/platform/x86/inspur-wmi.c

    68	
    69	/**
  > 70	 * Set Power Mode to EC RAM. If Power Mode value greater than 0x3,
    71	 * return error
    72	 * Method ID: 0x3
    73	 * Arg: 4 Bytes
    74	 * Byte [0]: Power Mode:
    75	 *         0x0: Balance Mode
    76	 *         0x1: Performance Mode
    77	 *         0x2: Power Saver Mode
    78	 * Return Value: 4 Bytes
    79	 * Byte [0]: Return Code
    80	 *         0x0: No Error
    81	 *         0x1: Error
    82	 */
    83	static ssize_t powermode_store(struct device *dev,
    84				       struct device_attribute *attr,
    85				       const char *buf, size_t count)
    86	{
    87		struct inspur_wmi_priv *priv = dev_get_drvdata(dev);
    88		int ret;
    89		u32 mode;
    90		u8 *ret_code;
    91	
    92		ret = kstrtoint(buf, 0, &mode);
    93		if (ret)
    94			return ret;
    95	
    96		ret = inspur_wmi_perform_query(priv->wdev,
    97					       INSPUR_WMI_SET_POWERMODE,
    98					       &mode, sizeof(mode), sizeof(mode));
    99	
   100		if (ret < 0)
   101			return ret;
   102	
   103		ret_code = (u8 *)(&mode);
   104		if (ret_code[0])
   105			return -EBADRQC;
   106	
   107		return count;
   108	}
   109	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki