Linus Walleij wrote at Friday, September 16, 2011 6:14 AM:
This adds a driver for the U300 pinmux portions of the system controller "SYSCON". It also serves as an example of how to use the pinmux subsystem. This driver also houses the platform data for the only supported platform.
...
diff --git a/arch/arm/mach-u300/core.c b/arch/arm/mach-u300/core.c
...
+/* Pinmux settings */ +static struct pinmux_map u300_pinmux_map[] = {
- /* anonymous maps for chip power and EMIFs */
- PINMUX_MAP_PRIMARY_SYS_HOG("POWER", "power"),
- PINMUX_MAP_PRIMARY_SYS_HOG("EMIF0", "emif0"),
- PINMUX_MAP_PRIMARY_SYS_HOG("EMIF1", "emif1"),
- /* per-device maps for MMC/SD, SPI and UART */
- PINMUX_MAP_PRIMARY("MMCSD", "mmc0", "mmci"),
- PINMUX_MAP_PRIMARY("SPI", "spi0", "pl022"),
- PINMUX_MAP_PRIMARY("UART0", "uart0", "uart0"),
+};
+struct u300_mux_hog {
- const char *name;
- struct device *dev;
- struct pinmux *pmx;
+};
+static struct u300_mux_hog u300_mux_hogs[] = {
- {
.name = "uart0",.dev = &uart0_device.dev,- },
- {
.name = "spi0",.dev = &pl022_device.dev,- },
- {
.name = "mmc0",.dev = &mmcsd_device.dev,- },
+};
+static int __init u300_pinmux_fetch(void) +{
- int i;
- for (i = 0; i < ARRAY_SIZE(u300_mux_hogs); i++) {
struct pinmux *pmx;int ret;pmx = pinmux_get(u300_mux_hogs[i].dev, NULL);if (IS_ERR(pmx)) {pr_err("u300: could not get pinmux hog %s\n",u300_mux_hogs[i].name);continue;}ret = pinmux_enable(pmx);if (ret) {pr_err("u300: could enable pinmux hog %s\n",u300_mux_hogs[i].name);continue;}u300_mux_hogs[i].pmx = pmx;- }
- return 0;
+} +subsys_initcall(u300_pinmux_fetch);
Why not just have the pinmux core support hogging on non-"system" mapping entries; then everything I quoted above except u300_pinmux_map[] could be deleted, and the "hog" flag set on the last 3 u300_pinmux_map[] entries.