From: kernel test robot <lkp@intel.com>
To: "Nuno Sá via B4 Relay" <devnull+nuno.sa.analog.com@kernel.org>,
linux-gpio@vger.kernel.org, linux-pwm@vger.kernel.org,
devicetree@vger.kernel.org, linux-input@vger.kernel.org
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
"Lee Jones" <lee@kernel.org>, "Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Uwe Kleine-König" <ukleinek@kernel.org>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Bartosz Golaszewski" <brgl@bgdev.pl>,
"Dmitry Torokhov" <dmitry.torokhov@gmail.com>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
"Liu Ying" <victor.liu@nxp.com>
Subject: Re: [PATCH 06/18] mfd: adp5585: add support for adp5589
Date: Fri, 14 Mar 2025 17:12:16 +0800 [thread overview]
Message-ID: <202503141607.g4arza1A-lkp@intel.com> (raw)
In-Reply-To: <20250313-dev-adp5589-fw-v1-6-20e80d4bd4ea@analog.com>
Hi Nuno,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 4d395cb071a343196ca524d3694790f06978fe91]
url: https://github.com/intel-lab-lkp/linux/commits/Nuno-S-via-B4-Relay/dt-bindings-mfd-adp5585-ease-on-the-required-properties/20250313-222511
base: 4d395cb071a343196ca524d3694790f06978fe91
patch link: https://lore.kernel.org/r/20250313-dev-adp5589-fw-v1-6-20e80d4bd4ea%40analog.com
patch subject: [PATCH 06/18] mfd: adp5585: add support for adp5589
config: riscv-randconfig-001-20250314 (https://download.01.org/0day-ci/archive/20250314/202503141607.g4arza1A-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250314/202503141607.g4arza1A-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/202503141607.g4arza1A-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/mfd/adp5585.c:48:41: warning: unused variable 'adp5589_volatile_regs' [-Wunused-const-variable]
48 | static const struct regmap_access_table adp5589_volatile_regs = {
| ^~~~~~~~~~~~~~~~~~~~~
>> drivers/mfd/adp5585.c:229:34: warning: unused variable 'adp5585_info' [-Wunused-const-variable]
229 | static const struct adp5585_info adp5585_info = {
| ^~~~~~~~~~~~
>> drivers/mfd/adp5585.c:239:34: warning: unused variable 'adp5585_01_info' [-Wunused-const-variable]
239 | static const struct adp5585_info adp5585_01_info = {
| ^~~~~~~~~~~~~~~
>> drivers/mfd/adp5585.c:249:34: warning: unused variable 'adp5585_02_info' [-Wunused-const-variable]
249 | static const struct adp5585_info adp5585_02_info = {
| ^~~~~~~~~~~~~~~
4 warnings generated.
vim +/adp5589_volatile_regs +48 drivers/mfd/adp5585.c
47
> 48 static const struct regmap_access_table adp5589_volatile_regs = {
49 .yes_ranges = adp5589_volatile_ranges,
50 .n_yes_ranges = ARRAY_SIZE(adp5589_volatile_ranges),
51 };
52
53 /*
54 * Chip variants differ in the default configuration of pull-up and pull-down
55 * resistors, and therefore have different default register values:
56 *
57 * - The -00, -01 and -03 variants (collectively referred to as
58 * ADP5585_REGMAP_00) have pull-up on all GPIO pins by default.
59 * - The -02 variant has no default pull-up or pull-down resistors.
60 * - The -04 variant has default pull-down resistors on all GPIO pins.
61 */
62
63 static const u8 adp5585_regmap_defaults_00[ADP5585_MAX_REG + 1] = {
64 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
65 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
66 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
67 /* 0x18 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
68 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
69 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
70 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
71 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00,
72 };
73
74 static const u8 adp5585_regmap_defaults_02[ADP5585_MAX_REG + 1] = {
75 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
76 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
77 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xc3,
78 /* 0x18 */ 0x03, 0x00, 0x03, 0x00, 0x00, 0x00, 0x00, 0x00,
79 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
80 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
81 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
82 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00,
83 };
84
85 static const u8 adp5585_regmap_defaults_04[ADP5585_MAX_REG + 1] = {
86 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
87 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
88 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x55,
89 /* 0x18 */ 0x05, 0x55, 0x01, 0x00, 0x00, 0x00, 0x00, 0x00,
90 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
91 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
92 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
93 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00,
94 };
95
96 static const u8 adp5589_regmap_defaults_00[ADP5589_MAX_REG + 1] = {
97 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
98 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
99 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
100 /* 0x18 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
101 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
102 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
103 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
104 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
105 /* 0x40 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
106 /* 0x48 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
107 };
108
109 static const u8 adp5589_regmap_defaults_01[ADP5589_MAX_REG + 1] = {
110 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
111 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
112 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
113 /* 0x18 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
114 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
115 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
116 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
117 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x20, 0x00, 0x00,
118 /* 0x40 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
119 /* 0x48 */ 0x00, 0x00, 0x00, 0x00, 0x20, 0x00, 0x00,
120 };
121
122 static const u8 adp5589_regmap_defaults_02[ADP5589_MAX_REG + 1] = {
123 /* 0x00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
124 /* 0x08 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
125 /* 0x10 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
126 /* 0x18 */ 0x00, 0x41, 0x01, 0x00, 0x11, 0x04, 0x00, 0x00,
127 /* 0x20 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
128 /* 0x28 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
129 /* 0x30 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
130 /* 0x38 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
131 /* 0x40 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
132 /* 0x48 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
133 };
134
135 enum adp5585_regmap_type {
136 ADP5585_REGMAP_00,
137 ADP5585_REGMAP_02,
138 ADP5585_REGMAP_04,
139 ADP5589_REGMAP_00,
140 ADP5589_REGMAP_01,
141 ADP5589_REGMAP_02,
142 };
143
144 static const struct regmap_config adp5585_regmap_configs[] = {
145 [ADP5585_REGMAP_00] = {
146 .reg_bits = 8,
147 .val_bits = 8,
148 .max_register = ADP5585_MAX_REG,
149 .volatile_table = &adp5585_volatile_regs,
150 .cache_type = REGCACHE_MAPLE,
151 .reg_defaults_raw = adp5585_regmap_defaults_00,
152 .num_reg_defaults_raw = sizeof(adp5585_regmap_defaults_00),
153 },
154 [ADP5585_REGMAP_02] = {
155 .reg_bits = 8,
156 .val_bits = 8,
157 .max_register = ADP5585_MAX_REG,
158 .volatile_table = &adp5585_volatile_regs,
159 .cache_type = REGCACHE_MAPLE,
160 .reg_defaults_raw = adp5585_regmap_defaults_02,
161 .num_reg_defaults_raw = sizeof(adp5585_regmap_defaults_02),
162 },
163 [ADP5585_REGMAP_04] = {
164 .reg_bits = 8,
165 .val_bits = 8,
166 .max_register = ADP5585_MAX_REG,
167 .volatile_table = &adp5585_volatile_regs,
168 .cache_type = REGCACHE_MAPLE,
169 .reg_defaults_raw = adp5585_regmap_defaults_04,
170 .num_reg_defaults_raw = sizeof(adp5585_regmap_defaults_04),
171 },
172 [ADP5589_REGMAP_00] = {
173 .reg_bits = 8,
174 .val_bits = 8,
175 .max_register = ADP5589_MAX_REG,
176 .volatile_table = &adp5585_volatile_regs,
177 .cache_type = REGCACHE_MAPLE,
178 .reg_defaults_raw = adp5589_regmap_defaults_00,
179 .num_reg_defaults_raw = sizeof(adp5589_regmap_defaults_00),
180 },
181 [ADP5589_REGMAP_01] = {
182 .reg_bits = 8,
183 .val_bits = 8,
184 .max_register = ADP5589_MAX_REG,
185 .volatile_table = &adp5585_volatile_regs,
186 .cache_type = REGCACHE_MAPLE,
187 .reg_defaults_raw = adp5589_regmap_defaults_01,
188 .num_reg_defaults_raw = sizeof(adp5589_regmap_defaults_01),
189 },
190 [ADP5589_REGMAP_02] = {
191 .reg_bits = 8,
192 .val_bits = 8,
193 .max_register = ADP5589_MAX_REG,
194 .volatile_table = &adp5585_volatile_regs,
195 .cache_type = REGCACHE_MAPLE,
196 .reg_defaults_raw = adp5589_regmap_defaults_02,
197 .num_reg_defaults_raw = sizeof(adp5589_regmap_defaults_02),
198 },
199 };
200
201 static const struct adp5585_regs adp5585_regs = {
202 .debounce_dis_a = ADP5585_DEBOUNCE_DIS_A,
203 .rpull_cfg_a = ADP5585_RPULL_CONFIG_A,
204 .gpo_data_a = ADP5585_GPO_DATA_OUT_A,
205 .gpo_out_a = ADP5585_GPO_OUT_MODE_A,
206 .gpio_dir_a = ADP5585_GPIO_DIRECTION_A,
207 .gpi_stat_a = ADP5585_GPI_STATUS_A,
208 .pwm_cfg = ADP5585_PWM_CFG,
209 .pwm_offt_low = ADP5585_PWM_OFFT_LOW,
210 .pwm_ont_low = ADP5585_PWM_ONT_LOW,
211 .gen_cfg = ADP5585_GENERAL_CFG,
212 .ext_cfg = ADP5585_PIN_CONFIG_C,
213 };
214
215 static const struct adp5585_regs adp5589_regs = {
216 .debounce_dis_a = ADP5589_DEBOUNCE_DIS_A,
217 .rpull_cfg_a = ADP5589_RPULL_CONFIG_A,
218 .gpo_data_a = ADP5589_GPO_DATA_OUT_A,
219 .gpo_out_a = ADP5589_GPO_OUT_MODE_A,
220 .gpio_dir_a = ADP5589_GPIO_DIRECTION_A,
221 .gpi_stat_a = ADP5589_GPI_STATUS_A,
222 .pwm_cfg = ADP5589_PWM_CFG,
223 .pwm_offt_low = ADP5589_PWM_OFFT_LOW,
224 .pwm_ont_low = ADP5589_PWM_ONT_LOW,
225 .gen_cfg = ADP5589_GENERAL_CFG,
226 .ext_cfg = ADP5589_PIN_CONFIG_D,
227 };
228
> 229 static const struct adp5585_info adp5585_info = {
230 .adp5585_devs = adp5585_devs,
231 .regmap_config = &adp5585_regmap_configs[ADP5585_REGMAP_00],
232 .n_devs = ARRAY_SIZE(adp5585_devs),
233 .id = ADP5585_MAN_ID_VALUE,
234 .regs = &adp5585_regs,
235 .max_rows = ADP5585_MAX_ROW_NUM,
236 .max_cols = ADP5585_MAX_COL_NUM,
237 };
238
> 239 static const struct adp5585_info adp5585_01_info = {
240 .adp5585_devs = adp5585_devs,
241 .regmap_config = &adp5585_regmap_configs[ADP5585_REGMAP_00],
242 .n_devs = ARRAY_SIZE(adp5585_devs),
243 .id = ADP5585_MAN_ID_VALUE,
244 .regs = &adp5585_regs,
245 .max_rows = ADP5585_MAX_ROW_NUM,
246 .max_cols = ADP5585_MAX_COL_NUM,
247 };
248
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-03-14 9:13 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-13 14:19 [PATCH 00/18] mfd: adp5585: support keymap events and drop legacy Input driver Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 01/18] dt-bindings: mfd: adp5585: ease on the required properties Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 02/18] mfd: adp5585: enable oscilator during probe Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 03/18] pwm: adp5585: don't control OSC_EN in the pwm driver Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 04/18] mfd: adp5585: make use of MFD_CELL_NAME() Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 05/18] dt-bindings: mfd: adp5585: document adp5589 I/O expander Nuno Sá via B4 Relay
2025-03-14 8:49 ` Krzysztof Kozlowski
2025-03-14 9:38 ` Nuno Sá
2025-03-17 7:41 ` Krzysztof Kozlowski
2025-03-17 9:30 ` Nuno Sá
2025-03-17 10:41 ` Krzysztof Kozlowski
2025-03-13 14:19 ` [PATCH 06/18] mfd: adp5585: add support for adp5589 Nuno Sá via B4 Relay
2025-03-14 9:12 ` kernel test robot [this message]
2025-03-14 9:23 ` kernel test robot
2025-03-14 9:32 ` Nuno Sá
2025-03-13 14:19 ` [PATCH 07/18] gpio: adp5585: add support for the ad5589 expander Nuno Sá via B4 Relay
2025-03-14 10:53 ` Linus Walleij
2025-03-13 14:19 ` [PATCH 08/18] pwm: adp5585: add support for adp5589 Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 09/18] dt-bindings: mfd: adp5585: add properties for input events Nuno Sá via B4 Relay
2025-03-17 15:53 ` Rob Herring
2025-03-17 16:08 ` Nuno Sá
2025-03-13 14:19 ` [PATCH 10/18] mfd: adp5585: add support for key events Nuno Sá via B4 Relay
2025-03-14 9:43 ` kernel test robot
2025-03-13 14:19 ` [PATCH 11/18] gpio: adp5585: support gpi events Nuno Sá via B4 Relay
2025-03-14 10:54 ` Linus Walleij
2025-03-13 14:19 ` [PATCH 12/18] Input: adp5585: Add Analog Devices ADP5585/89 support Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 13/18] Input: adp5589: remove the driver Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 14/18] mfd: adp5585: support getting vdd regulator Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 15/18] dt-bindings: mfd: adp5585: document reset gpio Nuno Sá via B4 Relay
2025-03-14 8:52 ` Krzysztof Kozlowski
2025-03-13 14:19 ` [PATCH 16/18] mfd: adp5585: add support for a reset pin Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 17/18] pwm: adp5585: make sure to include mod_devicetable.h Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 18/18] gpio: " Nuno Sá via B4 Relay
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=202503141607.g4arza1A-lkp@intel.com \
--to=lkp@intel.com \
--cc=brgl@bgdev.pl \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=devnull+nuno.sa.analog.com@kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=krzk@kernel.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=lee@kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=robh@kernel.org \
--cc=ukleinek@kernel.org \
--cc=victor.liu@nxp.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).