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.

-- 
nvpublic


_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to