December 2015 - U-Boot

Re: [U-Boot] [PATCH] Don't try to stat() a colon separated list of files in multi or script mode
by Simon Glass 03 Dec '15

03 Dec '15

Hi, On 3 December 2015 at 14:55, Nye Liu <nyet(a)nyet.org> wrote: > > On 11/19/2015 6:15 AM, Simon Glass wrote: >> Hi, >> >> On 18 November 2015 at 17:37, Nye Liu <nyet(a)nyet.org> wrote: >>> --- >>> tools/mkimage.c | 32 +++++++++++++++++++------------- >>> 1 file changed, 19 insertions(+), 13 deletions(-) >>> >> >> Can you please add a commit message explaining the problem, and what >> you are trying to do here? >> >>> diff --git a/tools/mkimage.c b/tools/mkimage.c >>> index 8af9d50..aa739b4 100644 >>> --- a/tools/mkimage.c >>> +++ b/tools/mkimage.c >>> @@ -311,21 +311,27 @@ NXTARG: ; >>> exit (retval); >>> } >>> >>> - dfd = open(params.datafile, O_RDONLY | O_BINARY); >>> - if (dfd < 0) { >>> - fprintf(stderr, "%s: Can't open %s: %s\n", >>> - params.cmdname, params.datafile, strerror(errno)); >>> - exit(EXIT_FAILURE); >>> - } >>> + if (params.skipcpy || >>> + !(params.type == IH_TYPE_MULTI || >>> + params.type == IH_TYPE_SCRIPT)) { >>> + dfd = open(params.datafile, O_RDONLY | O_BINARY); >>> + if (dfd < 0) { >>> + fprintf(stderr, "%s: Can't open %s: %s\n", >>> + params.cmdname, params.datafile, >>> + strerror(errno)); >>> + exit(EXIT_FAILURE); >>> + } >>> >>> - if (fstat(dfd, &sbuf) < 0) { >>> - fprintf(stderr, "%s: Can't stat %s: %s\n", >>> - params.cmdname, params.datafile, strerror(errno)); >>> - exit(EXIT_FAILURE); >>> - } >>> + if (fstat(dfd, &sbuf) < 0) { >>> + fprintf(stderr, "%s: Can't stat %s: %s\n", >>> + params.cmdname, params.datafile, >>> + strerror(errno)); >>> + exit(EXIT_FAILURE); >>> + } >>> >>> - params.file_size = sbuf.st_size + tparams->header_size; >>> - close(dfd); >>> + params.file_size = sbuf.st_size + tparams->header_size; >>> + close(dfd); >>> + } >>> >>> /* >>> * In case there an header with a variable >>> -- >>> 2.6.2 > > Meaning, you don't know what it does, or you won't accept the patch w/o it? > > In short, > > "mkimage -T multi -d file1:file2:file3" > > no longer works. > > It stats the file "file1:file2:file3" (and fails) before opening each > file separately. > Yes, please put this explanation in version 2 of your patch. Regards, Simon

1 0

[U-Boot] [PATCH v5 0/8] dm: pci: tegra: Convert Tegra PCI to driver model
by Simon Glass 03 Dec '15

03 Dec '15

This series converts all Tegra boards to use driver model for PCI. The net effect should be no change in functionality. A few additional features are added to make this possible: - Helper functions to support accessing 8- and 16-bit values within a 32-bit word - Fixing a build error for CONFIG_CMD_PCI_ENUM - Decoding the PCI ranges property such that configuration ranges are ignored - Supporting bus-master devices on boards where RAM does not start at 0 This series is tested on beaver. It is available at u-boot-dm/tegra-working. Changes in v5: - Squash in Stephen's fixes from https://patchwork.ozlabs.org/patch/544727 Changes in v4: - Fix Kconig typo in commit message - Drop CONFIG_CMD_PCI_ENUM for p2371-2180 also - Fix merge conflict resolution error in v3 - Add compatible string for T210 (from Stephen Warren) Changes in v3: - Rebase onto tegra/master - Enable DM_PCI for Tegra 210 family too Changes in v2: - Update commit message to explain that the feature is not important - Drop the feature from tegra boards - Rename 'addr' to 'size' - Correct logic for use of gd->pci_ram_top - Update commit message to mention future work - Use the device_is_on_pci_bus() API - Leave pci_skip_dev() at the bottom of the file to reduce the diff size Simon Glass (8): dm: tegra: pci: Move CONFIG_PCI_TEGRA to Kconfig dm: pci: Avoid a driver model build error with CONFIG_CMD_PCI_ENUM dm: pci: Set up the SDRAM mapping correctly dm: pci: Support decoding ranges with duplicate entries dm: pci: Add functions to emulate 8- and 16-bit access dm: pci: Add a function to get the controller for a bus dm: pci: Add a function to find the regions for a PCI bus dm: tegra: pci: Convert tegra boards to driver model for PCI arch/arm/mach-tegra/Kconfig | 1 + arch/arm/mach-tegra/board2.c | 4 + common/cmd_pci.c | 4 + configs/apalis_t30_defconfig | 1 + configs/beaver_defconfig | 1 + configs/cardhu_defconfig | 1 + configs/jetson-tk1_defconfig | 1 + configs/p2371-2180_defconfig | 1 + configs/trimslice_defconfig | 1 + drivers/pci/Kconfig | 10 + drivers/pci/pci-uclass.c | 105 +++++++++- drivers/pci/pci_tegra.c | 477 ++++++++++++++----------------------------- include/configs/apalis_t30.h | 2 - include/configs/beaver.h | 2 - include/configs/cardhu.h | 2 - include/configs/jetson-tk1.h | 2 - include/configs/p2371-2180.h | 2 - include/configs/trimslice.h | 2 - include/fdtdec.h | 4 - include/pci.h | 51 +++++ lib/fdtdec.c | 4 - 21 files changed, 327 insertions(+), 351 deletions(-) -- 2.6.0.rc2.230.g3dd15c0

2 12

[U-Boot] [PATCHv2 0/9] ARM: socfpga: Add minimal support for Arria10
by dinguyen＠opensource.altera.com 03 Dec '15

03 Dec '15

From: Dinh Nguyen <dinguyen(a)opensource.altera.com> Hi, This is v2 of the patch series that adds minimal support for Altera's Arria10 platform. v2: - Removes the need for a separate /mach-socfpga/arria10 directory - Tries to re-use alot of the Arria5/Cyclone5 code - Removes the usage of unions bitfields in the SDRAM defines v1: The series builds for the Altera Arria10 SoCDK, but is not entirely functional on the hardware yet. This series really just add the defines, build and Kconfig layout for Arria10. There are few TODO after this series: - Add DTS file for the devkit - Add clock manager - The Arria10 Pin Mux requires the FPGA to get programmed before the SDRAM controller can be usable. So another patch is needed to make use of the fpga manager to program the FPGA within the limited size of the OCRAM. Thanks, Dinh Dinh Nguyen (9): arm: socfpga: arria10: add system manager defines arm: socfpga: arria10: add reset manager for Arria10 arm: socfpga: arria10: add sdram defines for Arria10 arm: socfpga: arria10: add misc functions for Arria10 arm: socfpga: arria10: add socfpga_arria10_socdk config arm: socfpga: arria10: add board files for the Arria10 SoCDK arm: socfpga: arria10: add socfpga_arria10_defconfig arm: socfpga: arria10: add config option build for arria10 arm: socfpga: remove building scan manager arch/arm/Kconfig | 4 +- arch/arm/mach-socfpga/Kconfig | 10 + arch/arm/mach-socfpga/Makefile | 7 +- arch/arm/mach-socfpga/include/mach/reset_manager.h | 66 ++++ arch/arm/mach-socfpga/include/mach/sdram_a10.h | 380 +++++++++++++++++++++ .../mach-socfpga/include/mach/system_manager_a10.h | 157 +++++++++ arch/arm/mach-socfpga/misc.c | 56 ++- arch/arm/mach-socfpga/reset_manager.c | 47 ++- board/altera/arria10-socdk/Kconfig | 18 + board/altera/arria10-socdk/Makefile | 7 + board/altera/arria10-socdk/socfpga.c | 24 ++ configs/socfpga_arria10_defconfig | 11 + include/configs/socfpga_arria10_socdk.h | 157 +++++++++ 13 files changed, 937 insertions(+), 7 deletions(-) create mode 100644 arch/arm/mach-socfpga/include/mach/sdram_a10.h create mode 100644 arch/arm/mach-socfpga/include/mach/system_manager_a10.h create mode 100644 board/altera/arria10-socdk/Kconfig create mode 100644 board/altera/arria10-socdk/Makefile create mode 100644 board/altera/arria10-socdk/socfpga.c create mode 100644 configs/socfpga_arria10_defconfig create mode 100644 include/configs/socfpga_arria10_socdk.h -- 2.6.2

5 24

[U-Boot] [PATCH] iocon: Disable FIT_VERBOSE
by Tom Rini 03 Dec '15

03 Dec '15

In order to fit into image constraints again, remove this feature. Signed-off-by: Tom Rini <trini(a)konsulko.com> --- configs/iocon_defconfig | 1 + include/configs/iocon.h | 2 -- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/configs/iocon_defconfig b/configs/iocon_defconfig index bc5e026..a6532d1 100644 --- a/configs/iocon_defconfig +++ b/configs/iocon_defconfig @@ -1,6 +1,7 @@ CONFIG_PPC=y CONFIG_4xx=y CONFIG_TARGET_IOCON=y +CONFIG_FIT=y CONFIG_AUTOBOOT_KEYED=y CONFIG_AUTOBOOT_STOP_STR=" " # CONFIG_CMD_ELF is not set diff --git a/include/configs/iocon.h b/include/configs/iocon.h index 385a644..8663c17 100644 --- a/include/configs/iocon.h +++ b/include/configs/iocon.h @@ -35,8 +35,6 @@ #undef CONFIG_ZERO_BOOTDELAY_CHECK /* ignore keypress on bootdelay==0 */ /* new uImage format support */ -#define CONFIG_FIT -#define CONFIG_FIT_VERBOSE /* enable fit_format_{error,warning}() */ #define CONFIG_FIT_DISABLE_SHA256 #define CONFIG_ENV_IS_IN_FLASH /* use FLASH for environment vars */ -- 1.7.9.5

2 4

[U-Boot] [PATCH 0/8] x86: fsp: Move platform-specific config to chipset directory
by Bin Meng 03 Dec '15

03 Dec '15

All FSP spec v1.0 complaint FSP binary uses struct fspinit_rtbuf as defined by the 1.0 spec, however there are FSPs that do not follow 1.0 spec, like Intel FSP for 3rd generation Intel Core and Intel Celeron processors with mobile Intel HM76 and QM77 chipsets platform (formerly Chief River Platform: Ivy Bridge and Panther Point). Although Intel website says this FSP conforms to FSP v1.0 specification which defines the UPD usage, it is not really the case. This might possible due to that FSP predates the 1.0 spec. Also future FSP binary that is complaint to v1.1 spec defines an optional paltform-specific runtime data in the struct fspinit_rtbuf. Besides this fspinit_rtbuf, the IvyBridge FSP does not support UPD either, so that current codes in arch/x86/lib/fsp/fsp_support.c won't work for that FSP. We need some flexibility, hence move those platform-specific config to chipset directory. Right now we still use hardcoded 'const struct' table (in flash) to pass the Azalia (Intel HD Audio) verb table to FSP, because different FSP may use different verb table format (at least for Queensbay and BayTrail), we cannot handle this in the common FSP support codes. But with this series, we are now able to get Azalia verb table from device tree, just like what we did for overriding UPD data configuration. This can be done in future patch set. Bin Meng (8): x86: fsp: Simplify fsp_continue() x86: fsp: Avoid cast stack_top in struct shared_data x86: fsp: Add boot_mode as a member of struct shared_data x86: fsp: Rename shared_data to fsp_config_data x86: fsp: Rename update_fsp_upd() and change its signature x86: fsp: Move struct fspinit_rtbuf definition to chipset header x86: fsp: Move VPD/UPD verification to update_fsp_configs() x86: queensbay: Remove invalid comments in update_fsp_upd() arch/x86/cpu/baytrail/fsp_configs.c | 27 ++++++++- arch/x86/cpu/queensbay/fsp_configs.c | 30 ++++++++-- .../include/asm/arch-baytrail/fsp/fsp_configs.h | 21 +++++++ .../include/asm/arch-queensbay/fsp/fsp_configs.h | 21 +++++++ arch/x86/include/asm/fsp/fsp_api.h | 2 +- arch/x86/include/asm/fsp/fsp_platform.h | 15 ----- arch/x86/include/asm/fsp/fsp_support.h | 20 +++---- arch/x86/lib/fsp/fsp_support.c | 64 +++++----------------- 8 files changed, 112 insertions(+), 88 deletions(-) create mode 100644 arch/x86/include/asm/arch-baytrail/fsp/fsp_configs.h create mode 100644 arch/x86/include/asm/arch-queensbay/fsp/fsp_configs.h delete mode 100644 arch/x86/include/asm/fsp/fsp_platform.h -- 1.8.2.1

2 22

[U-Boot] Next for x86?
by Simon Glass 03 Dec '15

03 Dec '15

Hi, I'm wondering what might come next for U-Boot x86 support. The PCI conversion to driver model is coming along nicely. The ACPI support is only partially there but it is a good start. What else? More platforms? Other features? Better support for existing platforms? Regards, Simon

3 6

[U-Boot] [PATCH] arm: socfpga: fix up a questionable macro for SDMMC
by dinguyen＠opensource.altera.com 03 Dec '15

03 Dec '15

From: Dinh Nguyen <dinguyen(a)opensource.altera.com> Not sure what made this macro questionable, but edit the macro to be similar to what is used in Linux. Signed-off-by: Dinh Nguyen <dinguyen(a)opensource.altera.com> --- arch/arm/mach-socfpga/include/mach/system_manager.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm/mach-socfpga/include/mach/system_manager.h b/arch/arm/mach-socfpga/include/mach/system_manager.h index 618c92f..c0566b0 100644 --- a/arch/arm/mach-socfpga/include/mach/system_manager.h +++ b/arch/arm/mach-socfpga/include/mach/system_manager.h @@ -201,9 +201,8 @@ struct socfpga_system_manager { #define SYSMGR_FPGAINTF_NAND (1 << 4) #define SYSMGR_FPGAINTF_SDMMC (1 << 5) -/* FIXME: This is questionable macro. */ #define SYSMGR_SDMMC_CTRL_SET(smplsel, drvsel) \ - ((((drvsel) << 0) & 0x7) | (((smplsel) << 3) & 0x38)) + ((((smplsel) & 0x7) << 3) | (((drvsel) & 0x7) << 0)) /* EMAC Group Bit definitions */ #define SYSMGR_EMACGRP_CTRL_PHYSEL_ENUM_GMII_MII 0x0 -- 2.6.2

4 4

[U-Boot] [PATCH] distro_bootcmd: do not try to fetch unused bootfile for PXE
by Stefan Brüns 03 Dec '15

03 Dec '15

pxe get derives the pxelinux config file name from the bootfile name, but the bootfile itself is never used and might not even exist. Disable bootfile autoload to avoid the delay. Signed-off-by: Stefan Brüns <stefan.bruens(a)rwth-aachen.de> --- include/config_distro_bootcmd.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/config_distro_bootcmd.h b/include/config_distro_bootcmd.h index 66264ce..c9d4b25 100644 --- a/include/config_distro_bootcmd.h +++ b/include/config_distro_bootcmd.h @@ -176,7 +176,9 @@ #define BOOTENV_DEV_PXE(devtypeu, devtypel, instance) \ "bootcmd_pxe=" \ BOOTENV_RUN_USB_INIT \ - "dhcp; " \ + "env exists autoload && setenv autoload_save ${autoload}; " \ + "setenv autoload no; dhcp; " \ + "env exists autoload_save && setenv autoload ${autoload_save}; " \ "if pxe get; then " \ "pxe boot; " \ "fi\0" -- 2.1.4

2 1

[U-Boot] [PATCH v7 00/34] sf: MTD support
by Jagan Teki 03 Dec '15

03 Dec '15

This series is combination of mtd and sf tunning stuff in previous version patches.[1][2] This is whole patch series for add mtd support to spi-flash framework and related stuff. The idea is to introduce the spi-nor flash framework which similar to Linux with driver-model support. Detail changes: - drivers/mtd/spi/sf_probe.c: spi-flash to spi drivers interface(dm and non-dm) - drivers/mtd/spi/sf_ops.c: Core spi-flash functionalities. - spi_flash ops and dm_spi_ops are not needed as flash opertaion are common for dm and non-dm via MTD Changes in v7: - Rebase to master - Added MTD core support to dataflash - Few patch bisectable separations Changes in v6, v5, v4, v3, v2: - One patch bisectable separation - Rebase to master - added newly mtd stuff patches. Testing: $ git clone git://git.denx.de/u-boot-spi.git $ cd u-boot-spi $ git checkout -b spi-nor-mtd origin/next-spi-nor-mtd [1] http://u-boot.10912.n7.nabble.com/PATCH-v6-00-23-sf-MTD-support-td233769.ht… [2] http://lists.denx.de/pipermail/u-boot/2015-October/229857.html thanks! Jagan. Jagan Teki (34): sf: spi_flash_validate_params => spi_flash_scan sf: Move spi_flash_scan code to sf_ops sf: Move read_id code to sf_ops sf: probe: Code cleanup sf: Use static for file-scope functions sf: Fix Makefile sf: Use simple name for register access functions sf: Use flash function pointers in dm_spi_flash_ops sf: Flash power up read-only based on idcode0 sf: Use static for file-scope functions sf: Remove unneeded header includes sf: probe: Use spi_flash_scan in dm-spi-flash sf: Re-factorize spi_flash_probe_tail code dm-sf: Re-factorize spi_flash_std_probe code zynq: Enable CONFIG_SPL_MTD_SUPPORT sf: Add MTD support to spi_flash sf: Use mtd_info ops instead of spi_flash ops cmd_sf: Use mtd->size instead of flash->size sf: Use mtd->erasesize instead of flash->erase_size dm-sf: use mtd_ops, drop dm_spi_flash_ops sf: Use MTD lock operations sf: Add MTD support for non-dm spi_flash interface sf: probe: Minor cleanup sf: Drop SNOR_F_SST_WR flash->flags sf: Remove unneeded SST_BP and SST_WP sf: ops: Fix missing break on spansion read_bar sf: Drop SPI_FLASH_MTD driver configs: Remove CONFIG_SPI_FLASH_MTD sf: dataflash: Remove unneeded spi data sf: dataflash: Move flash id detection into jedec_probe sf: dataflash: Fix add_dataflash return logic sf: dataflash: Add MTD core support sf: dataflash: Rename sf_dataflash.c to mtd_dataflash.c mtd: dataflash: Minor cleanups common/cmd_sf.c | 16 +- drivers/mtd/spi/Kconfig | 14 +- drivers/mtd/spi/Makefile | 9 +- .../mtd/spi/{sf_dataflash.c => mtd_dataflash.c} | 402 ++++++----- drivers/mtd/spi/sf-uclass.c | 39 -- drivers/mtd/spi/sf_internal.h | 74 +- drivers/mtd/spi/sf_mtd.c | 104 --- drivers/mtd/spi/sf_ops.c | 769 ++++++++++++++++----- drivers/mtd/spi/sf_probe.c | 508 ++------------ include/configs/aristainetos-common.h | 1 - include/configs/gw_ventana.h | 1 - include/configs/ls1021aqds.h | 2 +- include/configs/socfpga_common.h | 1 - include/configs/zynq-common.h | 1 + include/spi_flash.h | 163 ++--- 15 files changed, 926 insertions(+), 1178 deletions(-) rename drivers/mtd/spi/{sf_dataflash.c => mtd_dataflash.c} (64%) delete mode 100644 drivers/mtd/spi/sf_mtd.c -- 1.9.1

4 76

[U-Boot] [PATCH] dm: core: Distinguish between OF_TRANSLATE and SPL_OF_TRANSLATE
by Michal Simek 03 Dec '15

03 Dec '15

Kconfig description is the same for both. Extend SPL one to indicate that this option is for SPL only. Signed-off-by: Michal Simek <michal.simek(a)xilinx.com> --- drivers/core/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/core/Kconfig b/drivers/core/Kconfig index 15681df6d379..b68a26ffcea3 100644 --- a/drivers/core/Kconfig +++ b/drivers/core/Kconfig @@ -136,7 +136,7 @@ config OF_TRANSLATE smaller in size than fdt_translate_address(). config SPL_OF_TRANSLATE - bool "Translate addresses using fdt_translate_address" + bool "Translate addresses using fdt_translate_address in SPL" depends on SPL_DM && SPL_OF_CONTROL default n help -- 1.9.1

2 2