* [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext'
@ 2019-08-24 22:03 kbuild test robot
2019-08-24 22:58 ` Jacek Anaszewski
0 siblings, 1 reply; 4+ messages in thread
From: kbuild test robot @ 2019-08-24 22:03 UTC (permalink / raw)
To: Jacek Anaszewski; +Cc: kbuild-all, linux-leds
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=unknown-8bit, Size: 17871 bytes --]
tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/j.anaszewski/linux-leds.git for-next
head: fd81d7e946c6bdb86dbf0bd88fee3e1a545e7979
commit: b2b998c0f944993c9ef435569651e407d607af41 [15/49] leds: class: Improve LED and LED flash class registration API
config: i386-randconfig-h002-201934 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-10) 7.4.0
reproduce:
git checkout b2b998c0f944993c9ef435569651e407d607af41
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
ld: drivers/input/keyboard/applespi.o: in function `applespi_probe':
>> drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext'
vim +1789 drivers/input/keyboard/applespi.c
038b1a05eae666 Ronald Tschalär 2019-07-15 1620
038b1a05eae666 Ronald Tschalär 2019-07-15 1621 static int applespi_probe(struct spi_device *spi)
038b1a05eae666 Ronald Tschalär 2019-07-15 1622 {
038b1a05eae666 Ronald Tschalär 2019-07-15 1623 struct applespi_data *applespi;
038b1a05eae666 Ronald Tschalär 2019-07-15 1624 acpi_handle spi_handle = ACPI_HANDLE(&spi->dev);
038b1a05eae666 Ronald Tschalär 2019-07-15 1625 acpi_status acpi_sts;
038b1a05eae666 Ronald Tschalär 2019-07-15 1626 int sts, i;
038b1a05eae666 Ronald Tschalär 2019-07-15 1627 unsigned long long gpe, usb_status;
038b1a05eae666 Ronald Tschalär 2019-07-15 1628
038b1a05eae666 Ronald Tschalär 2019-07-15 1629 /* check if the USB interface is present and enabled already */
038b1a05eae666 Ronald Tschalär 2019-07-15 1630 acpi_sts = acpi_evaluate_integer(spi_handle, "UIST", NULL, &usb_status);
038b1a05eae666 Ronald Tschalär 2019-07-15 1631 if (ACPI_SUCCESS(acpi_sts) && usb_status) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1632 /* let the USB driver take over instead */
038b1a05eae666 Ronald Tschalär 2019-07-15 1633 dev_info(&spi->dev, "USB interface already enabled\n");
038b1a05eae666 Ronald Tschalär 2019-07-15 1634 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1635 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1636
038b1a05eae666 Ronald Tschalär 2019-07-15 1637 /* allocate driver data */
038b1a05eae666 Ronald Tschalär 2019-07-15 1638 applespi = devm_kzalloc(&spi->dev, sizeof(*applespi), GFP_KERNEL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1639 if (!applespi)
038b1a05eae666 Ronald Tschalär 2019-07-15 1640 return -ENOMEM;
038b1a05eae666 Ronald Tschalär 2019-07-15 1641
038b1a05eae666 Ronald Tschalär 2019-07-15 1642 applespi->spi = spi;
038b1a05eae666 Ronald Tschalär 2019-07-15 1643
038b1a05eae666 Ronald Tschalär 2019-07-15 1644 INIT_WORK(&applespi->work, applespi_worker);
038b1a05eae666 Ronald Tschalär 2019-07-15 1645
038b1a05eae666 Ronald Tschalär 2019-07-15 1646 /* store the driver data */
038b1a05eae666 Ronald Tschalär 2019-07-15 1647 spi_set_drvdata(spi, applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1648
038b1a05eae666 Ronald Tschalär 2019-07-15 1649 /* create our buffers */
038b1a05eae666 Ronald Tschalär 2019-07-15 1650 applespi->tx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE,
038b1a05eae666 Ronald Tschalär 2019-07-15 1651 GFP_KERNEL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1652 applespi->tx_status = devm_kmalloc(&spi->dev, APPLESPI_STATUS_SIZE,
038b1a05eae666 Ronald Tschalär 2019-07-15 1653 GFP_KERNEL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1654 applespi->rx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE,
038b1a05eae666 Ronald Tschalär 2019-07-15 1655 GFP_KERNEL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1656 applespi->msg_buf = devm_kmalloc_array(&spi->dev, MAX_PKTS_PER_MSG,
038b1a05eae666 Ronald Tschalär 2019-07-15 1657 APPLESPI_PACKET_SIZE,
038b1a05eae666 Ronald Tschalär 2019-07-15 1658 GFP_KERNEL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1659
038b1a05eae666 Ronald Tschalär 2019-07-15 1660 if (!applespi->tx_buffer || !applespi->tx_status ||
038b1a05eae666 Ronald Tschalär 2019-07-15 1661 !applespi->rx_buffer || !applespi->msg_buf)
038b1a05eae666 Ronald Tschalär 2019-07-15 1662 return -ENOMEM;
038b1a05eae666 Ronald Tschalär 2019-07-15 1663
038b1a05eae666 Ronald Tschalär 2019-07-15 1664 /* set up our spi messages */
038b1a05eae666 Ronald Tschalär 2019-07-15 1665 applespi_setup_read_txfrs(applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1666 applespi_setup_write_txfrs(applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1667
038b1a05eae666 Ronald Tschalär 2019-07-15 1668 /* cache ACPI method handles */
038b1a05eae666 Ronald Tschalär 2019-07-15 1669 acpi_sts = acpi_get_handle(spi_handle, "SIEN", &applespi->sien);
038b1a05eae666 Ronald Tschalär 2019-07-15 1670 if (ACPI_FAILURE(acpi_sts)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1671 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1672 "Failed to get SIEN ACPI method handle: %s\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1673 acpi_format_exception(acpi_sts));
038b1a05eae666 Ronald Tschalär 2019-07-15 1674 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1675 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1676
038b1a05eae666 Ronald Tschalär 2019-07-15 1677 acpi_sts = acpi_get_handle(spi_handle, "SIST", &applespi->sist);
038b1a05eae666 Ronald Tschalär 2019-07-15 1678 if (ACPI_FAILURE(acpi_sts)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1679 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1680 "Failed to get SIST ACPI method handle: %s\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1681 acpi_format_exception(acpi_sts));
038b1a05eae666 Ronald Tschalär 2019-07-15 1682 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1683 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1684
038b1a05eae666 Ronald Tschalär 2019-07-15 1685 /* switch on the SPI interface */
038b1a05eae666 Ronald Tschalär 2019-07-15 1686 sts = applespi_setup_spi(applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1687 if (sts)
038b1a05eae666 Ronald Tschalär 2019-07-15 1688 return sts;
038b1a05eae666 Ronald Tschalär 2019-07-15 1689
038b1a05eae666 Ronald Tschalär 2019-07-15 1690 sts = applespi_enable_spi(applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1691 if (sts)
038b1a05eae666 Ronald Tschalär 2019-07-15 1692 return sts;
038b1a05eae666 Ronald Tschalär 2019-07-15 1693
038b1a05eae666 Ronald Tschalär 2019-07-15 1694 /* setup the keyboard input dev */
038b1a05eae666 Ronald Tschalär 2019-07-15 1695 applespi->keyboard_input_dev = devm_input_allocate_device(&spi->dev);
038b1a05eae666 Ronald Tschalär 2019-07-15 1696
038b1a05eae666 Ronald Tschalär 2019-07-15 1697 if (!applespi->keyboard_input_dev)
038b1a05eae666 Ronald Tschalär 2019-07-15 1698 return -ENOMEM;
038b1a05eae666 Ronald Tschalär 2019-07-15 1699
038b1a05eae666 Ronald Tschalär 2019-07-15 1700 applespi->keyboard_input_dev->name = "Apple SPI Keyboard";
038b1a05eae666 Ronald Tschalär 2019-07-15 1701 applespi->keyboard_input_dev->phys = "applespi/input0";
038b1a05eae666 Ronald Tschalär 2019-07-15 1702 applespi->keyboard_input_dev->dev.parent = &spi->dev;
038b1a05eae666 Ronald Tschalär 2019-07-15 1703 applespi->keyboard_input_dev->id.bustype = BUS_SPI;
038b1a05eae666 Ronald Tschalär 2019-07-15 1704
038b1a05eae666 Ronald Tschalär 2019-07-15 1705 applespi->keyboard_input_dev->evbit[0] =
038b1a05eae666 Ronald Tschalär 2019-07-15 1706 BIT_MASK(EV_KEY) | BIT_MASK(EV_LED) | BIT_MASK(EV_REP);
038b1a05eae666 Ronald Tschalär 2019-07-15 1707 applespi->keyboard_input_dev->ledbit[0] = BIT_MASK(LED_CAPSL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1708
038b1a05eae666 Ronald Tschalär 2019-07-15 1709 input_set_drvdata(applespi->keyboard_input_dev, applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1710 applespi->keyboard_input_dev->event = applespi_event;
038b1a05eae666 Ronald Tschalär 2019-07-15 1711
038b1a05eae666 Ronald Tschalär 2019-07-15 1712 for (i = 0; i < ARRAY_SIZE(applespi_scancodes); i++)
038b1a05eae666 Ronald Tschalär 2019-07-15 1713 if (applespi_scancodes[i])
038b1a05eae666 Ronald Tschalär 2019-07-15 1714 input_set_capability(applespi->keyboard_input_dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1715 EV_KEY, applespi_scancodes[i]);
038b1a05eae666 Ronald Tschalär 2019-07-15 1716
038b1a05eae666 Ronald Tschalär 2019-07-15 1717 for (i = 0; i < ARRAY_SIZE(applespi_controlcodes); i++)
038b1a05eae666 Ronald Tschalär 2019-07-15 1718 if (applespi_controlcodes[i])
038b1a05eae666 Ronald Tschalär 2019-07-15 1719 input_set_capability(applespi->keyboard_input_dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1720 EV_KEY, applespi_controlcodes[i]);
038b1a05eae666 Ronald Tschalär 2019-07-15 1721
038b1a05eae666 Ronald Tschalär 2019-07-15 1722 for (i = 0; i < ARRAY_SIZE(applespi_fn_codes); i++)
038b1a05eae666 Ronald Tschalär 2019-07-15 1723 if (applespi_fn_codes[i].to)
038b1a05eae666 Ronald Tschalär 2019-07-15 1724 input_set_capability(applespi->keyboard_input_dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1725 EV_KEY, applespi_fn_codes[i].to);
038b1a05eae666 Ronald Tschalär 2019-07-15 1726
038b1a05eae666 Ronald Tschalär 2019-07-15 1727 input_set_capability(applespi->keyboard_input_dev, EV_KEY, KEY_FN);
038b1a05eae666 Ronald Tschalär 2019-07-15 1728
038b1a05eae666 Ronald Tschalär 2019-07-15 1729 sts = input_register_device(applespi->keyboard_input_dev);
038b1a05eae666 Ronald Tschalär 2019-07-15 1730 if (sts) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1731 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1732 "Unable to register keyboard input device (%d)\n", sts);
038b1a05eae666 Ronald Tschalär 2019-07-15 1733 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1734 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1735
038b1a05eae666 Ronald Tschalär 2019-07-15 1736 /*
038b1a05eae666 Ronald Tschalär 2019-07-15 1737 * The applespi device doesn't send interrupts normally (as is described
038b1a05eae666 Ronald Tschalär 2019-07-15 1738 * in its DSDT), but rather seems to use ACPI GPEs.
038b1a05eae666 Ronald Tschalär 2019-07-15 1739 */
038b1a05eae666 Ronald Tschalär 2019-07-15 1740 acpi_sts = acpi_evaluate_integer(spi_handle, "_GPE", NULL, &gpe);
038b1a05eae666 Ronald Tschalär 2019-07-15 1741 if (ACPI_FAILURE(acpi_sts)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1742 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1743 "Failed to obtain GPE for SPI slave device: %s\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1744 acpi_format_exception(acpi_sts));
038b1a05eae666 Ronald Tschalär 2019-07-15 1745 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1746 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1747 applespi->gpe = (int)gpe;
038b1a05eae666 Ronald Tschalär 2019-07-15 1748
038b1a05eae666 Ronald Tschalär 2019-07-15 1749 acpi_sts = acpi_install_gpe_handler(NULL, applespi->gpe,
038b1a05eae666 Ronald Tschalär 2019-07-15 1750 ACPI_GPE_LEVEL_TRIGGERED,
038b1a05eae666 Ronald Tschalär 2019-07-15 1751 applespi_notify, applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1752 if (ACPI_FAILURE(acpi_sts)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1753 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1754 "Failed to install GPE handler for GPE %d: %s\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1755 applespi->gpe, acpi_format_exception(acpi_sts));
038b1a05eae666 Ronald Tschalär 2019-07-15 1756 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1757 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1758
038b1a05eae666 Ronald Tschalär 2019-07-15 1759 applespi->suspended = false;
038b1a05eae666 Ronald Tschalär 2019-07-15 1760
038b1a05eae666 Ronald Tschalär 2019-07-15 1761 acpi_sts = acpi_enable_gpe(NULL, applespi->gpe);
038b1a05eae666 Ronald Tschalär 2019-07-15 1762 if (ACPI_FAILURE(acpi_sts)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1763 dev_err(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1764 "Failed to enable GPE handler for GPE %d: %s\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1765 applespi->gpe, acpi_format_exception(acpi_sts));
038b1a05eae666 Ronald Tschalär 2019-07-15 1766 acpi_remove_gpe_handler(NULL, applespi->gpe, applespi_notify);
038b1a05eae666 Ronald Tschalär 2019-07-15 1767 return -ENODEV;
038b1a05eae666 Ronald Tschalär 2019-07-15 1768 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1769
038b1a05eae666 Ronald Tschalär 2019-07-15 1770 /* trigger touchpad setup */
038b1a05eae666 Ronald Tschalär 2019-07-15 1771 applespi_init(applespi, false);
038b1a05eae666 Ronald Tschalär 2019-07-15 1772
038b1a05eae666 Ronald Tschalär 2019-07-15 1773 /*
038b1a05eae666 Ronald Tschalär 2019-07-15 1774 * By default this device is not enabled for wakeup; but USB keyboards
038b1a05eae666 Ronald Tschalär 2019-07-15 1775 * generally are, so the expectation is that by default the keyboard
038b1a05eae666 Ronald Tschalär 2019-07-15 1776 * will wake the system.
038b1a05eae666 Ronald Tschalär 2019-07-15 1777 */
038b1a05eae666 Ronald Tschalär 2019-07-15 1778 device_wakeup_enable(&spi->dev);
038b1a05eae666 Ronald Tschalär 2019-07-15 1779
038b1a05eae666 Ronald Tschalär 2019-07-15 1780 /* set up keyboard-backlight */
038b1a05eae666 Ronald Tschalär 2019-07-15 1781 sts = applespi_get_saved_bl_level(applespi);
038b1a05eae666 Ronald Tschalär 2019-07-15 1782 if (sts >= 0)
038b1a05eae666 Ronald Tschalär 2019-07-15 1783 applespi_set_bl_level(&applespi->backlight_info, sts);
038b1a05eae666 Ronald Tschalär 2019-07-15 1784
038b1a05eae666 Ronald Tschalär 2019-07-15 1785 applespi->backlight_info.name = "spi::kbd_backlight";
038b1a05eae666 Ronald Tschalär 2019-07-15 1786 applespi->backlight_info.default_trigger = "kbd-backlight";
038b1a05eae666 Ronald Tschalär 2019-07-15 1787 applespi->backlight_info.brightness_set = applespi_set_bl_level;
038b1a05eae666 Ronald Tschalär 2019-07-15 1788
038b1a05eae666 Ronald Tschalär 2019-07-15 @1789 sts = devm_led_classdev_register(&spi->dev, &applespi->backlight_info);
038b1a05eae666 Ronald Tschalär 2019-07-15 1790 if (sts)
038b1a05eae666 Ronald Tschalär 2019-07-15 1791 dev_warn(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1792 "Unable to register keyboard backlight class dev (%d)\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1793 sts);
038b1a05eae666 Ronald Tschalär 2019-07-15 1794
038b1a05eae666 Ronald Tschalär 2019-07-15 1795 /* set up debugfs entries for touchpad dimensions logging */
038b1a05eae666 Ronald Tschalär 2019-07-15 1796 applespi->debugfs_root = debugfs_create_dir("applespi", NULL);
038b1a05eae666 Ronald Tschalär 2019-07-15 1797 if (IS_ERR(applespi->debugfs_root)) {
038b1a05eae666 Ronald Tschalär 2019-07-15 1798 if (PTR_ERR(applespi->debugfs_root) != -ENODEV)
038b1a05eae666 Ronald Tschalär 2019-07-15 1799 dev_warn(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1800 "Error creating debugfs root entry (%ld)\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1801 PTR_ERR(applespi->debugfs_root));
038b1a05eae666 Ronald Tschalär 2019-07-15 1802 } else {
038b1a05eae666 Ronald Tschalär 2019-07-15 1803 struct dentry *ret;
038b1a05eae666 Ronald Tschalär 2019-07-15 1804
038b1a05eae666 Ronald Tschalär 2019-07-15 1805 ret = debugfs_create_bool("enable_tp_dim", 0600,
038b1a05eae666 Ronald Tschalär 2019-07-15 1806 applespi->debugfs_root,
038b1a05eae666 Ronald Tschalär 2019-07-15 1807 &applespi->debug_tp_dim);
038b1a05eae666 Ronald Tschalär 2019-07-15 1808 if (IS_ERR(ret))
038b1a05eae666 Ronald Tschalär 2019-07-15 1809 dev_dbg(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1810 "Error creating debugfs entry enable_tp_dim (%ld)\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1811 PTR_ERR(ret));
038b1a05eae666 Ronald Tschalär 2019-07-15 1812
038b1a05eae666 Ronald Tschalär 2019-07-15 1813 ret = debugfs_create_file("tp_dim", 0400,
038b1a05eae666 Ronald Tschalär 2019-07-15 1814 applespi->debugfs_root, applespi,
038b1a05eae666 Ronald Tschalär 2019-07-15 1815 &applespi_tp_dim_fops);
038b1a05eae666 Ronald Tschalär 2019-07-15 1816 if (IS_ERR(ret))
038b1a05eae666 Ronald Tschalär 2019-07-15 1817 dev_dbg(&applespi->spi->dev,
038b1a05eae666 Ronald Tschalär 2019-07-15 1818 "Error creating debugfs entry tp_dim (%ld)\n",
038b1a05eae666 Ronald Tschalär 2019-07-15 1819 PTR_ERR(ret));
038b1a05eae666 Ronald Tschalär 2019-07-15 1820 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1821
038b1a05eae666 Ronald Tschalär 2019-07-15 1822 return 0;
038b1a05eae666 Ronald Tschalär 2019-07-15 1823 }
038b1a05eae666 Ronald Tschalär 2019-07-15 1824
:::::: The code at line 1789 was first introduced by commit
:::::: 038b1a05eae6666b731920e46f47d8e2332e07ff Input: add Apple SPI keyboard and trackpad driver
:::::: TO: Ronald Tschalär <ronald@innovation.ch>
:::::: CC: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 28807 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' 2019-08-24 22:03 [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' kbuild test robot @ 2019-08-24 22:58 ` Jacek Anaszewski 2019-08-27 6:46 ` Life is hard, and then you die 0 siblings, 1 reply; 4+ messages in thread From: Jacek Anaszewski @ 2019-08-24 22:58 UTC (permalink / raw) To: Ronald Tschalär Cc: kbuild test robot, kbuild-all, linux-leds, Dmitry Torokhov Hi Ronald, You're missing LEDS_CLASS dependency in Kconfig: diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig index 8e9c3ea9d5e7..96285e071ee1 100644 --- a/drivers/input/keyboard/Kconfig +++ b/drivers/input/keyboard/Kconfig @@ -76,6 +76,7 @@ config KEYBOARD_APPLESPI depends on ACPI && EFI depends on SPI depends on X86 || COMPILE_TEST + depends on LEDS_CLASS help Say Y here if you are running Linux on any Apple MacBook8,1 or later, or any MacBookPro13,* or MacBookPro14,*. On 8/25/19 12:03 AM, kbuild test robot wrote: > tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/j.anaszewski/linux-leds.git for-next > head: fd81d7e946c6bdb86dbf0bd88fee3e1a545e7979 > commit: b2b998c0f944993c9ef435569651e407d607af41 [15/49] leds: class: Improve LED and LED flash class registration API > config: i386-randconfig-h002-201934 (attached as .config) > compiler: gcc-7 (Debian 7.4.0-10) 7.4.0 > reproduce: > git checkout b2b998c0f944993c9ef435569651e407d607af41 > # save the attached .config to linux build tree > make ARCH=i386 > > If you fix the issue, kindly add following tag > Reported-by: kbuild test robot <lkp@intel.com> > > All errors (new ones prefixed by >>): > > ld: drivers/input/keyboard/applespi.o: in function `applespi_probe': >>> drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' > > vim +1789 drivers/input/keyboard/applespi.c > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1620 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1621 static int applespi_probe(struct spi_device *spi) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1622 { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1623 struct applespi_data *applespi; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1624 acpi_handle spi_handle = ACPI_HANDLE(&spi->dev); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1625 acpi_status acpi_sts; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1626 int sts, i; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1627 unsigned long long gpe, usb_status; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1628 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1629 /* check if the USB interface is present and enabled already */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1630 acpi_sts = acpi_evaluate_integer(spi_handle, "UIST", NULL, &usb_status); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1631 if (ACPI_SUCCESS(acpi_sts) && usb_status) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1632 /* let the USB driver take over instead */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1633 dev_info(&spi->dev, "USB interface already enabled\n"); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1634 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1635 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1636 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1637 /* allocate driver data */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1638 applespi = devm_kzalloc(&spi->dev, sizeof(*applespi), GFP_KERNEL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1639 if (!applespi) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1640 return -ENOMEM; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1641 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1642 applespi->spi = spi; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1643 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1644 INIT_WORK(&applespi->work, applespi_worker); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1645 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1646 /* store the driver data */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1647 spi_set_drvdata(spi, applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1648 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1649 /* create our buffers */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1650 applespi->tx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1651 GFP_KERNEL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1652 applespi->tx_status = devm_kmalloc(&spi->dev, APPLESPI_STATUS_SIZE, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1653 GFP_KERNEL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1654 applespi->rx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1655 GFP_KERNEL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1656 applespi->msg_buf = devm_kmalloc_array(&spi->dev, MAX_PKTS_PER_MSG, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1657 APPLESPI_PACKET_SIZE, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1658 GFP_KERNEL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1659 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1660 if (!applespi->tx_buffer || !applespi->tx_status || > 038b1a05eae666 Ronald Tschalär 2019-07-15 1661 !applespi->rx_buffer || !applespi->msg_buf) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1662 return -ENOMEM; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1663 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1664 /* set up our spi messages */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1665 applespi_setup_read_txfrs(applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1666 applespi_setup_write_txfrs(applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1667 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1668 /* cache ACPI method handles */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1669 acpi_sts = acpi_get_handle(spi_handle, "SIEN", &applespi->sien); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1670 if (ACPI_FAILURE(acpi_sts)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1671 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1672 "Failed to get SIEN ACPI method handle: %s\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1673 acpi_format_exception(acpi_sts)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1674 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1675 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1676 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1677 acpi_sts = acpi_get_handle(spi_handle, "SIST", &applespi->sist); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1678 if (ACPI_FAILURE(acpi_sts)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1679 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1680 "Failed to get SIST ACPI method handle: %s\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1681 acpi_format_exception(acpi_sts)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1682 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1683 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1684 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1685 /* switch on the SPI interface */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1686 sts = applespi_setup_spi(applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1687 if (sts) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1688 return sts; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1689 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1690 sts = applespi_enable_spi(applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1691 if (sts) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1692 return sts; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1693 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1694 /* setup the keyboard input dev */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1695 applespi->keyboard_input_dev = devm_input_allocate_device(&spi->dev); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1696 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1697 if (!applespi->keyboard_input_dev) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1698 return -ENOMEM; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1699 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1700 applespi->keyboard_input_dev->name = "Apple SPI Keyboard"; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1701 applespi->keyboard_input_dev->phys = "applespi/input0"; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1702 applespi->keyboard_input_dev->dev.parent = &spi->dev; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1703 applespi->keyboard_input_dev->id.bustype = BUS_SPI; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1704 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1705 applespi->keyboard_input_dev->evbit[0] = > 038b1a05eae666 Ronald Tschalär 2019-07-15 1706 BIT_MASK(EV_KEY) | BIT_MASK(EV_LED) | BIT_MASK(EV_REP); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1707 applespi->keyboard_input_dev->ledbit[0] = BIT_MASK(LED_CAPSL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1708 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1709 input_set_drvdata(applespi->keyboard_input_dev, applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1710 applespi->keyboard_input_dev->event = applespi_event; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1711 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1712 for (i = 0; i < ARRAY_SIZE(applespi_scancodes); i++) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1713 if (applespi_scancodes[i]) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1714 input_set_capability(applespi->keyboard_input_dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1715 EV_KEY, applespi_scancodes[i]); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1716 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1717 for (i = 0; i < ARRAY_SIZE(applespi_controlcodes); i++) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1718 if (applespi_controlcodes[i]) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1719 input_set_capability(applespi->keyboard_input_dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1720 EV_KEY, applespi_controlcodes[i]); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1721 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1722 for (i = 0; i < ARRAY_SIZE(applespi_fn_codes); i++) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1723 if (applespi_fn_codes[i].to) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1724 input_set_capability(applespi->keyboard_input_dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1725 EV_KEY, applespi_fn_codes[i].to); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1726 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1727 input_set_capability(applespi->keyboard_input_dev, EV_KEY, KEY_FN); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1728 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1729 sts = input_register_device(applespi->keyboard_input_dev); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1730 if (sts) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1731 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1732 "Unable to register keyboard input device (%d)\n", sts); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1733 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1734 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1735 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1736 /* > 038b1a05eae666 Ronald Tschalär 2019-07-15 1737 * The applespi device doesn't send interrupts normally (as is described > 038b1a05eae666 Ronald Tschalär 2019-07-15 1738 * in its DSDT), but rather seems to use ACPI GPEs. > 038b1a05eae666 Ronald Tschalär 2019-07-15 1739 */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1740 acpi_sts = acpi_evaluate_integer(spi_handle, "_GPE", NULL, &gpe); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1741 if (ACPI_FAILURE(acpi_sts)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1742 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1743 "Failed to obtain GPE for SPI slave device: %s\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1744 acpi_format_exception(acpi_sts)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1745 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1746 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1747 applespi->gpe = (int)gpe; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1748 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1749 acpi_sts = acpi_install_gpe_handler(NULL, applespi->gpe, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1750 ACPI_GPE_LEVEL_TRIGGERED, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1751 applespi_notify, applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1752 if (ACPI_FAILURE(acpi_sts)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1753 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1754 "Failed to install GPE handler for GPE %d: %s\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1755 applespi->gpe, acpi_format_exception(acpi_sts)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1756 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1757 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1758 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1759 applespi->suspended = false; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1760 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1761 acpi_sts = acpi_enable_gpe(NULL, applespi->gpe); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1762 if (ACPI_FAILURE(acpi_sts)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1763 dev_err(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1764 "Failed to enable GPE handler for GPE %d: %s\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1765 applespi->gpe, acpi_format_exception(acpi_sts)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1766 acpi_remove_gpe_handler(NULL, applespi->gpe, applespi_notify); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1767 return -ENODEV; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1768 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1769 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1770 /* trigger touchpad setup */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1771 applespi_init(applespi, false); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1772 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1773 /* > 038b1a05eae666 Ronald Tschalär 2019-07-15 1774 * By default this device is not enabled for wakeup; but USB keyboards > 038b1a05eae666 Ronald Tschalär 2019-07-15 1775 * generally are, so the expectation is that by default the keyboard > 038b1a05eae666 Ronald Tschalär 2019-07-15 1776 * will wake the system. > 038b1a05eae666 Ronald Tschalär 2019-07-15 1777 */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1778 device_wakeup_enable(&spi->dev); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1779 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1780 /* set up keyboard-backlight */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1781 sts = applespi_get_saved_bl_level(applespi); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1782 if (sts >= 0) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1783 applespi_set_bl_level(&applespi->backlight_info, sts); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1784 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1785 applespi->backlight_info.name = "spi::kbd_backlight"; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1786 applespi->backlight_info.default_trigger = "kbd-backlight"; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1787 applespi->backlight_info.brightness_set = applespi_set_bl_level; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1788 > 038b1a05eae666 Ronald Tschalär 2019-07-15 @1789 sts = devm_led_classdev_register(&spi->dev, &applespi->backlight_info); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1790 if (sts) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1791 dev_warn(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1792 "Unable to register keyboard backlight class dev (%d)\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1793 sts); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1794 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1795 /* set up debugfs entries for touchpad dimensions logging */ > 038b1a05eae666 Ronald Tschalär 2019-07-15 1796 applespi->debugfs_root = debugfs_create_dir("applespi", NULL); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1797 if (IS_ERR(applespi->debugfs_root)) { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1798 if (PTR_ERR(applespi->debugfs_root) != -ENODEV) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1799 dev_warn(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1800 "Error creating debugfs root entry (%ld)\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1801 PTR_ERR(applespi->debugfs_root)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1802 } else { > 038b1a05eae666 Ronald Tschalär 2019-07-15 1803 struct dentry *ret; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1804 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1805 ret = debugfs_create_bool("enable_tp_dim", 0600, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1806 applespi->debugfs_root, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1807 &applespi->debug_tp_dim); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1808 if (IS_ERR(ret)) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1809 dev_dbg(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1810 "Error creating debugfs entry enable_tp_dim (%ld)\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1811 PTR_ERR(ret)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1812 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1813 ret = debugfs_create_file("tp_dim", 0400, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1814 applespi->debugfs_root, applespi, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1815 &applespi_tp_dim_fops); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1816 if (IS_ERR(ret)) > 038b1a05eae666 Ronald Tschalär 2019-07-15 1817 dev_dbg(&applespi->spi->dev, > 038b1a05eae666 Ronald Tschalär 2019-07-15 1818 "Error creating debugfs entry tp_dim (%ld)\n", > 038b1a05eae666 Ronald Tschalär 2019-07-15 1819 PTR_ERR(ret)); > 038b1a05eae666 Ronald Tschalär 2019-07-15 1820 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1821 > 038b1a05eae666 Ronald Tschalär 2019-07-15 1822 return 0; > 038b1a05eae666 Ronald Tschalär 2019-07-15 1823 } > 038b1a05eae666 Ronald Tschalär 2019-07-15 1824 > > :::::: The code at line 1789 was first introduced by commit > :::::: 038b1a05eae6666b731920e46f47d8e2332e07ff Input: add Apple SPI keyboard and trackpad driver > > :::::: TO: Ronald Tschalär <ronald@innovation.ch> > :::::: CC: Dmitry Torokhov <dmitry.torokhov@gmail.com> > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation > -- Best regards, Jacek Anaszewski ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' 2019-08-24 22:58 ` Jacek Anaszewski @ 2019-08-27 6:46 ` Life is hard, and then you die 2019-08-27 19:39 ` Jacek Anaszewski 0 siblings, 1 reply; 4+ messages in thread From: Life is hard, and then you die @ 2019-08-27 6:46 UTC (permalink / raw) To: Jacek Anaszewski Cc: kbuild test robot, kbuild-all, linux-leds, Dmitry Torokhov Hi Jacek, On Sun, Aug 25, 2019 at 12:58:59AM +0200, Jacek Anaszewski wrote: > > You're missing LEDS_CLASS dependency in Kconfig: > > diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig > index 8e9c3ea9d5e7..96285e071ee1 100644 > --- a/drivers/input/keyboard/Kconfig > +++ b/drivers/input/keyboard/Kconfig > @@ -76,6 +76,7 @@ config KEYBOARD_APPLESPI > depends on ACPI && EFI > depends on SPI > depends on X86 || COMPILE_TEST > + depends on LEDS_CLASS > help > Say Y here if you are running Linux on any Apple MacBook8,1 or > later, > or any MacBookPro13,* or MacBookPro14,*. This patch is already present in v5.3-rc4 - I can't tell from the below what version you were testing, but apparently an older one? > On 8/25/19 12:03 AM, kbuild test robot wrote: > > tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/j.anaszewski/linux-leds.git for-next > > head: fd81d7e946c6bdb86dbf0bd88fee3e1a545e7979 > > commit: b2b998c0f944993c9ef435569651e407d607af41 [15/49] leds: class: Improve LED and LED flash class registration API > > config: i386-randconfig-h002-201934 (attached as .config) > > compiler: gcc-7 (Debian 7.4.0-10) 7.4.0 > > reproduce: > > git checkout b2b998c0f944993c9ef435569651e407d607af41 > > # save the attached .config to linux build tree > > make ARCH=i386 > > > > If you fix the issue, kindly add following tag > > Reported-by: kbuild test robot <lkp@intel.com> > > > > All errors (new ones prefixed by >>): > > > > ld: drivers/input/keyboard/applespi.o: in function `applespi_probe': > >>> drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' > > > > vim +1789 drivers/input/keyboard/applespi.c > > > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1620 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1621 static int applespi_probe(struct spi_device *spi) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1622 { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1623 struct applespi_data *applespi; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1624 acpi_handle spi_handle = ACPI_HANDLE(&spi->dev); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1625 acpi_status acpi_sts; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1626 int sts, i; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1627 unsigned long long gpe, usb_status; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1628 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1629 /* check if the USB interface is present and enabled already */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1630 acpi_sts = acpi_evaluate_integer(spi_handle, "UIST", NULL, &usb_status); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1631 if (ACPI_SUCCESS(acpi_sts) && usb_status) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1632 /* let the USB driver take over instead */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1633 dev_info(&spi->dev, "USB interface already enabled\n"); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1634 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1635 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1636 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1637 /* allocate driver data */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1638 applespi = devm_kzalloc(&spi->dev, sizeof(*applespi), GFP_KERNEL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1639 if (!applespi) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1640 return -ENOMEM; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1641 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1642 applespi->spi = spi; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1643 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1644 INIT_WORK(&applespi->work, applespi_worker); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1645 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1646 /* store the driver data */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1647 spi_set_drvdata(spi, applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1648 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1649 /* create our buffers */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1650 applespi->tx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1651 GFP_KERNEL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1652 applespi->tx_status = devm_kmalloc(&spi->dev, APPLESPI_STATUS_SIZE, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1653 GFP_KERNEL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1654 applespi->rx_buffer = devm_kmalloc(&spi->dev, APPLESPI_PACKET_SIZE, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1655 GFP_KERNEL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1656 applespi->msg_buf = devm_kmalloc_array(&spi->dev, MAX_PKTS_PER_MSG, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1657 APPLESPI_PACKET_SIZE, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1658 GFP_KERNEL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1659 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1660 if (!applespi->tx_buffer || !applespi->tx_status || > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1661 !applespi->rx_buffer || !applespi->msg_buf) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1662 return -ENOMEM; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1663 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1664 /* set up our spi messages */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1665 applespi_setup_read_txfrs(applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1666 applespi_setup_write_txfrs(applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1667 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1668 /* cache ACPI method handles */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1669 acpi_sts = acpi_get_handle(spi_handle, "SIEN", &applespi->sien); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1670 if (ACPI_FAILURE(acpi_sts)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1671 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1672 "Failed to get SIEN ACPI method handle: %s\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1673 acpi_format_exception(acpi_sts)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1674 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1675 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1676 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1677 acpi_sts = acpi_get_handle(spi_handle, "SIST", &applespi->sist); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1678 if (ACPI_FAILURE(acpi_sts)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1679 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1680 "Failed to get SIST ACPI method handle: %s\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1681 acpi_format_exception(acpi_sts)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1682 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1683 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1684 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1685 /* switch on the SPI interface */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1686 sts = applespi_setup_spi(applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1687 if (sts) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1688 return sts; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1689 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1690 sts = applespi_enable_spi(applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1691 if (sts) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1692 return sts; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1693 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1694 /* setup the keyboard input dev */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1695 applespi->keyboard_input_dev = devm_input_allocate_device(&spi->dev); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1696 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1697 if (!applespi->keyboard_input_dev) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1698 return -ENOMEM; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1699 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1700 applespi->keyboard_input_dev->name = "Apple SPI Keyboard"; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1701 applespi->keyboard_input_dev->phys = "applespi/input0"; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1702 applespi->keyboard_input_dev->dev.parent = &spi->dev; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1703 applespi->keyboard_input_dev->id.bustype = BUS_SPI; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1704 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1705 applespi->keyboard_input_dev->evbit[0] = > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1706 BIT_MASK(EV_KEY) | BIT_MASK(EV_LED) | BIT_MASK(EV_REP); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1707 applespi->keyboard_input_dev->ledbit[0] = BIT_MASK(LED_CAPSL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1708 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1709 input_set_drvdata(applespi->keyboard_input_dev, applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1710 applespi->keyboard_input_dev->event = applespi_event; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1711 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1712 for (i = 0; i < ARRAY_SIZE(applespi_scancodes); i++) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1713 if (applespi_scancodes[i]) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1714 input_set_capability(applespi->keyboard_input_dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1715 EV_KEY, applespi_scancodes[i]); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1716 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1717 for (i = 0; i < ARRAY_SIZE(applespi_controlcodes); i++) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1718 if (applespi_controlcodes[i]) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1719 input_set_capability(applespi->keyboard_input_dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1720 EV_KEY, applespi_controlcodes[i]); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1721 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1722 for (i = 0; i < ARRAY_SIZE(applespi_fn_codes); i++) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1723 if (applespi_fn_codes[i].to) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1724 input_set_capability(applespi->keyboard_input_dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1725 EV_KEY, applespi_fn_codes[i].to); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1726 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1727 input_set_capability(applespi->keyboard_input_dev, EV_KEY, KEY_FN); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1728 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1729 sts = input_register_device(applespi->keyboard_input_dev); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1730 if (sts) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1731 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1732 "Unable to register keyboard input device (%d)\n", sts); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1733 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1734 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1735 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1736 /* > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1737 * The applespi device doesn't send interrupts normally (as is described > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1738 * in its DSDT), but rather seems to use ACPI GPEs. > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1739 */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1740 acpi_sts = acpi_evaluate_integer(spi_handle, "_GPE", NULL, &gpe); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1741 if (ACPI_FAILURE(acpi_sts)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1742 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1743 "Failed to obtain GPE for SPI slave device: %s\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1744 acpi_format_exception(acpi_sts)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1745 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1746 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1747 applespi->gpe = (int)gpe; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1748 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1749 acpi_sts = acpi_install_gpe_handler(NULL, applespi->gpe, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1750 ACPI_GPE_LEVEL_TRIGGERED, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1751 applespi_notify, applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1752 if (ACPI_FAILURE(acpi_sts)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1753 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1754 "Failed to install GPE handler for GPE %d: %s\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1755 applespi->gpe, acpi_format_exception(acpi_sts)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1756 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1757 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1758 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1759 applespi->suspended = false; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1760 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1761 acpi_sts = acpi_enable_gpe(NULL, applespi->gpe); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1762 if (ACPI_FAILURE(acpi_sts)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1763 dev_err(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1764 "Failed to enable GPE handler for GPE %d: %s\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1765 applespi->gpe, acpi_format_exception(acpi_sts)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1766 acpi_remove_gpe_handler(NULL, applespi->gpe, applespi_notify); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1767 return -ENODEV; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1768 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1769 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1770 /* trigger touchpad setup */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1771 applespi_init(applespi, false); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1772 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1773 /* > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1774 * By default this device is not enabled for wakeup; but USB keyboards > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1775 * generally are, so the expectation is that by default the keyboard > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1776 * will wake the system. > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1777 */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1778 device_wakeup_enable(&spi->dev); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1779 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1780 /* set up keyboard-backlight */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1781 sts = applespi_get_saved_bl_level(applespi); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1782 if (sts >= 0) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1783 applespi_set_bl_level(&applespi->backlight_info, sts); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1784 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1785 applespi->backlight_info.name = "spi::kbd_backlight"; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1786 applespi->backlight_info.default_trigger = "kbd-backlight"; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1787 applespi->backlight_info.brightness_set = applespi_set_bl_level; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1788 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 @1789 sts = devm_led_classdev_register(&spi->dev, &applespi->backlight_info); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1790 if (sts) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1791 dev_warn(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1792 "Unable to register keyboard backlight class dev (%d)\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1793 sts); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1794 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1795 /* set up debugfs entries for touchpad dimensions logging */ > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1796 applespi->debugfs_root = debugfs_create_dir("applespi", NULL); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1797 if (IS_ERR(applespi->debugfs_root)) { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1798 if (PTR_ERR(applespi->debugfs_root) != -ENODEV) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1799 dev_warn(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1800 "Error creating debugfs root entry (%ld)\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1801 PTR_ERR(applespi->debugfs_root)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1802 } else { > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1803 struct dentry *ret; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1804 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1805 ret = debugfs_create_bool("enable_tp_dim", 0600, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1806 applespi->debugfs_root, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1807 &applespi->debug_tp_dim); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1808 if (IS_ERR(ret)) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1809 dev_dbg(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1810 "Error creating debugfs entry enable_tp_dim (%ld)\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1811 PTR_ERR(ret)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1812 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1813 ret = debugfs_create_file("tp_dim", 0400, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1814 applespi->debugfs_root, applespi, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1815 &applespi_tp_dim_fops); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1816 if (IS_ERR(ret)) > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1817 dev_dbg(&applespi->spi->dev, > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1818 "Error creating debugfs entry tp_dim (%ld)\n", > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1819 PTR_ERR(ret)); > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1820 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1821 > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1822 return 0; > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1823 } > > 038b1a05eae666 Ronald Tschalär 2019-07-15 1824 > > > > :::::: The code at line 1789 was first introduced by commit > > :::::: 038b1a05eae6666b731920e46f47d8e2332e07ff Input: add Apple SPI keyboard and trackpad driver > > > > :::::: TO: Ronald Tschalär <ronald@innovation.ch> > > :::::: CC: Dmitry Torokhov <dmitry.torokhov@gmail.com> > > > > --- > > 0-DAY kernel test infrastructure Open Source Technology Center > > https://lists.01.org/pipermail/kbuild-all Intel Corporation > > > > -- > Best regards, > Jacek Anaszewski Cheers, Ronald ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' 2019-08-27 6:46 ` Life is hard, and then you die @ 2019-08-27 19:39 ` Jacek Anaszewski 0 siblings, 0 replies; 4+ messages in thread From: Jacek Anaszewski @ 2019-08-27 19:39 UTC (permalink / raw) To: Life is hard, and then you die Cc: kbuild test robot, kbuild-all, linux-leds, Dmitry Torokhov On 8/27/19 8:46 AM, Life is hard, and then you die wrote: > > Hi Jacek, > > On Sun, Aug 25, 2019 at 12:58:59AM +0200, Jacek Anaszewski wrote: >> >> You're missing LEDS_CLASS dependency in Kconfig: >> >> diff --git a/drivers/input/keyboard/Kconfig b/drivers/input/keyboard/Kconfig >> index 8e9c3ea9d5e7..96285e071ee1 100644 >> --- a/drivers/input/keyboard/Kconfig >> +++ b/drivers/input/keyboard/Kconfig >> @@ -76,6 +76,7 @@ config KEYBOARD_APPLESPI >> depends on ACPI && EFI >> depends on SPI >> depends on X86 || COMPILE_TEST >> + depends on LEDS_CLASS >> help >> Say Y here if you are running Linux on any Apple MacBook8,1 or >> later, >> or any MacBookPro13,* or MacBookPro14,*. > > This patch is already present in v5.3-rc4 - I can't tell from the > below what version you were testing, but apparently an older one? I received this message from kbuild test robot <lkp@intel.com>, that tested for-next branch of linux-leds.git which is based on rc1. But I see it fixed indeed in mainline, so that's no longer an issue. -- Best regards, Jacek Anaszewski ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-27 19:39 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-08-24 22:03 [j.anaszewski-leds:for-next 15/49] drivers/input/keyboard/applespi.c:1789: undefined reference to `devm_led_classdev_register_ext' kbuild test robot 2019-08-24 22:58 ` Jacek Anaszewski 2019-08-27 6:46 ` Life is hard, and then you die 2019-08-27 19:39 ` Jacek Anaszewski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox