From: kernel test robot <lkp@intel.com>
To: Oleksij Rempel <o.rempel@pengutronix.de>,
Rob Herring <robh+dt@kernel.org>,
Jonathan Cameron <jic23@kernel.org>
Cc: kbuild-all@lists.01.org, Oleksij Rempel <o.rempel@pengutronix.de>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Pengutronix Kernel Team <kernel@pengutronix.de>,
David Jander <david@protonic.nl>,
Robin van der Gracht <robin@protonic.nl>,
linux-iio@vger.kernel.org
Subject: Re: [PATCH v3 3/3] iio: adc: add ADC driver for the TI TSC2046 controller
Date: Sat, 20 Mar 2021 00:35:23 +0800 [thread overview]
Message-ID: <202103200032.OAOTRy35-lkp@intel.com> (raw)
In-Reply-To: <20210319144509.7627-4-o.rempel@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 6672 bytes --]
Hi Oleksij,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linux/master]
[also build test WARNING on linus/master v5.12-rc3 next-20210319]
[cannot apply to iio/togreg]
[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/0day-ci/linux/commits/Oleksij-Rempel/mainline-ti-tsc2046-adc-driver/20210319-224746
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a74e6a014c9d4d4161061f770c9b4f98372ac778
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.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/0day-ci/linux/commit/63e96b25ae609f5659a28132f77d353cc7ecbd84
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Oleksij-Rempel/mainline-ti-tsc2046-adc-driver/20210319-224746
git checkout 63e96b25ae609f5659a28132f77d353cc7ecbd84
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/device.h:15,
from include/linux/spi/spi.h:10,
from drivers/iio/adc/ti-tsc2046.c:11:
drivers/iio/adc/ti-tsc2046.c: In function 'tsc2046_adc_probe':
>> drivers/iio/adc/ti-tsc2046.c:621:16: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
19 | #define dev_fmt(fmt) fmt
| ^~~
drivers/iio/adc/ti-tsc2046.c:621:3: note: in expansion of macro 'dev_err'
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ^~~~~~~
drivers/iio/adc/ti-tsc2046.c:621:77: note: format string is defined here
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ~^
| |
| int
| %ld
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for FRAME_POINTER
Depends on DEBUG_KERNEL && (M68K || UML || SUPERH) || ARCH_WANT_FRAME_POINTERS
Selected by
- FAULT_INJECTION_STACKTRACE_FILTER && FAULT_INJECTION_DEBUG_FS && STACKTRACE_SUPPORT && !X86_64 && !MIPS && !PPC && !S390 && !MICROBLAZE && !ARM && !ARC && !X86
vim +621 drivers/iio/adc/ti-tsc2046.c
609
610 static int tsc2046_adc_probe(struct spi_device *spi)
611 {
612 const struct tsc2046_adc_dcfg *dcfg;
613 struct device *dev = &spi->dev;
614 struct tsc2046_adc_priv *priv;
615 struct iio_dev *indio_dev;
616 struct iio_trigger *trig;
617 const char *name;
618 int ret;
619
620 if (spi->max_speed_hz > TI_TSC2046_MAX_CLK_FREQ) {
> 621 dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
622 spi->max_speed_hz, TI_TSC2046_MAX_CLK_FREQ);
623 return -EINVAL;
624 }
625
626 dcfg = device_get_match_data(dev);
627 if (!dcfg)
628 return -EINVAL;
629
630 spi->bits_per_word = 8;
631 spi->mode &= ~SPI_MODE_X_MASK;
632 spi->mode |= SPI_MODE_0;
633 ret = spi_setup(spi);
634 if (ret < 0)
635 return dev_err_probe(dev, ret, "Error in spi setup\n");
636
637 indio_dev = devm_iio_device_alloc(dev, sizeof(*priv));
638 if (!indio_dev)
639 return -ENOMEM;
640
641 priv = iio_priv(indio_dev);
642 priv->dcfg = dcfg;
643
644 spi_set_drvdata(spi, indio_dev);
645
646 priv->spi = spi;
647
648 name = devm_kasprintf(dev, GFP_KERNEL, "%s-%s",
649 TI_TSC2046_NAME, dev_name(dev));
650
651 indio_dev->name = name;
652 indio_dev->modes = INDIO_DIRECT_MODE | INDIO_BUFFER_TRIGGERED;
653 indio_dev->channels = dcfg->channels;
654 indio_dev->num_channels = dcfg->num_channels;
655 indio_dev->info = &tsc2046_adc_info;
656
657 tsc2046_adc_parse_fwnode(priv);
658
659 ret = tsc2046_adc_setup_spi_msg(priv);
660 if (ret)
661 return ret;
662
663 mutex_init(&priv->slock);
664
665 /* TODO: remove IRQ_NOAUTOEN after needed patches are mainline */
666 irq_set_status_flags(spi->irq, IRQ_NOAUTOEN);
667 ret = devm_request_irq(dev, spi->irq, &tsc2046_adc_irq,
668 0, name, indio_dev);
669 if (ret)
670 return ret;
671
672 trig = devm_iio_trigger_alloc(dev, "touchscreen-%s", indio_dev->name);
673 if (!trig)
674 return -ENOMEM;
675
676 priv->trig = trig;
677 trig->dev.parent = indio_dev->dev.parent;
678 iio_trigger_set_drvdata(trig, indio_dev);
679 trig->ops = &tsc2046_adc_trigger_ops;
680
681 spin_lock_init(&priv->trig_lock);
682 hrtimer_init(&priv->trig_timer, CLOCK_MONOTONIC,
683 HRTIMER_MODE_REL_SOFT);
684 priv->trig_timer.function = tsc2046_adc_trig_more;
685
686 ret = devm_iio_trigger_register(dev, trig);
687 if (ret) {
688 dev_err(dev, "failed to register trigger\n");
689 return ret;
690 }
691
692 ret = devm_iio_triggered_buffer_setup(dev, indio_dev, NULL,
693 &tsc2046_adc_trigger_handler, NULL);
694 if (ret) {
695 dev_err(dev, "Failed to setup triggered buffer\n");
696 return ret;
697 }
698
699 /* set default trigger */
700 indio_dev->trig = iio_trigger_get(priv->trig);
701
702 ret = devm_iio_device_register(dev, indio_dev);
703 if (ret) {
704 dev_err(dev, "Failed to register iio device\n");
705 return ret;
706 }
707
708 return 0;
709 }
710
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 63789 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v3 3/3] iio: adc: add ADC driver for the TI TSC2046 controller
Date: Sat, 20 Mar 2021 00:35:23 +0800 [thread overview]
Message-ID: <202103200032.OAOTRy35-lkp@intel.com> (raw)
In-Reply-To: <20210319144509.7627-4-o.rempel@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 6839 bytes --]
Hi Oleksij,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linux/master]
[also build test WARNING on linus/master v5.12-rc3 next-20210319]
[cannot apply to iio/togreg]
[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/0day-ci/linux/commits/Oleksij-Rempel/mainline-ti-tsc2046-adc-driver/20210319-224746
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git a74e6a014c9d4d4161061f770c9b4f98372ac778
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 9.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/0day-ci/linux/commit/63e96b25ae609f5659a28132f77d353cc7ecbd84
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Oleksij-Rempel/mainline-ti-tsc2046-adc-driver/20210319-224746
git checkout 63e96b25ae609f5659a28132f77d353cc7ecbd84
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=ia64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/device.h:15,
from include/linux/spi/spi.h:10,
from drivers/iio/adc/ti-tsc2046.c:11:
drivers/iio/adc/ti-tsc2046.c: In function 'tsc2046_adc_probe':
>> drivers/iio/adc/ti-tsc2046.c:621:16: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
19 | #define dev_fmt(fmt) fmt
| ^~~
drivers/iio/adc/ti-tsc2046.c:621:3: note: in expansion of macro 'dev_err'
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ^~~~~~~
drivers/iio/adc/ti-tsc2046.c:621:77: note: format string is defined here
621 | dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
| ~^
| |
| int
| %ld
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for FRAME_POINTER
Depends on DEBUG_KERNEL && (M68K || UML || SUPERH) || ARCH_WANT_FRAME_POINTERS
Selected by
- FAULT_INJECTION_STACKTRACE_FILTER && FAULT_INJECTION_DEBUG_FS && STACKTRACE_SUPPORT && !X86_64 && !MIPS && !PPC && !S390 && !MICROBLAZE && !ARM && !ARC && !X86
vim +621 drivers/iio/adc/ti-tsc2046.c
609
610 static int tsc2046_adc_probe(struct spi_device *spi)
611 {
612 const struct tsc2046_adc_dcfg *dcfg;
613 struct device *dev = &spi->dev;
614 struct tsc2046_adc_priv *priv;
615 struct iio_dev *indio_dev;
616 struct iio_trigger *trig;
617 const char *name;
618 int ret;
619
620 if (spi->max_speed_hz > TI_TSC2046_MAX_CLK_FREQ) {
> 621 dev_err(dev, "SPI max_speed_hz is too high: %d Hz. Max supported freq is %d Hz\n",
622 spi->max_speed_hz, TI_TSC2046_MAX_CLK_FREQ);
623 return -EINVAL;
624 }
625
626 dcfg = device_get_match_data(dev);
627 if (!dcfg)
628 return -EINVAL;
629
630 spi->bits_per_word = 8;
631 spi->mode &= ~SPI_MODE_X_MASK;
632 spi->mode |= SPI_MODE_0;
633 ret = spi_setup(spi);
634 if (ret < 0)
635 return dev_err_probe(dev, ret, "Error in spi setup\n");
636
637 indio_dev = devm_iio_device_alloc(dev, sizeof(*priv));
638 if (!indio_dev)
639 return -ENOMEM;
640
641 priv = iio_priv(indio_dev);
642 priv->dcfg = dcfg;
643
644 spi_set_drvdata(spi, indio_dev);
645
646 priv->spi = spi;
647
648 name = devm_kasprintf(dev, GFP_KERNEL, "%s-%s",
649 TI_TSC2046_NAME, dev_name(dev));
650
651 indio_dev->name = name;
652 indio_dev->modes = INDIO_DIRECT_MODE | INDIO_BUFFER_TRIGGERED;
653 indio_dev->channels = dcfg->channels;
654 indio_dev->num_channels = dcfg->num_channels;
655 indio_dev->info = &tsc2046_adc_info;
656
657 tsc2046_adc_parse_fwnode(priv);
658
659 ret = tsc2046_adc_setup_spi_msg(priv);
660 if (ret)
661 return ret;
662
663 mutex_init(&priv->slock);
664
665 /* TODO: remove IRQ_NOAUTOEN after needed patches are mainline */
666 irq_set_status_flags(spi->irq, IRQ_NOAUTOEN);
667 ret = devm_request_irq(dev, spi->irq, &tsc2046_adc_irq,
668 0, name, indio_dev);
669 if (ret)
670 return ret;
671
672 trig = devm_iio_trigger_alloc(dev, "touchscreen-%s", indio_dev->name);
673 if (!trig)
674 return -ENOMEM;
675
676 priv->trig = trig;
677 trig->dev.parent = indio_dev->dev.parent;
678 iio_trigger_set_drvdata(trig, indio_dev);
679 trig->ops = &tsc2046_adc_trigger_ops;
680
681 spin_lock_init(&priv->trig_lock);
682 hrtimer_init(&priv->trig_timer, CLOCK_MONOTONIC,
683 HRTIMER_MODE_REL_SOFT);
684 priv->trig_timer.function = tsc2046_adc_trig_more;
685
686 ret = devm_iio_trigger_register(dev, trig);
687 if (ret) {
688 dev_err(dev, "failed to register trigger\n");
689 return ret;
690 }
691
692 ret = devm_iio_triggered_buffer_setup(dev, indio_dev, NULL,
693 &tsc2046_adc_trigger_handler, NULL);
694 if (ret) {
695 dev_err(dev, "Failed to setup triggered buffer\n");
696 return ret;
697 }
698
699 /* set default trigger */
700 indio_dev->trig = iio_trigger_get(priv->trig);
701
702 ret = devm_iio_device_register(dev, indio_dev);
703 if (ret) {
704 dev_err(dev, "Failed to register iio device\n");
705 return ret;
706 }
707
708 return 0;
709 }
710
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 63789 bytes --]
next prev parent reply other threads:[~2021-03-19 16:37 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-19 14:45 [PATCH v3 0/3] mainline ti tsc2046 adc driver Oleksij Rempel
2021-03-19 14:45 ` [PATCH v3 1/3] dt-bindings:iio:adc: add generic settling-time-us and oversampling-ratio channel properties Oleksij Rempel
2021-03-19 14:45 ` [PATCH v3 2/3] dt-bindings:iio:adc: add documentation for TI TSC2046 controller Oleksij Rempel
2021-03-19 14:45 ` [PATCH v3 3/3] iio: adc: add ADC driver for the " Oleksij Rempel
2021-03-19 16:35 ` kernel test robot [this message]
2021-03-19 16:35 ` kernel test robot
2021-03-19 17:42 ` Andy Shevchenko
2021-03-22 10:30 ` Oleksij Rempel
2021-03-22 13:41 ` Andy Shevchenko
2021-03-22 14:08 ` Oleksij Rempel
2021-03-20 15:46 ` Jonathan Cameron
2021-03-22 11:56 ` Oleksij Rempel
2021-03-22 14:27 ` Jonathan Cameron
2021-03-29 7:38 ` Oleksij Rempel
2021-03-29 10:23 ` Jonathan Cameron
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=202103200032.OAOTRy35-lkp@intel.com \
--to=lkp@intel.com \
--cc=andy.shevchenko@gmail.com \
--cc=david@protonic.nl \
--cc=devicetree@vger.kernel.org \
--cc=jic23@kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=kernel@pengutronix.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=o.rempel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=robin@protonic.nl \
/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.