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

["Jingoo Han" <jingoohan1@gmail.com>]

On Friday, July 31, 2015 9:25 PM, 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

Hi Kishon,

Looks good! :-)

Reviewed-by: Jingoo Han <jingoohan1@gmail.com>

Best regards,
Jingoo Han

> 
>  drivers/pci/host/pci-dra7xx.c |   95 ++++++++++++++++++++++++++++++++++++++++-
>  1 file changed, 94 insertions(+), 1 deletion(-)
> 
> --
> 1.7.9.5