Message ID | 1359618000-19069-8-git-send-email-nikita@compulab.co.il |
---|---|
State | Rejected |
Delegated to: | Anatolij Gustschin |
Headers | show |
Following the discussion in "Create an API for safely accessing BMP header fields", this patch should be reverted to V1: http://patchwork.ozlabs.org/patch/207950/ On 01/31/2013 09:40 AM, Nikita Kiryanov wrote: > Add support for loading splash image from NAND > > Cc: Anatolij Gustschin <agust@denx.de> > Cc: Jeroen Hofstee <jeroen@myspectrum.nl> > Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il> > Signed-off-by: Igor Grinberg <grinberg@compulab.co.il> > --- > Changes in V2: > - Added fix to bmp_load_addr to comply with what happens in lcd.c > following patch number 3. > > board/cm_t35/cm_t35.c | 65 ++++++++++++++++++++++++++++++++++++++++++++++ > include/configs/cm_t35.h | 4 +++ > 2 files changed, 69 insertions(+) > > diff --git a/board/cm_t35/cm_t35.c b/board/cm_t35/cm_t35.c > index 8f3d735..5fd71f9 100644 > --- a/board/cm_t35/cm_t35.c > +++ b/board/cm_t35/cm_t35.c > @@ -33,7 +33,9 @@ > #include <net.h> > #include <i2c.h> > #include <usb.h> > +#include <nand.h> > #include <twl4030.h> > +#include <bmp_layout.h> > #include <linux/compiler.h> > > #include <asm/io.h> > @@ -75,6 +77,69 @@ static u32 gpmc_nand_config[GPMC_MAX_REG] = { > 0, > }; > > +#ifdef CONFIG_LCD > +#ifdef CONFIG_CMD_NAND > +static int splash_load_from_nand(u32 bmp_load_addr) > +{ > + struct bmp_header *bmp_hdr; > + int res, splash_screen_nand_offset = 0x100000; > + size_t bmp_size, bmp_header_size = sizeof(struct bmp_header); > + > + if (bmp_load_addr + bmp_header_size >= gd->start_addr_sp) > + goto splash_address_too_high; > + > + res = nand_read_skip_bad(&nand_info[nand_curr_device], > + splash_screen_nand_offset, &bmp_header_size, > + (u_char *)bmp_load_addr); > + if (res < 0) > + return res; > + > + bmp_hdr = (struct bmp_header *)bmp_load_addr; > + bmp_size = le32_to_cpu(bmp_hdr->file_size); > + > + if (bmp_load_addr + bmp_size >= gd->start_addr_sp) > + goto splash_address_too_high; > + > + return nand_read_skip_bad(&nand_info[nand_curr_device], > + splash_screen_nand_offset, &bmp_size, > + (u_char *)bmp_load_addr); > + > +splash_address_too_high: > + printf("Error: splashimage address too high. Data overwrites U-Boot " > + "and/or placed beyond DRAM boundaries.\n"); > + > + return -1; > +} > +#else > +static inline int splash_load_from_nand(void) > +{ > + return -1; > +} > +#endif /* CONFIG_CMD_NAND */ > + > +int board_splash_screen_prepare(void) > +{ > + char *env_splashimage_value; > + u32 bmp_load_addr; > + > + env_splashimage_value = getenv("splashimage"); > + if (env_splashimage_value == NULL) > + return -1; > + > + bmp_load_addr = simple_strtoul(env_splashimage_value, 0, 16); > + if (bmp_load_addr == 0) { > + printf("Error: bad splashimage address specified\n"); > + return -1; > + } > + > + /* Fix bmp_load_addr to the alignment that lcd.c expects */ > + if (bmp_load_addr % 4 != 2) > + bmp_load_addr += (bmp_load_addr % 4) ?: 2; > + > + return splash_load_from_nand(bmp_load_addr); > +} > +#endif /* CONFIG_LCD */ > + > /* > * Routine: board_init > * Description: hardware init. > diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h > index 46d35ba..53c4e3c 100644 > --- a/include/configs/cm_t35.h > +++ b/include/configs/cm_t35.h > @@ -344,5 +344,9 @@ > #define LCD_BPP LCD_COLOR16 > > #define CONFIG_LCD > +#define CONFIG_SPLASH_SCREEN > +#define CONFIG_CMD_BMP > +#define CONFIG_BMP_16BPP > +#define CONFIG_SPLASH_SCREEN_PREPARE > > #endif /* __CONFIG_H */ >
diff --git a/board/cm_t35/cm_t35.c b/board/cm_t35/cm_t35.c index 8f3d735..5fd71f9 100644 --- a/board/cm_t35/cm_t35.c +++ b/board/cm_t35/cm_t35.c @@ -33,7 +33,9 @@ #include <net.h> #include <i2c.h> #include <usb.h> +#include <nand.h> #include <twl4030.h> +#include <bmp_layout.h> #include <linux/compiler.h> #include <asm/io.h> @@ -75,6 +77,69 @@ static u32 gpmc_nand_config[GPMC_MAX_REG] = { 0, }; +#ifdef CONFIG_LCD +#ifdef CONFIG_CMD_NAND +static int splash_load_from_nand(u32 bmp_load_addr) +{ + struct bmp_header *bmp_hdr; + int res, splash_screen_nand_offset = 0x100000; + size_t bmp_size, bmp_header_size = sizeof(struct bmp_header); + + if (bmp_load_addr + bmp_header_size >= gd->start_addr_sp) + goto splash_address_too_high; + + res = nand_read_skip_bad(&nand_info[nand_curr_device], + splash_screen_nand_offset, &bmp_header_size, + (u_char *)bmp_load_addr); + if (res < 0) + return res; + + bmp_hdr = (struct bmp_header *)bmp_load_addr; + bmp_size = le32_to_cpu(bmp_hdr->file_size); + + if (bmp_load_addr + bmp_size >= gd->start_addr_sp) + goto splash_address_too_high; + + return nand_read_skip_bad(&nand_info[nand_curr_device], + splash_screen_nand_offset, &bmp_size, + (u_char *)bmp_load_addr); + +splash_address_too_high: + printf("Error: splashimage address too high. Data overwrites U-Boot " + "and/or placed beyond DRAM boundaries.\n"); + + return -1; +} +#else +static inline int splash_load_from_nand(void) +{ + return -1; +} +#endif /* CONFIG_CMD_NAND */ + +int board_splash_screen_prepare(void) +{ + char *env_splashimage_value; + u32 bmp_load_addr; + + env_splashimage_value = getenv("splashimage"); + if (env_splashimage_value == NULL) + return -1; + + bmp_load_addr = simple_strtoul(env_splashimage_value, 0, 16); + if (bmp_load_addr == 0) { + printf("Error: bad splashimage address specified\n"); + return -1; + } + + /* Fix bmp_load_addr to the alignment that lcd.c expects */ + if (bmp_load_addr % 4 != 2) + bmp_load_addr += (bmp_load_addr % 4) ?: 2; + + return splash_load_from_nand(bmp_load_addr); +} +#endif /* CONFIG_LCD */ + /* * Routine: board_init * Description: hardware init. diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h index 46d35ba..53c4e3c 100644 --- a/include/configs/cm_t35.h +++ b/include/configs/cm_t35.h @@ -344,5 +344,9 @@ #define LCD_BPP LCD_COLOR16 #define CONFIG_LCD +#define CONFIG_SPLASH_SCREEN +#define CONFIG_CMD_BMP +#define CONFIG_BMP_16BPP +#define CONFIG_SPLASH_SCREEN_PREPARE #endif /* __CONFIG_H */