* Konstantin Baydarov <kbaidarov@xxxxxxxxxxxxx> [120620 07:18]: > > So,the same API set (omap_control_readl/omap_control_writel) was added to omap-control-core.c. > > If omap-control-core.c should only service users from driver/ directory, than I agree - we can remove > omap_control_readl/omap_control_writel from omap-control-core.c. > But IIUC you are agree to "switch" arch/arm/mach-omap2/id.c from control.c to omap-control-core.c. If arch/arm/mach-omap2/id.c is switched to control.c, then I guess all arch/arm/mach-omap2/*.c should be "switched" to omap-control-core.c as well. But this means that omap-control-core.c should provide omap_control_readl/omap_control_writel API. Can't you just add something like int omap_ctrl_read_status(void) that id.c can use? Tony