Hi Kevin

you might want to consider renaming 

OMAP3_PRM_VC_VDD_MPU_ID

etc. to something like:

OMAP3_VC_VDD_MPU_ID

and to define those in some vc.h header file.

Also, I'd suggest moving the struct omap_prm_vc_ops assignments out of the 
prm*.c code into the vcXXXX_data.c files, ideally into some initcall, and 
renaming the struct omap_prm_vc_ops to simply struct omap_vc_ops.

As long as the VC's low-level registers are in the PRM, those reads/writes 
should happen through PRM code, IMHO.


- Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to