Hi Heikki, kernel test robot noticed the following build errors: [auto build test ERROR on usb/usb-testing] [also build test ERROR on usb/usb-next usb/usb-linus chrome-platform/for-next chrome-platform/for-firmware-next driver-core/driver-core-testing driver-core/driver-core-next driver-core/driver-core-linus linus/master v6.8-rc3 next-20240207] [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/Heikki-Krogerus/usb-roles-Link-the-switch-to-its-connector/20240207-230017 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing patch link: https://lore.kernel.org/r/20240207145851.1603237-3-heikki.krogerus%40linux.intel.com patch subject: [PATCH 2/2] platform/chrome: cros_ec_typec: Make sure the USB role switch has PLD config: m68k-allmodconfig (https://download.01.org/0day-ci/archive/20240208/202402081136.sve0cViZ-lkp@xxxxxxxxx/config) compiler: m68k-linux-gcc (GCC) 13.2.0 reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240208/202402081136.sve0cViZ-lkp@xxxxxxxxx/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@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202402081136.sve0cViZ-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): drivers/platform/chrome/cros_ec_typec.c: In function 'cros_typec_parse_port_props': >> drivers/platform/chrome/cros_ec_typec.c:75:34: error: invalid use of undefined type 'struct acpi_device' 75 | if (adev && !adev->pld_crc) | ^~ drivers/platform/chrome/cros_ec_typec.c:76:29: error: invalid use of undefined type 'struct acpi_device' 76 | adev->pld_crc = to_acpi_device_node(fwnode)->pld_crc; | ^~ drivers/platform/chrome/cros_ec_typec.c:76:68: error: invalid use of undefined type 'struct acpi_device' 76 | adev->pld_crc = to_acpi_device_node(fwnode)->pld_crc; | ^~ vim +75 drivers/platform/chrome/cros_ec_typec.c 23 24 #define DP_PORT_VDO (DP_CONF_SET_PIN_ASSIGN(BIT(DP_PIN_ASSIGN_C) | BIT(DP_PIN_ASSIGN_D)) | \ 25 DP_CAP_DFP_D | DP_CAP_RECEPTACLE) 26 27 static int cros_typec_parse_port_props(struct typec_capability *cap, 28 struct fwnode_handle *fwnode, 29 struct device *dev) 30 { 31 struct fwnode_handle *sw_fwnode; 32 const char *buf; 33 int ret; 34 35 memset(cap, 0, sizeof(*cap)); 36 ret = fwnode_property_read_string(fwnode, "power-role", &buf); 37 if (ret) { 38 dev_err(dev, "power-role not found: %d\n", ret); 39 return ret; 40 } 41 42 ret = typec_find_port_power_role(buf); 43 if (ret < 0) 44 return ret; 45 cap->type = ret; 46 47 ret = fwnode_property_read_string(fwnode, "data-role", &buf); 48 if (ret) { 49 dev_err(dev, "data-role not found: %d\n", ret); 50 return ret; 51 } 52 53 ret = typec_find_port_data_role(buf); 54 if (ret < 0) 55 return ret; 56 cap->data = ret; 57 58 /* Try-power-role is optional. */ 59 ret = fwnode_property_read_string(fwnode, "try-power-role", &buf); 60 if (ret) { 61 dev_warn(dev, "try-power-role not found: %d\n", ret); 62 cap->prefer_role = TYPEC_NO_PREFERRED_ROLE; 63 } else { 64 ret = typec_find_power_role(buf); 65 if (ret < 0) 66 return ret; 67 cap->prefer_role = ret; 68 } 69 70 /* Assing the USB role switch the correct pld_crc if it's missing. */ 71 sw_fwnode = fwnode_find_reference(fwnode, "usb-role-switch", 0); 72 if (!IS_ERR_OR_NULL(sw_fwnode)) { 73 struct acpi_device *adev = to_acpi_device_node(sw_fwnode); 74 > 75 if (adev && !adev->pld_crc) 76 adev->pld_crc = to_acpi_device_node(fwnode)->pld_crc; 77 fwnode_handle_put(sw_fwnode); 78 } 79 80 cap->fwnode = fwnode; 81 82 return 0; 83 } 84 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
