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 X-Spam-Level: X-Spam-Status: No, score=-4.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7430DC4363A for ; Tue, 27 Oct 2020 10:57:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1626B22263 for ; Tue, 27 Oct 2020 10:57:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1603796225; bh=HzIkvC83+y+GZokmoVZEDhfMbPbSNT/Gp0cxE0BYOIo=; h=From:To:Cc:Subject:In-Reply-To:References:Date:List-ID:From; b=sbv1E38UXvm09ThMvuWcnNbuhTHcF9PFssIlhIQqW0C7a17SGGt+P1S6lgf+TBKb8 xooIkgvhDd0GBlxxMl/FYlcUeHIIEZBclS79Y2djTc5PQIhRwDnhkEpWvkVbGTvrjw OmLiqkvxwMKyOdgfa8SLW0T0cZOUoE05oZWVwLL0= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729393AbgJ0K5E (ORCPT ); Tue, 27 Oct 2020 06:57:04 -0400 Received: from mail.kernel.org ([198.145.29.99]:37900 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725811AbgJ0K5E (ORCPT ); Tue, 27 Oct 2020 06:57:04 -0400 Received: from saruman (88-113-213-94.elisa-laajakaista.fi [88.113.213.94]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 9871A2224E; Tue, 27 Oct 2020 10:57:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1603796223; bh=HzIkvC83+y+GZokmoVZEDhfMbPbSNT/Gp0cxE0BYOIo=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=bbDpiDwbQi0UUVH+z6bPC9RcCJ28fi3s7MxPxRyHQq9f1Kc99Nvn1xhoddukknhgt vwtTuTUkYBf7eYhKPsocuwWMNPoqaWjzO9qDOI3RWoKbdNwEgTHjkGnElJdP5BrgJa VrZm4Ku77aIu/IASpngKl8HkzClie2pmnESl+1Po= From: Felipe Balbi To: Artur Petrosyan , Minas Harutyunyan , Greg Kroah-Hartman , "linux-usb@vger.kernel.org" Cc: John Youn , Douglas Anderson Subject: Re: [PATCH v2 00/15] usb: dwc2: Fix and improve power saving modes. In-Reply-To: <0247ace2-841d-1044-37cd-5ec40997bbab@synopsys.com> References: <20201011135035.7C61DA005E@mailhost.synopsys.com> <2edb35af-bf0b-bfab-69c7-9b7f0cac6a69@synopsys.com> <875z6wc8z3.fsf@kernel.org> <0247ace2-841d-1044-37cd-5ec40997bbab@synopsys.com> Date: Tue, 27 Oct 2020 12:56:59 +0200 Message-ID: <87zh48aqdg.fsf@kernel.org> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org Hi, Artur Petrosyan writes: > Hi Felipe, > > On 10/27/2020 13:29, Felipe Balbi wrote: >> >> Hi, >> >> Minas Harutyunyan writes: >>> On 10/11/2020 5:50 PM, Artur Petrosyan wrote: >>>> This patch set fixes/improves partial power down, hibernation power >>>> saving modes and adds support for host/device clock gating. >>>> >>>> Changes from V1: >>>> - Added new patches. >>>> 1. usb: dwc2: Add support for dwc2 host/device clock gating. >>>> 2. usb: dwc2: Add exit power saving mode before removing driver. >>>> 3. usb: dwc2: Fix HPRT0.PrtSusp bit setting for HiKey 960 board. >>>> >>>> - Updated patches >>>> 1. usb: dwc2: Fix/update enter/exit partial power down. >>>> - Updated the patch name from "usb: dwc2: Fix >>>> dwc2_restore_device_registers() function." to "usb: dwc2: >>>> Fix/update enter/exit partial power down." >>>> - Updated entering and exiting partial power down function >>>> flow. Before there was a lot of confusions with core >>>> entering to partial power down in device or host mode. >>>> - Added "rem_wakeup" for host exiting from Partial Power >>>> Down mode from host remote wakeup flow. According to >>>> programming guide in host mode, port power must be >>>> turned on when wakeup is detected. >>>> - Added "in_ppd" flag to indicate the core state after >>>> entering into Partial Power Down mode. >>>> 2. usb: dwc2: Fix wakeup detected and session request interrupt handlers. >>>> - According to programming guide added partial power >>>> down exit flow in wakeup detected interrupt handler. >>>> - Added clock gating exit flow from wakeup detected >>>> and session request interrupt handlers. >>>> 3. usb: dwc2: Fix suspend state in host mode for partial power down. >>>> - Added "dwc2_port_suspend" and "dwc2_port_resume" functions to >>>> "core.h" header file >>>> - Updated "USB_PORT_FEAT_RESET" flow when core receives port >>>> reset in Partial Power Down or Hibernation state. >>>> - Added return "-ENODEV" if core is in suspend state >>>> in "_dwc2_hcd_urb_enqueue" function to avoid port >>>> reset issue, when an external hub is connected. >>>> 4. usb: dwc2: Add part. power down exit from dwc2_conn_id_status_change(). >>>> - Instead of clearing registers inline for exiting partial power down >>>> mode, now calling the "dwc2_exit_partial_power_down" function without >>>> restoring the backup registers. >>>> 5. usb: dwc2: Update dwc2_handle_usb_suspend_intr function. >>>> - Added changes suggested by Douglas Anderson from commit >>>> "usb: dwc2: Get rid of useless error checks for >>>> hibernation/partial power down" >>>> 6. usb: dwc2: Fix hibernation between host and device modes. >>>> - Added setting of "hsotg->bus_suspended" flag to 0 because before if >>>> core exited from "GPWRDN_STS_CHGINT" interrupt in host mode the flag >>>> remained true not letting enter to hibernation next time. >>>> 7. usb: dwc2: Clear fifo_map when resetting core. >>>> - Added "static inline void dwc2_clear_fifo_map()" helper >>>> function to clear fifo_map with peripheral or dual role mode. >>>> - Added a dummy version of "dwc2_clear_fifo_map()" helper >>>> for host-only mode. >>>> 8. usb: dwc2: Add power saving mode support from system issued suspend/resume. >>>> - Updated the patch name from "usb: dwc2: Add enter/exit hibernation >>>> from system scheduled suspend" to "usb: dwc2: Add power saving mode >>>> support from system issued suspend/resume." >>>> - Fixed issue related to "spinlock already unlocked" reported by Doug >>>> Anderson. >>>> >>>> - Abandoned patches. >>>> 1. usb: dwc2: Add port conn. sts. checking in _dwc2_hcd_resume() function. >>>> - The changes of this patch are moved to the "usb: dwc2: Add power >>>> saving mode support from system issued suspend/resume" patch. >>>> 2. usb: dwc2: Add flag and debug messages for Partial Power Down mode. >>>> - The changes of this patch are moved to the "usb: dwc2: Fix/update >>>> enter/exit partial power down." patch. >>>> >>>> >>>> Artur Petrosyan (15): >>>> usb: dwc2: Fix/update enter/exit partial power down. >>>> usb: dwc2: Add support for dwc2 host/device clock gating. >>>> usb: dwc2: Fix wakeup detected and session request interrupt handlers. >>>> usb: dwc2: Fix suspend state in host mode for partial power down. >>>> usb: dwc2: Add part. power down exit from >>>> dwc2_conn_id_status_change(). >>>> usb: dwc2: Reset DEVADDR after exiting gadget hibernation. >>>> usb: dwc2: Add default param to control power optimization. >>>> usb: dwc2: Update dwc2_handle_usb_suspend_intr function. >>>> usb: dwc2: Fix hibernation between host and device modes. >>>> usb: dwc2: Fix HPRT0.PrtSusp bit setting for HiKey 960 board. >>>> usb: dwc2: Allow exiting hibernation from gpwrdn rst detect >>>> usb: dwc2: Clear fifo_map when resetting core. >>>> usb: dwc2: Clear GINTSTS_RESTOREDONE bit after restore is generated. >>>> usb: dwc2: Add power saving mode support from system issued >>>> suspend/resume >>>> usb: dwc2: Add exit power saving mode before removing driver >> >> where are all the patches? I only got the cover letter. When resending, >> please collect Minas' acked-by. >> > > I am not sure how to answer to the question. The patch series including > all the patches has been successfully sent to LKML. My colleagues has > received them. Didn't you get e-mail? because I can see that your e-mail > address is listed in "To" list of all the patches in the series. oh, I see what happened, this is the same series where patches were not sent as a reply to the cover letter. Now I understand. > Also related to collection of Minas' acked-by. Do I need to add the > acked-by tag to all the patches or only in the 0 patch? Because the > acked-by tag is now done for only the 0 patch. Every patch. Patch 0 doesn't go into git ;-) -- balbi