Re: [PATCH] ssb: main.c: This patch removes unnecessary return statement using spatch tool

From: kbuild test robot
Date: Sat Jan 07 2017 - 08:49:56 EST


Hi Rahul,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc2 next-20170106]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Rahul-Krishnan/ssb-main-c-This-patch-removes-unnecessary-return-statement-using-spatch-tool/20170107-205451
config: i386-randconfig-r0-201701 (attached as .config)
compiler: gcc-5 (Debian 5.4.1-2) 5.4.1 20160904
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

drivers/ssb/main.c:1:8: error: expected '=', ',', ';', 'asm' or '__attribute__' before '/' token
patch /home/rahul/git/kernels/staging/drivers//*
^
drivers/ssb/main.c:6:43: error: stray '@' in program
* Copyright 2006, 2007, Michael Buesch <m@xxxxxxx>
^
In file included from include/linux/ioport.h:12:0,
from include/linux/device.h:16,
from include/linux/ssb/ssb.h:4,
from drivers/ssb/ssb_private.h:4,
from drivers/ssb/main.c:11:
>> include/linux/compiler.h:110:34: warning: 'struct ftrace_branch_data' declared inside parameter list
void ftrace_likely_update(struct ftrace_branch_data *f, int val, int expect);
^
>> include/linux/compiler.h:110:34: warning: its scope is only this definition or declaration, which is probably not what you want
include/linux/err.h: In function 'IS_ERR':
include/linux/compiler.h:117:18: error: variable '______f' has initializer but incomplete type
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/compiler.h:117:18: error: unknown field 'func' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/compiler.h:121:13: warning: excess elements in struct initializer
.func = __func__, \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/compiler.h:121:13: note: (near initialization for '______f')
.func = __func__, \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/compiler.h:117:18: error: unknown field 'file' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/err.h:35:40: warning: excess elements in struct initializer
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/err.h:35:40: note: (near initialization for '______f')
include/linux/compiler.h:117:18: error: unknown field 'line' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/err.h:35:40: warning: excess elements in struct initializer
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/err.h:35:40: note: (near initialization for '______f')
include/linux/compiler.h:120:5: error: storage size of '______f' isn't known
______f = { \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
#define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO)
^
include/linux/err.h:35:9: note: in expansion of macro 'IS_ERR_VALUE'
return IS_ERR_VALUE((unsigned long)ptr);
^
include/linux/compiler.h:120:5: warning: unused variable '______f' [-Wunused-variable]
______f = { \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
include/linux/err.h:21:25: note: in expansion of macro 'unlikely'
--
^
include/linux/compiler.h:121:13: note: (near initialization for '______f')
.func = __func__, \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
arch/x86/include/asm/preempt.h:98:9: note: in expansion of macro 'unlikely'
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
include/linux/compiler.h:117:18: error: unknown field 'file' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
arch/x86/include/asm/preempt.h:98:9: note: in expansion of macro 'unlikely'
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
arch/x86/include/asm/preempt.h:98:67: warning: excess elements in struct initializer
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
arch/x86/include/asm/preempt.h:98:67: note: (near initialization for '______f')
include/linux/compiler.h:117:18: error: unknown field 'line' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
arch/x86/include/asm/preempt.h:98:9: note: in expansion of macro 'unlikely'
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
arch/x86/include/asm/preempt.h:98:67: warning: excess elements in struct initializer
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
arch/x86/include/asm/preempt.h:98:67: note: (near initialization for '______f')
include/linux/compiler.h:120:5: error: storage size of '______f' isn't known
______f = { \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
arch/x86/include/asm/preempt.h:98:9: note: in expansion of macro 'unlikely'
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
include/linux/compiler.h:120:5: warning: unused variable '______f' [-Wunused-variable]
______f = { \
^
include/linux/compiler.h:139:58: note: in expansion of macro '__branch_check__'
# define unlikely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
^
arch/x86/include/asm/preempt.h:98:9: note: in expansion of macro 'unlikely'
return unlikely(raw_cpu_read_4(__preempt_count) == preempt_offset);
^
include/asm-generic/qspinlock.h: In function 'queued_spin_lock':
include/linux/compiler.h:117:18: error: variable '______f' has initializer but incomplete type
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/linux/compiler.h:117:18: error: unknown field 'func' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/linux/compiler.h:121:13: warning: excess elements in struct initializer
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/linux/compiler.h:121:13: note: (near initialization for '______f')
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/linux/compiler.h:117:18: error: unknown field 'file' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
>> include/asm-generic/qspinlock.h:101:21: warning: excess elements in struct initializer
if (likely(val == 0))
^
include/asm-generic/qspinlock.h:101:21: note: (near initialization for '______f')
include/linux/compiler.h:117:18: error: unknown field 'line' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
>> include/asm-generic/qspinlock.h:101:21: warning: excess elements in struct initializer
if (likely(val == 0))
^
include/asm-generic/qspinlock.h:101:21: note: (near initialization for '______f')
include/linux/compiler.h:120:5: error: storage size of '______f' isn't known
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/linux/compiler.h:120:5: warning: unused variable '______f' [-Wunused-variable]
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qspinlock.h:101:6: note: in expansion of macro 'likely'
if (likely(val == 0))
^
include/asm-generic/qrwlock.h: In function 'queued_read_trylock':
include/linux/compiler.h:117:18: error: variable '______f' has initializer but incomplete type
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:117:18: error: unknown field 'func' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:121:13: warning: excess elements in struct initializer
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:121:13: note: (near initialization for '______f')
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:117:18: error: unknown field 'file' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
>> include/asm-generic/qrwlock.h:83:32: warning: excess elements in struct initializer
if (likely(!(cnts & _QW_WMASK))) {
^
include/asm-generic/qrwlock.h:83:32: note: (near initialization for '______f')
include/linux/compiler.h:117:18: error: unknown field 'line' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
>> include/asm-generic/qrwlock.h:83:32: warning: excess elements in struct initializer
if (likely(!(cnts & _QW_WMASK))) {
^
include/asm-generic/qrwlock.h:83:32: note: (near initialization for '______f')
include/linux/compiler.h:120:5: error: storage size of '______f' isn't known
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:120:5: warning: unused variable '______f' [-Wunused-variable]
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:83:6: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK))) {
^
include/linux/compiler.h:117:18: error: variable '______f' has initializer but incomplete type
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/linux/compiler.h:117:18: error: unknown field 'func' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/linux/compiler.h:121:13: warning: excess elements in struct initializer
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/linux/compiler.h:121:13: note: (near initialization for '______f')
.func = __func__, \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/linux/compiler.h:117:18: error: unknown field 'file' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/asm-generic/qrwlock.h:85:33: warning: excess elements in struct initializer
if (likely(!(cnts & _QW_WMASK)))
^
include/asm-generic/qrwlock.h:85:33: note: (near initialization for '______f')
include/linux/compiler.h:117:18: error: unknown field 'line' specified in initializer
static struct ftrace_branch_data \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/asm-generic/qrwlock.h:85:33: warning: excess elements in struct initializer
if (likely(!(cnts & _QW_WMASK)))
^
include/asm-generic/qrwlock.h:85:33: note: (near initialization for '______f')
include/linux/compiler.h:120:5: error: storage size of '______f' isn't known
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
^
include/linux/compiler.h:120:5: warning: unused variable '______f' [-Wunused-variable]
______f = { \
^
include/linux/compiler.h:136:56: note: in expansion of macro '__branch_check__'
# define likely(x) (__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
^
include/asm-generic/qrwlock.h:85:7: note: in expansion of macro 'likely'
if (likely(!(cnts & _QW_WMASK)))
..

vim +212 drivers/ssb/main.c

3ba6018a Michael Buesch 2009-11-23 196 *
3ba6018a Michael Buesch 2009-11-23 197 * After freezing no device driver will be handling a device
3ba6018a Michael Buesch 2009-11-23 198 * on this bus anymore. ssb_devices_thaw() must be called after
3ba6018a Michael Buesch 2009-11-23 199 * a successful freeze to reactivate the devices.
3ba6018a Michael Buesch 2009-11-23 200 *
3ba6018a Michael Buesch 2009-11-23 201 * @bus: The bus.
3ba6018a Michael Buesch 2009-11-23 202 * @ctx: Context structure. Pass this to ssb_devices_thaw().
3ba6018a Michael Buesch 2009-11-23 203 */
3ba6018a Michael Buesch 2009-11-23 204 int ssb_devices_freeze(struct ssb_bus *bus, struct ssb_freeze_context *ctx)
61e115a5 Michael Buesch 2007-09-18 205 {
3ba6018a Michael Buesch 2009-11-23 206 struct ssb_device *sdev;
3ba6018a Michael Buesch 2009-11-23 207 struct ssb_driver *sdrv;
3ba6018a Michael Buesch 2009-11-23 208 unsigned int i;
3ba6018a Michael Buesch 2009-11-23 209
3ba6018a Michael Buesch 2009-11-23 210 memset(ctx, 0, sizeof(*ctx));
3ba6018a Michael Buesch 2009-11-23 211 ctx->bus = bus;
3ba6018a Michael Buesch 2009-11-23 @212 SSB_WARN_ON(bus->nr_devices > ARRAY_SIZE(ctx->device_frozen));
61e115a5 Michael Buesch 2007-09-18 213
61e115a5 Michael Buesch 2007-09-18 214 for (i = 0; i < bus->nr_devices; i++) {
3ba6018a Michael Buesch 2009-11-23 215 sdev = ssb_device_get(&bus->devices[i]);
3ba6018a Michael Buesch 2009-11-23 216
3ba6018a Michael Buesch 2009-11-23 217 if (!sdev->dev || !sdev->dev->driver ||
3ba6018a Michael Buesch 2009-11-23 218 !device_is_registered(sdev->dev)) {
3ba6018a Michael Buesch 2009-11-23 219 ssb_device_put(sdev);
61e115a5 Michael Buesch 2007-09-18 220 continue;

:::::: The code at line 212 was first introduced by commit
:::::: 3ba6018aa314559c5867138a8173b068268a70db ssb: Fix SPROM writing

:::::: TO: Michael Buesch <mb@xxxxxxxxx>
:::::: CC: John W. Linville <linville@xxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip