linux-kernel - Re: [PATCH 3/4] usb: host: ehci.h: remove duplicated return

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 9 Dec 2015 18:47:13 -0300
From:	"Geyslan G. Bem" <geyslan@...il.com>
To:	kbuild test robot <lkp@...el.com>
Cc:	kbuild-all@...org, Peter Senna Tschudin <peter.senna@...il.com>,
	Alan Stern <stern@...land.harvard.edu>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	linux-usb@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 3/4] usb: host: ehci.h: remove duplicated return

2015-12-09 18:24 GMT-03:00 kbuild test robot <lkp@...el.com>:
> Hi Geyslan,
>
> [auto build test ERROR on usb/usb-testing]
> [also build test ERROR on v4.4-rc4 next-20151209]
>
> url:    https://github.com/0day-ci/linux/commits/Geyslan-G-Bem/usb-host-ehci-h-cleanup-header-file/20151210-040115
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
> config: x86_64-randconfig-s0-12100345 (attached as .config)
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=x86_64
>
> All errors (new ones prefixed by >>):
>
>    In file included from drivers/usb/chipidea/host.c:29:0:
>    drivers/usb/chipidea/../host/ehci.h: In function 'ehci_port_speed':
>>> drivers/usb/chipidea/../host/ehci.h:668:3: error: label at end of compound statement
>       default:
Missed the break. Coul be removed too.

>       ^
>
> vim +668 drivers/usb/chipidea/../host/ehci.h
>
> ^1da177e Linus Torvalds 2005-04-16  662                 case 0:
> ^1da177e Linus Torvalds 2005-04-16  663                         return 0;
> ^1da177e Linus Torvalds 2005-04-16  664                 case 1:
> 288ead45 Alan Stern     2010-03-04  665                         return USB_PORT_STAT_LOW_SPEED;
> ^1da177e Linus Torvalds 2005-04-16  666                 case 2:
> 50691528 Geyslan G. Bem 2015-12-09  667                 /* fall through to default function return */
> ^1da177e Linus Torvalds 2005-04-16 @668                 default:
> ^1da177e Linus Torvalds 2005-04-16  669                 }
> ^1da177e Linus Torvalds 2005-04-16  670         }
> 288ead45 Alan Stern     2010-03-04  671         return USB_PORT_STAT_HIGH_SPEED;
>
> :::::: The code at line 668 was first introduced by commit
> :::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
>
> :::::: TO: Linus Torvalds <torvalds@...970.osdl.org>
> :::::: CC: Linus Torvalds <torvalds@...970.osdl.org>
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

> No, please leave it as-is, it helps to understand what is going on here
> easier.

Ok. Let it go. :)

> thanks,

> greg k-h



-- 
Regards,

Geyslan G. Bem
hackingbits.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/