summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStephen Warren <swarren@nvidia.com>2013-06-13 17:13:11 -0600
committerAnatolij Gustschin <agust@denx.de>2013-07-01 20:11:33 +0200
commit5af7d0f090b9fe2464d7980841b940846a547716 (patch)
tree48086351669807c5f8427677c621a7eebb7b88a0
parentb1d8654b41e8cbc4a9c12356ecabad30386a3a7e (diff)
downloadu-boot-imx-5af7d0f090b9fe2464d7980841b940846a547716.zip
u-boot-imx-5af7d0f090b9fe2464d7980841b940846a547716.tar.gz
u-boot-imx-5af7d0f090b9fe2464d7980841b940846a547716.tar.bz2
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>
-rw-r--r--common/lcd.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/common/lcd.c b/common/lcd.c
index 3a60484..c9a589e 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");