Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-24 Thread Mike Looijmans
On 23-11-15 23:02, Moritz Fischer wrote: Hi Mike, thanks for your feedback. I put what I think you suggested inline below. On Thu, Nov 19, 2015 at 11:25 PM, Mike Looijmans wrote: On 19-11-15 23:07, Moritz Fischer wrote: @@ -457,19 +457,26 @@ static int zynq_fpga_probe(struct

Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-24 Thread Mike Looijmans
On 23-11-15 23:02, Moritz Fischer wrote: Hi Mike, thanks for your feedback. I put what I think you suggested inline below. On Thu, Nov 19, 2015 at 11:25 PM, Mike Looijmans wrote: On 19-11-15 23:07, Moritz Fischer wrote: @@ -457,19 +457,26 @@ static int

Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-23 Thread Moritz Fischer
Hi Mike, thanks for your feedback. I put what I think you suggested inline below. On Thu, Nov 19, 2015 at 11:25 PM, Mike Looijmans wrote: > On 19-11-15 23:07, Moritz Fischer wrote: >> @@ -457,19 +457,26 @@ static int zynq_fpga_probe(struct platform_device >> *pdev) >> return

Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-23 Thread Moritz Fischer
Hi Mike, thanks for your feedback. I put what I think you suggested inline below. On Thu, Nov 19, 2015 at 11:25 PM, Mike Looijmans wrote: > On 19-11-15 23:07, Moritz Fischer wrote: >> @@ -457,19 +457,26 @@ static int zynq_fpga_probe(struct platform_device >> *pdev) >>

Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-19 Thread Mike Looijmans
On 19-11-15 23:07, Moritz Fischer wrote: Replaced constant clock_{enable,disable} calls with pm_runtime hooks for suspend and resume to avoid constant clk_enable / clk_disable. Acked-by: Alan Tull Signed-off-by: Moritz Fischer --- Changes: v1: - Removed superfluous #ifdef CONFIG_PM as

[PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-19 Thread Moritz Fischer
Replaced constant clock_{enable,disable} calls with pm_runtime hooks for suspend and resume to avoid constant clk_enable / clk_disable. Acked-by: Alan Tull Signed-off-by: Moritz Fischer --- Changes: v1: - Removed superfluous #ifdef CONFIG_PM as suggested by Michal - Changed commit message

[PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-19 Thread Moritz Fischer
Replaced constant clock_{enable,disable} calls with pm_runtime hooks for suspend and resume to avoid constant clk_enable / clk_disable. Acked-by: Alan Tull Signed-off-by: Moritz Fischer --- Changes: v1: - Removed superfluous #ifdef

Re: [PATCH v2] fpga: zynq-fpga: Enable pm_runtime (suspend, resume)

2015-11-19 Thread Mike Looijmans
On 19-11-15 23:07, Moritz Fischer wrote: Replaced constant clock_{enable,disable} calls with pm_runtime hooks for suspend and resume to avoid constant clk_enable / clk_disable. Acked-by: Alan Tull Signed-off-by: Moritz Fischer ---