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

5 Jun 2017


      On Friday 02 June 2017 08:30 PM, Sam 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
Will you be able to test on BeagleBoneBlack as well?
Thanks and regards,
Lokesh