From: kernel test robot <lkp@intel.com>
To: Mauro Carvalho Chehab <mchehab@kernel.org>,
Jonathan Corbet <corbet@lwn.net>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
linux-media@vger.kernel.org,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Linux Doc Mailing List <linux-doc@vger.kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 27/27] scripts: kernel-doc: validate kernel-doc markup with the actual names
Date: Mon, 16 Nov 2020 23:42:49 +0800 [thread overview]
Message-ID: <202011162356.2dksRicD-lkp@intel.com> (raw)
In-Reply-To: <7b013fef4b0a45bddc5f1a5593a282baceb13b0c.1605521731.git.mchehab+huawei@kernel.org>
[-- Attachment #1: Type: text/plain, Size: 11019 bytes --]
Hi Mauro,
I love your patch! Perhaps something to improve:
[auto build test WARNING on next-20201116]
[cannot apply to drm-intel/for-linux-next s390/features tip/timers/core tip/irq/core tip/sched/core linus/master hnaz-linux-mm/master v5.10-rc4 v5.10-rc3 v5.10-rc2 v5.10-rc4]
[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/Mauro-Carvalho-Chehab/net-phy-fix-kernel-doc-markups/20201116-191847
base: 034307507118f7e1b18f8403c85af2216da2dc94
config: arm-randconfig-r006-20201116 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project c044709b8fbea2a9a375e4173a6bd735f6866c0c)
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
# install arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://github.com/0day-ci/linux/commit/11cd9cdaab434fff7a5edbee8d8d4ce9ef11acf4
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Mauro-Carvalho-Chehab/net-phy-fix-kernel-doc-markups/20201116-191847
git checkout 11cd9cdaab434fff7a5edbee8d8d4ce9ef11acf4
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
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 >>):
arch/arm/common/locomo.c:369: warning: Function parameter or member 'me' not described in '__locomo_probe'
arch/arm/common/locomo.c:369: warning: Function parameter or member 'mem' not described in '__locomo_probe'
arch/arm/common/locomo.c:369: warning: Function parameter or member 'irq' not described in '__locomo_probe'
>> arch/arm/common/locomo.c:369: warning: expecting prototype for locomo_probe(). Prototype was for __locomo_probe() instead
vim +369 arch/arm/common/locomo.c
b38d950d3aedf90 John Lenz 2005-09-08 353
4ebf2d00260bac5 Pavel Machek 2006-03-15 354
^1da177e4c3f415 Linus Torvalds 2005-04-16 355 /**
^1da177e4c3f415 Linus Torvalds 2005-04-16 356 * locomo_probe - probe for a single LoCoMo chip.
^1da177e4c3f415 Linus Torvalds 2005-04-16 357 * @phys_addr: physical address of device.
^1da177e4c3f415 Linus Torvalds 2005-04-16 358 *
^1da177e4c3f415 Linus Torvalds 2005-04-16 359 * Probe for a LoCoMo chip. This must be called
^1da177e4c3f415 Linus Torvalds 2005-04-16 360 * before any other locomo-specific code.
^1da177e4c3f415 Linus Torvalds 2005-04-16 361 *
^1da177e4c3f415 Linus Torvalds 2005-04-16 362 * Returns:
^1da177e4c3f415 Linus Torvalds 2005-04-16 363 * %-ENODEV device not found.
^1da177e4c3f415 Linus Torvalds 2005-04-16 364 * %-EBUSY physical address already marked in-use.
^1da177e4c3f415 Linus Torvalds 2005-04-16 365 * %0 successful.
^1da177e4c3f415 Linus Torvalds 2005-04-16 366 */
^1da177e4c3f415 Linus Torvalds 2005-04-16 367 static int
^1da177e4c3f415 Linus Torvalds 2005-04-16 368 __locomo_probe(struct device *me, struct resource *mem, int irq)
^1da177e4c3f415 Linus Torvalds 2005-04-16 @369 {
ac609d266e4af4e Eric Miao 2010-02-04 370 struct locomo_platform_data *pdata = me->platform_data;
^1da177e4c3f415 Linus Torvalds 2005-04-16 371 struct locomo *lchip;
^1da177e4c3f415 Linus Torvalds 2005-04-16 372 unsigned long r;
^1da177e4c3f415 Linus Torvalds 2005-04-16 373 int i, ret = -ENODEV;
^1da177e4c3f415 Linus Torvalds 2005-04-16 374
d2a02b93cf78205 Russell King 2006-03-20 375 lchip = kzalloc(sizeof(struct locomo), GFP_KERNEL);
^1da177e4c3f415 Linus Torvalds 2005-04-16 376 if (!lchip)
^1da177e4c3f415 Linus Torvalds 2005-04-16 377 return -ENOMEM;
^1da177e4c3f415 Linus Torvalds 2005-04-16 378
^1da177e4c3f415 Linus Torvalds 2005-04-16 379 spin_lock_init(&lchip->lock);
^1da177e4c3f415 Linus Torvalds 2005-04-16 380
^1da177e4c3f415 Linus Torvalds 2005-04-16 381 lchip->dev = me;
^1da177e4c3f415 Linus Torvalds 2005-04-16 382 dev_set_drvdata(lchip->dev, lchip);
^1da177e4c3f415 Linus Torvalds 2005-04-16 383
^1da177e4c3f415 Linus Torvalds 2005-04-16 384 lchip->phys = mem->start;
^1da177e4c3f415 Linus Torvalds 2005-04-16 385 lchip->irq = irq;
ac609d266e4af4e Eric Miao 2010-02-04 386 lchip->irq_base = (pdata) ? pdata->irq_base : NO_IRQ;
^1da177e4c3f415 Linus Torvalds 2005-04-16 387
^1da177e4c3f415 Linus Torvalds 2005-04-16 388 /*
^1da177e4c3f415 Linus Torvalds 2005-04-16 389 * Map the whole region. This also maps the
^1da177e4c3f415 Linus Torvalds 2005-04-16 390 * registers for our children.
^1da177e4c3f415 Linus Torvalds 2005-04-16 391 */
^1da177e4c3f415 Linus Torvalds 2005-04-16 392 lchip->base = ioremap(mem->start, PAGE_SIZE);
^1da177e4c3f415 Linus Torvalds 2005-04-16 393 if (!lchip->base) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 394 ret = -ENOMEM;
^1da177e4c3f415 Linus Torvalds 2005-04-16 395 goto out;
^1da177e4c3f415 Linus Torvalds 2005-04-16 396 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 397
^1da177e4c3f415 Linus Torvalds 2005-04-16 398 /* locomo initialize */
^1da177e4c3f415 Linus Torvalds 2005-04-16 399 locomo_writel(0, lchip->base + LOCOMO_ICR);
^1da177e4c3f415 Linus Torvalds 2005-04-16 400 /* KEYBOARD */
^1da177e4c3f415 Linus Torvalds 2005-04-16 401 locomo_writel(0, lchip->base + LOCOMO_KEYBOARD + LOCOMO_KIC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 402
^1da177e4c3f415 Linus Torvalds 2005-04-16 403 /* GPIO */
^1da177e4c3f415 Linus Torvalds 2005-04-16 404 locomo_writel(0, lchip->base + LOCOMO_GPO);
2a52efb2cecf782 Thomas Kunze 2008-04-29 405 locomo_writel((LOCOMO_GPIO(1) | LOCOMO_GPIO(2) | LOCOMO_GPIO(13) | LOCOMO_GPIO(14))
^1da177e4c3f415 Linus Torvalds 2005-04-16 406 , lchip->base + LOCOMO_GPE);
2a52efb2cecf782 Thomas Kunze 2008-04-29 407 locomo_writel((LOCOMO_GPIO(1) | LOCOMO_GPIO(2) | LOCOMO_GPIO(13) | LOCOMO_GPIO(14))
^1da177e4c3f415 Linus Torvalds 2005-04-16 408 , lchip->base + LOCOMO_GPD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 409 locomo_writel(0, lchip->base + LOCOMO_GIE);
^1da177e4c3f415 Linus Torvalds 2005-04-16 410
e44237818500257 Richard Purdie 2006-06-26 411 /* Frontlight */
^1da177e4c3f415 Linus Torvalds 2005-04-16 412 locomo_writel(0, lchip->base + LOCOMO_FRONTLIGHT + LOCOMO_ALS);
^1da177e4c3f415 Linus Torvalds 2005-04-16 413 locomo_writel(0, lchip->base + LOCOMO_FRONTLIGHT + LOCOMO_ALD);
4ebf2d00260bac5 Pavel Machek 2006-03-15 414
^1da177e4c3f415 Linus Torvalds 2005-04-16 415 /* Longtime timer */
^1da177e4c3f415 Linus Torvalds 2005-04-16 416 locomo_writel(0, lchip->base + LOCOMO_LTINT);
^1da177e4c3f415 Linus Torvalds 2005-04-16 417 /* SPI */
1b0d76cb9d5a655 H Hartley Sweeten 2010-03-22 418 locomo_writel(0, lchip->base + LOCOMO_SPI + LOCOMO_SPIIE);
^1da177e4c3f415 Linus Torvalds 2005-04-16 419
^1da177e4c3f415 Linus Torvalds 2005-04-16 420 locomo_writel(6 + 8 + 320 + 30 - 10, lchip->base + LOCOMO_ASD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 421 r = locomo_readl(lchip->base + LOCOMO_ASD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 422 r |= 0x8000;
^1da177e4c3f415 Linus Torvalds 2005-04-16 423 locomo_writel(r, lchip->base + LOCOMO_ASD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 424
^1da177e4c3f415 Linus Torvalds 2005-04-16 425 locomo_writel(6 + 8 + 320 + 30 - 10 - 128 + 4, lchip->base + LOCOMO_HSD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 426 r = locomo_readl(lchip->base + LOCOMO_HSD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 427 r |= 0x8000;
^1da177e4c3f415 Linus Torvalds 2005-04-16 428 locomo_writel(r, lchip->base + LOCOMO_HSD);
^1da177e4c3f415 Linus Torvalds 2005-04-16 429
^1da177e4c3f415 Linus Torvalds 2005-04-16 430 locomo_writel(128 / 8, lchip->base + LOCOMO_HSC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 431
^1da177e4c3f415 Linus Torvalds 2005-04-16 432 /* XON */
^1da177e4c3f415 Linus Torvalds 2005-04-16 433 locomo_writel(0x80, lchip->base + LOCOMO_TADC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 434 udelay(1000);
^1da177e4c3f415 Linus Torvalds 2005-04-16 435 /* CLK9MEN */
^1da177e4c3f415 Linus Torvalds 2005-04-16 436 r = locomo_readl(lchip->base + LOCOMO_TADC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 437 r |= 0x10;
^1da177e4c3f415 Linus Torvalds 2005-04-16 438 locomo_writel(r, lchip->base + LOCOMO_TADC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 439 udelay(100);
^1da177e4c3f415 Linus Torvalds 2005-04-16 440
^1da177e4c3f415 Linus Torvalds 2005-04-16 441 /* init DAC */
^1da177e4c3f415 Linus Torvalds 2005-04-16 442 r = locomo_readl(lchip->base + LOCOMO_DAC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 443 r |= LOCOMO_DAC_SCLOEB | LOCOMO_DAC_SDAOEB;
^1da177e4c3f415 Linus Torvalds 2005-04-16 444 locomo_writel(r, lchip->base + LOCOMO_DAC);
^1da177e4c3f415 Linus Torvalds 2005-04-16 445
^1da177e4c3f415 Linus Torvalds 2005-04-16 446 r = locomo_readl(lchip->base + LOCOMO_VER);
^1da177e4c3f415 Linus Torvalds 2005-04-16 447 printk(KERN_INFO "LoCoMo Chip: %lu%lu\n", (r >> 8), (r & 0xff));
^1da177e4c3f415 Linus Torvalds 2005-04-16 448
^1da177e4c3f415 Linus Torvalds 2005-04-16 449 /*
^1da177e4c3f415 Linus Torvalds 2005-04-16 450 * The interrupt controller must be initialised before any
^1da177e4c3f415 Linus Torvalds 2005-04-16 451 * other device to ensure that the interrupts are available.
^1da177e4c3f415 Linus Torvalds 2005-04-16 452 */
ac609d266e4af4e Eric Miao 2010-02-04 453 if (lchip->irq != NO_IRQ && lchip->irq_base != NO_IRQ)
^1da177e4c3f415 Linus Torvalds 2005-04-16 454 locomo_setup_irq(lchip);
^1da177e4c3f415 Linus Torvalds 2005-04-16 455
^1da177e4c3f415 Linus Torvalds 2005-04-16 456 for (i = 0; i < ARRAY_SIZE(locomo_devices); i++)
^1da177e4c3f415 Linus Torvalds 2005-04-16 457 locomo_init_one_child(lchip, &locomo_devices[i]);
^1da177e4c3f415 Linus Torvalds 2005-04-16 458 return 0;
^1da177e4c3f415 Linus Torvalds 2005-04-16 459
^1da177e4c3f415 Linus Torvalds 2005-04-16 460 out:
^1da177e4c3f415 Linus Torvalds 2005-04-16 461 kfree(lchip);
^1da177e4c3f415 Linus Torvalds 2005-04-16 462 return ret;
^1da177e4c3f415 Linus Torvalds 2005-04-16 463 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 464
---
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: 31215 bytes --]
next prev parent reply other threads:[~2020-11-16 15:44 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-16 10:17 [PATCH v4 00/27]Fix several bad kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:17 ` [PATCH v4 01/27] net: phy: fix " Mauro Carvalho Chehab
2020-11-16 10:17 ` [PATCH v4 02/27] net: datagram: fix some " Mauro Carvalho Chehab
2020-11-16 10:20 ` Kirill Tkhai
2020-11-16 10:17 ` [PATCH v4 03/27] net: core: " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 04/27] s390: fix " Mauro Carvalho Chehab
2020-11-16 10:25 ` Cornelia Huck
2020-11-16 10:38 ` Vineeth Vijayan
2020-11-16 12:04 ` Vineeth Vijayan
2020-11-16 10:18 ` [PATCH v4 05/27] drm: fix some " Mauro Carvalho Chehab
2020-11-16 11:37 ` Jani Nikula
2020-11-16 19:48 ` Daniel Vetter
2020-11-16 10:18 ` [PATCH v4 06/27] HSI: fix a kernel-doc markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 07/27] IB: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:36 ` Gustavo A. R. Silva
2020-11-23 23:45 ` Jason Gunthorpe
2020-12-01 11:39 ` Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 08/27] parport: fix a kernel-doc markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 09/27] rapidio: fix kernel-doc a markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 10/27] video: fix some kernel-doc markups Mauro Carvalho Chehab
2020-11-16 15:36 ` Daniel Vetter
2020-11-16 16:38 ` Mauro Carvalho Chehab
2020-11-16 17:24 ` Daniel Vetter
2020-11-16 18:11 ` Sam Ravnborg
2020-11-16 19:43 ` Daniel Vetter
2020-11-16 18:42 ` Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 11/27] fs: fix " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 12/27] jbd2: " Mauro Carvalho Chehab
2020-11-20 3:38 ` Theodore Y. Ts'o
2020-11-16 10:18 ` [PATCH v4 13/27] pstore/zone: fix a kernel-doc markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 14/27] completion: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 11:36 ` Peter Zijlstra
2020-11-16 10:18 ` [PATCH v4 15/27] firmware: stratix10-svc: " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 16/27] connector: fix a kernel-doc markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 17/27] lib/crc7: " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 18/27] hrtimer: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 19/27] genirq: " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 20/27] list: fix a typo at the kernel-doc markup Mauro Carvalho Chehab
2020-11-16 19:57 ` Paul E. McKenney
2020-11-16 10:18 ` [PATCH v4 21/27] memblock: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 22/27] w1: fix a kernel-doc markup Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 23/27] resource: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 24/27] shed: fix kernel-doc markup Mauro Carvalho Chehab
2020-11-16 12:34 ` Vincent Guittot
2020-11-16 10:18 ` [PATCH v4 25/27] mm: fix kernel-doc markups Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 26/27] selftests: kselftest_harness.h: partially " Mauro Carvalho Chehab
2020-11-16 10:18 ` [PATCH v4 27/27] scripts: kernel-doc: validate kernel-doc markup with the actual names Mauro Carvalho Chehab
2020-11-16 15:06 ` kernel test robot
2020-11-16 15:42 ` kernel test robot [this message]
2020-11-16 15:51 ` kernel test robot
2020-11-17 22:19 ` [PATCH v4 00/27]Fix several bad kernel-doc markups Jakub Kicinski
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=202011162356.2dksRicD-lkp@intel.com \
--to=lkp@intel.com \
--cc=clang-built-linux@googlegroups.com \
--cc=corbet@lwn.net \
--cc=kbuild-all@lists.01.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@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).