diff options
| author | Stephen Warren <swarren@nvidia.com> | 2013-06-13 17:13:11 -0600 |
|---|---|---|
| committer | Anatolij Gustschin <agust@denx.de> | 2013-07-01 20:11:33 +0200 |
| commit | 5af7d0f090b9fe2464d7980841b940846a547716 (patch) | |
| tree | 48086351669807c5f8427677c621a7eebb7b88a0 /common | |
| parent | b1d8654b41e8cbc4a9c12356ecabad30386a3a7e (diff) | |
| download | talos-obmc-uboot-5af7d0f090b9fe2464d7980841b940846a547716.tar.gz talos-obmc-uboot-5af7d0f090b9fe2464d7980841b940846a547716.zip | |
lcd: remove unaligned access in lcd_dt_simplefb_configure_node()
Some ARM compilers may emit code that makes unaligned accesses when
faced with constructs such as:
const char format[] = "r5g6b5";
Make this data static since it doesn't chagne; the compiler will simply
place it into the .rodata section directly, and avoid any unaligned run-
time initialization.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Diffstat (limited to 'common')
| -rw-r--r-- | common/lcd.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/lcd.c b/common/lcd.c index 3a60484eea..c9a589e976 100644 --- a/common/lcd.c +++ b/common/lcd.c @@ -1193,7 +1193,7 @@ static int lcd_dt_simplefb_configure_node(void *blob, int off) u32 stride; fdt32_t cells[2]; int ret; - const char format[] = + static const char format[] = #if LCD_BPP == LCD_COLOR16 "r5g6b5"; #else @@ -1239,8 +1239,8 @@ static int lcd_dt_simplefb_configure_node(void *blob, int off) int lcd_dt_simplefb_add_node(void *blob) { - const char compat[] = "simple-framebuffer"; - const char disabled[] = "disabled"; + static const char compat[] = "simple-framebuffer"; + static const char disabled[] = "disabled"; int off, ret; off = fdt_add_subnode(blob, 0, "framebuffer"); |
