linux-kernel - Re: [PATCH v5 0/3] dra7xx: Add PM support to PCIe

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <55CADC62.7060007@ti.com>
Date:	Wed, 12 Aug 2015 11:10:50 +0530
From:	Kishon Vijay Abraham I <kishon@...com>
To:	Bjorn Helgaas <bhelgaas@...gle.com>
CC:	<linux-omap@...r.kernel.org>, <linux-pci@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <jingoohan1@...il.com>,
	<nsekhar@...com>
Subject: Re: [PATCH v5 0/3] dra7xx: Add PM support to PCIe



On Wednesday 12 August 2015 02:22 AM, Bjorn Helgaas wrote:
> On Fri, Jul 31, 2015 at 05:55:09PM +0530, Kishon Vijay Abraham I wrote:
>> This series adds PM support to pci-dra7xx so that PCI clocks can be disabled
>> during suspend and enabled back during resume without affecting
>> PCI functionality.
>>
>> Changes from v4:
>> *) Fixed a bug caused by sending incomplete patch.
>>
>> Changes from v3:
>> *) Fix compilation errors when individual patches are applied
>>
>> Changes from v2:
>> *) Used SET_SYSTEM_SLEEP_PM_OPS and SET_NOIRQ_SYSTEM_SLEEP_PM_OPS for
>>    populating PM ops.
>>
>> Changes from v1:
>> *) Moved resetting and setting of MSE bit to pci-dra7xx.
>>
>> The comment to reset and set ISE is not done now since I don't have a card
>> with IO space. Once I get to test that, I'll post a separate patch for
>> handling that.
>>
>> Kishon Vijay Abraham I (3):
>>   PCI: host: pci-dra7xx: Disable pm_runtime on get_sync failure
>>   PCI: host: pci-dra7xx: add pm support to pci dra7xx
>>   PCI: host: pci-dra7xx: Idle the module by disabling MSE bit
>>
>>  drivers/pci/host/pci-dra7xx.c |   95 ++++++++++++++++++++++++++++++++++++++++-
>>  1 file changed, 94 insertions(+), 1 deletion(-)
> 
> Applied with Jingoo's Reviewed-by to pci/host-dra7xx for v4.3, thanks!
> 
> I tweaked the changelogs as follows:
> 
> 
> commit 0e2bdb0e7abf4b5170874e415ec42df547916dd3
> Author: Kishon Vijay Abraham I <kishon@...com>
> Date:   Fri Jul 31 17:55:10 2015 +0530
> 
>     PCI: dra7xx: Disable pm_runtime on get_sync failure
>     
>     Fix the error handling when pm_runtime_get_sync() fails.
>     
>     If pm_runtime_get_sync() fails, call pm_runtime_disable() so there are no
>     unbalanced pm_runtime_enable() calls.
>     
>     Signed-off-by: Kishon Vijay Abraham I <kishon@...com>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@...gle.com>
>     Reviewed-by: Jingoo Han <jingoohan1@...il.com>
> 
> commit e52eb445ea1d97bf7fb92d2297e487a305392136
> Author: Kishon Vijay Abraham I <kishon@...com>
> Date:   Fri Jul 31 17:55:11 2015 +0530
> 
>     PCI: dra7xx: Add PM support
>     
>     Add PM support to pci-dra7xx so PCI clocks can be disabled during suspend
>     and enabled during resume without affecting PCI functionality.
>     
>     Signed-off-by: Kishon Vijay Abraham I <kishon@...com>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@...gle.com>
>     Reviewed-by: Jingoo Han <jingoohan1@...il.com>
> 
> commit 389c7094ec241ee8ebe358ba10fe392018692c8c
> Author: Kishon Vijay Abraham I <kishon@...com>
> Date:   Fri Jul 31 17:55:12 2015 +0530
> 
>     PCI: dra7xx: Clear MSE bit during suspend so clocks will idle
>     
>     DRA7xx requires the MSE bit to be cleared to set the master in standby
>     mode.  (In DRA7xx TRM_vE, section 24.9.4.5.2.2.1 PCIe Controller Master
>     Standby Behavior advises to use the clearing of the local MSE bit to set
>     the master in standby.  Without this some of the clocks do not idle).
>     
>     Clear the MSE bit on suspend and enable it on resume.  Clearing MSE bit is
>     required to get clocks to be idled after suspend.
>     
>     Signed-off-by: Kishon Vijay Abraham I <kishon@...com>
>     Signed-off-by: Sekhar Nori <nsekhar@...com>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@...gle.com>
>     Reviewed-by: Jingoo Han <jingoohan1@...il.com>

Thanks Bjorn!

-Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/