Hi Rajeshwari,
On Fri, Sep 20, 2013 at 3:32 AM, Rajeshwari Birje <
rajeshwari.bi...@gmail.com> wrote:
> Hi Simon
>
> Thank you for coments.
>
> Regards,
> Rajeshwari Shinde
>
> On Thu, Sep 19, 2013 at 11:27 AM, Simon Glass <s...@chromium.org> wrote:
>
>> Hi Rajeshwari,
>>
>> On Wed, Sep 11, 2013 at 4:01 AM, Rajeshwari S Shinde <
>> rajeshwar...@samsung.com> wrote:
>>
>> > From: Akshay Saraswat <aksha...@samsung.com>
>> >
>> > Currently, part of TZPC init code for Exynos5 starts setting DECPROT
>> > from the base address 0x10100000 upto 0x10190000 but in case of
>> > Exynos5420 we need it to start from 0x0100E0000 and keep end address
>> > same as 0x10190000.
>> >
>> > Signed-off-by: Rajeshwari S Shinde <rajeshwar...@samsung.com>
>> > Signed-off-by: Akshay Saraswat <aksha...@samsung.com>
>> > ---
>> > Changes in V2:
>> > - None
>> > Changes in V3:
>> > - None
>> > arch/arm/cpu/armv7/exynos/tzpc.c | 7 ++++++-
>> > 1 file changed, 6 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/arch/arm/cpu/armv7/exynos/tzpc.c
>> > b/arch/arm/cpu/armv7/exynos/tzpc.c
>> > index 395077c..1102596 100644
>> > --- a/arch/arm/cpu/armv7/exynos/tzpc.c
>> > +++ b/arch/arm/cpu/armv7/exynos/tzpc.c
>> > @@ -23,7 +23,12 @@ void tzpc_init(void)
>> > else if (cpu_is_exynos4())
>> > end = start + ((EXYNOS4_NR_TZPC_BANKS - 1) *
>> > TZPC_BASE_OFFSET);
>> >
>> > - for (addr = start; addr <= end; addr += TZPC_BASE_OFFSET) {
>> > + if (proid_is_exynos5420())
>> > + addr = start - 0x20000;
>> >
>>
>> Shouldn't this be in a #define somewhere?
>>
> Instead of having a #define we get the start address via
> samsung_get_base_tzpc, since there exception in case of 5420 where we need
> to start from 0x100E0000 we put a if condition.
>
Does that mean that the address returned by samsung_get_base_tzpc() should
be different for 5250 and 5420?
>>
>> > + else
>> > + addr = start;
>> > +
>> > + for (; addr <= end; addr += TZPC_BASE_OFFSET) {
>> > tzpc = (struct exynos_tzpc *)addr;
>> >
>> > if (addr == start)
>> > --
>> > 1.7.12.4
>> >
>> >
>>
>> Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
