Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751077AbdHRLFD (ORCPT <rfc822;w@1wt.eu>);
        Fri, 18 Aug 2017 07:05:03 -0400
Received: from mga03.intel.com ([134.134.136.65]:60013 "EHLO mga03.intel.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1750920AbdHRLFB (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 18 Aug 2017 07:05:01 -0400
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.41,392,1498546800"; 
   d="scan'208";a="891369767"
Subject: Re: [PATCH v5 1/1] usb:host:xhci support option to disable the xHCI
 USB2 HW LPM
To: "Thang Q. Nguyen" <tqnguyen@apm.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>,
        Mathias Nyman <mathias.nyman@intel.com>, linux-usb@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
References: <1502166477-23897-1-git-send-email-tqnguyen@apm.com>
 <CAKrQpSt_JuiXPFxTT7ApYFN+6u5AmreJB4PFAGzMRh12FpcnHA@mail.gmail.com>
Cc: Phong Vo <pvo@apm.com>, Loc Ho <lho@apm.com>,
        Tung Nguyen <tunguyen@apm.com>, patches <patches@apm.com>,
        Iyappan Subramanian <isubramanian@apm.com>
From: Mathias Nyman <mathias.nyman@linux.intel.com>
Message-ID: <5996CAA7.2050304@linux.intel.com>
Date: Fri, 18 Aug 2017 14:08:23 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.8.0
MIME-Version: 1.0
In-Reply-To: <CAKrQpSt_JuiXPFxTT7ApYFN+6u5AmreJB4PFAGzMRh12FpcnHA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 837
Lines: 26

On 17.08.2017 06:19, Thang Q. Nguyen wrote:
> On Tue, Aug 8, 2017 at 11:27 AM, Thang Q. Nguyen <tqnguyen@apm.com> wrote:
>> XHCI specification 1.1 does not require xHCI-compliant controllers
>> to always enable hardware USB2 LPM. However, the current xHCI
>> driver always enable it when seeing HLC=1.
>> This patch supports an option for users to control disabling
>> USB2 Hardware LPM via DT/ACPI attribute.
>> This option is needed in case user would like to disable this
>> feature. For example, their xHCI controller has its USB2 HW LPM
>> broken.
>>
>> Signed-off-by: Tung Nguyen <tunguyen@apm.com>
>> Signed-off-by: Thang Q. Nguyen <tqnguyen@apm.com>
>> Acked-by: Rob Herring <robh@kernel.org>
>> ---
...
>>
>
> Do you have any comments about this patch? Thanks.
> --

Looks good,
will send forward after 4.14-rc1

Thanks
Mathias