[U-Boot] [PATCH 01/10 V6] EXYNOS5: Create a common board file

[Rajeshwari Birje]

Hi Przemyslaw

On Fri, Nov 8, 2013 at 1:43 PM, Przemyslaw Marczak
<p.marczak at samsung.com> wrote:
> Hello Rajeshwari,
>
>
> On 10/29/2013 08:23 AM, Rajeshwari S Shinde wrote:
>>
>> Create a common board.c file for all functions which are common across
>> all EXYNOS5 platforms.
>>
>> exynos_init function is provided for platform specific code.
>>
>> Signed-off-by: Rajeshwari S Shinde <rajeshwari.s at samsung.com>
>> Acked-by: Simon Glass <sjg at chromium.org>
>> ---
>> Changes in V2:
>>         - None
>> Changes in V3:
>>         - None
>> Changes in V4:
>>         - Added check for the compilation of MAX77686 pmic.
>> Changes in V5:
>>         - Moved board_eth_init and board_mmc_init in case of
>>         device tree support
>> Changes in V6:
>>         - None.
>>   arch/arm/include/asm/arch-exynos/board.h |  17 ++
>>   board/samsung/common/Makefile            |   4 +
>>   board/samsung/common/board.c             | 405
>> +++++++++++++++++++++++++++++++
>>   board/samsung/smdk5250/exynos5-dt.c      | 361
>> +--------------------------
>>   board/samsung/smdk5250/smdk5250.c        | 182 +-------------
>>   include/configs/exynos5250-dt.h          |   2 +
>>   6 files changed, 435 insertions(+), 536 deletions(-)
>>   create mode 100644 arch/arm/include/asm/arch-exynos/board.h
>>   create mode 100644 board/samsung/common/board.c
>>
>
> The name of exynos5 common file can't be just "board.c" since it is common
> only for Exynos5 boards and is placed in Samsung common.
> We have few Exynos4 boards in u-boot and in future we can introduce common
> board file for them too.
> So it is better to add some additional word to this file name. This also
> needs to change config name or add checking SOC version in common Makefile.
>
Cannot we use the same file even for exynos4 boards.

-- 
Regards,
Rajeshwari Shinde
> Thank you,
> --
> Przemyslaw Marczak
> Samsung R&D Institute Poland
> Samsung Electronics
> p.marczak at samsung.com
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot