From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752596AbdGJGdQ (ORCPT ); Mon, 10 Jul 2017 02:33:16 -0400 Received: from mail-co1nam03on0063.outbound.protection.outlook.com ([104.47.40.63]:11920 "EHLO NAM03-CO1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751163AbdGJGdN (ORCPT ); Mon, 10 Jul 2017 02:33:13 -0400 Authentication-Results: spf=pass (sender IP is 149.199.60.100) smtp.mailfrom=xilinx.com; embeddedor.com; dkim=none (message not signed) header.d=none;embeddedor.com; dmarc=bestguesspass action=none header.from=xilinx.com; Subject: Re: [PATCH] usb: gadget: udc-xilinx: compress return logic into one line To: "Gustavo A. R. Silva" , Felipe Balbi , Greg Kroah-Hartman , Michal Simek , =?UTF-8?Q?S=c3=b6ren_Brinkmann?= CC: , , References: <20170710030458.GA14639@embeddedgus> From: Michal Simek Message-ID: <73e25e50-c65a-bd43-d51c-1291a9168cd2@xilinx.com> Date: Mon, 10 Jul 2017 08:32:55 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170710030458.GA14639@embeddedgus> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-RCIS-Action: ALLOW X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.1.0.1062-23184.005 X-TM-AS-User-Approved-Sender: Yes;Yes X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:149.199.60.100;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(39450400003)(39400400002)(39840400002)(39860400002)(39850400002)(39410400002)(2980300002)(438002)(24454002)(199003)(189002)(65826007)(6636002)(2950100002)(4326008)(54356999)(38730400002)(305945005)(50986999)(31696002)(76176999)(86362001)(50466002)(8936002)(54906002)(77096006)(478600001)(36756003)(2906002)(9786002)(5660300001)(8676002)(356003)(81166006)(230700001)(53546010)(47776003)(23676002)(106466001)(63266004)(229853002)(189998001)(31686004)(4001350100001)(83506001)(626005)(33646002)(36386004)(107986001);DIR:OUT;SFP:1101;SCL:1;SRVR:BY2PR0201MB1591;H:xsj-pvapsmtpgw02;FPR:;SPF:Pass;MLV:sfv;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2NAM02FT018;1:Rf/20bipoLpv+BVjl189zejJisrBV4HWQABSmo9PTIstOxn3tB7SqxLV7Vkq+xGT7iOUo0cruEMJvJgaG0MeIEHlF09M9pdhz9x8xAKeX2aCgSRDUKxvNIpDR5UYLcEJYg4bkto4JE1nOaPcrB+TxR7jb6xyEwFgHoWjtbaBtEY3uVonaJyts8miYLWND6bg9ZJZWQ/60iAkrbnZgfRn4LmbYi9p1nMGUuYVqAyRcVdPzb6OUfPgm/8XNCcpUMV1Lg509wZi4R1CtGtK/y3eBKeTa/w6mrORXFrAtSOh9n39wJTl8r0j05QJd0cVATUOrr1R9vSpm+phrmfGaedkufCpo/l/7DMqvYZs4EIJC4OAN9cd7LvLRU9ZnlVTRD6PEBO/SQe28i9HxmxVk0NQNBV1zMBdGHaHJWaF5piena9bj6VYKDnI6bx6EfBQBtwC6UFeswSXw/blBWn62IEXllUoPROEqpThi5unM42d0JbudGEo67wBrMg6ch5+PFxYzi5BOZuuxiddqMXQ5vuA7bpN7wLxGrPszbPZGPTtpO1KK4JMxII1GgdtGUbA8IaJP513dM2GqVe5FKxTM+lV2CZTEQysXrcWSO0aAe1n3WvxkflQ7YZXoblBl8+zBXOLAE18HseTUiEceAVWgMNUAPbOHH9L9/Ar5MOL3N3MDZ5csS8PSifp4HnxIGn/zSVC+B8aORoPxqs6x3VIutzhv5WFSGfB8aRC773yFncBHPdk3JhjkVW+TjZw+qUG3VmSUEugu1zVCWGWM0InA/u2BQAU7g/EdW0YG4DNmy1T/wVlGg8Pspum0aXgjhwS4eun6unQaKeBgWSwBKDIiOgKNoAlmseNTJiPDzwoPBus4TM= X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 28c70c2a-9e4c-41e5-bdee-08d4c75d8826 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(8251501002)(2017030254075)(300000503095)(300135400095)(2017052603031)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);SRVR:BY2PR0201MB1591; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCWTJQUjAyMDFNQjE1OTE7MzpzSVFJY2w0dnhPSVp5T0JIS2wybVVYcWcz?= =?utf-8?B?UDV3cGhLQmdSNjBraEtwWWFFM284UmI2eUJocFdWdVVzc3V2RWJkQU5EQ24w?= =?utf-8?B?Q2hocTIzZTNZdDFIZVBhUG05MWFzQThLYVJYR3plSHYxWHZnbW1UNTZ2c1hE?= =?utf-8?B?cHpvNHFJWmk1VWpIL2Y3bndtbnJHaUtkVHVnQ0RpWk1IM1FKa0o0d1dwQVg5?= =?utf-8?B?eUs0NHoyZzNlb2MzdXpjcy9BdzM4T212dHpBeElwNEJScFZJMXUydDVOYTBF?= =?utf-8?B?b0ZXcWQydnp4ZTRmMFZJZzh0RVJXbkNqT0hwRHh2eS9lUVRqYjV5SnROZzhW?= =?utf-8?B?dzk3L3VlUHh5R1pBUTdORGJVZUVUMlpzbXl2c05rMjArWHNyK1p5YUYxRXYw?= =?utf-8?B?djh4UzZxQWFxbTR0ZENzdlJnek5QYVF3bE5ORTk1eEJ2a3c1elBTcHBrS1dZ?= =?utf-8?B?dnBlUGdmeU0wYzdickZpd1N0eG5pU3VFSUl0VCtJMm45b1RyellZQUJwWEFM?= =?utf-8?B?RnNKSFpPME5QTHFCQTVDMGorajE5RFUvLzVXSWhtamEvS3RHbWgwbTg0NHh6?= =?utf-8?B?elo4ZWkzTWQza203dzJjV21qQ3JjVlQyazlLM2FxQnJaRjRrY2VHOFBRcnJj?= =?utf-8?B?N00wV1VzL3Y4SFBWNEM2endEYzNuRWNmcFJEL3FmblBlVHIrYTlHT0NKMVNn?= =?utf-8?B?ZlNaejJDanpMS0h2ejc2TFl6cFhld05jeWtvbm5SZzh0RWhEWEsyd3FlUTh4?= =?utf-8?B?cTk1RE1uNTN4b3pQNy84U3UyY1NvRjhtYzVBRDdYc3hkYzZaU1lFekJlditL?= =?utf-8?B?Tnh4ZWUwL1ZuZHl2VE8vMXhZMXgvT3VOK1FkaE5YL3YyUTVDUVdqdXhXeDI3?= =?utf-8?B?KzJuS2k2dG1YcTBsTmR2ZzVYdmRGOU4xbndtVUNrYnEwWUsreG1DbW0vU0V2?= =?utf-8?B?b1hEaTNDbi9pS1NoM1Nsa0dIbnYxUjdIa2hRK0ZmZFZ0bHkwdVAva2pld2xG?= =?utf-8?B?R0xCdWFzdS82UnVFSy9CWjZiaVpwTzRMU1BoWTluQm1xTXBzdSs2QkQ0VDZ3?= =?utf-8?B?c1pDa1FIMW80ei96cDJvZy83RitjU0JYK2FhSXN1eXBoeHVMbXJSZmNPTWI2?= =?utf-8?B?Yk54VDh0TGZXbHVqTUxHT3VCSzFqZUYzdXZFbmowN3ZYVURJcXVmOGlaSFg2?= =?utf-8?B?TmdiNEw4Zk9hRWtMaWNLMUxMVnBzaThSVjlaOFh6MmdhNDJ3L01ROTYyM2h1?= =?utf-8?B?a2xaN0JFdERiREMraVNyVUtiQVFwVVlSQWFqRFZ6NEsrWGo1cTQ5c1BuOG43?= =?utf-8?B?cmhrWGtSU2t5Zld3PT0=?= X-MS-TrafficTypeDiagnostic: BY2PR0201MB1591: X-Microsoft-Exchange-Diagnostics: 1;BY2PR0201MB1591;25:ae5YNco5QNPzySGzdtDzbgbjrDhDQ7/U1TzqNXte8CBcUalRC4lZ79UGDaozx8ks02Mqc+4KxrZ6c8svPusnYYOvF/vgDDD4hYZDgqwdPAG6Y+/jPBudcsJqQJNznEeYoK+PibM/goaQojmCXLo9PSslYUzTM0aNFc70gS499wNl5d3pM0yjfHP2fQh01jS5vm4ww2zjADwA4k4b1+76GhDXWyCzFceFp/j+PG19Lu+5CFgAah1YMDZgR3ozYaK4rbuIDv4II0mpikEgHJ46pdDGpFRwQ1kxSx9LO6q1k2xxcRogrVWpBFRKE+0rxKKkZXTT5gcZG1gJN+Tq9Cb67UcXbHdmrEebU3obfBsaTnUPmMVtHMmCiYNPzGM/jZuNNNozN2N3dNmBMxPHj2POI1IjX28gX14DVJDSYwQf7Y8/OMDkoJjqteU/d94RoXxCDRMfhXKu8/SAaGuemnBIw8EySbieB17LG4DE09TchBgUivVnJD9+vOM4AHJk4rPEYtuJfj8sw7TJ865PsuCQwRpWtaIAM+Kjf0EIvLboepHmUZCO/bmYjYlUcH5DVZlQUFVwyfS2lya3Jg4mvn5t7TouFw/8590y3uTEDpxubDXI12NmZC+JxDiyoNUEjznTjQQdRGo0GOWeeD/be8ZRY2V9lPUFCrKtUetBMSefZMPV4C8OVlEYChGaX4nQi/mU8tFVq6zTsMahBN9njtDbwjrshdllWhGl3KxK8OkTYsjN/G54udoz0ISP5i0YWyLlJivzRqPIzKkG3tJitBemaY3RQYlXgxCWZnYmK1SJ01CdHS4jx5mkjpmXA+gls1FqLgl4vH3AF3rCq6d+1OAZOZFgCCF6YdoTqLHj/58E9Tpr6N9MraJ28hyn47xFPNdG3ymVZT+R9J3/SJ6764bRfhmilF24nuBh4nuWyjdw6Ss= X-Microsoft-Exchange-Diagnostics: 1;BY2PR0201MB1591;31:XGjmbiPGtYqhF2FA9ttsEZVi6peV1VN4QlWXkqxcFLixAWVDJe0NnoqJfUHeKezq27nqwPvb/I2cx+tzKPlrqVlD8k34a1bY1xrvC9RQQvnnH1epgcqiA04IXmtcv81NfcwNsGVnYM+chrWVhbQHEN/syw/uarLe1sV+FN9NHKBL4xhzoZx0nUePj8VCKHLWnyvHyFrXQ4S5VUhk9uUIhBYZrWAiR2axWgyag2O8lB5PfeCsnmAgxBLSu4kNOB1fbZznFOP9DhtSAHbWqn/j5Sbwldx4P5uFmbXjANEhCLMCbAVJHzey0xxl6b5wDimNTjDSbn6KqufU3jeOduwx5IO0sGZtWfSxLTrzVGZBGpu7QX/3udkTC0c0xOGtl8ayv/s6k53Io5aPBnbjVwwytEFOg8gsrs8m+3dTTfrBbbd742UHGkrZlbYoAj2Ln0Tam63s4OIpCRidH3iO6tYJnggqnp6hXNEyuNhIODemXslJ7mCTGzb8+A82iatyvTBoByx42pEBgPcNGWMhOAMMuSaWch9VyhIzhfwaacdoi7yUXosj9P+dB0hSXZsDtp3KNfrBTzdQEH9J0xmussfFvqv+ttDTawQPgAkMi8VofIkg5dL190Bvve1fmr4hLPVOF1iSCOLPxUBOsIYdbvAE1GoxrjAJss6erxsWDwJJD6UUz9n2wyJxuSMjlbUshnJ9h6BJHFq0LPl78dXw3ewHjw== X-Microsoft-Exchange-Diagnostics: 1;BY2PR0201MB1591;20:PPxMmhqDAPmhpOvPVxWghUhnXBfm+YxYTwdZ65nyMmnqth2kAQ4KSuDS9jwDzsjaGGyy11nlTThqeS+xCLlKgpK163NrAj95B9g3luAtt/PPpvBm0rC0uT0i1GA89xbQCpo1e5vY96l+9avzVTM+2t1ryr0EVAgVtxRhoUOhN3fEz63J2LbpSYyi34v06lcigyQUSaGYpu07uE+7H7/Gz7whTgfGkeGtZsTRIjoeQO9vdnIEBvK1zz/1pvmRnGkBRvkVpPSQyOZEF4Z80i+Fe8S7YHCRhGITojb9kup0KsIoO/wT4z2fS49thYXB5NYk1pQCBNsFrO6z6pUEolEfwqZx0BkmtrZvL60oPPUvlyub516J1vtmSKYhIQTiMMT5g5L6F3ZTHr6i6R5XdzTH2YIi2McgQN1+oYhMWD7sBKtGd0ieg/OWotKHaRzOcH8pFdTwVJLNYOpDd2iBYx6gZvzFAzLpzKFWsJghyunodfBjF/loX79BtAevqbzToo/k X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(236129657087228)(192813158149592)(247924648384137); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(2017060910075)(13018025)(5005006)(13016025)(8121501046)(3002001)(10201501046)(100000703101)(100105400095)(93006095)(93004095)(6055026)(6041248)(20161123555025)(20161123558100)(20161123564025)(20161123560025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:BY2PR0201MB1591;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:BY2PR0201MB1591; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCWTJQUjAyMDFNQjE1OTE7NDptcXJqV29zbzd0OGRGSVRSYlpDdG1DSTd4?= =?utf-8?B?K1YreUhkRTZkczkya2RhM3pEeVdTNWdmOC81eWoxT2huU1lFRUNrZlR5THNF?= =?utf-8?B?UzJwRm5sV0M4MXcwUnRXc1pnMGdWUDJjNTFNamd0aW03alAyWXVVeTdRam5Y?= =?utf-8?B?UjBPaUl3U3dBZHVzV1E5dzBDK3Y4YTNKWDh1RTBEY2FEelUxOGFLSWRJS3Nj?= =?utf-8?B?RG4vSXlFOE56ZHpNWm1iRFNkR1NzR1hqYUFkRzJnVlVBUGVGOFRVUWpFMG03?= =?utf-8?B?dkRvYmdPaDBMSFZPTDdpNndWNTFEWUYxTDk2bzF4QmorWHIwRnJxL0hRckJX?= =?utf-8?B?RnlWNDcyTVZpVi9Ja2VWM01XL1UzbjVYTHBwYXVkNk5WMDJzZUdaOUFOcit4?= =?utf-8?B?SGhOdmZmR2ZHTVNVNHhRWEQ2NzMvQnppdFR6N0t0aW5CVmtYM3V5N2FnbVZJ?= =?utf-8?B?T2NkcVVINHpQTDUvaEhObE1UcVFuZS8ycm9WR0lKRkxmOUtINnBOQWszQjZh?= =?utf-8?B?b09TcVZYdWJvWnZjbWVsbGdCdjJIa01oRXVwUEtOZVRZdy9WV2EvYWZxQml5?= =?utf-8?B?ZlRlY3NLbUJ0bXdPa2xMN0xMYWtoUUVrVkxBbE9Ka1hIRjh0R0FabFpkcTU2?= =?utf-8?B?bTAzWWVqSzY2cEs1Y2pJbnRpZ2JtekREQlIzeHFDcFRrRG1vVWNqcjFkWlpJ?= =?utf-8?B?VXNjL0ZuOWNRSVBoSFViRW1lOW1YaDhaNGxGUElpTDBOSWovT0NmcEQ0VUxG?= =?utf-8?B?WkVRZEp2cUF3VUJ1SGkwSUh0ZXliZnVFOHNHSGR2SHNGdmYwaitOL2sreHJt?= =?utf-8?B?b3ZBeENmTnQ0QmVKUWYwMi9XK3BKTlVhRzlHbXY2cmh5SkV0R015QmlvNmFZ?= =?utf-8?B?aGtKWHZZbThBQitCTEpXNEtlM2pTUndHYWVMc2F6MllrL2p6Y2dxVHVkaEtI?= =?utf-8?B?YjVtZE9NVlhvbHllRjZ0K3NyVm54ZStOVnlXSzFVZDB2SHhTVlZVVGdtQmlR?= =?utf-8?B?N3dicWhkSFprbDBzS0FkZEl3dk5jUXVydVpSRzVKSnFIdVE3YWhZazFiU1R0?= =?utf-8?B?ZjJCN3psdlZENWZrMnVYeWFiaCtqZUtaOEM5S2ExeCtCSUJTVmdRYnc1cmRG?= =?utf-8?B?OTc4bFlGNjZnbGFCUllia05kWmZwQTJJOCtHQzlvdXBrdVJoZE81UnJTdjQr?= =?utf-8?B?UVF3V0ZKNGJZR1h0TGJrTllGd1BNNmFkeEVFZnVLT1RuU1M4RGFYQzBGcUhp?= =?utf-8?B?a0ZXWlNJVkdRek1iMVpQT3Y3cHVrVmtqMTVPTXpvVnAzWXdld2JqREZxTVI3?= =?utf-8?B?cFNXa2JsajNuSWNWWHU2U2t0ZWpGTmoxWGJac1hIcDhLUU5uMDc1d0JFK05n?= =?utf-8?B?NEVEQzhsVUYySy9LRXFmUTJWVlliTGFrakdGT09CVFlNajRFQmRBRzM1OU40?= =?utf-8?B?b1ZybnVkNEJocjIwRFdhL0dWSkVNdnVXMUxHeXdmN2t4c3ZacWhqcTRocnhZ?= =?utf-8?B?S2NFLzkvMFVvUE91em0vcXE0NGM5Q2kxL2hFRGZrbWYzMExQUmtKWG1GaXFK?= =?utf-8?B?N1ZyTEJHeUFwQnNyam9hVDJnWFY3ZkJqajhWeEVHTjJqRDJtOUVNeHdweGFa?= =?utf-8?B?cnpJZmhIc2h3eUdlMHdBUmVmQU5Bb3NxTm9HM2ozeksxVGFtL2JwRE0yTC9N?= =?utf-8?B?ZittVkJlNXl0ZTlIUEl0d0JNeXYrZ1pXV3p6RGVWZUErTTN5SmEwa0lYUmM1?= =?utf-8?B?SzVBRXoxcjAyWVFNQjR1NFpUT24vYk5BSjZ0YmF6SkRGN2x3RlpHeVFRRjV6?= =?utf-8?B?WC91YStlMTZ1TkhmUTYvY2hHdE5ISE9tQTJGbkFZOWxwSzFRPT0=?= X-Forefront-PRVS: 03648EFF89 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCWTJQUjAyMDFNQjE1OTE7MjM6Snk3RStiTlB1cC9DR1NLNTNkQWo1ZWwv?= =?utf-8?B?dnJSbys5R1I4NXFqTGxHS3BRMmJEV09udXlBSndaWjE1YzNKcGVsenRZdE5r?= =?utf-8?B?SUpVMjNNNGNHdU9CRWRMR3YvQUtDTjhna0VwdU9jZ0JEL1V0dTFaTHlFY3Rs?= =?utf-8?B?R291TjNpTXhQUXBhREZRdExyTDE2K2dnOU5sSUZzYVpZNzIwZ3puM28yWThV?= =?utf-8?B?bTZJbGhFU2VtRHFWQmFPc2w5cHZaM1NzZVF4MDdLNVptd01KNU9kMHc2YVQr?= =?utf-8?B?NzJ4K25KS1J4NVhENk85Q2V2MHlhVjRXMHFUYTdMVWVmZWh5aWFSYUZ2aHV1?= =?utf-8?B?UnJqZ1QrMVFvd2pobGtNb0dlVkUwYkZ6Nk04Ym9CY0k5aXhseEw4WUMwOTJZ?= =?utf-8?B?N08xQkZ2NEFCT3dQVFVpYkNzSDdTS3k2Nzh1UDdoMUtXNlBwYnA5UjRXMmJV?= =?utf-8?B?ckNhcVVPN1IzcFkvSkk0WWl5a0ZhYndLTU5uS2svdm9pdCtNWUNPTXk4ejBL?= =?utf-8?B?M1c0QVgyMDF0ZnRYbHh0SXFKZC9ZQStXRGovdjdhNE5qaXdaSDFweGtQSkxJ?= =?utf-8?B?Uzg1ZDhESmt6dElJa3l4RDBzWHRvVUNrdGJFTnUxWUlXTm01dkJoZ0lnelUw?= =?utf-8?B?TFgrMGRZRWpSTGYwRU5ISmFGT1YwR25HZzUyL3N1ZWZVaEU0aHhncTkxYnBZ?= =?utf-8?B?QXBVQWpaTDNvODh2K2I1cUI5N0FqWjQreVN1T0t2aWV5cHJHOTg3RUtNSWM5?= =?utf-8?B?c1FqOXQwd05oTHk3QnI5d3JmclVBSm10cldWV3FPZVhpTERtbkxzUlRhVVpo?= =?utf-8?B?bjZMcHNmclpqcXZTS1NhL2Fod0FMTCt6TmJhcWxVRkJjRXE5aXBtZHQxanBq?= =?utf-8?B?bTZoODVYQm9FemhrUnJpNExWdm9TSzVzTGhOaWpHd2xGR1A2bVhNVnB2dk4w?= =?utf-8?B?aTFtSDcyRGlCM1ZnRmNUK2FaRjgxMldOdmtwRy9vVjViSitFblNnTGM0Qnph?= =?utf-8?B?dm5EczFYOEszdmdVVStuTnczM1RnakdVUUE4V2paWldmaG1RNGJyYkw5MlFK?= =?utf-8?B?dGFaQ1FNK1FOZmY0a2wxNzU1TDdRdFk1ZHFOSlpoVURJVnV1aVBlWmRFTjJi?= =?utf-8?B?cXFxRzVXR05lN0NDZUNKNDFpN2t3VmFZOERVSUk4WmF0L0pyTDRPSzJoYWtL?= =?utf-8?B?MlF0d2cwajhSQmpic1RHMkFNNmVUT2QwV3Z3ZTNpQVdXZFV1ckRnSGtMa3Zj?= =?utf-8?B?ZkNBRE5SV1QxVmpYOXlSMWFpd2tRQWVubm9kYnJ0QjB0U0kva3YwSXc4Y3hz?= =?utf-8?B?aWNDRkxNdVRRclAwSURsZEVUTW5wUnZvMFRaeGlGMUErNVNzNkpuc0hqanNP?= =?utf-8?B?ZnVGem9WR1ZYTkxiMEt0S215d1JRWU1oTGh4cUxUOExUc3QwTDV4YW9VSGpx?= =?utf-8?B?cmMybDkyOGZ3aXYxVWZiQXlDYlUrcVAyZzVFZFdwdndldmhWM0E3bEN4Sm9R?= =?utf-8?B?QVJHVFdhS3RUWjRyOGt2Mmk4cmtqeCtrTU0vVnNNamd5MEVwY2V5Z3hpQkZi?= =?utf-8?B?b1RDRWlNMk5kTEtPZW1nTDV5Y2dzNm5sVlBqQWtzSmJhOFFBb2NGRXhVcWdH?= =?utf-8?B?YTB2Rk92R0YzYU82RmJpTDFIeGxNR2NENEU5L3lCR1F1dU5sRTRqalVxS3c9?= =?utf-8?Q?=3D?= X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCWTJQUjAyMDFNQjE1OTE7NjpUMExyVUxJQkNNMXBqbzRIRU1xV3pHYWFB?= =?utf-8?B?T1l3d3k3N2FvQW1pNkMyRnE2cXkyMXZOeEUzbEpUc1RnU2JwcG5DZHNDYnZy?= =?utf-8?B?a2w2OGErL1lxejFZTmpGMUlsRGh5S004eUJCcklwc1BPbW5JL2J1Skl4eG1U?= =?utf-8?B?aUcvV2RJZDZpRko4ejJLYmlFWGdNZ1ArUDBSdnRBWjkvQ2o2TndnUHJUVDd3?= =?utf-8?B?YU9kcVpuNXU0Ym92OTFMdW05Vm5UNWVUWWt4TzVtQWUvdHVxMGxtRXo4SWhl?= =?utf-8?B?SDlRano1K0FGaC9vQVFYcndYd1laNzBnRUwrMjlGV3RyQkNscEJxT1F0UFVr?= =?utf-8?B?T24rRnRVUjBwQWtHZWp2RFc4K045VmpEaFpENUJXdjk3SHZubkxETDYzMkli?= =?utf-8?B?bU5oSFFpeDR6QS9OV2Y1Qm1GTU4vOVdxanFEWkZUeVhlU0wvOXZYZXluQUhM?= =?utf-8?B?YkJJTWVra0RHSkd5Ri80aEdBR1c4STBxKzREbGhxcUE4ckVuS1Fpc3dkcnFB?= =?utf-8?B?M1dWRDdVNHZKZ0xpQUowQzFFK3F6dElaelc3STArTFJIREdkTVdQUHgvQWor?= =?utf-8?B?aGZqVmRoZ1Y5TTliU2FlOWIxQkVKOWN2M2sxbnNMTkpOMkFLbkJlN1BBb0lP?= =?utf-8?B?YnpLV1RuYXU0M3pUL1lkOGhSTkhqTEx6ajlFWmRwdVhxb0Qva1RwRkJ1MUNF?= =?utf-8?B?WGN4UnhlN0JYZ2VuV0l3aE9HODlFSDJsUENIWGQyUUpqcVFoeTVma1FpNEUy?= =?utf-8?B?b0NTWWoxMzJUS1d5RnFNZjRmNnNnOERsbnhNLzBlYTNXU1hCVW1uUlpZNk1r?= =?utf-8?B?dHd0SGRUcWN6TnozUFpUcTBHZFdrZXoxa1RKazY0YmJzaU5JeFg1UTNXMVl5?= =?utf-8?B?THl2dFZXSUVsa1o4SE02RDVCZlpNYnVlbVMwVFhuYVhQdW1LOTNoeWVnVWpq?= =?utf-8?B?WGJQUlJvYmtLR3dzbGVEUE1TOTRLdWxzU2ZLeUFaQjRGOFpKS3l6QW51YWs1?= =?utf-8?B?ZWpoR0xKVzhlNDF2QWlINTBKK3RCOUgrRWlSUVQ3VVZPM3Q2VzJQTGpoNTQ1?= =?utf-8?B?L2pwaDBtWk55R2U4aUpLNUR2VzdVdzVneUtmYkxtdXlPNmlVVlBBNS9iZjND?= =?utf-8?B?b1dhN0hHNzhsb3pNUTYvaWh3RzJZQVZlc2xtY20rZWtTNzB6ZnplRXBhMVVs?= =?utf-8?B?T2lBMHo0QzkveTM4UDY4eEovbTZHNm1LZWlwanZRTHZJZ0E1Z0RSWnRIdkY4?= =?utf-8?B?Z0VHcVQrdTF1aUNGOVE0NUNKNkRJVHhoSDRiZklqNFFCaGVVU29nL2pRaDZv?= =?utf-8?B?QXNqakROSVEyMzAwTE5VcEpaaCtFSXcvUTFBenUvYTVWTXhDQmFDSlU3aWZP?= =?utf-8?Q?YRbdP2JKd?= X-Microsoft-Exchange-Diagnostics: 1;BY2PR0201MB1591;5:cqGcLzXBY8pOtNYaozEhj98gaR/xz2ovrr0SwFJZR9+JDTzVeunC30ICbvJx8X9OdcKqUN2EKSV5qo2unBxkQHSSPdKAxNYSdHbJkzdeSWMkUWJETxOBhH8SMHKSQVmtm95JunuRBsHTKBY6QD4NUEF9tzi3QjdEsVG6xZBoxxSP/LZtP3kk1Dyv12mM1o2YRs2lWvdCpifm6EEpj6Oo0qalerQNLeN372fY6aWvBtKwCVYpjR5pYWr187XogqdW6CcnhZnoZLymOm0bEyarkTthRo+hlXrJSXOtqmUv+f9ewJE5r/g9Oc1CyFwdu2iVKFEbtT9jyGECzDMsrZnBi4TgZkjnHppa1omxnKfgCNMy9Kdgo+DmDJJtEElv2tnmN4obtFcP0/O47HtSfdESqIjWOj6w037xtnd9+iWndB3294YybMm+7aCx51WhgdIqBp/SrE7Mu/UCxBZAYpha+90Z0UX/a6kd7K9cCjn89h+491iSC3mo8cwZ6COykqof;24:NomJHppyp+J4DQuKjTBmj7APJmGbTwXraDpJAF2Khhd0wP4vrzUe55A0mGtPuABfIamElcvStEFBZQlNiZixX13O5QnCBzzm3qfA1Vf+oVs= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;BY2PR0201MB1591;7:Ta0REwVlrP/VH0KzvfdrhNkKlYSR9vELcSFkiGJXANZg97ldVqhZtxLCpEGbrDpJ584Kozj00fBAt5XyQ5t3xsUo1UhNEiWkB0D+1SQaIcvUSp1xJUHQLWoeL9nAC9za4pEv/wkXzwktK3Ct4exX7rqjmHZdhjF/tTMtZpHsWHLfdsgmYNMxtpCUB7JsBOyMIe5hOi5qt4Ifit/k6z09clWwQxdqqf6PJ4hwlydc4sfhq/5EFggY2V6pXVv9BJunIb62ADvMqjYMkbCxn820FvY2DAyEWVIg55PTiWvVQyYQnZh5nuLaDEzWlBWKo+BfRcdnIPBMljnMg/DVNnLciByXdkWJd9eSrmCSN268CBCnBguWEFiArJjPIxtuotkmHVz0VSC77XFva2/zPXG9crC+W3GJkwIFNleUAMI0S2Y/qcCtqshtN5vYVQFS9nWRyi3ICHCsP5vb5TGN052zcWWcft5z56XJbKahDQ10AW08mFfkaGUdUxePSwc8dh2xyM3ApFqS420DxEkZDBkp61X49lFOp4q4qL2zrF8UqsbVqTZwUC/g+4sCLMXrbaj8lQgZS90xXh53VACEFed1ZhJdcV5jor8V7hOoH0qnjFrezsyPn6KNapr8pY4qUFrU2zntkITig8JOE3rKMupwxThME6i1IUMDXIgy5/w8OYtn8Y9wBoNGwA3A8ftHakotjVGEE1j5+KiXj0zxbN6aMdK0N7I3H0rUSbCUlh8DiKr4ODdaAXTitc4QEIXLh9gXIkZG08XN1f6daJ4l4FytnyZU3/jMw+J4kPlbMFInEXk= X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jul 2017 06:33:08.2139 (UTC) X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.60.100];Helo=[xsj-pvapsmtpgw02] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR0201MB1591 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10.7.2017 05:04, Gustavo A. R. Silva wrote: > Simplify return logic to avoid unnecessary variable assignment. > > This issue was detected using Coccinelle and the following > semantic patch: > > @@ > local idexpression ret; > expression e; > @@ > > -ret = > +return > e; > -return ret; > > Signed-off-by: Gustavo A. R. Silva > --- > drivers/usb/gadget/udc/udc-xilinx.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/usb/gadget/udc/udc-xilinx.c b/drivers/usb/gadget/udc/udc-xilinx.c > index de207a9..552389d 100644 > --- a/drivers/usb/gadget/udc/udc-xilinx.c > +++ b/drivers/usb/gadget/udc/udc-xilinx.c > @@ -1217,14 +1217,13 @@ static const struct usb_ep_ops xusb_ep_ops = { > static int xudc_get_frame(struct usb_gadget *gadget) > { > struct xusb_udc *udc; > - int frame; > > if (!gadget) > return -ENODEV; > > udc = to_udc(gadget); > - frame = udc->read_fn(udc->addr + XUSB_FRAMENUM_OFFSET); > - return frame; > + > + return udc->read_fn(udc->addr + XUSB_FRAMENUM_OFFSET); > } > > /** > Acked-by: Michal Simek Thanks, Michal From mboxrd@z Thu Jan 1 00:00:00 1970 From: michal.simek@xilinx.com (Michal Simek) Date: Mon, 10 Jul 2017 08:32:55 +0200 Subject: [PATCH] usb: gadget: udc-xilinx: compress return logic into one line In-Reply-To: <20170710030458.GA14639@embeddedgus> References: <20170710030458.GA14639@embeddedgus> Message-ID: <73e25e50-c65a-bd43-d51c-1291a9168cd2@xilinx.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 10.7.2017 05:04, Gustavo A. R. Silva wrote: > Simplify return logic to avoid unnecessary variable assignment. > > This issue was detected using Coccinelle and the following > semantic patch: > > @@ > local idexpression ret; > expression e; > @@ > > -ret = > +return > e; > -return ret; > > Signed-off-by: Gustavo A. R. Silva > --- > drivers/usb/gadget/udc/udc-xilinx.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/usb/gadget/udc/udc-xilinx.c b/drivers/usb/gadget/udc/udc-xilinx.c > index de207a9..552389d 100644 > --- a/drivers/usb/gadget/udc/udc-xilinx.c > +++ b/drivers/usb/gadget/udc/udc-xilinx.c > @@ -1217,14 +1217,13 @@ static const struct usb_ep_ops xusb_ep_ops = { > static int xudc_get_frame(struct usb_gadget *gadget) > { > struct xusb_udc *udc; > - int frame; > > if (!gadget) > return -ENODEV; > > udc = to_udc(gadget); > - frame = udc->read_fn(udc->addr + XUSB_FRAMENUM_OFFSET); > - return frame; > + > + return udc->read_fn(udc->addr + XUSB_FRAMENUM_OFFSET); > } > > /** > Acked-by: Michal Simek Thanks, Michal