* Rajendra Nayak <rnayak@xxxxxx> [111114 04:05]:
> --- /dev/null
> +++ b/drivers/pinctrl/pinmux-omap.c
> @@ -0,0 +1,735 @@
> +
> +/* omap4 core pads */
> +static const struct pinctrl_pin_desc omap4_core_pads[] = {
> + PINCTRL_PIN(0, "c12"),
> + PINCTRL_PIN(1, "d12"),
> + PINCTRL_PIN(2, "c13"),
...
Let's not even plan on adding this data here. We already have
about 6000 lines of pinmux data for omaps even without this patch.
Instead, just write something as a separate patch that populates
omap pinmux data from the existing data until we have the data
coming from DT.
> +/* omap4 wkup pads */
> +static const struct pinctrl_pin_desc omap4_wkup_pads[] = {
> + PINCTRL_PIN(0, "c12"),
> + PINCTRL_PIN(1, "d12"),
> + PINCTRL_PIN(2, "c13"),
...
This too needs to go.
> +static const unsigned hdmi_pins[] = {
> + 44, /* HDMI_HPD */
> + 45, /* HDMI_CEC */
> + 46, /* HDMI_DDC_SCL */
> + 47, /* HDMI_DDC_SDA */
> +};
> +
> +static const unsigned hdmi_muxmodes[] = {
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> +};
> +
> +static const unsigned sdmmc5_pins[] = {
> + 132, /* WLAN SDIO: MMC5 CMD */
> + 133, /* WLAN SDIO: MMC5 CLK */
> + 134, /* WLAN SDIO: MMC5 DAT0 */
> + 135, /* WLAN SDIO: MMC5 DAT1 */
> + 136, /* WLAN SDIO: MMC5 DAT2 */
> + 137, /* WLAN SDIO: MMC5 DAT3 */
> +};
> +
> +static const unsigned sdmmc5_muxmodes[] = {
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> + OMAP_MUX_MODE0,
> +};
> +
> +static const unsigned dispc2_pins[] = {
> + 145, /* dispc2_data23 */
> + 146, /* dispc2_data22 */
> + 147, /* dispc2_data21 */
...
And all of the above.. The same goes for all the static package
specific data.
> +/* handle 8 bit registers too, for now only worry about 16bit ones*/
> +static u16 omap_pmx_read(struct omap_pmx *opmx, unsigned pin)
> +{
> + return __raw_readw(opmx->virtbase + (pin * 2));
> +}
> +
> +static void omap_pmx_write(u16 val, struct omap_pmx *opmx, unsigned pin)
> +{
> + __raw_writew(val, opmx->virtbase + (pin * 2));
> +}
Might as well fix this one since we already have it working.
The rest looks OK to me as a minimal driver. I don't know if
we should still populate it with platform_data though, that
seems already broken as it's omap4 only. Might as well go with
the DT support to start with.
Regards,
Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
