On Mon, Jun 03, 2013 at 05:19:44PM -0700, Rohit Vaswani wrote: > + sc1_base_ptr = of_iomap(dn, 0); > + if (sc1_base_ptr) { > + writel_relaxed(0, sc1_base_ptr + VDD_SC1_ARRAY_CLAMP_GFS_CTL); > + writel_relaxed(0, sc1_base_ptr + SCSS_CPU1CORE_RESET); > + writel_relaxed(3, sc1_base_ptr + SCSS_DBG_STATUS_CORE_PWRDUP); > + mb(); > + iounmap(sc1_base_ptr); If you need to fiddle with power rails and resets for your secondary core, you don't need any of the pen_release stuff, and you really should get rid of it. The pen_release stuff is only there for platforms where there's no proper way of controlling the secondary CPUs except by using a software method. -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html