From patchwork Sat Oct 26 01:04:06 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jack Pham X-Patchwork-Id: 11213321 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 055AA1515 for ; Sat, 26 Oct 2019 01:04:37 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D3DFE20679 for ; Sat, 26 Oct 2019 01:04:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="ZjM3Rkp2"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="MJ0aAPM+" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726105AbfJZBEf (ORCPT ); Fri, 25 Oct 2019 21:04:35 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:41482 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725887AbfJZBEf (ORCPT ); Fri, 25 Oct 2019 21:04:35 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 9B69160AA8; Sat, 26 Oct 2019 01:04:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1572051874; bh=3yaT7z9yykYv3jW7oWej3mtZutvEctXePluI1eVXsLk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ZjM3Rkp2k/mi5LvMEziuSvFH9FTuvUXVFtlcV0TCTSscMYOM+Sx8ys5hqEsEEBNl0 G/SEPVIKd+9jbJp2OsL1D/qqwaI0KowIbClYte8nKCNogJHcvrq8sp5f9fMgxOdUoa +bD3tvb9Mr53pvS3Db6M6rqYIiuSxWf1lCxFGcl0= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.7 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_INVALID,DKIM_SIGNED,SPF_NONE autolearn=no autolearn_force=no version=3.4.0 Received: from jackp-linux.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: jackp@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 8A3286076A; Sat, 26 Oct 2019 01:04:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1572051873; bh=3yaT7z9yykYv3jW7oWej3mtZutvEctXePluI1eVXsLk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=MJ0aAPM+d7Q+3x7rnL37DGAq3l7mTTHiz/O6s+rKuyrBKYzzY4/ARFkdlaA/FicyO mfrj1oPkoTSwL5gohmm3hvlZmlgStutXPlrKr9jgr0WrL7eY4OCNPgtZ4Ypu00V4s7 lsOm4QjiijVsT2xhCW9k5GRX5YGcii4kvvS0wKEk= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 8A3286076A Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=jackp@codeaurora.org From: Jack Pham To: Felipe Balbi , Greg Kroah-Hartman Cc: linux-usb@vger.kernel.org, Jack Pham Subject: [PATCH v2 2/2] usb: gadget: composite: Support more than 500mA MaxPower Date: Fri, 25 Oct 2019 18:04:06 -0700 Message-Id: <20191026010406.24071-1-jackp@codeaurora.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20191023065753.32722-1-jackp@codeaurora.org> References: <20191023065753.32722-1-jackp@codeaurora.org> MIME-Version: 1.0 Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org USB 3.x SuperSpeed peripherals can draw up to 900mA of VBUS power when in configured state. However, if a configuration wanting to take advantage of this is added with MaxPower greater than 500 (currently possible if using a ConfigFS gadget) the composite driver fails to accommodate this for a couple reasons: - usb_gadget_vbus_draw() when called from set_config() and composite_resume() will be passed the MaxPower value without regard for the current connection speed, resulting in a violation for USB 2.0 since the max is 500mA. - the bMaxPower of the configuration descriptor would be incorrectly encoded, again if the connection speed is only at USB 2.0 or below, likely wrapping around UINT8_MAX since the 2mA multiplier corresponds to a maximum of 510mA. Fix these by adding checks against the current gadget->speed when the c->MaxPower value is used and appropriately limit based on whether it is currently at a low-/full-/high- or super- speed connection. Incidentally, 900 is not divisible by 8, so even for Superspeed the bMaxPower needs to be capped at 896mA, otherwise due to the round-up division a MaxPower of 900mA will result in an encoded value of 0x71. When a host stack (including Linux and Windows) enumerates this on a single port root hub, it reads this value back and decodes (multiplies by 8) to get 904mA which is strictly greater than 900mA that is typically budgeted for that port, causing it to reject the configuration. N.B. USB 3.2 Gen N x 2 allows for up to 1500mA but there doesn't seem to be any any peripheral controller supported by Linux that does two lane operation, so for now keeping the clamp at 900 should be fine. Signed-off-by: Jack Pham --- v2: Fix typos in commit text and reworded the blurb about rounding drivers/usb/gadget/composite.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/usb/gadget/composite.c b/drivers/usb/gadget/composite.c index e1db94d1fe2e..92ce3018f482 100644 --- a/drivers/usb/gadget/composite.c +++ b/drivers/usb/gadget/composite.c @@ -438,9 +438,10 @@ static u8 encode_bMaxPower(enum usb_device_speed speed, if (!val) return 0; if (speed < USB_SPEED_SUPER) - return DIV_ROUND_UP(val, 2); + return DIV_ROUND_UP(min(val, 500U), 2); else - return DIV_ROUND_UP(val, 8); + /* USB 3.x supports 900mA, but that isn't divisible by 8... */ + return DIV_ROUND_UP(min(val, 896U), 8); } static int config_buf(struct usb_configuration *config, @@ -852,6 +853,10 @@ static int set_config(struct usb_composite_dev *cdev, /* when we return, be sure our power usage is valid */ power = c->MaxPower ? c->MaxPower : CONFIG_USB_GADGET_VBUS_DRAW; + if (gadget->speed < USB_SPEED_SUPER) + power = min(power, 500U); + else + power = min(power, 900U); done: usb_gadget_vbus_draw(gadget, power); if (result >= 0 && cdev->delayed_status) @@ -2289,6 +2294,10 @@ void composite_resume(struct usb_gadget *gadget) } maxpower = cdev->config->MaxPower; + if (gadget->speed < USB_SPEED_SUPER) + maxpower = min_t(u16, maxpower, 500U); + else + maxpower = min_t(u16, maxpower, 900U); usb_gadget_vbus_draw(gadget, maxpower ? maxpower : CONFIG_USB_GADGET_VBUS_DRAW);