From: kernel test robot <yujie.liu@intel.com>
To: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: <oe-kbuild-all@lists.linux.dev>, <linux-kernel@vger.kernel.org>
Subject: drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'ndevres' not described in 'intel_th_alloc'
Date: Wed, 22 Nov 2023 10:44:28 +0800 [thread overview]
Message-ID: <202311220224.smGIP2Re-lkp@intel.com> (raw)
Hi Alexander,
kernel test robot noticed the following build warnings:
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 98b1cc82c4affc16f5598d4fa14b1858671b2263
commit: db73a059de00eed721f13051c0d6ff3e7de90fe8 intel_th: Rework resource passing between glue layers and core
config: x86_64-buildonly-randconfig-001-20231012 (https://download.01.org/0day-ci/archive/20231122/202311220224.smGIP2Re-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231122/202311220224.smGIP2Re-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 <yujie.liu@intel.com>
| Closes: https://lore.kernel.org/r/202311220224.smGIP2Re-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'drvdata' not described in 'intel_th_alloc'
>> drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'ndevres' not described in 'intel_th_alloc'
vim +812 drivers/hwtracing/intel_th/core.c
39f4034693b7c7b Alexander Shishkin 2015-09-22 802
39f4034693b7c7b Alexander Shishkin 2015-09-22 803 /**
39f4034693b7c7b Alexander Shishkin 2015-09-22 804 * intel_th_alloc() - allocate a new Intel TH device and its subdevices
39f4034693b7c7b Alexander Shishkin 2015-09-22 805 * @dev: parent device
db73a059de00eed Alexander Shishkin 2019-05-03 806 * @devres: resources indexed by th_mmio_idx
39f4034693b7c7b Alexander Shishkin 2015-09-22 807 * @irq: irq number
39f4034693b7c7b Alexander Shishkin 2015-09-22 808 */
39f4034693b7c7b Alexander Shishkin 2015-09-22 809 struct intel_th *
3321371b5d64847 Alexander Shishkin 2017-08-18 810 intel_th_alloc(struct device *dev, struct intel_th_drvdata *drvdata,
3321371b5d64847 Alexander Shishkin 2017-08-18 811 struct resource *devres, unsigned int ndevres, int irq)
39f4034693b7c7b Alexander Shishkin 2015-09-22 @812 {
39f4034693b7c7b Alexander Shishkin 2015-09-22 813 struct intel_th *th;
661b0df8489a35d Alexander Shishkin 2017-08-23 814 int err, r;
661b0df8489a35d Alexander Shishkin 2017-08-23 815
db73a059de00eed Alexander Shishkin 2019-05-03 816 if (ndevres < TH_MMIO_END)
db73a059de00eed Alexander Shishkin 2019-05-03 817 return ERR_PTR(-EINVAL);
39f4034693b7c7b Alexander Shishkin 2015-09-22 818
39f4034693b7c7b Alexander Shishkin 2015-09-22 819 th = kzalloc(sizeof(*th), GFP_KERNEL);
39f4034693b7c7b Alexander Shishkin 2015-09-22 820 if (!th)
39f4034693b7c7b Alexander Shishkin 2015-09-22 821 return ERR_PTR(-ENOMEM);
39f4034693b7c7b Alexander Shishkin 2015-09-22 822
39f4034693b7c7b Alexander Shishkin 2015-09-22 823 th->id = ida_simple_get(&intel_th_ida, 0, 0, GFP_KERNEL);
39f4034693b7c7b Alexander Shishkin 2015-09-22 824 if (th->id < 0) {
39f4034693b7c7b Alexander Shishkin 2015-09-22 825 err = th->id;
39f4034693b7c7b Alexander Shishkin 2015-09-22 826 goto err_alloc;
39f4034693b7c7b Alexander Shishkin 2015-09-22 827 }
39f4034693b7c7b Alexander Shishkin 2015-09-22 828
39f4034693b7c7b Alexander Shishkin 2015-09-22 829 th->major = __register_chrdev(0, 0, TH_POSSIBLE_OUTPUTS,
39f4034693b7c7b Alexander Shishkin 2015-09-22 830 "intel_th/output", &intel_th_output_fops);
39f4034693b7c7b Alexander Shishkin 2015-09-22 831 if (th->major < 0) {
39f4034693b7c7b Alexander Shishkin 2015-09-22 832 err = th->major;
39f4034693b7c7b Alexander Shishkin 2015-09-22 833 goto err_ida;
39f4034693b7c7b Alexander Shishkin 2015-09-22 834 }
39f4034693b7c7b Alexander Shishkin 2015-09-22 835 th->dev = dev;
3321371b5d64847 Alexander Shishkin 2017-08-18 836 th->drvdata = drvdata;
39f4034693b7c7b Alexander Shishkin 2015-09-22 837
db73a059de00eed Alexander Shishkin 2019-05-03 838 for (r = 0; r < ndevres; r++)
db73a059de00eed Alexander Shishkin 2019-05-03 839 th->resource[r] = devres[r];
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 840 th->num_resources = ndevres;
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 841 th->irq = irq;
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 842
d7b1787161b78a5 Alexander Shishkin 2016-02-15 843 dev_set_drvdata(dev, th);
d7b1787161b78a5 Alexander Shishkin 2016-02-15 844
142dfeb20209607 Alexander Shishkin 2016-06-22 845 pm_runtime_no_callbacks(dev);
142dfeb20209607 Alexander Shishkin 2016-06-22 846 pm_runtime_put(dev);
142dfeb20209607 Alexander Shishkin 2016-06-22 847 pm_runtime_allow(dev);
142dfeb20209607 Alexander Shishkin 2016-06-22 848
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 849 err = intel_th_populate(th);
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 850 if (err) {
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 851 /* free the subdevices and undo everything */
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 852 intel_th_free(th);
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 853 return ERR_PTR(err);
a753bfcfdb1f31d Alexander Shishkin 2017-08-10 854 }
39f4034693b7c7b Alexander Shishkin 2015-09-22 855
39f4034693b7c7b Alexander Shishkin 2015-09-22 856 return th;
39f4034693b7c7b Alexander Shishkin 2015-09-22 857
39f4034693b7c7b Alexander Shishkin 2015-09-22 858 err_ida:
39f4034693b7c7b Alexander Shishkin 2015-09-22 859 ida_simple_remove(&intel_th_ida, th->id);
39f4034693b7c7b Alexander Shishkin 2015-09-22 860
39f4034693b7c7b Alexander Shishkin 2015-09-22 861 err_alloc:
39f4034693b7c7b Alexander Shishkin 2015-09-22 862 kfree(th);
39f4034693b7c7b Alexander Shishkin 2015-09-22 863
39f4034693b7c7b Alexander Shishkin 2015-09-22 864 return ERR_PTR(err);
39f4034693b7c7b Alexander Shishkin 2015-09-22 865 }
39f4034693b7c7b Alexander Shishkin 2015-09-22 866 EXPORT_SYMBOL_GPL(intel_th_alloc);
39f4034693b7c7b Alexander Shishkin 2015-09-22 867
:::::: The code at line 812 was first introduced by commit
:::::: 39f4034693b7c7bd1fe4cb58c93259d600f55561 intel_th: Add driver infrastructure for Intel(R) Trace Hub devices
:::::: TO: Alexander Shishkin <alexander.shishkin@linux.intel.com>
:::::: CC: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2023-11-22 2:49 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-22 2:44 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2024-02-18 22:01 drivers/hwtracing/intel_th/core.c:812: warning: Function parameter or member 'ndevres' not described in 'intel_th_alloc' kernel test robot
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=202311220224.smGIP2Re-lkp@intel.com \
--to=yujie.liu@intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
/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