[U-Boot] [PATCH 2/4 V2] SMDK5250: Convert lowlevel_init.S to lowlevel_init.c

Minkyu Kang mk7.kang at samsung.com
Mon Jan 14 08:44:32 CET 2013


On 14/01/13 15:32, Rajeshwari Birje wrote:
> Hi Minkyu Kang,
> 
>> OK, but is this something you intend to fix, perhaps in a future
>> series? It seems like you need separate directories sooner rather than
>> later.I don't think this is board-specific code, but chip-specific.
> 
> Please do let me know your opinion on the above comment.
> 
> Regards,
> Rajeshwari Shinde.
> 
> On Fri, Jan 11, 2013 at 7:54 PM, Simon Glass <sjg at chromium.org> wrote:
>> Hi Rajeshwari,
>>
>> On Fri, Jan 11, 2013 at 2:43 AM, Rajeshwari Birje
>> <rajeshwari.birje at gmail.com> wrote:
>>> Hi Simon,
>>>
>>> Thank you for comments.
>>>
>>> On Thu, Jan 10, 2013 at 11:04 PM, Simon Glass <sjg at chromium.org> wrote:
>>>> Hi Rajeshwari,
>>>>
>>>> On Mon, Jan 7, 2013 at 5:08 AM, Rajeshwari Shinde
>>>> <rajeshwari.s at samsung.com> wrote:
>>>>> This patch converts lowlevel_init.S to lowlevel_init_c.c for
>>>>> SMDK5250.
>>>>> Lowlevel.S as of now added only for SMDK5250 and same can be
>>>>> extended to other SOC in future.
>>>>
>>>> Should perhaps also mention new feature (controllable memory reset for resume?)
>>> -OK
>>>>
>>>>>
>>>>> Signed-off-by: Rajeshwari Shinde <rajeshwari.s at samsung.com>
>>>>> ---
>>>>> Changes in V2:
>>>>>         - Renamed lowlevel_init.S to lowlevel.S and moved to
>>>>>         arch/arm/cpu/armv7/exynos/
>>>>>         - Moved power mode defines to power.h
>>>>>         - Added early serial support.
>>>>>         - Renamed mem_reset to reset.
>>>>>  arch/arm/cpu/armv7/exynos/Makefile       |    6 ++
>>>>>  arch/arm/cpu/armv7/exynos/lowlevel.S     |   35 ++++++++
>>>>>  arch/arm/include/asm/arch-exynos/power.h |    8 ++
>>>>>  board/samsung/smdk5250/Makefile          |    2 +-
>>>>>  board/samsung/smdk5250/dmc_common.c      |    4 +-
>>>>>  board/samsung/smdk5250/dmc_init_ddr3.c   |    6 +-
>>>>>  board/samsung/smdk5250/lowlevel_init.S   |   96 --------------------
>>>>>  board/samsung/smdk5250/lowlevel_init.c   |   81 +++++++++++++++++
>>>>
>>>> Any change we could move all of this to arch/arm/cpu/armv7/exynos...?
>>> We do not have a separate directory for exynos5 and exynos4 and if we
>>> add all these files in arch/arm/cpu/armv7/exynos it would break the
>>> compilation for EXYNOS4. Also Later versions of exynos5 have different
>>> memory and timing variants.
>>
>> OK, but is this something you intend to fix, perhaps in a future
>> series? It seems like you need separate directories sooner rather than
>> later.I don't think this is board-specific code, but chip-specific.
>>

We don't have to separate directory for exynos5 and exynos4.
We can check this by cpu_is_exynos5 or cpu_is_exynos4.
I think each SoCs have different memory and timing but the logic can be same.
If so we can move all of changes to SoC directory.

--
Thanks,
Minkyu Kang.


More information about the U-Boot mailing list