From patchwork Mon Feb 12 16:19:07 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Frank Li X-Patchwork-Id: 13553445 Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2058.outbound.protection.outlook.com [40.107.6.58]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DB1AE3D3A4; Mon, 12 Feb 2024 16:19:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.6.58 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707754773; cv=fail; b=k20fO8tfW9E37IHIkE6qX3zzeQBlDIP3TVO9zBEgrOXbmqHsxWel00AJA48A5A+1PwSH3IHGXri2h7/w0Ct7fkMQQLG/XkuCU6Pb8t0MAo47X0oDnqdUAoO6f0IWLeXgEblDiPRU+rXFBA5KSCIuX8my13OzzOpx7MqZbAQuWCk= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707754773; c=relaxed/simple; bh=mHut+6Z7GK4LUyPL5v1lGhnQZgR8R3h1UZy+qMslDkM=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=JSrF04+4o2RCL9eiD6Fi+gaRzFuwQLcDEfq3s7eZao0KSTJkBgdtfaf80qJiuVvOCiF+cy1Eh5VtL6JWyLcpw/7ye24PFy3vUOdeYP8yGhiwP39s8njVSeS2ZN3eIbirxS711by7dM4AFlmPyN/2BverNMKf8VSnSHi6DOLXv4I= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (1024-bit key) header.d=nxp.com header.i=@nxp.com header.b=FLTVyyKF; arc=fail smtp.client-ip=40.107.6.58 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=nxp.com header.i=@nxp.com header.b="FLTVyyKF" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nW4edLQhPGTC9Jb+oADkk1lxSW6vMOAIoxmsggrbR+S1eNF+U+eOwzWWEKg5Pu30guyPsVYzZ7Riubvk1rMAMwqY36ztGdkwJUhUAewYfHUC2OviJd8lb5AyXaMrZW8LF054eOJBgxjNXVORRKMS9bBwridh4t+NdATdVALsq7idroZP3dCSEvyI1fCzJh5r0zdqZNj+73frgHHYLjEBrkpW+Ik7Sr0BsPuyBRVJyL8gOaxtnDMjyYNeQcoZbRr0dtLYf2UQ/tVYMK3bkfO9liUR/ic5gr90DSk20hPfcBxF1QPghAzlXjgr3FnF0Vd4EuPhF1rGxdO/eKbTpzVuug== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ycTA0bP259Iv1t89syfXIMsWI8A3L6dfhyC//SX7E0w=; b=abTK2CGh4kIOTGPXmHG4cwL4EiIdvSaJLpcWGyMz6w9l7G/6KKWl76l12IsbDuNUDy5YhmoiDFEbQAlDMEdOU2xqx0Q/F6LBR4T8XELgHccfRpojQe/9GUnt2gglKb74dxMyoUaXIF+F9J1K8gkEHzw2CACEfqe9+cwmNg1G3JW3sSsRWaiN1dR1ukKzDHRzK0hhPnbnkQhXwNsZ3obNR/7RRtDyirCNdMJK9vG5ZkLlguUHPUBK3X/3q9KlhYVDryh5USPGN04y7JJciOiu2Q6Wj8pyrzn6VVP6oqCb3VpgPTbqDG9FSYso4XedJtGWP6xYx1L4FC3fDl15iU3BBQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ycTA0bP259Iv1t89syfXIMsWI8A3L6dfhyC//SX7E0w=; b=FLTVyyKF0VVLbIOeqcGLd5YFnX3WJlG4Pfxu3pVmOEoDdsom8t85fuu6HgVcyUlEi4K1FU5h+YHjMc7vP/Yx7w1MWjfnBdjkVNgnSbp+7pUh2hSoB42JAE4C78USkyltA6Yk3F3kuPMTikK4K8cyV5KVhlXbDn2TOKt7NjVW5Q0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) by DU2PR04MB9049.eurprd04.prod.outlook.com (2603:10a6:10:2e4::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7270.35; Mon, 12 Feb 2024 16:19:28 +0000 Received: from PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::c8b4:5648:8948:e85c]) by PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::c8b4:5648:8948:e85c%3]) with mapi id 15.20.7270.025; Mon, 12 Feb 2024 16:19:28 +0000 From: Frank Li Date: Mon, 12 Feb 2024 11:19:07 -0500 Subject: [PATCH v2 1/2] dt-bindings: usb: dwc3: drop 'snps,host-vbus-glitches-quirk' Message-Id: <20240212-vbus-glitch-v2-1-d71b73a82de1@nxp.com> References: <20240212-vbus-glitch-v2-0-d71b73a82de1@nxp.com> In-Reply-To: <20240212-vbus-glitch-v2-0-d71b73a82de1@nxp.com> To: Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Felipe Balbi , Thinh Nguyen Cc: linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, Frank Li X-Mailer: b4 0.13-dev-c87ef X-Developer-Signature: v=1; a=ed25519-sha256; t=1707754764; l=1157; i=Frank.Li@nxp.com; s=20240130; h=from:subject:message-id; bh=mHut+6Z7GK4LUyPL5v1lGhnQZgR8R3h1UZy+qMslDkM=; b=YWxhApEV4qMh+Se7urbVCn+BskRMRCI2ZXc3dxR5tytMV+ZKCWq3LJ+QUIPES0eaeYX6c81pV DFA+UOfrOdJBzNzHgqs6z5J1mGokV5v+aXmT0Z1xFZW6T4MvoDzYb3t X-Developer-Key: i=Frank.Li@nxp.com; a=ed25519; pk=I0L1sDUfPxpAkRvPKy7MdauTuSENRq+DnA+G4qcS94Q= X-ClientProxiedBy: SN4PR0501CA0001.namprd05.prod.outlook.com (2603:10b6:803:40::14) To PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9642:EE_|DU2PR04MB9049:EE_ X-MS-Office365-Filtering-Correlation-Id: f6623a91-db2d-439a-c88b-08dc2be66319 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Ni0wZvOaPmn/Xz+u0PSBsEzankk+g5ZOVSxWEHuRfbiR9NwWY+VDvTMof7S9IXmOC56pvZOzsrdT/ZTgCQXAhWOT4VIVokM/fIaZngjEFWjzuow29U8yLHRNWYsOMaBaMhTy3uidQN87dyet35Eg/HvjJMhrtywI/wrN87c+bwZZEqDbzdN+XZRrI7s7Uin8K/AWBUBB+sLvAOSXNykfBOOmukoKSXgy8Bs3EQAOvpOvoHVD6qbteiRSUhPJDpoFCGkU7oTEUTHaDyc//Ts3vEPF3hYxVuNpK8FEcLaEBphd4GyjRsNvI2dWEIKx6POi7kKl1FxGq3YjDGeG5ochAEEsNIttxfavlYrXEP/nSJWPQ5PJOMIi07rCymGb4uEcaLePg3VnhZnSGKK0dJBzGH2PDadQVn7YSfFWqdTzbKaLhblV4XZieKA+XwIIyYHCIIhP+yOan0ABTmaOnzxPkk6Uk3irYFcwcfcESwYhgd+d+HZbPiU8MsxmY4QTf8aoybs4DQc+Vjy7hu11/d/OTQ2YwXe150ykepWAoKLv0bgrRI7AtCNefzcovWrhZTXNlbztW9uijd3/wLHo5pcx32Z6pLGkpmIGGvr6RPA6c32TfO/Zj0UYz5sWJuw1wtIq X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAXPR04MB9642.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(376002)(346002)(136003)(366004)(39860400002)(396003)(230273577357003)(230922051799003)(64100799003)(1800799012)(451199024)(186009)(6512007)(478600001)(6486002)(41300700001)(8676002)(8936002)(4326008)(7416002)(5660300002)(2906002)(110136005)(38350700005)(66476007)(6506007)(6666004)(316002)(52116002)(66946007)(66556008)(83380400001)(2616005)(86362001)(26005)(38100700002)(36756003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?GalzcfBdkWeHtS3Jq8bxzH5wBoRS?= =?utf-8?q?EBkCjhW/In+Ry7FISjMcuB1mhI6DYnVaorIFsLd/Y3WyLCfg/f0wjePN0rLkZmsIh?= =?utf-8?q?Dj0LkYN6LAmQwJgm4fDEpzbA0Hppjs6ZOZBWnpUkJC0/mAxx5T1eiu4V+2otKojfj?= =?utf-8?q?dUwiibQzatcT04v1ZI3rGfSchQCUMwbo11XvZEbmrpUppkHLRrxEDmCXEEcklyugE?= =?utf-8?q?TekrtCaCT+SqOwpiSA9uoOn2mE3ieQyhGMKgIp9GzeKHxRjjXXK1jPAxsQZZvonzn?= =?utf-8?q?4SmS+Wgri9WDfB++IHaaCPxM50amY4oojKGCH44mXdhvof58AM1dfq8kn7AspTbkI?= =?utf-8?q?UPDIbOa9XdKtNBZN+zCQw7ggCoiUr7YI+sdvr6hgDiEOLQuunzmZEJMo/SrzGm1Wf?= =?utf-8?q?qi9DGJvpO+/U3oX/t1qt4xLEMFfwxfWkpmMeVv8twap6hsCWKl4FKoYLGqDh/Ws2w?= =?utf-8?q?AsxcFBwEyqGTaYMltiWprhOpRvpEYSq13YFLu2SO4m5rjRd4qrA0yGyyfiInB8ULj?= =?utf-8?q?D1VUyAO4z9R+PIPoX/3xxqikcwKgjgdsaINDhyLNc5fvwR1Uo8/guVqEj/EoyaldT?= =?utf-8?q?8cX0xfm2r3B2Jvsh6A2BsjCKd6yfLLWGCGweQWsqyaUIgbewRhZhjUTrOjb0o+pJj?= =?utf-8?q?OIFwcjJRW5Pw7ws6SMjy198WYSoSUVsFfVhpCto7Ay27TYjvNwGRBUhH2RCQWrmKF?= =?utf-8?q?LCJeqjqaiKWTSBAXIuGgkbI+aZDYf+u/kvydOB2TKyd6oH0iY9p2xcTT+np0wB39M?= =?utf-8?q?pdwDZ7QttjqSxkMXkvv/99LdOq9CqyXuFLEBqGX52b+zUDHnJAUQb3Wn4TyiP4JC4?= =?utf-8?q?fifr8CWwk3pZDA26nuccq3X5vfDs7CRCxEsj3VulDeAW64a2IxeGw2sfD4leOeIG6?= =?utf-8?q?DDiYBVs8MoQSWGxuh/yIG6Jx102kWQUUtf4bn+8Cx/WCQuroZh0/4VyrPYxPloiog?= =?utf-8?q?PiQDWlnIM7xhS/fsTSPhXFDgduDMRXzquulT/19T2Lg1wf0Mx0TPzq6PADFjqDC5W?= =?utf-8?q?nYWJR3A/mmBe41Pxleexj9dBdxk3wPBZN0yxL2BSjQWalVJcya5YbZUjgk6YjyzXk?= =?utf-8?q?v4pKf0leuvaU4EKwehEbriVx0PhMKV4W+Hq70tvQsdqFTM2+ctwMFp0VVGG+/Yrdg?= =?utf-8?q?RAwK84QsQav2p1NFUUrZxlMReUNUFLsfy7LfHqlrba9RPlW4NtewTLz+ClqFm/7nT?= =?utf-8?q?pcwf6IQvImfLMaUb0vbSOpB8eAIM31IDJNSaOKmlAC72Lm8CPuSPczT/i2SYWQXKJ?= =?utf-8?q?sy1SWRf9vHyK8Z2eu3aVJ//xObsWicgIk2h4M8mZcaGUCxtIVcba+EEFsWBb0eA+R?= =?utf-8?q?sEGiksiiSuWRQr0iQSBLpW820WeDOkhltlW6YSva+4Gu21LES2vQB2hUO62hS1NBO?= =?utf-8?q?ZDLzTyTEHb59atvmdudUe9Vqfe6ldx+TgejBPEVOXyfRo0QiS+dWW5bDbsNELA26+?= =?utf-8?q?Xnfg7GUuwB7+RrvhE+9eCq3e/zXesKFTS0ELL5z5AtRMeQLM04JSriD0=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: f6623a91-db2d-439a-c88b-08dc2be66319 X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Feb 2024 16:19:28.5941 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: WfynC3ST66XKeDQcDrh3MwO09sictSTa3VBxsGuX+3f4eRsVa3PbbKzrT7s4IpeHo5Bj31J8kkQPQrMr5R7m4A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB9049 Drop 'snps,host-vbus-glitches-quirk'. It is safer to keep vbus disabled before handing over to xhci driver. Needn't this property to control enable workaround. Apply workaround unconditional. Signed-off-by: Frank Li --- Documentation/devicetree/bindings/usb/snps,dwc3.yaml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml index 8f5d250070c78..203a1eb66691f 100644 --- a/Documentation/devicetree/bindings/usb/snps,dwc3.yaml +++ b/Documentation/devicetree/bindings/usb/snps,dwc3.yaml @@ -273,13 +273,6 @@ properties: with an external supply. type: boolean - snps,host-vbus-glitches-quirk: - description: - When set, power off all Root Hub ports immediately after - setting host mode to avoid vbus (negative) glitch happen in later - xhci reset. And the vbus will back to 5V automatically when reset done. - type: boolean - snps,is-utmi-l1-suspend: description: True when DWC3 asserts output signal utmi_l1_suspend_n, false when From patchwork Mon Feb 12 16:19:08 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Frank Li X-Patchwork-Id: 13553446 Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2058.outbound.protection.outlook.com [40.107.6.58]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 109BD3D552; Mon, 12 Feb 2024 16:19:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.6.58 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707754775; cv=fail; b=QpNgpfbACbilAEZdojVovty9JsGeT+Flqy8Q8rOaYPfHKxBJ1tg9MkF0v8y3K6+QytGZz4kRYBZfBGB4g3A/RJ2ThMrztKAFwMOxBXu/kY5YlyH1va8JlmuXOMEBW4Vqtdm7FdcNgC/1y/rKVP94geyADsevjKEvnixVSao4oxo= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707754775; c=relaxed/simple; bh=RVZanAIO2cvNYwqNO1H2+H1kfQCW0KHxJ3QV94uN8yE=; h=From:Date:Subject:Content-Type:Message-Id:References:In-Reply-To: To:Cc:MIME-Version; b=TnH23vePvQaSH4//BwpJFB43RcrDOKROo5vSMfXtnwGZp+lQUdl9U2bCP+em2OhTRI7Ue8jSqnpT8uk18i2NAIbp39EaWyqLOW4pgMq6X3dsxfrD1Xe5GFCASWxtr4hzTp549IjNzIUUTxZ1t2lsqfSAo9ePJzEq91xNRmEfTaQ= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com; spf=pass smtp.mailfrom=nxp.com; dkim=pass (1024-bit key) header.d=nxp.com header.i=@nxp.com header.b=qMxMH0Fk; arc=fail smtp.client-ip=40.107.6.58 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=nxp.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=nxp.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=nxp.com header.i=@nxp.com header.b="qMxMH0Fk" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CgAGTf6jHPeivSZd6gk/oXqAJ9m5O5NO/uH/3G4NE29/oudl8I5p1MNjetvvuSOGrmzZoje5tUa5ErEluqsJ3tUouaTKdcrIZhInLu7cfzU1nYrXW1iToJ5QBG1GYZSDRyeNulPXerJltdaK+gnApwwpjulU4wYztBx5zpeG4h7bnW1PyoWP9u/oaheezXh3C2K63w0RxZeARYiO26bK7vcV65SFhThQjyEdFF2rj0KmwQYf89d2swfQwnyn6PQssFdiEJklpi1Al7l3ItTEyZ+OnnbovalFal7YMW1dkghnZ5kbor43kg0jX/YUJKLry0nLQYzOB/m+eNzEs68ssA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=pjv7QnKdUodCbDCtWeMa8BZzh8+qG0XKvEheQHpbLE8=; b=mAlVImCzFYSzuM4GDwwWMXlC5EOjiEUOpsS3Ov2vicj6K4sY5K6NGjWiizJwe8jq8NpF7muoTBIg7oDqKLjXhirMFmN7DioA4xyvuSun+AMc01+oVvqUrY4E/cEiMx/hRUHb1wLK5e88su1FdNYMGaad6b5UWey4wT+5he4wooW2NIaPyJvYTXNm4AhGiqxexVhQum9kZMNqXMrH/PcFkMMjkISFY6JnXIZHaHiHYLQcj72L42CazCGc24U5xdHH9eUh+SD8YD6L4Ls8tHzVL3/RJANgobvlcqnepp6cO/tesSpDp8PQ5kiVv3+Z0JnO7fhG3yHteAoVEDbqPU9Uqg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=pjv7QnKdUodCbDCtWeMa8BZzh8+qG0XKvEheQHpbLE8=; b=qMxMH0Fk4t8T4euYIkQpIKS3HL0eStO30VZG5PfeCJqbYCpSHQe9vhxtEDh14aX0ogpy1T++c8NS0fQfwpDpdc0cbQau4f7IyDeEp/c2vcDfVn4IHWn4Z7Sdc5oDcv91ncvICQLyqAP5awNmYJhgFyijcyjY2ElSLjJINx/UHS0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) by DU2PR04MB9049.eurprd04.prod.outlook.com (2603:10a6:10:2e4::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7270.35; Mon, 12 Feb 2024 16:19:30 +0000 Received: from PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::c8b4:5648:8948:e85c]) by PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::c8b4:5648:8948:e85c%3]) with mapi id 15.20.7270.025; Mon, 12 Feb 2024 16:19:30 +0000 From: Frank Li Date: Mon, 12 Feb 2024 11:19:08 -0500 Subject: [PATCH v2 2/2] usb: dwc3: apply snps,host-vbus-glitches workaround unconditionally Message-Id: <20240212-vbus-glitch-v2-2-d71b73a82de1@nxp.com> References: <20240212-vbus-glitch-v2-0-d71b73a82de1@nxp.com> In-Reply-To: <20240212-vbus-glitch-v2-0-d71b73a82de1@nxp.com> To: Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Felipe Balbi , Thinh Nguyen Cc: linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, imx@lists.linux.dev, Frank Li X-Mailer: b4 0.13-dev-c87ef X-Developer-Signature: v=1; a=ed25519-sha256; t=1707754764; l=2477; i=Frank.Li@nxp.com; s=20240130; h=from:subject:message-id; bh=RVZanAIO2cvNYwqNO1H2+H1kfQCW0KHxJ3QV94uN8yE=; b=UDHEWzz3fU1LvjbSPirbztQdGXqjfNd3iHqO6GmfrBVppRfsEjnhT2hc3jYX53NlAmYGMNW9h +vnx1nUSV2lDyg9HTpsxJEoY3UkY+wjsL1LB48xPmGU9DqGtW7lA6q6 X-Developer-Key: i=Frank.Li@nxp.com; a=ed25519; pk=I0L1sDUfPxpAkRvPKy7MdauTuSENRq+DnA+G4qcS94Q= X-ClientProxiedBy: SN4PR0501CA0001.namprd05.prod.outlook.com (2603:10b6:803:40::14) To PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9642:EE_|DU2PR04MB9049:EE_ X-MS-Office365-Filtering-Correlation-Id: 11c26fc0-abe9-4f8c-b337-08dc2be66444 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9MvcPLGXj1UyHo5GTB/5dhHzpG2cjs4vUQh3/K9X3DTMlQgsoEeGbuP0iW5XmJcbihgGTXCHe1ItVLS/rcsGt6L+d84VeOxrf5AORURMzqXoPYkrBSgkfufbUL3HPE9/qjKeh1pSSGkBqonmk59gZSQMDodY+Mtpc8Z11+LYf37pL1bVuY9tMvTP7X8sEi5MI9Ll3UysBYQs0xAgItkmxE+CwxyfDAXGJg3MDcu079h/Mji9S4GrZsYHE+6cXcXHn9GowPw68Al6Z50rKwf3wV8phHgc54Ijh2fkPz1jYFZGC93ZwUJ6WZaLx4n3Iari5ev8GfekAqY5D3coOAXIu2oUvI+hEzOYx7oM0/BcwuGl7MOLgpeCMhPX+CbiLFUgYXokat3cwVboyaMVlw+BWRXoVadFtTrFJOTfRazQPbzE5cs56d2xgq5HWGgLi2e5aQD5a8F0wgRp5L+oPtZSlscaD5qySlShMffoWPqbcViiNrqB4GtiFB0YRULE5pXOYp21y8MmKNoCOBQRZMLNBxM0v8t/7KctvBAWm8EWwgXR1KtPK3LcEARAcTVSncQHk+YCZAkIQ0UNLRADeGzuckUB7BhXCIz9LS2ABLIj5qzV7sK19/gg7l3Q/7wM1hu5 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAXPR04MB9642.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(376002)(346002)(136003)(366004)(39860400002)(396003)(230922051799003)(64100799003)(1800799012)(451199024)(186009)(6512007)(478600001)(6486002)(41300700001)(8676002)(8936002)(4326008)(7416002)(5660300002)(2906002)(110136005)(38350700005)(66476007)(6506007)(6666004)(316002)(52116002)(66946007)(66556008)(83380400001)(2616005)(86362001)(26005)(38100700002)(36756003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?9rEMaZbaNe5j1+/FoV7N4ivsrgqH?= =?utf-8?q?JLjrdf9SXYo8J4TBVktbRuDbXhf8VlTietH9U+v2m4onXCvIOgdM3W9wy2JIm/egj?= =?utf-8?q?RmVol4Wcc/XS82BlByEmw3OYjTom1qOksLWJbuP3JVpFLiWtnqacBSS/28WteyQOt?= =?utf-8?q?ZM+Q7RFBXFcsbXirtdnANqEr+We6/D1F529oQT3HlqezXN/Xf1W07TPdlTeVBW6Rr?= =?utf-8?q?PWGEvI9LdrvdNiNa9Xvc40ollLdr5fBJ3bZZkPHmYcdHvOnQI8+hyOdOaddzo2vbJ?= =?utf-8?q?GjJsQE+eeELAa41sFLJJL1/miVxg+hXw3qrzJy4ZFpXurSUBGskcNcN1HHu41OHCJ?= =?utf-8?q?izyfGZUJecO4+ZXBSfvxKdO5bfYQGso3MLV7ET1+PAZj7Xso97kEt7JEdUb6uk5x5?= =?utf-8?q?+F77FcgRl08O5WCGpg569JfbggNGCpZUYzY6lcBK84d77U+QnM09WsgoFBRELPuos?= =?utf-8?q?iGttSlwfxjMbw2dMLFs5vvV8IJXaLgLGEsBT9FgrhtZMIX2IEhWjHNUBJP4lNVOmm?= =?utf-8?q?fO+SO52lXIRWlT02Dfy4EdOS2LLEo0lGR2acCAYxXWMZe2g5loFgnJxA3iZdhR1lv?= =?utf-8?q?Esuqjcotu8Cs+fp9j3bgRGkvNwvigU0IvBCIeQqknC3GOHmTUqBL4ROY4X/ERxjOT?= =?utf-8?q?0SxidNxM4u7/6w2qeXQZuLVPe80KrHinAahzGhgXh5nIARuNvzXrQGVAtw/3S0S1R?= =?utf-8?q?FAkg6d/tDt//BOlmL7oth/WV/JV0y6vMQS5AEO9AL/4DtTpr+aAjR09KzTkmgebn7?= =?utf-8?q?54mS9bpwt1lscmXhH5qv6P+qpz8lOkrsUYbAIyEkhLSHyRvTviMeVb8EYuMInrToG?= =?utf-8?q?unArHgnb2s9iYMvPfqWwWJoadHKqYVmkiAFPiYRCB9LNDerGAZIGsdS4jZo3IoEvl?= =?utf-8?q?i/2itMcAcTdalK8+/uCdlzjWmlpHQ/kJHG8HtqWfhdrB8dm/jAccUL0iWX8cAiJGp?= =?utf-8?q?ZFbRvGev3QsgQfTePdUc8ecOY8CzNFLSSapNrrADPPILAS0KU3pJRX6BwZ0sfFaEF?= =?utf-8?q?Hex4CYrgRDfL8C6nPY61uKcUH4FyFtzL1HxB1nBu0wUk1a6kRU2LIzayVM8zrs83Q?= =?utf-8?q?xiC6n/9r+TH7vGKFrKkxjS6ZcGeyvSa2dj1hG76fWqKk8BIF1Av5IOfECGRsG14F8?= =?utf-8?q?vpDg4X7VX8MxcQkEuiTl73NwVVNm1cuNMHzqz8tbQR9VWhVmoBaPZVGUz+pRSyUHy?= =?utf-8?q?cffWaJzHFBFpKqqvo2EtOF2Ax2ah9xoPN7oAzIk/riJ76f2Zv7VWcJUr+IEzX/vbY?= =?utf-8?q?+8YDeYPgKfPqXPZmP6qGivYrtlaRHkN8A5cuzbJFjs6LXG9dNNxrPSzfHHhwYmbyy?= =?utf-8?q?hyb1SN8hlIA4n9r+1OXsRqsbozAg6QvLdAVZRbh573/BXf79Mc32//Mt4TMl+gzco?= =?utf-8?q?zlJ2uoMrh1xRVAClaz6snD54u4v4d8LkP+gg5JGkMnM9/29jVWusJOX1099QKlXpA?= =?utf-8?q?k914Z+2fOKDxFYpmP8Ucbzci9qQfgra2+Z7mAboi2DvjQtemFSiNGdjk=3D?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 11c26fc0-abe9-4f8c-b337-08dc2be66444 X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Feb 2024 16:19:30.5521 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: dZjJkGXhNVIGpybPBEctOi2EJrUCbA3u9IrvctqR/Uwc4qlEWa+zo/seKT8lLCTmAQpx3+TKyO1bF18mOWsTnA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR04MB9049 Remove 'snps,host-vbus-glitches-quirk' and apply workaround unconditionally. It is safer to keep vbus disabled before handing over to xhci driver. So needn't 'snps,host-vbus-glitches' property to enable it and apply it unconditionally. Remove all host_vbus_glitches variable and call dwc3_power_off_all_roothub_ports() directly. Signed-off-by: Frank Li Acked-by: Thinh Nguyen --- drivers/usb/dwc3/core.c | 3 --- drivers/usb/dwc3/core.h | 2 -- drivers/usb/dwc3/host.c | 3 +-- 3 files changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index 3b68e8e45b8b9..3e55838c00014 100644 --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -1626,9 +1626,6 @@ static void dwc3_get_properties(struct dwc3 *dwc) dwc->dis_split_quirk = device_property_read_bool(dev, "snps,dis-split-quirk"); - dwc->host_vbus_glitches_quirk = device_property_read_bool(dev, - "snps,host-vbus-glitches-quirk"); - dwc->lpm_nyet_threshold = lpm_nyet_threshold; dwc->tx_de_emphasis = tx_de_emphasis; diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h index df544ec730d22..e3eea965e57bf 100644 --- a/drivers/usb/dwc3/core.h +++ b/drivers/usb/dwc3/core.h @@ -1132,7 +1132,6 @@ struct dwc3_scratchpad_array { * 2 - No de-emphasis * 3 - Reserved * @dis_metastability_quirk: set to disable metastability quirk. - * @host_vbus_glitches_quirk: set to avoid vbus glitch during xhci reset. * @dis_split_quirk: set to disable split boundary. * @wakeup_configured: set if the device is configured for remote wakeup. * @suspended: set to track suspend event due to U3/L2. @@ -1354,7 +1353,6 @@ struct dwc3 { unsigned tx_de_emphasis:2; unsigned dis_metastability_quirk:1; - unsigned host_vbus_glitches_quirk:1; unsigned dis_split_quirk:1; unsigned async_callbacks:1; diff --git a/drivers/usb/dwc3/host.c b/drivers/usb/dwc3/host.c index ae189b7a4f8b8..98964e8dea1f7 100644 --- a/drivers/usb/dwc3/host.c +++ b/drivers/usb/dwc3/host.c @@ -114,8 +114,7 @@ int dwc3_host_init(struct dwc3 *dwc) * Some platforms need to power off all Root hub ports immediately after DWC3 set to host * mode to avoid VBUS glitch happen when xhci get reset later. */ - if (dwc->host_vbus_glitches_quirk) - dwc3_power_off_all_roothub_ports(dwc); + dwc3_power_off_all_roothub_ports(dwc); irq = dwc3_host_get_irq(dwc); if (irq < 0)