All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: "Jean-François Lessard" <jefflessard3@gmail.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>
Cc: oe-kbuild-all@lists.linux.dev,
	"Geert Uytterhoeven" <geert@linux-m68k.org>,
	devicetree@vger.kernel.org, linux-leds@vger.kernel.org,
	linux-kernel@vger.kernel.org, "Andreas Färber" <afaerber@suse.de>,
	"Boris Gjenero" <boris.gjenero@gmail.com>,
	"Christian Hewitt" <christianshewitt@gmail.com>,
	"Heiner Kallweit" <hkallweit1@gmail.com>,
	"Paolo Sabatino" <paolo.sabatino@gmail.com>,
	"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>
Subject: Re: [PATCH v3 3/4] auxdisplay: Add TM16xx 7-segment LED matrix display controllers driver
Date: Wed, 27 Aug 2025 13:32:42 +0800	[thread overview]
Message-ID: <202508271344.WqQr2aa7-lkp@intel.com> (raw)
In-Reply-To: <20250820163120.24997-4-jefflessard3@gmail.com>

Hi Jean-François,

kernel test robot noticed the following build errors:

[auto build test ERROR on robh/for-next]
[also build test ERROR on linus/master v6.17-rc3 next-20250826]
[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/Jean-Fran-ois-Lessard/dt-bindings-vendor-prefixes-Add-fdhisi-titanmec-princeton-winrise-wxicore/20250821-003451
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link:    https://lore.kernel.org/r/20250820163120.24997-4-jefflessard3%40gmail.com
patch subject: [PATCH v3 3/4] auxdisplay: Add TM16xx 7-segment LED matrix display controllers driver
config: x86_64-randconfig-r112-20250827 (https://download.01.org/0day-ci/archive/20250827/202508271344.WqQr2aa7-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14+deb12u1) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250827/202508271344.WqQr2aa7-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/202508271344.WqQr2aa7-lkp@intel.com/

All errors (new ones prefixed by >>):

   ld: vmlinux.o: in function `tm16xx_i2c_write':
>> drivers/auxdisplay/tm16xx.c:1469: undefined reference to `i2c_transfer'
   ld: vmlinux.o: in function `tm16xx_i2c_read':
   drivers/auxdisplay/tm16xx.c:1497: undefined reference to `i2c_transfer'
   ld: vmlinux.o: in function `tm16xx_i2c_probe':
>> drivers/auxdisplay/tm16xx.c:1416: undefined reference to `i2c_get_match_data'
   ld: vmlinux.o: in function `tm16xx_i2c_register':
>> drivers/auxdisplay/tm16xx.c:1727: undefined reference to `i2c_register_driver'
   ld: vmlinux.o: in function `tm16xx_i2c_unregister':
>> drivers/auxdisplay/tm16xx.c:1732: undefined reference to `i2c_del_driver'


vim +1469 drivers/auxdisplay/tm16xx.c

  1401	
  1402	/* I2C specific code */
  1403	#if IS_ENABLED(CONFIG_I2C)
  1404	/**
  1405	 * tm16xx_i2c_probe() - Probe callback for I2C-attached controllers
  1406	 * @client: pointer to i2c_client
  1407	 *
  1408	 * Return: 0 on success, negative error code on failure
  1409	 */
  1410	static int tm16xx_i2c_probe(struct i2c_client *client)
  1411	{
  1412		const struct tm16xx_controller *controller;
  1413		struct tm16xx_display *display;
  1414		int ret;
  1415	
> 1416		controller = i2c_get_match_data(client);
  1417		if (!controller)
  1418			return -EINVAL;
  1419	
  1420		display = devm_kzalloc(&client->dev, sizeof(*display), GFP_KERNEL);
  1421		if (!display)
  1422			return -ENOMEM;
  1423	
  1424		display->client.i2c = client;
  1425		display->dev = &client->dev;
  1426		display->controller = controller;
  1427	
  1428		i2c_set_clientdata(client, display);
  1429	
  1430		ret = tm16xx_probe(display);
  1431		if (ret)
  1432			return ret;
  1433	
  1434		return 0;
  1435	}
  1436	
  1437	/**
  1438	 * tm16xx_i2c_remove() - Remove callback for I2C-attached controllers
  1439	 * @client: pointer to i2c_client
  1440	 */
  1441	static void tm16xx_i2c_remove(struct i2c_client *client)
  1442	{
  1443		struct tm16xx_display *display = i2c_get_clientdata(client);
  1444	
  1445		tm16xx_display_remove(display);
  1446	}
  1447	
  1448	/**
  1449	 * tm16xx_i2c_write() - I2C write helper for controller
  1450	 * @display: pointer to tm16xx_display structure
  1451	 * @data: command and data bytes to send
  1452	 * @len: number of bytes in @data
  1453	 *
  1454	 * Return: 0 on success, negative error code on failure
  1455	 */
  1456	static int tm16xx_i2c_write(struct tm16xx_display *display, u8 *data, size_t len)
  1457	{
  1458		dev_dbg(display->dev, "i2c_write %*ph", (char)len, data);
  1459	
  1460		/* expected sequence: S Command [A] Data [A] P */
  1461		struct i2c_msg msg = {
  1462			.addr = data[0] >> 1,
  1463			.flags = 0,
  1464			.len = len - 1,
  1465			.buf = &data[1],
  1466		};
  1467		int ret;
  1468	
> 1469		ret = i2c_transfer(display->client.i2c->adapter, &msg, 1);
  1470		if (ret < 0)
  1471			return ret;
  1472	
  1473		return (ret == 1) ? 0 : -EIO;
  1474	}
  1475	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  parent reply	other threads:[~2025-08-27  5:33 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-20 16:31 [PATCH v3 0/4] auxdisplay: Add TM16xx 7-segment LED matrix display controllers driver Jean-François Lessard
2025-08-20 16:31 ` [PATCH v3 1/4] dt-bindings: vendor-prefixes: Add fdhisi, titanmec, princeton, winrise, wxicore Jean-François Lessard
2025-08-20 20:08   ` Conor Dooley
2025-08-21 19:35     ` Jean-François Lessard
2025-08-21 20:13       ` Conor Dooley
2025-08-22  2:35         ` Jean-François Lessard
2025-08-20 16:31 ` [PATCH v3 2/4] dt-bindings: auxdisplay: add Titan Micro Electronics TM16xx Jean-François Lessard
2025-08-21  7:48   ` Krzysztof Kozlowski
2025-08-21 15:16     ` Jean-François Lessard
2025-08-22  6:44       ` Krzysztof Kozlowski
2025-08-22 13:32         ` Jean-François Lessard
2025-08-20 16:31 ` [PATCH v3 3/4] auxdisplay: Add TM16xx 7-segment LED matrix display controllers driver Jean-François Lessard
2025-08-21  7:43   ` Krzysztof Kozlowski
2025-08-21 16:42     ` Jean-François Lessard
2025-08-21  8:08   ` Andy Shevchenko
2025-08-21 19:04     ` Jean-François Lessard
2025-08-21 20:19       ` Andy Shevchenko
2025-08-22  2:20         ` Jean-François Lessard
2025-08-22  6:08           ` Andy Shevchenko
2025-08-22 13:50             ` Jean-François Lessard
2025-08-27  5:32   ` kernel test robot [this message]
2025-08-20 16:31 ` [PATCH v3 4/4] MAINTAINERS: Add entry for TM16xx driver Jean-François Lessard
2025-08-20 19:08   ` Andy Shevchenko
2025-08-20 19:51     ` Conor Dooley
2025-08-20 20:29       ` Andy Shevchenko
2025-08-21 17:40         ` Conor Dooley
2025-08-21 19:33           ` Andy Shevchenko
2025-08-21 19:35             ` Andy Shevchenko
2025-08-21 20:11               ` Conor Dooley
2025-08-21 20:23                 ` Andy Shevchenko

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=202508271344.WqQr2aa7-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=afaerber@suse.de \
    --cc=andy@kernel.org \
    --cc=boris.gjenero@gmail.com \
    --cc=christianshewitt@gmail.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=geert@linux-m68k.org \
    --cc=hkallweit1@gmail.com \
    --cc=jefflessard3@gmail.com \
    --cc=krzk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-leds@vger.kernel.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=paolo.sabatino@gmail.com \
    --cc=robh@kernel.org \
    /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.