Quoting Bjorn Andersson (2018-04-10 17:01:20) > On Thu 05 Apr 09:18 PDT 2018, Lina Iyer wrote: > > > > > diff --git a/drivers/soc/qcom/rpmh.c b/drivers/soc/qcom/rpmh.c > > new file mode 100644 > > index 000000000000..e3c7491e7baf > > --- /dev/null > > +++ b/drivers/soc/qcom/rpmh.c > > @@ -0,0 +1,253 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright (c) 2016-2018, The Linux Foundation. All rights reserved. > > + */ > > + > > +#include <linux/atomic.h> > > +#include <linux/interrupt.h> > > +#include <linux/jiffies.h> > > +#include <linux/kernel.h> > > +#include <linux/mailbox_client.h> > > +#include <linux/module.h> > > +#include <linux/of.h> > > +#include <linux/platform_device.h> > > +#include <linux/slab.h> > > +#include <linux/types.h> > > +#include <linux/wait.h> > > + > > +#include <soc/qcom/rpmh.h> > > + > > +#include "rpmh-internal.h" > > + > > +#define RPMH_MAX_MBOXES 2 > > +#define RPMH_TIMEOUT_MS 10000 > > Just define this in jiffies and you don't need to do msecs_to_jiffies() > every time you use it. > Put the msecs_to_jiffies() here if you want. The compiler will constant fold it to the final value, and then we get the same time in seconds no matter what the jiffies frequency is configured for. -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html