Hi Shawn, Thank you for the patch! Yet something to improve:
[auto build test ERROR on ljones-mfd/for-mfd-next] [also build test ERROR on v4.16-rc2 next-20180221] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Enric-Balletbo-i-Serra/platform-chrome-cros_ec-debugfs-and-sysfs-updates/20180222-074242 base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next config: i386-randconfig-s0-201807 (attached as .config) compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026 reproduce: # save the attached .config to linux build tree make ARCH=i386 All errors (new ones prefixed by >>): drivers/platform/chrome/cros_ec_debugfs.c: In function 'cros_ec_pdinfo_read': >> drivers/platform/chrome/cros_ec_debugfs.c:221:16: error: >> 'EC_USB_PD_MAX_PORTS' undeclared (first use in this function) char read_buf[EC_USB_PD_MAX_PORTS * 40], *p = read_buf; ^~~~~~~~~~~~~~~~~~~ drivers/platform/chrome/cros_ec_debugfs.c:221:16: note: each undeclared identifier is reported only once for each function it appears in drivers/platform/chrome/cros_ec_debugfs.c:221:7: warning: unused variable 'read_buf' [-Wunused-variable] char read_buf[EC_USB_PD_MAX_PORTS * 40], *p = read_buf; ^~~~~~~~ drivers/platform/chrome/cros_ec_debugfs.c:269:1: warning: control reaches end of non-void function [-Wreturn-type] } ^ vim +/EC_USB_PD_MAX_PORTS +221 drivers/platform/chrome/cros_ec_debugfs.c 215 216 static ssize_t cros_ec_pdinfo_read(struct file *file, 217 char __user *user_buf, 218 size_t count, 219 loff_t *ppos) 220 { > 221 char read_buf[EC_USB_PD_MAX_PORTS * 40], *p = read_buf; 222 struct cros_ec_debugfs *debug_info = file->private_data; 223 struct cros_ec_device *ec_dev = debug_info->ec->ec_dev; 224 225 struct { 226 struct cros_ec_command msg; 227 union { 228 struct ec_response_usb_pd_control_v1 resp; 229 struct ec_params_usb_pd_control params; 230 }; 231 } __packed ec_buf; 232 233 struct cros_ec_command *msg; 234 struct ec_response_usb_pd_control_v1 *resp; 235 struct ec_params_usb_pd_control *params; 236 237 int i; 238 239 msg = &ec_buf.msg; 240 params = (struct ec_params_usb_pd_control *)msg->data; 241 resp = (struct ec_response_usb_pd_control_v1 *)msg->data; 242 243 msg->command = EC_CMD_USB_PD_CONTROL; 244 msg->version = 1; 245 msg->insize = sizeof(*resp); 246 msg->outsize = sizeof(*params); 247 248 /* 249 * Read status from all PD ports until failure, typically caused 250 * by attempting to read status on a port that doesn't exist. 251 */ 252 for (i = 0; i < EC_USB_PD_MAX_PORTS; ++i) { 253 params->port = i; 254 params->role = 0; 255 params->mux = 0; 256 params->swap = 0; 257 258 if (cros_ec_cmd_xfer_status(ec_dev, msg) < 0) 259 break; 260 261 p += scnprintf(p, sizeof(read_buf) + read_buf - p, 262 "p%d: %s en:%.2x role:%.2x pol:%.2x\n", 263 i, resp->state, resp->enabled, resp->role, 264 resp->polarity); 265 } 266 267 return simple_read_from_buffer(user_buf, count, ppos, 268 read_buf, p - read_buf); 269 } 270 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip
