From: kernel test robot <lkp@intel.com>
To: frank zago <frank@zago.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-kernel@vger.kernel.org,
Bartosz Golaszewski <bgolaszewski@baylibre.com>,
Wolfram Sang <wsa-dev@sang-engineering.com>,
Johan Hovold <johan@kernel.org>,
linux-usb@vger.kernel.org, Lee Jones <lee.jones@linaro.org>,
Linus Walleij <linus.walleij@linaro.org>,
linux-gpio@vger.kernel.org, linux-i2c@vger.kernel.org
Cc: kbuild-all@lists.01.org
Subject: Re: [PATCH v6 3/4] i2c: ch341: add I2C MFD cell driver for the CH341
Date: Thu, 16 Jun 2022 17:18:39 +0800 [thread overview]
Message-ID: <202206161751.5icAiwNf-lkp@intel.com> (raw)
In-Reply-To: <20220616013747.126051-4-frank@zago.net>
Hi frank,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on lee-mfd/for-mfd-next]
[also build test ERROR on brgl/gpio/for-next wsa/i2c/for-next linus/master v5.19-rc2 next-20220615]
[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]
url: https://github.com/intel-lab-lkp/linux/commits/frank-zago/add-driver-for-the-WCH-CH341-in-I2C-GPIO-mode/20220616-094113
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
config: nios2-randconfig-r026-20220616 (https://download.01.org/0day-ci/archive/20220616/202206161751.5icAiwNf-lkp@intel.com/config)
compiler: nios2-linux-gcc (GCC) 11.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/46c68aee86d34ee630272146a73ad7c3147bb094
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review frank-zago/add-driver-for-the-WCH-CH341-in-I2C-GPIO-mode/20220616-094113
git checkout 46c68aee86d34ee630272146a73ad7c3147bb094
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=nios2 SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
nios2-linux-ld: drivers/mfd/ch341-core.o: in function `ch341_usb_probe':
>> drivers/mfd/ch341-core.c:46: undefined reference to `usb_find_common_endpoints'
drivers/mfd/ch341-core.c:46:(.text+0x8c): relocation truncated to fit: R_NIOS2_CALL26 against `usb_find_common_endpoints'
nios2-linux-ld: drivers/mfd/ch341-core.o: in function `ch341_usb_driver_init':
>> drivers/mfd/ch341-core.c:86: undefined reference to `usb_register_driver'
drivers/mfd/ch341-core.c:86:(.init.text+0x1c): relocation truncated to fit: R_NIOS2_CALL26 against `usb_register_driver'
nios2-linux-ld: drivers/i2c/busses/i2c-ch341.o: in function `no_data_xfer':
>> drivers/i2c/busses/i2c-ch341.c:81: undefined reference to `usb_bulk_msg'
drivers/i2c/busses/i2c-ch341.c:81:(.text+0xe0): relocation truncated to fit: R_NIOS2_CALL26 against `usb_bulk_msg'
>> nios2-linux-ld: drivers/i2c/busses/i2c-ch341.c:89: undefined reference to `usb_bulk_msg'
drivers/i2c/busses/i2c-ch341.c:89:(.text+0x134): relocation truncated to fit: R_NIOS2_CALL26 against `usb_bulk_msg'
nios2-linux-ld: drivers/i2c/busses/i2c-ch341.o: in function `ch341_i2c_xfer':
drivers/i2c/busses/i2c-ch341.c:278: undefined reference to `usb_bulk_msg'
drivers/i2c/busses/i2c-ch341.c:278:(.text+0x4e0): relocation truncated to fit: R_NIOS2_CALL26 against `usb_bulk_msg'
nios2-linux-ld: drivers/i2c/busses/i2c-ch341.c:290: undefined reference to `usb_bulk_msg'
drivers/i2c/busses/i2c-ch341.c:290:(.text+0x558): relocation truncated to fit: R_NIOS2_CALL26 against `usb_bulk_msg'
nios2-linux-ld: drivers/i2c/busses/i2c-ch341.o: in function `ch341_i2c_probe':
drivers/i2c/busses/i2c-ch341.c:357: undefined reference to `usb_bulk_msg'
drivers/i2c/busses/i2c-ch341.c:357:(.text+0x778): relocation truncated to fit: R_NIOS2_CALL26 against `usb_bulk_msg'
pahole: .tmp_vmlinux.btf: No such file or directory
.btf.vmlinux.bin.o: file not recognized: file format not recognized
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for MFD_CH341
Depends on HAS_IOMEM && USB
Selected by
- I2C_CH341 && I2C && HAS_IOMEM
vim +46 drivers/mfd/ch341-core.c
f33ee28d4d85a8 frank zago 2022-06-15 29
f33ee28d4d85a8 frank zago 2022-06-15 30 static int ch341_usb_probe(struct usb_interface *iface,
f33ee28d4d85a8 frank zago 2022-06-15 31 const struct usb_device_id *usb_id)
f33ee28d4d85a8 frank zago 2022-06-15 32 {
f33ee28d4d85a8 frank zago 2022-06-15 33 struct usb_endpoint_descriptor *bulk_out;
f33ee28d4d85a8 frank zago 2022-06-15 34 struct usb_endpoint_descriptor *bulk_in;
f33ee28d4d85a8 frank zago 2022-06-15 35 struct usb_endpoint_descriptor *intr_in;
f33ee28d4d85a8 frank zago 2022-06-15 36 struct ch341_ddata *ddata;
f33ee28d4d85a8 frank zago 2022-06-15 37 int ret;
f33ee28d4d85a8 frank zago 2022-06-15 38
f33ee28d4d85a8 frank zago 2022-06-15 39 ddata = devm_kzalloc(&iface->dev, sizeof(*ddata), GFP_KERNEL);
f33ee28d4d85a8 frank zago 2022-06-15 40 if (!ddata)
f33ee28d4d85a8 frank zago 2022-06-15 41 return -ENOMEM;
f33ee28d4d85a8 frank zago 2022-06-15 42
f33ee28d4d85a8 frank zago 2022-06-15 43 ddata->usb_dev = interface_to_usbdev(iface);
f33ee28d4d85a8 frank zago 2022-06-15 44 mutex_init(&ddata->usb_lock);
f33ee28d4d85a8 frank zago 2022-06-15 45
f33ee28d4d85a8 frank zago 2022-06-15 @46 ret = usb_find_common_endpoints(iface->cur_altsetting, &bulk_in,
f33ee28d4d85a8 frank zago 2022-06-15 47 &bulk_out, &intr_in, NULL);
f33ee28d4d85a8 frank zago 2022-06-15 48 if (ret) {
f33ee28d4d85a8 frank zago 2022-06-15 49 dev_err(&iface->dev, "Could not find all endpoints\n");
f33ee28d4d85a8 frank zago 2022-06-15 50 return -ENODEV;
f33ee28d4d85a8 frank zago 2022-06-15 51 }
f33ee28d4d85a8 frank zago 2022-06-15 52
f33ee28d4d85a8 frank zago 2022-06-15 53 ddata->ep_in = bulk_in->bEndpointAddress;
f33ee28d4d85a8 frank zago 2022-06-15 54 ddata->ep_out = bulk_out->bEndpointAddress;
f33ee28d4d85a8 frank zago 2022-06-15 55 ddata->ep_intr = intr_in->bEndpointAddress;
f33ee28d4d85a8 frank zago 2022-06-15 56 ddata->ep_intr_interval = intr_in->bInterval;
f33ee28d4d85a8 frank zago 2022-06-15 57
f33ee28d4d85a8 frank zago 2022-06-15 58 usb_set_intfdata(iface, ddata);
f33ee28d4d85a8 frank zago 2022-06-15 59
f33ee28d4d85a8 frank zago 2022-06-15 60 ret = mfd_add_devices(&iface->dev, PLATFORM_DEVID_AUTO, ch341_devs,
f33ee28d4d85a8 frank zago 2022-06-15 61 ARRAY_SIZE(ch341_devs), NULL, 0, NULL);
f33ee28d4d85a8 frank zago 2022-06-15 62 if (ret)
f33ee28d4d85a8 frank zago 2022-06-15 63 return dev_err_probe(&iface->dev, ret,
f33ee28d4d85a8 frank zago 2022-06-15 64 "Failed to register child devices\n");
f33ee28d4d85a8 frank zago 2022-06-15 65
f33ee28d4d85a8 frank zago 2022-06-15 66 return 0;
f33ee28d4d85a8 frank zago 2022-06-15 67 }
f33ee28d4d85a8 frank zago 2022-06-15 68
f33ee28d4d85a8 frank zago 2022-06-15 69 static void ch341_usb_disconnect(struct usb_interface *usb_if)
f33ee28d4d85a8 frank zago 2022-06-15 70 {
f33ee28d4d85a8 frank zago 2022-06-15 71 mfd_remove_devices(&usb_if->dev);
f33ee28d4d85a8 frank zago 2022-06-15 72 }
f33ee28d4d85a8 frank zago 2022-06-15 73
f33ee28d4d85a8 frank zago 2022-06-15 74 static const struct usb_device_id ch341_usb_table[] = {
f33ee28d4d85a8 frank zago 2022-06-15 75 { USB_DEVICE(0x1a86, 0x5512) },
f33ee28d4d85a8 frank zago 2022-06-15 76 { }
f33ee28d4d85a8 frank zago 2022-06-15 77 };
f33ee28d4d85a8 frank zago 2022-06-15 78 MODULE_DEVICE_TABLE(usb, ch341_usb_table);
f33ee28d4d85a8 frank zago 2022-06-15 79
f33ee28d4d85a8 frank zago 2022-06-15 80 static struct usb_driver ch341_usb_driver = {
f33ee28d4d85a8 frank zago 2022-06-15 81 .name = "ch341-mfd",
f33ee28d4d85a8 frank zago 2022-06-15 82 .id_table = ch341_usb_table,
f33ee28d4d85a8 frank zago 2022-06-15 83 .probe = ch341_usb_probe,
f33ee28d4d85a8 frank zago 2022-06-15 84 .disconnect = ch341_usb_disconnect,
f33ee28d4d85a8 frank zago 2022-06-15 85 };
f33ee28d4d85a8 frank zago 2022-06-15 @86 module_usb_driver(ch341_usb_driver);
f33ee28d4d85a8 frank zago 2022-06-15 87
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-06-16 9:19 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-16 1:37 [PATCH v6 0/4] add driver for the WCH CH341 in I2C/GPIO mode frank zago
2022-06-16 1:37 ` [PATCH v6 1/4] mfd: ch341: add core driver for the WCH CH341 in I2C/SPI/GPIO mode frank zago
2022-06-27 14:04 ` Lee Jones
2022-06-27 14:28 ` Greg Kroah-Hartman
2022-06-27 14:43 ` Lee Jones
2022-06-27 23:30 ` Frank Zago
2022-07-04 10:09 ` Lee Jones
2022-06-16 1:37 ` [PATCH v6 2/4] gpio: ch341: add GPIO MFD cell driver for the CH341 frank zago
2022-06-17 8:29 ` Linus Walleij
2022-06-20 10:25 ` Johan Hovold
2022-06-20 16:08 ` Andy Shevchenko
2022-06-21 6:57 ` Johan Hovold
2022-06-16 1:37 ` [PATCH v6 3/4] i2c: ch341: add I2C " frank zago
2022-06-16 5:09 ` Randy Dunlap
2022-06-16 9:18 ` kernel test robot [this message]
2022-06-16 11:10 ` kernel test robot
2022-06-16 1:37 ` [PATCH v6 4/4] docs: misc: add documentation for ch341 driver frank zago
2022-06-20 10:08 ` [PATCH v6 0/4] add driver for the WCH CH341 in I2C/GPIO mode Johan Hovold
2024-11-08 15:58 ` Matwey V. Kornilov
2024-11-08 19:11 ` Linus Walleij
2024-11-08 23:15 ` Frank Zago
2024-11-09 7:32 ` Matwey V. Kornilov
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=202206161751.5icAiwNf-lkp@intel.com \
--to=lkp@intel.com \
--cc=bgolaszewski@baylibre.com \
--cc=frank@zago.net \
--cc=gregkh@linuxfoundation.org \
--cc=johan@kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=lee.jones@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=wsa-dev@sang-engineering.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).