From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7E60DC433F5 for ; Tue, 29 Mar 2022 08:16:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=YTkImVVvC8YHGXf6bwtcX1C0pTyNUwpAk7S3XhR6HtU=; b=3sgHQrh0xbw3rL X6cnkAo/iBCSnAed7BcAFnc3fjZI8moRGY6aIZPws4VcmDaWuPvr1jT0SbGNaXjScedv2S+ktE6IO Ae5/9fkCZVpxfhgnNLvpwBLITkqRw+X7WAnmdEuyzh+GivNIiL6IZX9rNPzZEWI07tmTEvDMXCCse w244NCZ8aBFKq2QUIdnGMvYar0oDk6vNi1V1spXQgo3G71auiiNPn1p4EBEKDg11BGqxH/JZUQsT4 jXC5Ab0R+EyDRpDQgdqcrGHbg+7dx6WX3k02PcJutzTWNZmKXi27Ku1Zk7n8vz6kj/aaEB2wTXbWw G/Pp3RD8T2B/w68TB1JQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZ704-00BRCs-0k; Tue, 29 Mar 2022 08:14:48 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nZ6zz-00BRB5-Qz for linux-arm-kernel@lists.infradead.org; Tue, 29 Mar 2022 08:14:45 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1648541683; x=1680077683; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=8ynEFQsC8hbL57VrTSrxKWhCNhcRSSU9o87WmR6LjsI=; b=IIxc52lG5VcIPXR1SsKhX1rzDLko56eLh5NVaiG+fEW9TmhMKaQP1hyt JOsp/HDe4AqE85HZ8rAR9mLQlXsBFWx2U0JXh7KfoakTy2FlHH7YlKZDx f5dWPAuVUrYirnjnIBtuWsFa1rdDKwEVm2y32SNCMrAInEQ2uVEZhd3cP EHrciZqwGiMKGNsLwIJmInfZP2vXFPvj5fcA951TizRO1ZbEOP6O9ZueA rojW24nzQwtyR8VzkA30JY0aG0hfxQtPmhMe9fLMru7r5RO7EFF1Xxld0 2nikl0iGbz+tlcM5meBS35FQiWAXny6GVL3wTrTUh189Emdm+AtS8ZrdG Q==; X-IronPort-AV: E=Sophos;i="5.90,219,1643670000"; d="scan'208";a="22952252" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 29 Mar 2022 10:14:39 +0200 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Tue, 29 Mar 2022 10:14:39 +0200 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Tue, 29 Mar 2022 10:14:39 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1648541679; x=1680077679; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=8ynEFQsC8hbL57VrTSrxKWhCNhcRSSU9o87WmR6LjsI=; b=DyNOriQ8xNoiPX04hq2gOmXCeQ9ilxAfIVpqdhvhD9s7Rg0BYtoYc/ju aGQEdz01NBbgLK6WPC8WDJv7lGG4aTLa384GhP5wFqnJ4ixWuLA1ZsN5S Fgu4MsxZ21Z0rauJDruCAiohTWt0HsoeL/9vJoo4cHHtldyVZNusbNqcs xbzh1BB9AmwC2qrjNEojbcvCFAwbO/dBD6OltPzrCE8KarkndzDq3Dumh shy8JDKZzg/ktWrf8w4WWmJH+F9EgpX487hXmk40O73dtgpR/CVifrh8q 9uU+16tzaXF/e0RVqzwGhsszRSNxj0OvHchlgW0erNMMVBRyMWi+TQ8MS g==; X-IronPort-AV: E=Sophos;i="5.90,219,1643670000"; d="scan'208";a="22952251" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 29 Mar 2022 10:14:39 +0200 Received: from steina-w.localnet (unknown [10.123.49.12]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id E94F9280065; Tue, 29 Mar 2022 10:14:38 +0200 (CEST) From: Alexander Stein To: Peter Chen Cc: Peter Chen , Greg Kroah-Hartman , Shawn Guo , Sascha Hauer , Fabio Estevam , Pengutronix Kernel Team , NXP Linux Team , USB list , linux-arm-kernel@lists.infradead.org Subject: Re: (EXT) Re: [RFC 1/1] usb: chipidea: ci_hdrc_imx: disable runtime pm for HSIC interface Date: Tue, 29 Mar 2022 10:14:36 +0200 Message-ID: <1891703.PYKUYFuaPT@steina-w> Organization: TQ-Systems GmbH In-Reply-To: References: <20220302094239.3075014-1-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220329_011444_275368_1A019A83 X-CRM114-Status: GOOD ( 19.60 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Peter, Am Dienstag, 15. M=E4rz 2022, 02:23:23 CEST schrieb Peter Chen: > On Wed, Mar 2, 2022 at 5:42 PM Alexander Stein > = > wrote: > > With the add of power-domain support in commit 02f8eb40ef7b ("ARM: dts: > > imx7s: Add power domain for imx7d HSIC") runtime suspend will disable > > the power-domain. This prevents IRQs to occur when a new device is > > attached > > on a downstream hub. > > = > > Signed-off-by: Alexander Stein > > --- > > Our board TQMa7x + MBa7x (i.MX7 based) uses a HSIC link to mounted USB = HUB > > on usbh device. Cold plugging an USB mass storage device is working fin= e. > > But once the last non-HUB device is disconnected the ci_hdrc device goes > > into runtime suspend. > = > Would you please show the difference between cold boot and runtime > suspend after disconnecting > the last USB device? > = > - Power domain on/off status for HUB device > - Runtime suspend status at /sys entry for HUB device > - "/sys/..power/wakeup" /sys entry for HUB device I hope I got all entries you requested. For reference this is the bus topology: lsusb -t /: Bus 02.Port 1: Dev 1, Class=3Droot_hub, Driver=3Dci_hdrc/1p, 480M /: Bus 01.Port 1: Dev 1, Class=3Droot_hub, Driver=3Dci_hdrc/1p, 480M |__ Port 1: Dev 2, If 0, Class=3DHub, Driver=3Dhub/4p, 480M |__ Port 2: Dev 3, If 0, Class=3DMass Storage, Driver=3Dusb-storage= , 480M Bus 2 is a different connector and doesn't matter here. I'm disconnecting '= Dev = 3' in this scenario. After boot up with the bus as shown above: $ cat /sys/bus/usb/devices/1-1/power/wakeup disabled $ cat /sys/bus/usb/devices/1-1/power/runtime_status active $ cat /sys/kernel/debug/pm_genpd/usb-hsic-phy/current_state on After disconnecting Dev 3 from the bus ('usb 1-1.2: USB disconnect, device = number 3' in dmesg) the status changes as follows (without the patch): $ cat /sys/bus/usb/devices/1-1/power/wakeup disabled $ cat /sys/bus/usb/devices/1-1/power/runtime_status suspended $ cat /sys/kernel/debug/pm_genpd/usb-hsic-phy/current_state off-0 For the record, when applying the posted patch this changes into: $ cat /sys/bus/usb/devices/1-1/power/wakeup disabled $ cat /sys/bus/usb/devices/1-1/power/runtime_status suspended $ cat /sys/kernel/debug/pm_genpd/usb-hsic-phy/current_state on Regards, Alexander _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel