* Tony Lindgren <tony@xxxxxxxxxxx> [120907 10:29]: > * AnilKumar, Chimata <anilkumar@xxxxxx> [120906 20:27]: > > > @@ -1570,6 +1578,8 @@ static int serial_omap_remove(struct platform_device *dev) > > > pm_runtime_disable(&up->pdev->dev); > > > uart_remove_one_port(&serial_omap_reg, &up->port); > > > pm_qos_remove_request(&up->pm_qos_request); > > > + if (up->pins) > > > + pinctrl_put(up->pins); > > > > I think this can be removed if we use devm_pinctrl_get_select_default() > > above. > > Yeah will do, I was initially thinking that would cause issues remuxing > pins back to safe mode on unload, but we can still do that in > serial_omap_remove(). Sent now as a separate patch with subject "[PATCH] serial: omap: Request pins using pinctrl framework". 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