RE: [PATCH] char/tpm: Use struct dev_pm_ops for power management

2012-12-10 Thread Mathias LEBLANC
de...@lists.sourceforge.net; Kent Yoder Subject: Re: [PATCH] char/tpm: Use struct dev_pm_ops for power management On Thu, Dec 06, 2012 at 01:20:51AM +0100, Peter Huewe wrote: > This patch converts the suspend and resume functions for > tpm_i2c_stm_st33 to the new dev_pm_ops. Ignore my la

Re: [PATCH] char/tpm: Use struct dev_pm_ops for power management

2012-12-07 Thread Peter Hüwe
Am Donnerstag, 6. Dezember 2012, 17:27:02 schrieb Kent Yoder: > On Thu, Dec 06, 2012 at 01:20:51AM +0100, Peter Huewe wrote: > > This patch converts the suspend and resume functions for > > tpm_i2c_stm_st33 to the new dev_pm_ops. > > > > Signed-off-by: Peter Huewe > > One minor tweak, the PM f

Re: [PATCH] char/tpm: Use struct dev_pm_ops for power management

2012-12-06 Thread Kent Yoder
On Thu, Dec 06, 2012 at 01:20:51AM +0100, Peter Huewe wrote: > This patch converts the suspend and resume functions for > tpm_i2c_stm_st33 to the new dev_pm_ops. > > Signed-off-by: Peter Huewe One minor tweak, the PM funcs need to be inside CONFIG_PM_SLEEP to avoid warnings when compiled witho

Re: [PATCH] char/tpm: Use struct dev_pm_ops for power management

2012-12-06 Thread Kent Yoder
On Thu, Dec 06, 2012 at 01:20:51AM +0100, Peter Huewe wrote: > This patch converts the suspend and resume functions for > tpm_i2c_stm_st33 to the new dev_pm_ops. Ignore my last mail. I'll take a look at this. Thanks, Kent > Signed-off-by: Peter Huewe > --- > Note: > I'm sending this patch on

RE: [PATCH] char/tpm: Use struct dev_pm_ops for power management.

2012-08-07 Thread Peter.Huewe
Hi Kent, > Thanks Peter. One more request - can you roll this fix into the driver > patch itself and just add a note in the change log? Sorry I didn't > mention this before. Yes I'll do that. And while at it I'll also replace our i2c_transfer_nolock with the new (in 3.6rc-1) __i2c_transfer func

Re: [PATCH] char/tpm: Use struct dev_pm_ops for power management.

2012-08-06 Thread Kent Yoder
On Mon, Aug 06, 2012 at 09:58:59AM +0200, Peter Huewe wrote: > Make the tpm_i2c_infineon driver define its PM callbacks trough a > struct dev_pm_ops by using SIMPLE_DEV_PM_OPS instead of coding it > explicitly. > > This simplifies the code and allows the driver to use tpm_pm_suspend() > and tpm_pm