On Fri, 2015-06-26 at 15:44 +0800, Yuantian.Tang@freescale.com wrote:

+static void rcpm_v1_set_ip_power(bool enable, u32 *mask)
+{
+     if (enable)
+             setbits32(&rcpm_v1_regs->ippdexpcr, *mask);
+     else
+             clrbits32(&rcpm_v1_regs->ippdexpcr, *mask);
+}
+
+static void rcpm_v2_set_ip_power(bool enable, u32 *mask)
+{
+     if (enable)
+             setbits32(&rcpm_v2_regs->ippdexpcr[0], *mask);
+     else
+             clrbits32(&rcpm_v2_regs->ippdexpcr[0], *mask);
+}

Why do these take "u32 *mask" instead of "u32 mask"?

-Scott