* [U-Boot] [PATCH v4 1/3] mx6sabresd: Add eMMC specific environment to allow U-Boot update @ 2014-01-03 17:58 Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 2/3] wandboard: add Future Eletronics 7" WVGA LCD extension board Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage Otavio Salvador 0 siblings, 2 replies; 7+ messages in thread From: Otavio Salvador @ 2014-01-03 17:58 UTC (permalink / raw) To: u-boot A new 'update_emmc_firmware' target is added to allow for easy U-Boot update in the eMMC as it has secury boot partition and this needs specific handling on how to program the specific partition. Signed-off-by: Otavio Salvador <otavio@ossystems.com.br> --- Changes in v4: - Fix compilation error (Stefano) Changes in v3: None Changes in v2: None include/configs/mx6sabre_common.h | 23 +++++++++++++++++++++++ include/configs/mx6sabresd.h | 2 ++ 2 files changed, 25 insertions(+) diff --git a/include/configs/mx6sabre_common.h b/include/configs/mx6sabre_common.h index 21c848f..5ee7fa5 100644 --- a/include/configs/mx6sabre_common.h +++ b/include/configs/mx6sabre_common.h @@ -95,6 +95,28 @@ #define CONFIG_LOADADDR 0x12000000 #define CONFIG_SYS_TEXT_BASE 0x17800000 +#ifdef CONFIG_SUPPORT_EMMC_BOOT +#define EMMC_ENV \ + "emmcdev=2\0" \ + "update_emmc_firmware=" \ + "if test ${ip_dyn} = yes; then " \ + "setenv get_cmd dhcp; " \ + "else " \ + "setenv get_cmd tftp; " \ + "fi; " \ + "if ${get_cmd} ${update_sd_firmware_filename}; then " \ + "if mmc dev ${emmcdev} && " \ + "mmc open ${emmcdev} 1; then " \ + "setexpr fw_sz ${filesize} / 0x200; " \ + "setexpr fw_sz ${fw_sz} + 1; " \ + "mmc write ${loadaddr} 0x2 ${fw_sz}; " \ + "mmc close ${emmcdev} 1; " \ + "fi; " \ + "fi\0" +#else +#define EMMC_ENV "" +#endif + #define CONFIG_EXTRA_ENV_SETTINGS \ "script=boot.scr\0" \ "uimage=uImage\0" \ @@ -121,6 +143,7 @@ "mmc write ${loadaddr} 0x2 ${fw_sz}; " \ "fi; " \ "fi\0" \ + EMMC_ENV \ "mmcargs=setenv bootargs console=${console},${baudrate} " \ "root=${mmcroot}\0" \ "loadbootscript=" \ diff --git a/include/configs/mx6sabresd.h b/include/configs/mx6sabresd.h index 3229bc7..4919f53 100644 --- a/include/configs/mx6sabresd.h +++ b/include/configs/mx6sabresd.h @@ -23,6 +23,8 @@ #endif #define PHYS_SDRAM_SIZE (1u * 1024 * 1024 * 1024) +#define CONFIG_SUPPORT_EMMC_BOOT /* eMMC specific */ + #include "mx6sabre_common.h" #define CONFIG_SYS_FSL_USDHC_NUM 3 -- 1.8.5.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 2/3] wandboard: add Future Eletronics 7" WVGA LCD extension board 2014-01-03 17:58 [U-Boot] [PATCH v4 1/3] mx6sabresd: Add eMMC specific environment to allow U-Boot update Otavio Salvador @ 2014-01-03 17:58 ` Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage Otavio Salvador 1 sibling, 0 replies; 7+ messages in thread From: Otavio Salvador @ 2014-01-03 17:58 UTC (permalink / raw) To: u-boot This adds support for the 7" WVGA produced by Future Eletronics and make it dynamically detect if it is connected or not based on the touchscreen controller. Signed-off-by: Otavio Salvador <otavio@ossystems.com.br> --- Changes in v4: None Changes in v3: - rebased on imx/master Changes in v2: None board/wandboard/wandboard.c | 240 +++++++++++++++++++++++++++++++++++++++----- include/configs/wandboard.h | 35 ++++++- 2 files changed, 251 insertions(+), 24 deletions(-) diff --git a/board/wandboard/wandboard.c b/board/wandboard/wandboard.c index 0043bc6..7debf30 100644 --- a/board/wandboard/wandboard.c +++ b/board/wandboard/wandboard.c @@ -15,6 +15,7 @@ #include <asm/arch/sys_proto.h> #include <asm/gpio.h> #include <asm/imx-common/iomux-v3.h> +#include <asm/imx-common/mxc_i2c.h> #include <asm/imx-common/boot_mode.h> #include <asm/io.h> #include <asm/sizes.h> @@ -25,6 +26,7 @@ #include <miiphy.h> #include <netdev.h> #include <linux/fb.h> +#include <i2c.h> DECLARE_GLOBAL_DATA_PTR; @@ -39,6 +41,10 @@ DECLARE_GLOBAL_DATA_PTR; #define ENET_PAD_CTRL (PAD_CTL_PUS_100K_UP | \ PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm | PAD_CTL_HYS) +#define I2C_PAD_CTRL (PAD_CTL_PUS_100K_UP | \ + PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm | PAD_CTL_HYS | \ + PAD_CTL_ODE | PAD_CTL_SRE_FAST) + #define USDHC1_CD_GPIO IMX_GPIO_NR(1, 2) #define USDHC3_CD_GPIO IMX_GPIO_NR(3, 9) #define ETH_PHY_RESET IMX_GPIO_NR(3, 29) @@ -208,50 +214,236 @@ int board_phy_config(struct phy_device *phydev) } #if defined(CONFIG_VIDEO_IPUV3) -static struct fb_videomode const hdmi = { - .name = "HDMI", - .refresh = 60, - .xres = 1024, - .yres = 768, - .pixclock = 15385, - .left_margin = 220, - .right_margin = 40, - .upper_margin = 21, - .lower_margin = 7, - .hsync_len = 60, - .vsync_len = 10, - .sync = FB_SYNC_EXT, - .vmode = FB_VMODE_NONINTERLACED +struct i2c_pads_info i2c_pad_info = { + .scl = { + .i2c_mode = MX6_PAD_KEY_COL3__I2C2_SCL + | MUX_PAD_CTRL(I2C_PAD_CTRL), + .gpio_mode = MX6_PAD_KEY_COL3__GPIO4_IO12 + | MUX_PAD_CTRL(I2C_PAD_CTRL), + .gp = IMX_GPIO_NR(4, 12) + }, + .sda = { + .i2c_mode = MX6_PAD_KEY_ROW3__I2C2_SDA + | MUX_PAD_CTRL(I2C_PAD_CTRL), + .gpio_mode = MX6_PAD_KEY_ROW3__GPIO4_IO13 + | MUX_PAD_CTRL(I2C_PAD_CTRL), + .gp = IMX_GPIO_NR(4, 13) + } }; -int board_video_skip(void) -{ - int ret; +static iomux_v3_cfg_t const fwadapt_7wvga_pads[] = { + MX6_PAD_DI0_DISP_CLK__IPU1_DI0_DISP_CLK, + MX6_PAD_DI0_PIN2__IPU1_DI0_PIN02, /* HSync */ + MX6_PAD_DI0_PIN3__IPU1_DI0_PIN03, /* VSync */ + MX6_PAD_DI0_PIN4__IPU1_DI0_PIN04 + | MUX_PAD_CTRL(PAD_CTL_DSE_120ohm), /* Contrast */ + MX6_PAD_DI0_PIN15__IPU1_DI0_PIN15, /* DISP0_DRDY */ + + MX6_PAD_DISP0_DAT0__IPU1_DISP0_DATA00, + MX6_PAD_DISP0_DAT1__IPU1_DISP0_DATA01, + MX6_PAD_DISP0_DAT2__IPU1_DISP0_DATA02, + MX6_PAD_DISP0_DAT3__IPU1_DISP0_DATA03, + MX6_PAD_DISP0_DAT4__IPU1_DISP0_DATA04, + MX6_PAD_DISP0_DAT5__IPU1_DISP0_DATA05, + MX6_PAD_DISP0_DAT6__IPU1_DISP0_DATA06, + MX6_PAD_DISP0_DAT7__IPU1_DISP0_DATA07, + MX6_PAD_DISP0_DAT8__IPU1_DISP0_DATA08, + MX6_PAD_DISP0_DAT9__IPU1_DISP0_DATA09, + MX6_PAD_DISP0_DAT10__IPU1_DISP0_DATA10, + MX6_PAD_DISP0_DAT11__IPU1_DISP0_DATA11, + MX6_PAD_DISP0_DAT12__IPU1_DISP0_DATA12, + MX6_PAD_DISP0_DAT13__IPU1_DISP0_DATA13, + MX6_PAD_DISP0_DAT14__IPU1_DISP0_DATA14, + MX6_PAD_DISP0_DAT15__IPU1_DISP0_DATA15, + MX6_PAD_DISP0_DAT16__IPU1_DISP0_DATA16, + MX6_PAD_DISP0_DAT17__IPU1_DISP0_DATA17, + + MX6_PAD_SD4_DAT2__GPIO2_IO10 + | MUX_PAD_CTRL(NO_PAD_CTRL), /* DISP0_BKLEN */ + MX6_PAD_SD4_DAT3__GPIO2_IO11 + | MUX_PAD_CTRL(NO_PAD_CTRL), /* DISP0_VDDEN */ +}; - ret = ipuv3_fb_init(&hdmi, 0, IPU_PIX_FMT_RGB24); +struct display_info_t { + int bus; + int addr; + int pixfmt; + int (*detect)(struct display_info_t const *dev); + void (*enable)(struct display_info_t const *dev); + struct fb_videomode mode; +}; - if (ret) { - printf("HDMI cannot be configured: %d\n", ret); - return ret; - } +static int detect_hdmi(struct display_info_t const *dev) +{ + struct hdmi_regs *hdmi = (struct hdmi_regs *)HDMI_ARB_BASE_ADDR; + return readb(&hdmi->phy_stat0) & HDMI_DVI_STAT; +} +static void do_enable_hdmi(struct display_info_t const *dev) +{ imx_enable_hdmi_phy(); +} - return ret; +static int detect_i2c(struct display_info_t const *dev) +{ + return ((0 == i2c_set_bus_num(dev->bus)) && + (0 == i2c_probe(dev->addr))); +} + +static void enable_fwadapt_7wvga(struct display_info_t const *dev) +{ + imx_iomux_v3_setup_multiple_pads( + fwadapt_7wvga_pads, + ARRAY_SIZE(fwadapt_7wvga_pads)); + + gpio_direction_output(IMX_GPIO_NR(2, 10), 1); + gpio_direction_output(IMX_GPIO_NR(2, 11), 1); +} + +static struct display_info_t const displays[] = {{ + .bus = -1, + .addr = 0, + .pixfmt = IPU_PIX_FMT_RGB24, + .detect = detect_hdmi, + .enable = do_enable_hdmi, + .mode = { + .name = "HDMI", + .refresh = 60, + .xres = 1024, + .yres = 768, + .pixclock = 15385, + .left_margin = 220, + .right_margin = 40, + .upper_margin = 21, + .lower_margin = 7, + .hsync_len = 60, + .vsync_len = 10, + .sync = FB_SYNC_EXT, + .vmode = FB_VMODE_NONINTERLACED +} }, { + .bus = 0, + .addr = 0x10, + .pixfmt = IPU_PIX_FMT_RGB666, + .detect = detect_i2c, + .enable = enable_fwadapt_7wvga, + .mode = { + .name = "FWBADAPT-LCD-F07A-0102", + .refresh = 60, + .xres = 800, + .yres = 480, + .pixclock = 33260, + .left_margin = 128, + .right_margin = 128, + .upper_margin = 22, + .lower_margin = 22, + .hsync_len = 1, + .vsync_len = 1, + .sync = 0, + .vmode = FB_VMODE_NONINTERLACED +} } }; + +int board_video_skip(void) +{ + int i; + int ret; + int detected = 0; + char const *panel = getenv("panel"); + if (!panel) { + for (i = 0; i < ARRAY_SIZE(displays); i++) { + struct display_info_t const *dev = displays+i; + if (dev->detect && dev->detect(dev)) { + panel = dev->mode.name; + detected = 1; + break; + } + } + if (!panel) { + panel = displays[0].mode.name; + printf("No panel detected: default to %s\n", panel); + i = 0; + } + } else { + for (i = 0; i < ARRAY_SIZE(displays); i++) { + if (!strcmp(panel, displays[i].mode.name)) + break; + } + } + if (i < ARRAY_SIZE(displays)) { + ret = ipuv3_fb_init(&displays[i].mode, 0, + displays[i].pixfmt); + + if (!ret) { + displays[i].enable(displays+i); + printf("Display: %s (%ux%u) %s\n", + displays[i].mode.name, + displays[i].mode.xres, + displays[i].mode.yres, + (detected ? "[auto]" : "")); + } else + printf("LCD %s cannot be configured: %d\n", + displays[i].mode.name, ret); + } else { + printf("unsupported panel %s\n", panel); + return -EINVAL; + } + + return 0; } static void setup_display(void) { struct mxc_ccm_reg *mxc_ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR; + struct iomuxc *iomux = (struct iomuxc *)IOMUXC_BASE_ADDR; int reg; enable_ipu_clock(); imx_setup_hdmi(); + /* Turn on LDB0, LDB1, IPU,IPU DI0 clocks */ + reg = __raw_readl(&mxc_ccm->CCGR3); + reg |= MXC_CCM_CCGR3_LDB_DI0_MASK | MXC_CCM_CCGR3_LDB_DI1_MASK; + writel(reg, &mxc_ccm->CCGR3); + + /* set LDB0, LDB1 clk select to 011/011 */ + reg = readl(&mxc_ccm->cs2cdr); + reg &= ~(MXC_CCM_CS2CDR_LDB_DI0_CLK_SEL_MASK + | MXC_CCM_CS2CDR_LDB_DI1_CLK_SEL_MASK); + reg |= (3 << MXC_CCM_CS2CDR_LDB_DI0_CLK_SEL_OFFSET) + | (3 << MXC_CCM_CS2CDR_LDB_DI1_CLK_SEL_OFFSET); + writel(reg, &mxc_ccm->cs2cdr); + + reg = readl(&mxc_ccm->cscmr2); + reg |= MXC_CCM_CSCMR2_LDB_DI0_IPU_DIV | MXC_CCM_CSCMR2_LDB_DI1_IPU_DIV; + writel(reg, &mxc_ccm->cscmr2); + reg = readl(&mxc_ccm->chsccdr); reg |= (CHSCCDR_CLK_SEL_LDB_DI0 << MXC_CCM_CHSCCDR_IPU1_DI0_CLK_SEL_OFFSET); + reg |= (CHSCCDR_CLK_SEL_LDB_DI0 + << MXC_CCM_CHSCCDR_IPU1_DI1_CLK_SEL_OFFSET); writel(reg, &mxc_ccm->chsccdr); + + reg = IOMUXC_GPR2_BGREF_RRMODE_EXTERNAL_RES + | IOMUXC_GPR2_DI1_VS_POLARITY_ACTIVE_LOW + | IOMUXC_GPR2_DI0_VS_POLARITY_ACTIVE_LOW + | IOMUXC_GPR2_BIT_MAPPING_CH1_SPWG + | IOMUXC_GPR2_DATA_WIDTH_CH1_18BIT + | IOMUXC_GPR2_BIT_MAPPING_CH0_SPWG + | IOMUXC_GPR2_DATA_WIDTH_CH0_18BIT + | IOMUXC_GPR2_LVDS_CH0_MODE_DISABLED + | IOMUXC_GPR2_LVDS_CH1_MODE_ENABLED_DI0; + writel(reg, &iomux->gpr[2]); + + reg = readl(&iomux->gpr[3]); + reg = (reg & ~(IOMUXC_GPR3_LVDS1_MUX_CTL_MASK + | IOMUXC_GPR3_HDMI_MUX_CTL_MASK)) + | (IOMUXC_GPR3_MUX_SRC_IPU1_DI0 + << IOMUXC_GPR3_LVDS1_MUX_CTL_OFFSET); + writel(reg, &iomux->gpr[3]); + + /* Disable LCD backlight */ + imx_iomux_v3_setup_pad(MX6_PAD_DI0_PIN4__GPIO4_IO20); + gpio_direction_input(IMX_GPIO_NR(4, 20)); } #endif /* CONFIG_VIDEO_IPUV3 */ @@ -309,6 +501,8 @@ int board_init(void) /* address of boot parameters */ gd->bd->bi_boot_params = PHYS_SDRAM + 0x100; + setup_i2c(1, CONFIG_SYS_I2C_SPEED, 0x7f, &i2c_pad_info); + return 0; } diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h index ae8480d..5134b2e 100644 --- a/include/configs/wandboard.h +++ b/include/configs/wandboard.h @@ -56,6 +56,12 @@ #define CONFIG_LOADADDR 0x12000000 #define CONFIG_SYS_TEXT_BASE 0x17800000 +/* I2C Configs */ +#define CONFIG_CMD_I2C +#define CONFIG_SYS_I2C +#define CONFIG_SYS_I2C_MXC +#define CONFIG_SYS_I2C_SPEED 100000 + /* MMC Configuration */ #define CONFIG_FSL_ESDHC #define CONFIG_FSL_USDHC @@ -98,6 +104,7 @@ #define CONFIG_VIDEO_LOGO #define CONFIG_VIDEO_BMP_LOGO #define CONFIG_IPUV3_CLK 260000000 +#define CONFIG_CMD_HDMIDETECT #define CONFIG_IMX_HDMI #if defined(CONFIG_MX6DL) || defined(CONFIG_MX6S) @@ -135,7 +142,33 @@ "fi; " \ "fi\0" \ "mmcargs=setenv bootargs console=${console},${baudrate} " \ - "root=${mmcroot}\0" \ + "root=${mmcroot}; run videoargs\0" \ + "videoargs=" \ + "setenv nextcon 0; " \ + "if hdmidet; then " \ + "setenv bootargs ${bootargs} " \ + "video=mxcfb${nextcon}:dev=hdmi,1280x720M at 60," \ + "if=RGB24; " \ + "setenv fbmen fbmem=28M; " \ + "setexpr nextcon ${nextcon} + 1; " \ + "else " \ + "echo - no HDMI monitor;" \ + "fi; " \ + "i2c dev 0; " \ + "if i2c probe 0x10; then " \ + "setenv bootargs ${bootargs} " \ + "video=mxcfb${nextcon}:dev=lcd,800x480 at 60," \ + "if=RGB666; " \ + "if test 0 -eq ${nextcon}; then " \ + "setenv fbmem fbmem=10M; " \ + "else " \ + "setenv fbmem ${fbmem},10M; " \ + "fi; " \ + "setexpr nextcon ${nextcon} + 1; " \ + "else " \ + "echo '- no FWBADAPT-7WVGA-LCD-F07A-0102 display';" \ + "fi; " \ + "setenv bootargs ${bootargs} ${fbmem}\0" \ "loadbootscript=" \ "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \ "bootscript=echo Running bootscript from mmc ...; " \ -- 1.8.5.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage 2014-01-03 17:58 [U-Boot] [PATCH v4 1/3] mx6sabresd: Add eMMC specific environment to allow U-Boot update Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 2/3] wandboard: add Future Eletronics 7" WVGA LCD extension board Otavio Salvador @ 2014-01-03 17:58 ` Otavio Salvador 2014-01-03 22:43 ` Wolfgang Denk 1 sibling, 1 reply; 7+ messages in thread From: Otavio Salvador @ 2014-01-03 17:58 UTC (permalink / raw) To: u-boot From: John Weber <rjohnweber@gmail.com> Change the default environment to use zImage instead of uImage, this requires changes to the default environment to load a file named zImage instead of uImage, and to use the 'bootz' command instead of 'bootm' when booting the kernel. The zImage works for FSL Linux's kernel fork versions 3.0.35, 3.10.9, and 3.10.17; this also works fine for mainline kernels. There is no reason to continue using uImage for default environment. Signed-off-by: John Weber <rjohnweber@gmail.com> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br> --- Changes in v4: None Changes in v3: None Changes in v2: - Rework commit log (wd) include/configs/wandboard.h | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h index 5134b2e..3424124 100644 --- a/include/configs/wandboard.h +++ b/include/configs/wandboard.h @@ -115,7 +115,7 @@ #define CONFIG_EXTRA_ENV_SETTINGS \ "script=boot.scr\0" \ - "uimage=uImage\0" \ + "image=zImage\0" \ "console=ttymxc0\0" \ "splashpos=m,m\0" \ "fdt_high=0xffffffff\0" \ @@ -173,22 +173,22 @@ "fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${script};\0" \ "bootscript=echo Running bootscript from mmc ...; " \ "source\0" \ - "loaduimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${uimage}\0" \ + "loadimage=fatload mmc ${mmcdev}:${mmcpart} ${loadaddr} ${image}\0" \ "loadfdt=fatload mmc ${mmcdev}:${mmcpart} ${fdt_addr} ${fdt_file}\0" \ "mmcboot=echo Booting from mmc ...; " \ "run mmcargs; " \ "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \ "if run loadfdt; then " \ - "bootm ${loadaddr} - ${fdt_addr}; " \ + "bootz ${loadaddr} - ${fdt_addr}; " \ "else " \ "if test ${boot_fdt} = try; then " \ - "bootm; " \ + "bootz; " \ "else " \ "echo WARN: Cannot load the DT; " \ "fi; " \ "fi; " \ "else " \ - "bootm; " \ + "bootz; " \ "fi;\0" \ "netargs=setenv bootargs console=${console},${baudrate} " \ "root=/dev/nfs " \ @@ -200,19 +200,19 @@ "else " \ "setenv get_cmd tftp; " \ "fi; " \ - "${get_cmd} ${uimage}; " \ + "${get_cmd} ${image}; " \ "if test ${boot_fdt} = yes || test ${boot_fdt} = try; then " \ "if ${get_cmd} ${fdt_addr} ${fdt_file}; then " \ - "bootm ${loadaddr} - ${fdt_addr}; " \ + "bootz ${loadaddr} - ${fdt_addr}; " \ "else " \ "if test ${boot_fdt} = try; then " \ - "bootm; " \ + "bootz; " \ "else " \ "echo WARN: Cannot load the DT; " \ "fi; " \ "fi; " \ "else " \ - "bootm; " \ + "bootz; " \ "fi;\0" #define CONFIG_BOOTCOMMAND \ @@ -220,7 +220,7 @@ "if run loadbootscript; then " \ "run bootscript; " \ "else " \ - "if run loaduimage; then " \ + "if run loadimage; then " \ "run mmcboot; " \ "else run netboot; " \ "fi; " \ -- 1.8.5.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage 2014-01-03 17:58 ` [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage Otavio Salvador @ 2014-01-03 22:43 ` Wolfgang Denk 2014-01-04 19:36 ` Dennis Gilmore 2014-01-06 11:12 ` Otavio Salvador 0 siblings, 2 replies; 7+ messages in thread From: Wolfgang Denk @ 2014-01-03 22:43 UTC (permalink / raw) To: u-boot Dear Otavio Salvador, In message <1388771903-8730-3-git-send-email-otavio@ossystems.com.br> you wrote: > > ... There is no > reason to continue using uImage for default environment. Sorry, but this "There is no reason to continue using uImage" is a phrase which challenges my protest. Actually switching from uImage to zImage is a serious deterioration. uImages have a number of properties that are extremely useful, and especially in embedded environments; none of these are available for zImages. The only problem is that uImage support has never been correctly implemented in the ARM Linux kernel, and that fixes have never been accepted (for various [non-technical] reasons). Switching to zImage means to take the path of least resistance, and chose the worst of the possible solutions, technically. It would make much more sense to push forward and use FIT images, instead. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de When the bosses talk about improving productivity, they are never talking about themselves. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage 2014-01-03 22:43 ` Wolfgang Denk @ 2014-01-04 19:36 ` Dennis Gilmore 2014-01-05 12:52 ` Wolfgang Denk 2014-01-06 11:12 ` Otavio Salvador 1 sibling, 1 reply; 7+ messages in thread From: Dennis Gilmore @ 2014-01-04 19:36 UTC (permalink / raw) To: u-boot -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Happy new year Wolfgang El Fri, 03 Jan 2014 23:43:54 +0100 Wolfgang Denk <wd@denx.de> escribi?: > Dear Otavio Salvador, > > In message <1388771903-8730-3-git-send-email-otavio@ossystems.com.br> > you wrote: > > > > ... There is no > > reason to continue using uImage for default environment. > > Sorry, but this "There is no reason to continue using uImage" is a > phrase which challenges my protest. > > Actually switching from uImage to zImage is a serious deterioration. > uImages have a number of properties that are extremely useful, and > especially in embedded environments; none of these are available for > zImages. The only problem is that uImage support has never been > correctly implemented in the ARM Linux kernel, and that fixes have > never been accepted (for various [non-technical] reasons). > > Switching to zImage means to take the path of least resistance, and > chose the worst of the possible solutions, technically. > > It would make much more sense to push forward and use FIT images, > instead. It is all about the target use case, uImage and FIT images suit a useful purpose in an embedded use case. A wandboard is most definitely not an embedded device. as ARM is moving more and more into general computing, supporting general computing use cases is needed. all of the generic distro work I have done is primarily revolving around using raw images and raw initramfs's all supported fedora systems use a multiplatform kernel, wrapping it in uImage to work everywhere is not an available option. You are free to protest, but I believe you are wrong in doing so. Dennis -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJSyEryAAoJEH7ltONmPFDRimsP/1OT/XNWVCxhYOSYw2TMyqKt upfTDpUZ1O4zTwDjwaCG/5zgAK+NPLnrD9FCdsT4V8fH216BsCPJoDYt+Uv8P9AS ZSHgmw0d60qADtVQmcukuUMrtmZvRjJZXI7eb2M7LIKg0QNbBeU7bAqm+OPlAe0J 9cguVj+jREuPQ3nqeCOp9WVb/qVEmSRID7+3heqlMG9/LRoh5hmwjlPot2L/sgzt 3kN4RYKqs+ulTlG5eowOCoShdAs8W9rrs+Sw88JpUro2gd0RGr7NpIrZ009Fw0vl muT0pCDEar4Z+SZdUyONSKS/ZKsjNI8fqVTDIkpIo8X3YYn4AP4hYU7Fiuwoqoi5 F5z3ZvzRGTpPOEfmWlF25D23hCHqrp9LIRIYsQ0BIW9jeMSvPPm/KfdeVsBtS4Ib j0q8I+LvVDWaGlQFuUvwcsKtxEPxyO1c51/LSaXudb1aurPXtiVB84FlyavZZh4x GF5572bOaz+X78ZObQbtDtxXFaQ0F9EFn/kKgCrfCrKJktlpWfGjgEp0VOa1F+z3 CWbBJ5PzQKF00OEs+QYqKGpwPWTyJ3uFTpOrqXL74LSESGcDrxjddGe4EkKb6pA2 pTOCCT+RXzHvt/qavusI9uSSzykG6/PbmZqISjntAuueP9ROLqBy05gCuQq0qEBI VMrw3oP2dga71ozMrvrP =V53U -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage 2014-01-04 19:36 ` Dennis Gilmore @ 2014-01-05 12:52 ` Wolfgang Denk 0 siblings, 0 replies; 7+ messages in thread From: Wolfgang Denk @ 2014-01-05 12:52 UTC (permalink / raw) To: u-boot Dear Dennis, In message <20140104133617.43f703f2@anubis.ausil.us> you wrote: > > > Sorry, but this "There is no reason to continue using uImage" is a > > phrase which challenges my protest. ... > It is all about the target use case, uImage and FIT images suit a > useful purpose in an embedded use case. A wandboard is most definitely > not an embedded device. as ARM is moving more and more into general Well, this is another topic where opinios may vary a lot. I do consider the wandboard a typical embedded system. But this is not really important here. > computing, supporting general computing use cases is needed. all of the > generic distro work I have done is primarily revolving around using raw > images and raw initramfs's all supported fedora systems use a > multiplatform kernel, wrapping it in uImage to work everywhere is not It is really a pity that people like you ignore the capabilities of FIT images and fail to see which benefits they could bring to "general computing use cases". > an available option. You are free to protest, but I believe you are > wrong in doing so. I know that I'm on lost ground here, and I do not try to block this patch. All I'm asking is to change the commit message. If you do not see reasons for using uImage (or even better FIT images) this does not necessarily mean that there are no reasons. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Lack of skill dictates economy of style. - Joey Ramone ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage 2014-01-03 22:43 ` Wolfgang Denk 2014-01-04 19:36 ` Dennis Gilmore @ 2014-01-06 11:12 ` Otavio Salvador 1 sibling, 0 replies; 7+ messages in thread From: Otavio Salvador @ 2014-01-06 11:12 UTC (permalink / raw) To: u-boot Hello Wolfgang, On Fri, Jan 3, 2014 at 8:43 PM, Wolfgang Denk <wd@denx.de> wrote: > Dear Otavio Salvador, > > In message <1388771903-8730-3-git-send-email-otavio@ossystems.com.br> you wrote: >> >> ... There is no >> reason to continue using uImage for default environment. > > Sorry, but this "There is no reason to continue using uImage" is a > phrase which challenges my protest. > > Actually switching from uImage to zImage is a serious deterioration. > uImages have a number of properties that are extremely useful, and > especially in embedded environments; none of these are available for > zImages. The only problem is that uImage support has never been > correctly implemented in the ARM Linux kernel, and that fixes have > never been accepted (for various [non-technical] reasons). > > Switching to zImage means to take the path of least resistance, and > chose the worst of the possible solutions, technically. > > It would make much more sense to push forward and use FIT images, > instead. I understand FIT has many benefits against zImage but Wandboard default should be as easy and trivial for use as possible; I can rework the commit log one more time. See if below version is better: wandboard: Set default environment to use zImage Change the default environment to use zImage instead of uImage, this requires changes to the default environment to load a file named zImage instead of uImage, and to use the 'bootz' command instead of 'bootm' when booting the kernel. The zImage works for FSL Linux's kernel fork versions 3.0.35, 3.10.9, and 3.10.17; this also works fine for mainline kernels. Please reply and I send a new revision of the patch. -- Otavio Salvador O.S. Systems http://www.ossystems.com.br http://code.ossystems.com.br Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750 ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-01-06 11:12 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-01-03 17:58 [U-Boot] [PATCH v4 1/3] mx6sabresd: Add eMMC specific environment to allow U-Boot update Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 2/3] wandboard: add Future Eletronics 7" WVGA LCD extension board Otavio Salvador 2014-01-03 17:58 ` [U-Boot] [PATCH v4 3/3] wandboard: Set default environment to use zImage Otavio Salvador 2014-01-03 22:43 ` Wolfgang Denk 2014-01-04 19:36 ` Dennis Gilmore 2014-01-05 12:52 ` Wolfgang Denk 2014-01-06 11:12 ` Otavio Salvador
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.