Re: [PATCH] fpga: zynq-fpga: Avoid hammering clk_{enable/disable}.

2015-11-18 Thread atull
On Tue, 17 Nov 2015, Moritz Fischer wrote: > > Body has nothing specific what it is available only for CONFIG_PM that's > > why it should just work. > > You're right. I'll resubmit once other people took another look. Not > high priority ;-) > > Cheers, > > Moritz > Hi Moritz, Looks good t

Re: [PATCH] fpga: zynq-fpga: Avoid hammering clk_{enable/disable}.

2015-11-17 Thread Michal Simek
On 17.11.2015 18:11, Moritz Fischer wrote: > Replaced constant clock_{enable,disable} calls with pm_runtime > hooks. > > Signed-off-by: Moritz Fischer > --- > drivers/fpga/zynq-fpga.c | 79 > +--- > 1 file changed, 62 insertions(+), 17 deletions(-) >

Re: [PATCH] fpga: zynq-fpga: Avoid hammering clk_{enable/disable}.

2015-11-17 Thread Moritz Fischer
Michal, On Tue, Nov 17, 2015 at 9:53 AM, Michal Simek wrote: >> +#ifdef CONFIG_PM > > remove this Will do. > >> +static int zynq_fpga_runtime_suspend(struct device *dev) > > add __maybe_unused here Will do. >> +static int zynq_fpga_runtime_resume(struct device *dev) > > and here Will do. >>

[PATCH] fpga: zynq-fpga: Avoid hammering clk_{enable/disable}.

2015-11-17 Thread Moritz Fischer
Replaced constant clock_{enable,disable} calls with pm_runtime hooks. Signed-off-by: Moritz Fischer --- drivers/fpga/zynq-fpga.c | 79 +--- 1 file changed, 62 insertions(+), 17 deletions(-) diff --git a/drivers/fpga/zynq-fpga.c b/drivers/fpga/zynq-fpg