summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* i2c: pdm360ng: Drop use of CONFIG_I2C_HARDSimon Glass2017-05-151-30/+0
| | | | | | Drop use of this long-deprecated option. Signed-off-by: Simon Glass <sjg@chromium.org>
* i2c: cm5200: Drop use of CONFIG_I2C_HARDSimon Glass2017-05-151-3/+3
| | | | | | Drop use of this long-deprecated option. Signed-off-by: Simon Glass <sjg@chromium.org>
* i2c: powerpc: Remove use of CONFIG_HARD_I2CSimon Glass2017-05-1514-2268/+1
| | | | | | Drop use of this long-deprecated option for powerpc. Signed-off-by: Simon Glass <sjg@chromium.org>
* i2c: Drop use of CONFIG_I2C_HARDSimon Glass2017-05-1581-701/+211
| | | | | | | | | This option is pretty old. It predates CONFIG_SYS_I2C which is itself deprecated in favour of driver model. Disable it for all boards. Also drop I2C options which depend on this. Signed-off-by: Simon Glass <sjg@chromium.org>
* Merge branch 'master' of git://git.denx.de/u-boot-mipsTom Rini2017-05-1314-41/+61
|\
| * boston: Enable CONFIG_DISTRO_DEFAULTS in defconfigsPaul Burton2017-05-124-24/+4
| | | | | | | | | | | | | | | | | | | | | | | | CONFIG_DISTRO_DEFAULTS selects a number of things we want for Boston defconfigs & generally describes what we want - to be able to boot an arbitrary Linux distribution. Enable it in order to shorten the defconfigs & to automatically keep up with any changes in the choice of Kconfig symbols selected. Signed-off-by: Paul Burton <paul.burton@imgtec.com> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
| * boston: Bump CONFIG_SYS_BOOTM_LEN to 64MiBPaul Burton2017-05-121-0/+5
| | | | | | | | | | | | | | | | | | | | The default value of CONFIG_SYS_BOOTM_LEN is too small for typical boston Linux kernels. Increase the limit to 64MB, which covers current kernels with plenty of breathing room. Signed-off-by: Paul Burton <paul.burton@imgtec.com> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
| * boston: Setup memory ranges in FDT provided to LinuxPaul Burton2017-05-123-0/+29
| | | | | | | | | | | | | | | | | | | | | | The boston memory map isn't suited to the simple "all memory starting from 0" approach that the MIPS arch_fixup_fdt() implementation takes. Instead we need to indicate the first 256MiB of DDR from 0 and the rest from 0x90000000. Implement ft_board_setup to do that. Signed-off-by: Paul Burton <paul.burton@imgtec.com> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
| * boston: Move CM GCRs away from flashPaul Burton2017-05-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | Move the MIPS Coherence Manager (CM) Global Configuration Registers (GCRs) away from the region of the physical address space which the Boston board's parallel flash is found in, such that we can access all of flash without clobbering GCRs. Signed-off-by: Paul Burton <paul.burton@imgtec.com> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
| * MIPS: Make CM GCR base configurablePaul Burton2017-05-122-9/+12
| | | | | | | | | | | | | | | | | | Without adding a prompt for CONFIG_MIPS_CM_BASE, Kconfig doesn't allow defconfigs to set it. Provide the prompt in order to allow for that. Signed-off-by: Paul Burton <paul.burton@imgtec.com> Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
| * mips: bmips: add missing SFR NeufBox 4 configÁlvaro Fernández Rojas2017-05-121-0/+6
| | | | | | | | | | | | Fixes commit a186d26, which missed including SFR NeufBox config from bmips Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
| * dm: ram: remove unneeded brcm,bcm63268-mc idÁlvaro Fernández Rojas2017-05-121-5/+1
| | | | | | | | | | | | brcm,bcm63268.dtsi uses brcm,bcm6328-mc instead of brcm,bcm63268-mc Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
| * mips: bmips: bcm63268: fix brcm, bcm6328-mc sizeÁlvaro Fernández Rojas2017-05-121-1/+1
| | | | | | | | | | | | Shrink brcm,bcm6328-mc size to avoid overlapping with other controllers Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
| * mips: bmips: bcm6328: fix brcm, bcm6328-mc sizeÁlvaro Fernández Rojas2017-05-121-1/+1
| | | | | | | | | | | | Shrink brcm,bcm6328-mc size to avoid overlapping with other controllers Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
| * mips: bmips: bcm6358: fix brcm, bcm6358-mc sizeÁlvaro Fernández Rojas2017-05-121-1/+1
| | | | | | | | | | | | Shrink brcm,bcm6358-mc size to avoid overlapping with other controllers Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
* | davinci: omapl138_lcdk: switch to using TI_COMMON_CMD_OPTIONSSekhar Nori2017-05-121-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we support using TI_COMMON_CMD_OPTIONS, we dont have to enable a number of commands explicitly in the defconfig if we enable TI_COMMON_CMD_OPTIONS. Enable TI_COMMON_CMD_OPTIONS and regenerate the defconfig using "make savedefconfig". CMD_GPIO has been kept disabled because there is no GPIO support (yet) on OMAP-L138 LCDK. Signed-off-by: Sekhar Nori <nsekhar@ti.com> Reviewed-by: Tom Rini <trini@konsulko.com>
* | davinci: omapl138_lcdk: add support for TI_COMMON_CMD_OPTIONSSekhar Nori2017-05-121-0/+2
| | | | | | | | | | | | | | | | | | | | | | OMAP-L138 LCDK board can benefit from using the commonly used commands enabled by TI_COMMON_CMD_OPTIONS. Source the relevant Kconfig file so TI_COMMON_CMD_OPTIONS can be enabled for OMAP-L138 LCDK board in a future patch. Signed-off-by: Sekhar Nori <nsekhar@ti.com> Reviewed-by: Tom Rini <trini@konsulko.com>
* | net: Disable the format-extra-args warningTom Rini2017-05-121-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | We will see warnings such as: net/eth_common.c:57:61: warning: data argument not used by format string [-Wformat-extra-args] sprintf(enetvar, index ? "%s%daddr" : "%saddr", base_name, index); ~~~~~~~~ ^ With clang. In this case we do not want to re-write our code to be less compact as the above is intentional and readable. Add a comment above the disabling so that it's clear why we want that warning off. Signed-off-by: Tom Rini <trini@konsulko.com>
* | tpm: Fix comparison of unsigned expression warningTom Rini2017-05-121-2/+1
| | | | | | | | | | | | | | | | The function tpm_xfer returns int so make 'err' be int rather than uint32_t so that we can catch an error condition. Reported by clang-3.8. Signed-off-by: Tom Rini <trini@konsulko.com>
* | gpio-uclass.c: Fix comparison of unsigned expression warningTom Rini2017-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | We declare that gpio_base (which is the base for counting gpios, not an address) is unsigned. Therefore the comparison with >= 0 is always true. As the desire is to allow for this base number to be 0, we can just drop this check. Reported by clang-3.8. Cc: Simon Glass <sjg@chromium.org> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
* | cmd/led.c: Remove unnecessary check on 'cmd' valueTom Rini2017-05-121-2/+0
| | | | | | | | | | | | | | | | | | | | | | We first check that if argc is less than 2 we return CMD_RET_USAGE. We then see if argc is greater than 2 and if so call get_led_cmd() to set 'cmd' and otherwise set it to LEDST_COUNT (which will always be positive as it's an enum with 0 already assigned). Therefore the test on if cmd is less than 0 will always be false and simply be omitted. Reported by clang-3.8. Signed-off-by: Tom Rini <trini@konsulko.com>
* | cmd/io.c: Fix comparison of unsigned expression warningTom Rini2017-05-121-1/+2
| | | | | | | | | | | | | | | | The function cmd_get_data_size() returns an int and not unsigned. So we should assign it to an int rather than unsigned so that we can later compare the return value. Reported by clang-3.8. Signed-off-by: Tom Rini <trini@konsulko.com>
* | cmd/bdinfo.c: Fix unused function warningTom Rini2017-05-121-1/+1
| | | | | | | | | | | | | | On most architectures we do not call print_std_bdinfo() so mark it with __maybe_unused. Reported by clang-3.8. Signed-off-by: Tom Rini <trini@konsulko.com>
* | Kconfig: Drop CONFIG_EMAC_MDIO_PHY_NUMTom Rini2017-05-125-5/+0
| | | | | | | | | | | | | | This particular macro hasn't been used in the code for some time, remove these references that were missed. Signed-off-by: Tom Rini <trini@konsulko.com>
* | tools/genboardscfg.py: Make 'Supported' as known statusLokesh Vutla2017-05-121-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | As per MAINTAINERS[1] file description, 'Supported' is a valid status for a board. But buildman thinks 'Maintained' is the only valid state and complains about boards with 'Supported' status. Update buildman to accept 'Supported' as valid state. [1] http://git.denx.de/?p=u-boot.git;a=blob;f=MAINTAINERS;h=0962b47bf9057b22e93624e070c0204b893790dc;hb=HEAD#l10 Reported-by: Sekhar Nori <nsekhar@ti.com> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> Reviewed-by: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
* | asm-generic: global_data: change timebase_l/h to unsigned intPeng Fan2017-05-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change type of timebase_l/h to unsigned int. >From lib/time.c: ((uint64_t)gd->timebase_h << 32) | gd->timebase_l; This piece code is based on that timebase_h and timebase_l are 32bits width, so change the type to unsigned int. Signed-off-by: Peng Fan <peng.fan@nxp.com> Cc: Eddie Cai <eddie.cai.linux@gmail.com> Cc: Jagan Teki <jteki@openedev.com> Cc: York Sun <york.sun@nxp.com> Cc: "Robert P. J. Day" <rpjday@crashcourse.ca> Cc: Michal Simek <michal.simek@xilinx.com> Cc: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
* | arm: change tbu/l type to unsigned intPeng Fan2017-05-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Change tbu/l type to unsigned int. >From the timer file for arm, "(((unsigned long long)gd->arch.tbu) << 32) | gd->arch.tbl;" is used, This piece code is based on tbu/tbl is 32bits, so change the type to unsigned int. Signed-off-by: Peng Fan <peng.fan@nxp.com> Cc: Simon Glass <sjg@chromium.org> Cc: Tom Rini <trini@konsulko.com>
* | m5253demo: Fix static variable in non-static inline function warningTom Rini2017-05-121-2/+2
| | | | | | | | | | | | | | | | | | The function 'spin_wheel' is declared as inline, but not static and thus we see warnings that 'w' and 'p' are declared static in a non-static inline function. Correct this by marking spin_wheel as static inline. Cc: TsiChung Liew <Tsi-Chung.Liew@freescale.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | common: Only build cli_readline.o for CMDLINE on non-SPLTom Rini2017-05-121-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With gcc-6 and later we may get a warning such as: .../common/cli_readline.c:20:21: warning: ‘tab_seq’ defined but not used [-Wunused-const-variable=] static const char tab_seq[] = " "; /* used to expand TABs */ ^~~~~~~ .../common/cli_readline.c:19:19: warning: ‘erase_seq’ defined but not used [-Wunused-const-variable=] static const char erase_seq[] = "\b \b"; /* erase sequence */ ^~~~~~~~~ Because in SPL we're normally not doing interactive commands anyhow, so lets just not compile this at all in SPL. This also means that we need to correct the logic (and comment) about what the drivers/ddr/fsl/ and CONFIG_FSL_DDR_INTERACTIVE requires and this will be included in SPL there. Signed-off-by: Tom Rini <trini@konsulko.com>
* | net: uli526x: Fix unknown storage size errorTom Rini2017-05-121-1/+0
| | | | | | | | | | | | | | | | | | The variable netdev_ethtool_ops is not referenced, drop it. However with gcc-6 or later we fail to even compile as we do not have the required struct definition in U-Boot. Cc: Joe Hershberger <joe.hershberger@ni.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | net: phy: mv88e61xx: Fix uninitialized variable warningTom Rini2017-05-121-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | The variable 'res' may be unused uninitialized if our call to mv88e61xx_port_read (register read) fails and we goto the error handling section. In this case we set 'res' to -EIO to indicate why we failed. Cc: Joe Hershberger <joe.hershberger@ni.com> Cc: Chris Packham <judge.packham@gmail.com> Cc: Kevin Smith <kevin.smith@elecsyscorp.com> Cc: Prafulla Wadaskar <prafulla@marvell.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | net: eepro100: Fix unused variable warningTom Rini2017-05-121-6/+0
| | | | | | | | | | | | | | The variable i82557_config_cmd is never referenced, drop. Cc: Joe Hershberger <joe.hershberger@ni.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | video: ld9040: Fix unused variable warningsTom Rini2017-05-121-16/+0
| | | | | | | | | | | | | | | | The variables SEQ_SWRESET, SEQ_ELVSS_ON, SEQ_TEMP_SWIRE, SEQ_APON and SEQ_SLPIN are unreferenced, drop. Cc: Anatolij Gustschin <agust@denx.de> Signed-off-by: Tom Rini <trini@konsulko.com>
* | colibri_imx7: Fix unused variable warningTom Rini2017-05-121-16/+0
| | | | | | | | | | | | | | The variable usdhc3_emmc_pads is never referenced, drop. Cc: Stefan Agner <stefan.agner@toradex.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | apalis_imx6: Fix unused variable warningTom Rini2017-05-121-47/+0
| | | | | | | | | | | | | | | | The variable vga_pads is never referenced, drop. Cc: Max Krummenacher <max.krummenacher@toradex.com> Signed-off-by: Tom Rini <trini@konsulko.com> Acked-by: Max Krummenacher <max.krummenacher@toradex.com>
* | pcm058: Fix unused variable warningsTom Rini2017-05-121-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | The variable nfc_pads is only referenced when CONFIG_CMD_NAND is set, add a gaurd. The variable gpio_pads is never referenced, drop it. The variable usdhc4_pads are only referenced when we do not have CONFIG_CMD_NAND set and we are not doing an SPL build, modify the existing guard. Cc: Stefano Babic <sbabic@denx.de> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Stefano Babic <sbabic@denx.de>
* | gdsys: P1022: Fix unused variable warningsTom Rini2017-05-121-25/+24
| | | | | | | | | | | | | | | | | | The variables prg_stage2_prepare, prg_stage2_success and prg_stage_fail are only referenced when CCDM_SECOND_STAGE is set, move these to be by the existing guard. Cc: Dirk Eibach <eibach@gdsys.de> Signed-off-by: Tom Rini <trini@konsulko.com>
* | gw_ventana: Fix unused variable warningsTom Rini2017-05-122-29/+1
| | | | | | | | | | | | | | | | | | The variable nfc_pads is only referenced when CONFIG_CMD_NAND is set, move the existing guard and drop a now redundant comment. The variable gwproto_gpio_pads is never referenced, remove it. Cc: Tim Harvey <tharvey@gateworks.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | mx6ul_14x14_evk: Fix unused variable warningTom Rini2017-05-121-0/+2
| | | | | | | | | | | | | | | | | | | | The variable usdhc1_pads is only referenced during SPL builds, add a guard. Cc: Stefano Babic <sbabic@denx.de> Cc: Peng Fan <peng.fan@nxp.com> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Peng Fan <peng.fan@nxp.com> Reviewed-by: Stefano Babic <sbabic@denx.de>
* | mx6slevk: Fix unused variable warningTom Rini2017-05-121-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | The variable usdhc1_pads is only referenced during SPL builds, add a guard. Cc: Fabio Estevam <fabio.estevam@nxp.com> Cc: Peng Fan <peng.fan@nxp.com> Cc: Stefano Babic <sbabic@denx.de> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Peng Fan <peng.fan@nxp.com> Reviewed-by: Stefano Babic <sbabic@denx.de>
* | cgtqmx6eval: Fix unused variable warningTom Rini2017-05-121-0/+2
| | | | | | | | | | | | | | The variable usdhc2_pads is only referenced during SPL builds, add a guard. Cc: Otavio Salvador <otavio@ossystems.com.br> Signed-off-by: Tom Rini <trini@konsulko.com>
* | ot1200: Fix unused variable warningTom Rini2017-05-121-4/+0
| | | | | | | | | | | | | | | | The variable pwm_pad is never referenced, drop. Cc: Christian Gmeiner <christian.gmeiner@gmail.com> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
* | rk3036: Fix unused variable warningTom Rini2017-05-121-1/+1
| | | | | | | | | | | | | | | | | | The variable grf is only referenced if EARLY_DEBUG is defined so move the declaration to be under the existing guard. Cc: Simon Glass <sjg@chromium.org> Signed-off-by: Tom Rini <trini@konsulko.com> Reviewed-by: Simon Glass <sjg@chromium.org>
* | socrates: Fix a misleading indentation warningTom Rini2017-05-121-1/+1
| | | | | | | | | | | | | | | | | | | | With gcc-6 and later we see a warning about the fact that we have a construct of "if (test);\n\tstatement". Upon reviewing the code, the intention here is as the compiler suggests, we only want to execute the indented statement if the test was true. Cc: Sergei Poselenov <sposelenov@emcraft.com> Signed-off-by: Tom Rini <trini@konsulko.com>
* | common: env: remove superfluous assignmentxypron.glpk@gmx.de2017-05-121-1/+0
| | | | | | | | | | | | | | | | The value assigned to variable 'value' is never used. The problem was indicated by clang scan-build. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
* | tools: kwbimage fix build with OpenSSL 1.1.xJelle van der Waa2017-05-121-6/+30
| | | | | | | | | | | | | | | | | | | | The rsa_st struct has been made opaque in 1.1.x, add forward compatible code to access the n, e, d members of rsa_struct. EVP_MD_CTX_cleanup has been removed in 1.1.x and EVP_MD_CTX_reset should be called to reinitialise an already created structure. Signed-off-by: Jelle van der Waa <jelle@vdwaa.nl>
* | rsa: Fix build with OpenSSL 1.1.xJelle van der Waa2017-05-121-6/+38
| | | | | | | | | | | | | | | | The rsa_st struct has been made opaque in 1.1.x, add forward compatible code to access the n, e, d members of rsa_struct. EVP_MD_CTX_cleanup has been removed in 1.1.x and EVP_MD_CTX_reset should be called to reinitialise an already created structure.
* | net/arp: remove superfluous assignmentsxypron.glpk@gmx.de2017-05-121-3/+0
| | | | | | | | | | | | | | | | The value of variable pkt is never used. The problem was indicated by clang scan-build. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
* | lib/slre: remove superfluous assignmentxypron.glpk@gmx.de2017-05-121-1/+1
| | | | | | | | | | | | | | | | The value assigned to saved_offset is never used. The problem was indicated by clang scan-build. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
* | pci: avoid memory leakxypron.glpk@gmx.de2017-05-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | strdup uses malloc to allocate memory for str. If we cannot bind to the generic driver we should release the memory. The problem was indicated by clang scan-build. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
generated by cgit (git 2.34.1) at 2024-09-30 14:24:56 +0000