Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932209AbbEUKbd (ORCPT ); Thu, 21 May 2015 06:31:33 -0400 Received: from mail-bl2on0094.outbound.protection.outlook.com ([65.55.169.94]:60142 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932154AbbEUKba (ORCPT ); Thu, 21 May 2015 06:31:30 -0400 Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=xilinx.com; vger.kernel.org; dkim=none (message not signed) header.d=none; From: Subbaraya Sundeep Bhatta To: CC: , , , , "Subbaraya Sundeep Bhatta" Subject: [PATCH v2 2/3] usb: dwc3: gadget: return error if command sent to DGCMD register fails Date: Thu, 21 May 2015 15:46:47 +0530 Message-ID: <1432203408-5482-2-git-send-email-sbhatta@xilinx.com> X-Mailer: git-send-email 1.7.4 In-Reply-To: <1432203408-5482-1-git-send-email-sbhatta@xilinx.com> References: <1432203408-5482-1-git-send-email-sbhatta@xilinx.com> X-RCIS-Action: ALLOW X-TM-AS-Product-Ver: IMSS-7.1.0.1224-8.0.0.1202-21558.005 X-TM-AS-User-Approved-Sender: Yes;Yes X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11FD010;1:N9L/gIlzryuEFGOYW3aMVEQ01fIs6e9lqqQSJk6Eh5sj90yc7959JuvyKdhkHgcjh8XGWVJJqFXnEbFkxwYdVgX6huYfQcUdj6C2dcZKhL6i+sXP4NibYlMHFanQ+KQz0Gz5rtwWBGyiLem6dHoP0PGrY1NKXA4re7uVDrFimqxd2sQNQYhm6NcPoJUa68l0KpeVrVS8Itjoz9iFZqlFndTP9FNY6CvkxOzevnj2MW4CNAoUqJmuq84BXzcNK6TrxWXcCBcJvz34CDAjkbh3XTdEgZN1xtO1m2TN/fp1P9GXSCsbJdxol1lAgdhLZUOE X-Forefront-Antispam-Report: CIP:149.199.60.83;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(438002)(199003)(189002)(42186005)(63266004)(4001540100001)(36386004)(46102003)(33646002)(87936001)(50226001)(52956003)(46386002)(47776003)(64706001)(81156007)(106466001)(77156002)(50466002)(86362001)(48376002)(92566002)(45336002)(6806004)(2950100001)(76176999)(110136002)(50986999)(2351001)(103686003)(229853001)(189998001)(62966003)(5001860100001)(19580405001)(107886002)(19580395003)(5001830100001)(36756003)(5001960100002)(107986001)(4001430100001)(90966001);DIR:OUT;SFP:1101;SCL:1;SRVR:BY2FFO11HUB018;H:xsj-pvapsmtpgw01;FPR:;SPF:Pass;PTR:unknown-60-83.xilinx.com;A:1;MX:1;LANG:en; MIME-Version: 1.0 Content-Type: text/plain X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11HUB018;2:stPo2+bf58q6fIyqqOOkDd4q0pf6J6e0rp+k0w0mBQSqYVdw0+bgoqcfZG5EZUth;2:W4F1C2mD6eL0wyp3KLEKGudGhBrFzI7DGXCOfdZHsZZFewd7PF+X6yH5SIUEJjaQVu2dwv+BM1+3QOygk2FtY3EZvhqGNt7LGxT6nwf9p8R18Ua7BVEWzOsGLyVWly6KH9tmhFI671dzc7RGpnTj93cVITkcyWvcafu9JUhT0BkX5ktoPYvw6e78IZhRyR4FwQvBOmqTJgM2DY6i1NJySDGAuADYvPkjBSXWJyx22Gs=;6:SVd2lWpb7S3gJ9EO/BUfltmNG7rUnV5fPlZYCHoTuVIrYKBEIQxAJKj1wMrgCkZku5Mgfdl1XS4WY0vbWDcvI69ix+gdmg3c+bf4WpehgOwU13w3dNDbyeim1sTbsEAbGKkor4RTJEheAyQZl0PKpcdSUaufj9NW6Ek8kr5qsvxj9PhzxTe75lFTX5YfilIIxnHssilbqYtyJwZqAnF7JTp0wEh7Xuho0S4k4h3FqhTf2ruxTNykh4bUjXmslpsAKgXajOHDu5jhMvE4yygl6BDm5sF0bVP5M4+kZyX/6JI1k4F0Q9NNfN4siiscBAZZopuCfqgON7b/vj6azq+1Ln9nl42AdzhmzOZLsDxG4sKAV0VKDn/2awESTjHiST/yPaO8kKkf5DXt50lH4UZR+/+iWgImUELtmyoGZWPgdNQzB3no2L7EcI/OlAam0sczTaTRW0KqdYta9asN4J5qqWz2kHlpk+J+DOHCC00HYSeZYCCamq4G5u19dqyPqJ7i X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY2FFO11HUB018; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:BY2FFO11HUB018;BCL:0;PCL:0;RULEID:;SRVR:BY2FFO11HUB018; X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11HUB018;3:hr0FwOT8FHqXTkqt1SIrUvLz/Mlpevp/MGQP1irBdqBz/tE9eqSgvOrMufdZSIn9yao6Nv/uxMYdcxx7PXEyHzwrDVc/Y7xh1UfaHXENhB9oDIpkKzwK8CXdIbBMVHaFgZlDxJGrqm9TmUxaJ9bTjAxchDQKQCbe3/wn12BHm7clrtrw2oGMpe3dB02Avz4W+UrmMxvMhMskzepoDl3YvbKp6Fnqt0LnGEyXLjWwePgXZ/k9w6/rdIKhZOnC/f9qT/NR11OjmE8rAeUZ14J7uTnHQzMAK8vx8ehSNdgbpYY= X-Forefront-PRVS: 0583A86C08 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BY2FFO11HUB018;9:TDOSlIm/zcr7hXIhz+hH+ba5A0Rwc8GeTUTfcRY0O?= =?us-ascii?Q?dCNkRN6kFuln7eZD2I7IgvOkQTaMGV3UcnRVlQNz/mv6UqlNxwdOAqj1INJg?= =?us-ascii?Q?BSpm1uorFkLPzEcWUuUsyWrnoCla6G2EoxZR7Km1jP9xZo9nlBhDSArrFhh1?= =?us-ascii?Q?uzPl6KqmWQmrSmhsDBGfNsWoGWudt5Cn02Wm14zhHJ5tcsosQLmRkDvhM6AL?= =?us-ascii?Q?mAmVfDSzm5wlZuLPMkGfSaes81lDtl2tW9UI4HJOp+IHntcLxjQFxR8+zB4y?= =?us-ascii?Q?fYx4QTD4WgoZvMQ+O6PoAYZpl7ScPU4aZ9Qm4TBidGUgJTrggKH90yiJ/83o?= =?us-ascii?Q?D/+Arx5yQ/qPEJDd7aSlpcjTm3kz7Ov1LdI2tLj7sDLkQoBuRKy/YjMxV6Gq?= =?us-ascii?Q?9snakSDBQy/mMRpRa1KPkKAl2qi5VpmleJ9pU9kd12nmPRdi3XrGc9D0fZiG?= =?us-ascii?Q?Zi/LZzCg1WI0cMXiFgfZERsRE37e1eL2iA+MTviRIqadRMzW3A9vZKqMshsR?= =?us-ascii?Q?ty+xeQWwOmebe4ENG3Czx0C4QvC8lNtG6qTlw7EV9LA5OBqQs9pTsQ1NxwoB?= =?us-ascii?Q?xVT/8hrmPAVPSdWKy0vGv2HoAp86m4XEgzggELM2HzHOP9Gb0ooRYANLai/s?= =?us-ascii?Q?lfV5IDtGiSNeIyasTzgX8GJ9kzzfuyHUwxLL9jTTumH6i6xZ+pm2eozBsmG6?= =?us-ascii?Q?Bq6EFFMluX3ypvWSQWlPYCMKNS4QToSkQi40VD6jNPkx6rlqDZHNe3+I9BmZ?= =?us-ascii?Q?BuWNKgMuUSsZs3gF5w94BYHP0goLSklgHQC/ffHsjjW9Uej9pYAhQXawJLoy?= =?us-ascii?Q?AGgZ5FeXiyxnofXa49ENjkJnps9t3xAo6624tHotSLz0RAs0ZQVhApO+m1D9?= =?us-ascii?Q?8Cwj46BRcq126Z5qlHhQDPIc5mBpehu42wlVlxp+YQr7JHij+3IcKyDdAQBF?= =?us-ascii?Q?DLkE+s1Vzu1vzBnYRRCrpM/iMVA4AYLVaxhZUPYrtgFK8m2sAU6FUs50QUiA?= =?us-ascii?Q?Iw9EYYVtuuYNh6Wta2A4bJ9xc8c145FVhPIMDmyzXeLzju9n6rKJ1neq6Mx4?= =?us-ascii?Q?1TDrh+brx4j7BUkffU2riBEn+Uh3qgIa3W25gb25Hb0GNvMXedqbu6Bpk/8k?= =?us-ascii?Q?RN9JTzYWZqYh1sq7ROu+pdMPSj4N8ic?= X-Microsoft-Exchange-Diagnostics: 1;BY2FFO11HUB018;3:K99s2Qt4fhY7sNz8fEev7eNVKBQnzGd+6D0MD1Ja597YNdYLAicuf3H6mYsgU8Ti8nBvV7NmONYLnPCgdppzlSQPcRyJdYubhwOui2sST6OQmLv9m4gukDCBqU4wnxtM4sOqRtYF2QTt8a2ybkpXkQ==;10:Z/NQAIUbCMcZc0upDOUMjIa9LDwGT+67Whn5xfn3totJSQsdYBhFuuOfRna6od2JBjJFsRwycwaZWV8Jnf5rPDl8JGtLx4mbJ9mv0VVg8Sk= X-OriginatorOrg: xilinx.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2015 10:16:56.6768 (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.83];Helo=[xsj-pvapsmtpgw01] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2FFO11HUB018 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1089 Lines: 34 We need to return error to caller if command is not sent to controller succesfully. Signed-off-by: Subbaraya Sundeep Bhatta Fixes: b09bb64239c8 (usb: dwc3: gadget: implement Global Command support) Cc: #v3.5+ --- v2 changes: Added Fixes and Cc in commit message. drivers/usb/dwc3/gadget.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index 8946c32..fcbe120 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -291,6 +291,8 @@ int dwc3_send_gadget_generic_command(struct dwc3 *dwc, unsigned cmd, u32 param) dwc3_trace(trace_dwc3_gadget, "Command Complete --> %d", DWC3_DGCMD_STATUS(reg)); + if (DWC3_DGCMD_STATUS(reg)) + return -EINVAL; return 0; } -- 1.7.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/