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: 52+ 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á
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 01/18] dt-bindings: mfd: adp5585: ease on the required properties Nuno Sá
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 02/18] mfd: adp5585: enable oscilator during probe Nuno Sá
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 04/18] mfd: adp5585: make use of MFD_CELL_NAME() Nuno Sá
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 05/18] dt-bindings: mfd: adp5585: document adp5589 I/O expander Nuno Sá
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 09/18] dt-bindings: mfd: adp5585: add properties for input events Nuno Sá
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 13/18] Input: adp5589: remove the driver Nuno Sá
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 14/18] mfd: adp5585: support getting vdd regulator Nuno Sá
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 15/18] dt-bindings: mfd: adp5585: document reset gpio Nuno Sá
2025-03-13 14:19 ` 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á
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 17/18] pwm: adp5585: make sure to include mod_devicetable.h Nuno Sá
2025-03-13 14:19 ` Nuno Sá via B4 Relay
2025-03-13 14:19 ` [PATCH 18/18] gpio: " Nuno Sá
2025-03-13 14:19 ` 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.