Re: [PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-06-23 Thread Maulik Shah
Reviewed-by: Maulik Shah Thanks, Maulik On 5/28/2020 8:18 PM, Douglas Anderson wrote: The write_tcs_reg_sync() may be called after timekeeping is suspended so it's not OK to use ktime. The readl_poll_timeout_atomic() macro implicitly uses ktime. This was causing a warning at suspend time.

Re: [PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-06-18 Thread Doug Anderson
Bjorn and Andy, On Thu, May 28, 2020 at 7:48 AM Douglas Anderson wrote: > > The write_tcs_reg_sync() may be called after timekeeping is suspended > so it's not OK to use ktime. The readl_poll_timeout_atomic() macro > implicitly uses ktime. This was causing a warning at suspend time. > > Change

Re: [PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-05-29 Thread Doug Anderson
Hi, On Thu, May 28, 2020 at 3:44 PM Stephen Boyd wrote: > > Quoting Douglas Anderson (2020-05-28 07:48:34) > > The write_tcs_reg_sync() may be called after timekeeping is suspended > > so it's not OK to use ktime. The readl_poll_timeout_atomic() macro > > implicitly uses ktime. This was

Re: [PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-05-28 Thread Stephen Boyd
Quoting Douglas Anderson (2020-05-28 07:48:34) > The write_tcs_reg_sync() may be called after timekeeping is suspended > so it's not OK to use ktime. The readl_poll_timeout_atomic() macro > implicitly uses ktime. This was causing a warning at suspend time. > > Change to just loop 100 times

[PATCH] soc: qcom: rpmh-rsc: Don't use ktime for timeout in write_tcs_reg_sync()

2020-05-28 Thread Douglas Anderson
The write_tcs_reg_sync() may be called after timekeeping is suspended so it's not OK to use ktime. The readl_poll_timeout_atomic() macro implicitly uses ktime. This was causing a warning at suspend time. Change to just loop 100 times with a delay of 1 us between loops. This may give a