From: kernel test robot <lkp@intel.com>
To: Jon Hunter <jonathanh@nvidia.com>,
Jean Delvare <jdelvare@suse.com>,
Guenter Roeck <linux@roeck-us.net>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzk@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Thierry Reding <thierry.reding@gmail.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-hwmon@vger.kernel.org,
devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
Ninad Malwade <nmalwade@nvidia.com>,
Rajkumar Kasirajan <rkasirajan@nvidia.com>,
Jon Hunter <jonathanh@nvidia.com>
Subject: Re: [PATCH V3 3/4] hwmon: ina3221: Add support for channel summation disable
Date: Fri, 22 Sep 2023 05:11:22 +0800 [thread overview]
Message-ID: <202309220454.kCi2xD48-lkp@intel.com> (raw)
In-Reply-To: <20230921130818.21247-4-jonathanh@nvidia.com>
Hi Jon,
kernel test robot noticed the following build warnings:
[auto build test WARNING on groeck-staging/hwmon-next]
[also build test WARNING on robh/for-next linus/master v6.6-rc2 next-20230921]
[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/Jon-Hunter/dt-bindings-hwmon-ina3221-Add-ti-summation-disable/20230922-011119
base: https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
patch link: https://lore.kernel.org/r/20230921130818.21247-4-jonathanh%40nvidia.com
patch subject: [PATCH V3 3/4] hwmon: ina3221: Add support for channel summation disable
config: m68k-allyesconfig (https://download.01.org/0day-ci/archive/20230922/202309220454.kCi2xD48-lkp@intel.com/config)
compiler: m68k-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230922/202309220454.kCi2xD48-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/202309220454.kCi2xD48-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/hwmon/ina3221.c:109: warning: Function parameter or member 'summation_disable' not described in 'ina3221_input'
>> drivers/hwmon/ina3221.c:134: warning: Function parameter or member 'debugfs' not described in 'ina3221_data'
>> drivers/hwmon/ina3221.c:134: warning: Function parameter or member 'summation_channel_control' not described in 'ina3221_data'
vim +109 drivers/hwmon/ina3221.c
7cb6dcff1956ec Andrew F. Davis 2016-06-10 97
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 98 /**
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 99 * struct ina3221_input - channel input source specific information
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 100 * @label: label of channel input source
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 101 * @shunt_resistor: shunt resistor value of channel input source
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 102 * @disconnected: connection status of channel input source
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 103 */
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 104 struct ina3221_input {
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 105 const char *label;
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 106 int shunt_resistor;
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 107 bool disconnected;
ac0a832fd3617c Ninad Malwade 2023-09-21 108 bool summation_disable;
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 @109 };
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 110
7cb6dcff1956ec Andrew F. Davis 2016-06-10 111 /**
7cb6dcff1956ec Andrew F. Davis 2016-06-10 112 * struct ina3221_data - device specific information
323aeb0eb5d9a6 Nicolin Chen 2018-11-05 113 * @pm_dev: Device pointer for pm runtime
7cb6dcff1956ec Andrew F. Davis 2016-06-10 114 * @regmap: Register map of the device
7cb6dcff1956ec Andrew F. Davis 2016-06-10 115 * @fields: Register fields of the device
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 116 * @inputs: Array of channel input source specific structures
87625b24986bc2 Nicolin Chen 2018-11-05 117 * @lock: mutex lock to serialize sysfs attribute accesses
59d608e152e582 Nicolin Chen 2018-09-29 118 * @reg_config: Register value of INA3221_CONFIG
2057bdfb7184e9 Nicolin Chen 2019-10-16 119 * @summation_shunt_resistor: equivalent shunt resistor value for summation
43dece162de047 Nicolin Chen 2019-01-17 120 * @single_shot: running in single-shot operating mode
7cb6dcff1956ec Andrew F. Davis 2016-06-10 121 */
7cb6dcff1956ec Andrew F. Davis 2016-06-10 122 struct ina3221_data {
323aeb0eb5d9a6 Nicolin Chen 2018-11-05 123 struct device *pm_dev;
7cb6dcff1956ec Andrew F. Davis 2016-06-10 124 struct regmap *regmap;
7cb6dcff1956ec Andrew F. Davis 2016-06-10 125 struct regmap_field *fields[F_MAX_FIELDS];
a9e9dd9c6de5d8 Nicolin Chen 2018-10-01 126 struct ina3221_input inputs[INA3221_NUM_CHANNELS];
87625b24986bc2 Nicolin Chen 2018-11-05 127 struct mutex lock;
ac0a832fd3617c Ninad Malwade 2023-09-21 128 struct dentry *debugfs;
59d608e152e582 Nicolin Chen 2018-09-29 129 u32 reg_config;
2057bdfb7184e9 Nicolin Chen 2019-10-16 130 int summation_shunt_resistor;
ac0a832fd3617c Ninad Malwade 2023-09-21 131 u32 summation_channel_control;
43dece162de047 Nicolin Chen 2019-01-17 132
43dece162de047 Nicolin Chen 2019-01-17 133 bool single_shot;
7cb6dcff1956ec Andrew F. Davis 2016-06-10 @134 };
7cb6dcff1956ec Andrew F. Davis 2016-06-10 135
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-09-21 21:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-21 13:08 [PATCH V3 0/4] hwmon: ina3221: Add selective summation support Jon Hunter
2023-09-21 13:08 ` [PATCH V3 1/4] dt-bindings: hwmon: ina3221: Convert to json-schema Jon Hunter
2023-09-22 21:01 ` Rob Herring
2023-09-25 10:46 ` Jon Hunter
2023-09-26 9:18 ` Conor Dooley
2023-09-26 12:02 ` Jon Hunter
2023-09-21 13:08 ` [PATCH V3 2/4] dt-bindings: hwmon: ina3221: Add ti,summation-disable Jon Hunter
2023-09-22 21:06 ` Rob Herring
2023-09-25 10:50 ` Jon Hunter
2023-09-21 13:08 ` [PATCH V3 3/4] hwmon: ina3221: Add support for channel summation disable Jon Hunter
2023-09-21 21:11 ` kernel test robot [this message]
2023-09-21 13:08 ` [PATCH V3 4/4] arm64: tegra: Add power-sensors for Tegra234 boards Jon Hunter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202309220454.kCi2xD48-lkp@intel.com \
--to=lkp@intel.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=jdelvare@suse.com \
--cc=jonathanh@nvidia.com \
--cc=krzk@kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=nmalwade@nvidia.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=rkasirajan@nvidia.com \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).