devicetree.vger.kernel.org archive mirror
 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 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).