On Fri, Apr 05, 2019 at 04:24:24PM +0100, Sean Young wrote:
> On Wed, Feb 27, 2019 at 01:16:06PM -0600, Brad Love wrote:
> + pr_info("%s(): resetting 160xxx demod\n", __func__);
> > + /* TODO: not sure this is proper place to reset once only */
> > + pvr2_issue_simple_cmd(hdw,
> > + FX2CMD_HCW_DEMOD_RESET_PIN |
> > + (1 << 8) |
> > + ((0) << 16));
> > + msleep(10);
>
> usleep_range() is preferred (for delays <= 10), I think. Maybe 10ms is too
> long anyway and msleep(1) is enough.
Sorry that was wrong. usleep_range() is preferred 1ms - 20ms, see:
https://www.kernel.org/doc/Documentation/timers/timers-howto.txt
Sean
