On Wed, Oct 12, 2011 at 09:54:27AM -0600, Stephen Warren wrote: > Use ioremap to obtain access to registers instead of using static > mappings. This reduces the number of users of the static mappings, which > will eventually allow them to be removed. > > Note that on Tegra30, the number of register "banks" will decrease to 2, > and the packing of specific bits into registers will change significantly. > That's why this change adds the "*_bank" fields to the pingroup tables, > rather than implementing some more hard-coded scheme. > > Also, completely remove the implementation of suspend/resume; Tegra doesn't > yet support suspend/resume, and the implementation is complex for the > general pinmux driver: > > * Not all registers are used within each bank, so we probably shouldn't > just iterate over every register in the bank, and save/restore it, > since that would mean touching undefined registers. > > * Registers are shared between pingroups, so we can't simply iterate over > each pingroup, and save/restore the registers it uses. > > It'd probably be best have probe() calculate a bitmask of actually-used > registers for each bank, and have suspend/resume iterate over those > bitmaps. > > Oh, and Real Soon Now, I should be looking into converting this driver to > the new pinmux/pinctrl subsystem, so I didn't want to put too much work > into the current incarnation. > > v2: s/space/bank/ to match comments on reg_* fields in pinmux.h. > Re-order bank/reg parameters to pg_readl/pg_writel. > > Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> Thanks, applied to for-3.2/features. -Olof -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
