| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202301150728.AatPPcx9-lkp@intel.com>
Date: Sun, 15 Jan 2023 07:01:27 +0800
From: kernel test robot <lkp@...el.com>
To: Arnd Bergmann <arnd@...db.de>
Cc: llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
linux-kernel@...r.kernel.org
Subject: arch/arm/mach-pxa/eseries.c:46:13: warning: no previous prototype
for function 'eseries_fixup'
Hi Arnd,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 7c698440524117dca7534592db0e7f465ae4d0bb
commit: 8c1fb11b8a77dc489a8ef6d96c38c1297b629d06 ARM: s3c: enable s3c24xx multiplatform support
date: 9 months ago
config: arm-randconfig-r032-20230115
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 8d9828ef5aa9688500657d36cd2aefbe12bbd162)
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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=8c1fb11b8a77dc489a8ef6d96c38c1297b629d06
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 8c1fb11b8a77dc489a8ef6d96c38c1297b629d06
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
>> arch/arm/mach-pxa/eseries.c:46:13: warning: no previous prototype for function 'eseries_fixup' [-Wmissing-prototypes]
void __init eseries_fixup(struct tag *tags, char **cmdline)
^
arch/arm/mach-pxa/eseries.c:46:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init eseries_fixup(struct tag *tags, char **cmdline)
^
static
>> arch/arm/mach-pxa/eseries.c:75:5: warning: no previous prototype for function 'eseries_tmio_enable' [-Wmissing-prototypes]
int eseries_tmio_enable(struct platform_device *dev)
^
arch/arm/mach-pxa/eseries.c:75:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int eseries_tmio_enable(struct platform_device *dev)
^
static
>> arch/arm/mach-pxa/eseries.c:88:5: warning: no previous prototype for function 'eseries_tmio_disable' [-Wmissing-prototypes]
int eseries_tmio_disable(struct platform_device *dev)
^
arch/arm/mach-pxa/eseries.c:88:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int eseries_tmio_disable(struct platform_device *dev)
^
static
>> arch/arm/mach-pxa/eseries.c:95:5: warning: no previous prototype for function 'eseries_tmio_suspend' [-Wmissing-prototypes]
int eseries_tmio_suspend(struct platform_device *dev)
^
arch/arm/mach-pxa/eseries.c:95:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int eseries_tmio_suspend(struct platform_device *dev)
^
static
>> arch/arm/mach-pxa/eseries.c:101:5: warning: no previous prototype for function 'eseries_tmio_resume' [-Wmissing-prototypes]
int eseries_tmio_resume(struct platform_device *dev)
^
arch/arm/mach-pxa/eseries.c:101:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int eseries_tmio_resume(struct platform_device *dev)
^
static
>> arch/arm/mach-pxa/eseries.c:108:6: warning: no previous prototype for function 'eseries_get_tmio_gpios' [-Wmissing-prototypes]
void eseries_get_tmio_gpios(void)
^
arch/arm/mach-pxa/eseries.c:108:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void eseries_get_tmio_gpios(void)
^
static
6 warnings generated.
--
>> arch/arm/mach-pxa/colibri-pxa320.c:240:13: warning: no previous prototype for function 'colibri_pxa320_init' [-Wmissing-prototypes]
void __init colibri_pxa320_init(void)
^
arch/arm/mach-pxa/colibri-pxa320.c:240:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init colibri_pxa320_init(void)
^
static
1 warning generated.
--
>> arch/arm/mach-pxa/balloon3.c:85:12: warning: no previous prototype for function 'parse_balloon3_features' [-Wmissing-prototypes]
int __init parse_balloon3_features(char *arg)
^
arch/arm/mach-pxa/balloon3.c:85:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int __init parse_balloon3_features(char *arg)
^
static
1 warning generated.
--
>> arch/arm/mach-pxa/corgi_pm.c:137:15: warning: no previous prototype for function 'corgipm_read_devdata' [-Wmissing-prototypes]
unsigned long corgipm_read_devdata(int type)
^
arch/arm/mach-pxa/corgi_pm.c:137:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
unsigned long corgipm_read_devdata(int type)
^
static
1 warning generated.
vim +/eseries_fixup +46 arch/arm/mach-pxa/eseries.c
3abcd199db312a Ian Molton 2007-11-19 44
3abcd199db312a Ian Molton 2007-11-19 45 /* Only e800 has 128MB RAM */
1c2f87c22566cd Laura Abbott 2014-04-13 @46 void __init eseries_fixup(struct tag *tags, char **cmdline)
3abcd199db312a Ian Molton 2007-11-19 47 {
3abcd199db312a Ian Molton 2007-11-19 48 if (machine_is_e800())
1c2f87c22566cd Laura Abbott 2014-04-13 49 memblock_add(0xa0000000, SZ_128M);
3abcd199db312a Ian Molton 2007-11-19 50 else
1c2f87c22566cd Laura Abbott 2014-04-13 51 memblock_add(0xa0000000, SZ_64M);
3abcd199db312a Ian Molton 2007-11-19 52 }
3abcd199db312a Ian Molton 2007-11-19 53
fdabc466f335bc Linus Walleij 2020-01-23 54 static struct gpiod_lookup_table e7xx_gpio_vbus_gpiod_table __maybe_unused = {
fdabc466f335bc Linus Walleij 2020-01-23 55 .dev_id = "gpio-vbus",
fdabc466f335bc Linus Walleij 2020-01-23 56 .table = {
fdabc466f335bc Linus Walleij 2020-01-23 57 GPIO_LOOKUP("gpio-pxa", GPIO_E7XX_USB_DISC,
fdabc466f335bc Linus Walleij 2020-01-23 58 "vbus", GPIO_ACTIVE_HIGH),
fdabc466f335bc Linus Walleij 2020-01-23 59 GPIO_LOOKUP("gpio-pxa", GPIO_E7XX_USB_PULLUP,
fdabc466f335bc Linus Walleij 2020-01-23 60 "pullup", GPIO_ACTIVE_LOW),
fdabc466f335bc Linus Walleij 2020-01-23 61 { },
fdabc466f335bc Linus Walleij 2020-01-23 62 },
ebcce7b13be837 Ian Molton 2008-08-19 63 };
ebcce7b13be837 Ian Molton 2008-08-19 64
0a137a1a38c64d Arnd Bergmann 2016-01-29 65 static struct platform_device e7xx_gpio_vbus __maybe_unused = {
133dce0687b4c3 Dmitry Baryshkov 2011-02-14 66 .name = "gpio-vbus",
133dce0687b4c3 Dmitry Baryshkov 2011-02-14 67 .id = -1,
133dce0687b4c3 Dmitry Baryshkov 2011-02-14 68 };
133dce0687b4c3 Dmitry Baryshkov 2011-02-14 69
36033422639913 Ian Molton 2008-08-24 70 struct pxaficp_platform_data e7xx_ficp_platform_data = {
c4bd01727380ce Marek Vasut 2009-07-17 71 .gpio_pwdown = GPIO_E7XX_IR_OFF,
36033422639913 Ian Molton 2008-08-24 72 .transceiver_cap = IR_SIRMODE | IR_OFF,
36033422639913 Ian Molton 2008-08-24 73 };
36033422639913 Ian Molton 2008-08-24 74
b1ae1b7b274f67 Ian Molton 2008-09-26 @75 int eseries_tmio_enable(struct platform_device *dev)
b1ae1b7b274f67 Ian Molton 2008-09-26 76 {
b1ae1b7b274f67 Ian Molton 2008-09-26 77 /* Reset - bring SUSPEND high before PCLR */
b1ae1b7b274f67 Ian Molton 2008-09-26 78 gpio_set_value(GPIO_ESERIES_TMIO_SUSPEND, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 79 gpio_set_value(GPIO_ESERIES_TMIO_PCLR, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 80 msleep(1);
b1ae1b7b274f67 Ian Molton 2008-09-26 81 gpio_set_value(GPIO_ESERIES_TMIO_SUSPEND, 1);
b1ae1b7b274f67 Ian Molton 2008-09-26 82 msleep(1);
b1ae1b7b274f67 Ian Molton 2008-09-26 83 gpio_set_value(GPIO_ESERIES_TMIO_PCLR, 1);
b1ae1b7b274f67 Ian Molton 2008-09-26 84 msleep(1);
b1ae1b7b274f67 Ian Molton 2008-09-26 85 return 0;
b1ae1b7b274f67 Ian Molton 2008-09-26 86 }
b1ae1b7b274f67 Ian Molton 2008-09-26 87
b1ae1b7b274f67 Ian Molton 2008-09-26 @88 int eseries_tmio_disable(struct platform_device *dev)
b1ae1b7b274f67 Ian Molton 2008-09-26 89 {
b1ae1b7b274f67 Ian Molton 2008-09-26 90 gpio_set_value(GPIO_ESERIES_TMIO_SUSPEND, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 91 gpio_set_value(GPIO_ESERIES_TMIO_PCLR, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 92 return 0;
b1ae1b7b274f67 Ian Molton 2008-09-26 93 }
b1ae1b7b274f67 Ian Molton 2008-09-26 94
b1ae1b7b274f67 Ian Molton 2008-09-26 @95 int eseries_tmio_suspend(struct platform_device *dev)
b1ae1b7b274f67 Ian Molton 2008-09-26 96 {
b1ae1b7b274f67 Ian Molton 2008-09-26 97 gpio_set_value(GPIO_ESERIES_TMIO_SUSPEND, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 98 return 0;
b1ae1b7b274f67 Ian Molton 2008-09-26 99 }
b1ae1b7b274f67 Ian Molton 2008-09-26 100
b1ae1b7b274f67 Ian Molton 2008-09-26 @101 int eseries_tmio_resume(struct platform_device *dev)
b1ae1b7b274f67 Ian Molton 2008-09-26 102 {
b1ae1b7b274f67 Ian Molton 2008-09-26 103 gpio_set_value(GPIO_ESERIES_TMIO_SUSPEND, 1);
b1ae1b7b274f67 Ian Molton 2008-09-26 104 msleep(1);
b1ae1b7b274f67 Ian Molton 2008-09-26 105 return 0;
b1ae1b7b274f67 Ian Molton 2008-09-26 106 }
b1ae1b7b274f67 Ian Molton 2008-09-26 107
b1ae1b7b274f67 Ian Molton 2008-09-26 @108 void eseries_get_tmio_gpios(void)
b1ae1b7b274f67 Ian Molton 2008-09-26 109 {
b1ae1b7b274f67 Ian Molton 2008-09-26 110 gpio_request(GPIO_ESERIES_TMIO_SUSPEND, NULL);
b1ae1b7b274f67 Ian Molton 2008-09-26 111 gpio_request(GPIO_ESERIES_TMIO_PCLR, NULL);
b1ae1b7b274f67 Ian Molton 2008-09-26 112 gpio_direction_output(GPIO_ESERIES_TMIO_SUSPEND, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 113 gpio_direction_output(GPIO_ESERIES_TMIO_PCLR, 0);
b1ae1b7b274f67 Ian Molton 2008-09-26 114 }
b1ae1b7b274f67 Ian Molton 2008-09-26 115
:::::: The code at line 46 was first introduced by commit
:::::: 1c2f87c22566cd057bc8cde10c37ae9da1a1bb76 ARM: 8025/1: Get rid of meminfo
:::::: TO: Laura Abbott <lauraa@...eaurora.org>
:::::: CC: Russell King <rmk+kernel@....linux.org.uk>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests
View attachment "config" of type "text/plain" (147191 bytes)
Powered by blists - more mailing lists