Re: [U-Boot] [U-Boot, v2] arm: omap: Unify get_device_type() function

On Fri, Jun 02, 2017 at 06:00:00PM +0300, Semen Protsenko wrote:

Refactor OMAP3/4/5 code so that we have only one get_device_type() function for all platforms.

Details:

• Add ctrl variable for AM33xx and OMAP3 platforms (like it's done for OMAP4/5), so we can obtain status register in common way
• For now ctrl structure for AM33xx/OMAP3 contains only status register address
• Run hw_data_init() in order to assign ctrl to proper structure
• Remove DEVICE_MASK and DEVICE_GP definitions as they are not used (DEVICE_TYPE_MASK and GP_DEVICE are used instead)
• Guard structs in omap_common.h with #ifndefs, because otherwise including omap_common.h in am33xx board files breaks compilation

Buildman script was run for all OMAP boards. Result output: arm: (for 38/616 boards) all +352.5 bss -1.4 data +3.5 rodata +300.0 spl/u-boot-spl:all +284.7 spl/u-boot-spl:data +2.2 spl/u-boot-spl:rodata +252.0 spl/u-boot-spl:text +30.5 text +50.4 (no errors to report)

Tested on AM57x EVM and BeagleBoard xM.

Signed-off-by: Sam Protsenko semen.protsenko@linaro.org Reviewed-by: Lokesh Vutla lokeshvutla@ti.com

Applied to u-boot/master, thanks!