From mboxrd@z Thu Jan  1 00:00:00 1970
From: Przemyslaw Marczak <p.marczak@samsung.com>
Date: Wed, 11 Dec 2013 14:48:31 +0100
Subject: [U-Boot] [PATCH 05/10] lib: tizen: add Tizen 16bpp logo support.
In-Reply-To: <20131211105613.201446ff68c75747d836fab7@samsung.com>
References: <1386093806-2948-1-git-send-email-p.marczak@samsung.com>
	<1386093806-2948-6-git-send-email-p.marczak@samsung.com>
	<20131211105613.201446ff68c75747d836fab7@samsung.com>
Message-ID: <52A86D2F.6000001@samsung.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Inha,

On 12/11/2013 02:56 AM, Inha Song wrote:
>
> Hi, Marczak
>
> I think, this patch can make unalgined access.
>
> (maybe.. this code)
>> if (vid->vl_bpix == 4)
>> 	vid->logo_addr = (ulong)tizen_hd_logo_16bpp;
>> else
>> 	vid->logo_addr = (ulong)tizen_hd_logo;
>
> Compiled using arm-linux-gnueabi (4.6.3 version) and
> When I tested using TRTAS2 board,
>
> I can face data abort error message like this,
>
> //////
> data abort
>
>      MAYBE you should read doc/README.arm-unaligned-accesses
>
> pc : [<7de5d150>]          lr : [<00000244>]
> sp : 78e4beb8  ip : 7de9f921     fp : 43e00020
> r10: 43e3a048  r9 : 78e4bf20     r8 : 11000000
> r7 : 11400000  r6 : 00000000     r5 : 000002d0  r4 : 00000001
> r3 : 00000064  r2 : 0000004d     r1 : 00000004  r0 : 7de9f921
> Flags: nZCv  IRQs off  FIQs on  Mode SVC_32
> Resetting CPU ...
>
> resetting ...
> ///////
>
> Thank you.
>
> ---- Original message ----
>
> On Tue, 03 Dec 2013 19:03:21 +0100
> Przemyslaw Marczak <p.marczak@samsung.com> wrote:
>
>> Signed-off-by: Przemyslaw Marczak <p.marczak@samsung.com>
>> ---
>> This is big size patch. Please follow link:
>> http://www.denx.de/wiki/pub/U-Boot/TooBigPatches/0005-lib-tizen-add-Tizen-16bpp-logo-support.patch
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot
>
>

Actually I used GCC 4.5. Linaro Gcc versions >= 4.6 has unaligned acces 
enabled by default. Maybe flag -mno-unaligned-access will avoid this. I 
will check this issue.

Regards.

-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com