Received: by 2002:a05:7412:40d:b0:e2:908c:2ebd with SMTP id 13csp525807rdf; Tue, 21 Nov 2023 08:53:57 -0800 (PST) X-Google-Smtp-Source: AGHT+IGPDCcr4N5Znxv1Fao837IKCQfkvo3P0aJls3xyQECZOI5A3hHX14tIl0DZqbYIXikOPWVZ X-Received: by 2002:a05:6808:1704:b0:3ae:1672:16ec with SMTP id bc4-20020a056808170400b003ae167216ecmr13719185oib.11.1700585637475; Tue, 21 Nov 2023 08:53:57 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700585637; cv=none; d=google.com; s=arc-20160816; b=ehGplFFaUDfvQyXoSLUVCa8MxjgbT/f/FqOTFIs7G0sSF+rSprbl6+wu5pehmPjZTN 3ji7z4i9GPcl3ITj3bSgaKRLoi9iz67U/hw1qh8ZS7cnHStcNDGcVw8rhs2VNyTdyml+ ipEuKHSvNsk7g8anlg4o2oVb14hnTqu2e6xKY9vuOj2yysVh0luejODVyc1oX1KRoXSV 1jH6zeyldypiHmwQ4MFYqK5e1at0iQ+N4x7b2l50Ata6vltkRQH/KSgfgIS6rTV7mc3i j8MUmq5oMRSr1E1bdQq4cgm9kkPa39/PXrLIdk0Nxaqb41IyGio3oXLhTI7Fq5l3mBNE 5DIw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=WuZIIfJHEW22RHsZgV3ZVUEweekvh02GYhDMlQQsjPk=; fh=SkDy7L6g6hCwtJRZfN1rJhznv2Cd+aGc82JdYRBmQjM=; b=HPXhSlDOG2898d0LllbgjK+EmyCmYSbvTxqaIC06AXN5FvThcUBwdUrPYFTBRnAYUA FNzo010nVeisIF+H1NWRq4aErLUB/cUhX7b6Ul28kXFrFQjsDXb0S5+hYYEFqsxciwFI F5Kb9pQd1tu/qxfGzWUheIXqlRgPL8rlZREGfkYxhsvBhyRKD21n0pueX3E5B55Vdebd 2ueHAKOr/FcDbTWS3szR2vkQ/j+RyPfGZEspVjWPI0v05PB8+H7o3NCAbRGff9XXVFA1 GdC/f74hApMljrKRSwll1FTnmgvxxlArsHCs0vFb9Kw9Ei1tgcKMYGIZjiVxU7OSlBxo /dqw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Ra6Vhd55; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from groat.vger.email (groat.vger.email. [23.128.96.35]) by mx.google.com with ESMTPS id l187-20020a6325c4000000b005c1b5a76098si10041735pgl.771.2023.11.21.08.53.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Nov 2023 08:53:57 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) client-ip=23.128.96.35; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Ra6Vhd55; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.35 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id 9ACAF80C3A19; Tue, 21 Nov 2023 08:53:54 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230340AbjKUQxj (ORCPT + 99 others); Tue, 21 Nov 2023 11:53:39 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46862 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229524AbjKUQxi (ORCPT ); Tue, 21 Nov 2023 11:53:38 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9ECFD92 for ; Tue, 21 Nov 2023 08:53:35 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4CE4CC433C7; Tue, 21 Nov 2023 16:53:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700585615; bh=dXMSeMNK+WEBDV6rnNty0bayoAnlHmRDxFemjBJL70E=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=Ra6Vhd55uJUnkSGETGcDSGW77NyER+qrJBIyENP3uAPYGiRUemWipmmhMGJika2xK OTrvmNCYHM2k6oEaRQqR30ujYKgxmVSXp0Cygnya952ww30PVtxhv01Sgm5Meo06Zw DvXk7eEo7SnZEE+J1XCKENfBU3gUTJhVA1FG5bOkGbZasJKZ+8H8c1+dC1heCKtFhK A2fO2twFLiA+Fpf3daT6uxwCp+STjsR39x4g8T+1YcB/wTIIa5asKzcv6Y3/CuVoGX t/BumSB2tJTR85lpdGri4xW6sGKdpZYAVhXnqe6ufmYrjPMhBl2Q3okB/M4JShskYu I0dIeaL3sUNgg== Message-ID: Date: Tue, 21 Nov 2023 18:53:28 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 6/7] usb: cdns3-ti: signal reset-on-resume to xHCI for J7200 platform Content-Language: en-US To: =?UTF-8?Q?Th=C3=A9o_Lebrun?= , Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Peter Chen , Pawel Laszczak , Nishanth Menon , Vignesh Raghavendra , Tero Kristo Cc: linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, =?UTF-8?Q?Gr=C3=A9gory_Clement?= References: <20231120-j7200-usb-suspend-v2-0-038c7e4a3df4@bootlin.com> <20231120-j7200-usb-suspend-v2-6-038c7e4a3df4@bootlin.com> From: Roger Quadros In-Reply-To: <20231120-j7200-usb-suspend-v2-6-038c7e4a3df4@bootlin.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-1.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Tue, 21 Nov 2023 08:53:54 -0800 (PST) On 20/11/2023 19:06, Théo Lebrun wrote: > Pass CDNS3_RESET_ON_RESUME as platform data to cdns3 host role. It will > in turn pass it down to xHCI platform data as XHCI_RESET_ON_RESUME. > > Avoid this warning on resume: > > [ 16.017462] xhci-hcd xhci-hcd.1.auto: xHC error in resume, USBSTS 0x401, Reinit > > When used, remote wakeup is not expected to work. > > Only focus J7200 as other SoC are untested. > > Signed-off-by: Théo Lebrun > --- > drivers/usb/cdns3/cdns3-ti.c | 19 +++++++++++++++++-- > 1 file changed, 17 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/cdns3/cdns3-ti.c b/drivers/usb/cdns3/cdns3-ti.c > index 84f93c2fcd5c..7d56a1acbc54 100644 > --- a/drivers/usb/cdns3/cdns3-ti.c > +++ b/drivers/usb/cdns3/cdns3-ti.c > @@ -16,6 +16,7 @@ > #include > #include > #include > +#include "core.h" > > /* USB Wrapper register offsets */ > #define USBSS_PID 0x0 > @@ -128,6 +129,7 @@ static int cdns_ti_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct device_node *node = pdev->dev.of_node; > + const struct of_dev_auxdata *auxdata; > struct cdns_ti *data; > unsigned long rate; > int error, i; > @@ -177,7 +179,8 @@ static int cdns_ti_probe(struct platform_device *pdev) > > cdns_ti_init_hw(data); > > - error = of_platform_populate(node, NULL, NULL, dev); > + auxdata = of_device_get_match_data(dev); > + error = of_platform_populate(node, NULL, auxdata, dev); > if (error) { > dev_err(dev, "failed to create children: %d\n", error); > return error; > @@ -222,8 +225,20 @@ static const struct dev_pm_ops cdns_ti_pm_ops = { > > #endif /* CONFIG_PM */ > > +static struct cdns3_platform_data cdns_ti_j7200_pdata = { > + .quirks = CDNS3_RESET_ON_RESUME, > +}; We will need to introduce a new data structure "struct cdns_ti_platform_data" and add platform specific details like "reset_on_resume" to it. This is to address what Krzysztof pointed to in patch 4. > + > +static const struct of_dev_auxdata cdns_ti_j7200_auxdata[] = { > + { > + .compatible = "cdns,usb3", > + .platform_data = &cdns_ti_j7200_pdata, > + }, > + {}, > +}; > + > static const struct of_device_id cdns_ti_of_match[] = { > - { .compatible = "ti,j7200-usb", }, > + { .compatible = "ti,j7200-usb", .data = cdns_ti_j7200_auxdata, }, Here we should pass "struct cdns_ti_platform_data" > { .compatible = "ti,j721e-usb", }, > { .compatible = "ti,am64-usb", }, > {}, > -- cheers, -roger