]> git.sur5r.net Git - u-boot/log
u-boot
7 years agoARM: dts: OMAP5+: Update spl specific dts
Lokesh Vutla [Fri, 16 Jun 2017 08:42:58 +0000 (14:12 +0530)]
ARM: dts: OMAP5+: Update spl specific dts

Now that we can specify DT nodes that can be used in spl, mark
all necessary nodes as u-boot,dm-spl.

Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoARM: dts: am43xx: Update spl specific dts
Lokesh Vutla [Fri, 16 Jun 2017 08:42:57 +0000 (14:12 +0530)]
ARM: dts: am43xx: Update spl specific dts

Now that we can specify DT nodes that can be used in spl, mark
all necessary nodes as u-boot,dm-spl.

Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoti816x: Add additional boot device detection logic
Tom Rini [Thu, 15 Jun 2017 17:57:00 +0000 (13:57 -0400)]
ti816x: Add additional boot device detection logic

It has been observed that between PG1.0 and PG2.0/2.1 depending on
which device we boot from, we may see a different value here than is
documented in the TRM.  Update the values for NAND and MMC1 based on
real life usage on each revision.

Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agoarm: omap: Extract OMAP5 boot environment to separate file
Semen Protsenko [Wed, 14 Jun 2017 18:34:23 +0000 (21:34 +0300)]
arm: omap: Extract OMAP5 boot environment to separate file

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoMerge git://www.denx.de/git/u-boot-marvell
Tom Rini [Fri, 23 Jun 2017 12:23:14 +0000 (08:23 -0400)]
Merge git://www.denx.de/git/u-boot-marvell

7 years agopinctrl: a3700: Fix the issue that gpio controller is registered with wrong node id
Ken Ma [Thu, 22 Jun 2017 09:13:36 +0000 (17:13 +0800)]
pinctrl: a3700: Fix the issue that gpio controller is registered with wrong node id

In armada_37xx_gpiochip_register, the return value of fdtdec_get_bool
should be true when gpio-controller is found; current codes makes a
wrong inverse return value judgement, this patch fixes it.

Signed-off-by: Ken Ma <make@marvell.com>
Cc: Stefan Roese <sr@denx.de>
Cc: Kostya Porotchkin <kostap@marvell.com>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Nadav Haklai <nadavh@marvell.com>
Cc: Wilson Ding <dingwei@marvell.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Signed-off-by: Stefan Roese <sr@denx.de>
7 years agopinctrl: a3700: Fix uart2 group selection register mask
Ken Ma [Thu, 22 Jun 2017 09:13:35 +0000 (17:13 +0800)]
pinctrl: a3700: Fix uart2 group selection register mask

If north bridge selection register bit1 is clear, pins [10:8] are for
SDIO0 Resetn, Wakeup, and PDN while if bit1 is set, pins [10:8]are for
GPIO; when bit1 is clear, pin 9 and pin 10 can be used for uart2 RTSn
and CTSn, so bit1 should be added to uart2 group and it must be set
for both "gpio" and "uart" functions of uart2 group.

Signed-off-by: Ken Ma <make@marvell.com>
Cc: Stefan Roese <sr@denx.de>
Cc: Kostya Porotchkin <kostap@marvell.com>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Nadav Haklai <nadavh@marvell.com>
Cc: Wilson Ding <dingwei@marvell.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Signed-off-by: Stefan Roese <sr@denx.de>
7 years agox86: Remove CONFIG_USB_MAX_CONTROLLER_COUNT
Bin Meng [Thu, 22 Jun 2017 06:42:40 +0000 (14:42 +0800)]
x86: Remove CONFIG_USB_MAX_CONTROLLER_COUNT

As all x86 boards have been switched over to use DM USB, remove
CONFIG_USB_MAX_CONTROLLER_COUNT which is not used by DM USB.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Tested-by: Stefan Roese <sr@denx.de>
7 years agox86: minnowmax: Configure GPIO pins to turn on USB ports VBUS
Bin Meng [Fri, 16 Jun 2017 13:31:46 +0000 (06:31 -0700)]
x86: minnowmax: Configure GPIO pins to turn on USB ports VBUS

GPIO bank E pin 8 & 9 are used to control the on-board two USB ports
VBUS on/off. Let's configure them in the misc_init_r().

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Stefan Roese <sr@denx.de>
Tested-by: Stefan Roese <sr@denx.de>
7 years agoMerge branch 'master' of git://git.denx.de/u-boot-samsung
Tom Rini [Wed, 21 Jun 2017 12:01:07 +0000 (08:01 -0400)]
Merge branch 'master' of git://git.denx.de/u-boot-samsung

7 years agoMerge branch 'master' of git://git.denx.de/u-boot-tegra
Tom Rini [Wed, 21 Jun 2017 12:00:04 +0000 (08:00 -0400)]
Merge branch 'master' of git://git.denx.de/u-boot-tegra

7 years agoMerge tag 'xilinx-for-v2017.07' of git://www.denx.de/git/u-boot-microblaze
Tom Rini [Wed, 21 Jun 2017 11:57:37 +0000 (07:57 -0400)]
Merge tag 'xilinx-for-v2017.07' of git://www.denx.de/git/u-boot-microblaze

Xilinx changes for v2017.07

ZynqMP:
- config cleanup
- SD LS mode support
- psu_init* cleanup
- unmap OCM
- Support for SMC

Zynq:
- add ddrc to Kconfig
- add topic-miamilite board support

7 years agoMAINTAINERS: Add myself as i.MX co-maintainer
Fabio Estevam [Tue, 6 Jun 2017 12:18:43 +0000 (09:18 -0300)]
MAINTAINERS: Add myself as i.MX co-maintainer

I would like to help Stefano Babic as a co-maintainer of
U-Boot i.MX.

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
Acked-by: Stefano Babic <sbabic@denx.de>
Acked-by: Jagan Teki <jagan@amarulasolutions.com>
7 years agosunxi: Correct select's of SPL_STACK_R and SPL_SYS_MALLOC_SIMPLE
Tom Rini [Wed, 21 Jun 2017 11:54:46 +0000 (07:54 -0400)]
sunxi: Correct select's of SPL_STACK_R and SPL_SYS_MALLOC_SIMPLE

On ARCH_SUNXI we've been selecting these targets for a long time if
SUPPORT_SPL is set.  However, Lichee Pi Zero is the first platform we've
added that does support SPL but does not build SPL and has exposed a
latent bug.  Both of these symbols depend on SPL not SUPPORT_SPL, so we
need to update our select here otherwise we get a Kconfig warning.

Fixes: f02abb0608fe ("sunxi: add support for Lichee Pi Zero")
Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agoARM: rmobile: Add missing config bits
Marek Vasut [Tue, 20 Jun 2017 13:17:19 +0000 (15:17 +0200)]
ARM: rmobile: Add missing config bits

The commit "fb82fe385173 configs: Resync defconfigs" resynced all config
files.  This exposed two latent issues with the Gen3 boards in that we
had not been setting CONFIG_DEFAULT_FDT_FILE correctly and had not been
setting CONFIG_CMD_MMC.  Fix both of these.

Fixes: fb82fe385173 ("configs: Resync defconfigs")
Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agoARM: dts: omap3: Fix dts->dtb typo
Marek Vasut [Tue, 20 Jun 2017 12:39:31 +0000 (14:39 +0200)]
ARM: dts: omap3: Fix dts->dtb typo

Trivial, fix typo.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Tom Rini <trini@konsulko.com>
7 years agobch: Fix build on FreeBSD host
Emmanuel Vadot [Tue, 20 Jun 2017 07:02:29 +0000 (09:02 +0200)]
bch: Fix build on FreeBSD host

endian.h on FreeBSD system exist in sys/ subdirectory.
FreeBSD already have a fls function defined in strings.h which is included
in string.h if __BSD_VISIBLE is defined, as a check for this.

Signed-off-by: Emmanuel Vadot <manu@bidouilliste.com>
7 years agotegra: mmc: Set the bus width correctly
Simon Glass [Thu, 8 Jun 2017 03:11:48 +0000 (21:11 -0600)]
tegra: mmc: Set the bus width correctly

The driver currently does not reset bit 5 of the hostctl register even if
the MMC stack requests it. Then means that once a bus width of 8 is
selected it is not possible to change it back to 1. This breaks
'mmc rescan' which needs to start off with a bus width of 1.

The problem was surfaced by enabling CONFIG_DM_MMC_OPS on tegra. Without
this option the MMC stack fully reinits the driver on a 'mmc rescan'.
But with this option driver model does not re-probe a driver once it has
been probed once.

Fix the driver to honour the request.

Signed-off-by: Simon Glass <sjg@chromium.org>
Tested-by: Peter Chubb <peter.chubb@data61.csiro.au>
Tested-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
7 years agoarm: zynq: Add support for the topic-miamilite system-on-module
Mike Looijmans [Wed, 7 Jun 2017 05:42:38 +0000 (07:42 +0200)]
arm: zynq: Add support for the topic-miamilite system-on-module

The topic-miamilite SoM contains a Zynq xc7z010 SoC, 1GB DDR3L RAM,
64MB dual-parallel QSPI NOR flash and clock sources.

Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm: zynq: Move CONFIG_SF_DUAL_FLASH to defconfig
Mike Looijmans [Wed, 7 Jun 2017 05:42:37 +0000 (07:42 +0200)]
arm: zynq: Move CONFIG_SF_DUAL_FLASH to defconfig

Move the only use of CONFIG_SF_DUAL_FLASH to defconfig. This makes the
associated topic_miamiplus.h header obsolete, so remove that as well.

Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Check pmufw version
Michal Simek [Tue, 7 Feb 2017 13:32:26 +0000 (14:32 +0100)]
arm64: zynqmp: Check pmufw version

If PMUFW version is not v0.3 then panic.
ZynqMP switch to CCF based clock driver which requires
PMUFW to be present at certain version.
This patch ensure that you use correct and tested PMUFW
binary.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agofpga: zynqmppl: Reuse invoke_smc routine
Siva Durga Prasad Paladugu [Fri, 17 Feb 2017 10:46:01 +0000 (16:16 +0530)]
fpga: zynqmppl: Reuse invoke_smc routine

Reuse invoke_smc() routine which is already defined
instead of duplicating same at multiple places.

Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Define routines for mmio write and read
Siva Durga Prasad Paladugu [Wed, 1 Feb 2017 19:40:46 +0000 (01:10 +0530)]
arm64: zynqmp: Define routines for mmio write and read

Define routines of mmio write and read functionalities
for zynqmp platform.

Also do not call SMC from SPL because SPL is running before ATF in EL3
that's why SMCs can't be called because there is nothing to call.
zynqmp_mmio*() are doing direct read/write accesses and this patch does
the same. PMUFW is up and running at this time and there is a way to talk
to pmufw via IPI but there is no reason to implement IPI stuff in SPL if
we need just simple read for getting clock driver to work.

Also make invoke_smc as global so that it can be reused in
multile places where ever possible.

Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm: zynq: Add Kconfig option for any DDR specific initialization
Siva Durga Prasad Paladugu [Fri, 12 May 2017 09:34:11 +0000 (15:04 +0530)]
arm: zynq: Add Kconfig option for any DDR specific initialization

Add Kconfig option for ddr init as this might be required
in cases like ddr less systems where we want to skip ddrc
init and this option is useful for it.

Signed-off-by: Siva Durga Prasad Paladugu <sivadur@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Do not map unused OCM/TCM region
Michal Simek [Sat, 25 Feb 2017 17:17:11 +0000 (18:17 +0100)]
arm64: zynqmp: Do not map unused OCM/TCM region

When OCM or TCM is protected this mapping still exist and it is causing access
violation.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Add comment about level shifter mode v1
Michal Simek [Thu, 2 Mar 2017 10:02:55 +0000 (11:02 +0100)]
arm64: zynqmp: Add comment about level shifter mode v1

Silicon v1 didn't support SD boot mode with level shifter.
Because system can't boot any error message is not shown
that's why comment is just a record if someone tries to debug it.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Add empty sleep.h file for psu_init* compilation
Michal Simek [Thu, 25 May 2017 14:08:43 +0000 (16:08 +0200)]
arm64: zynqmp: Add empty sleep.h file for psu_init* compilation

psu_init* contain sleep.h header which is not present in u-boot.
Instead of keep comment sleep.h in psu_init* it is easier to add empty
file which is included.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoblock: sata: ceva: drop extraneous netdev.h include
Peter Robinson [Sun, 14 May 2017 20:47:02 +0000 (21:47 +0100)]
block: sata: ceva: drop extraneous netdev.h include

Drop include of netdev.h as it's a SATA driver not a network driver.

Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Decrease MALLOC size for SPL
Michal Simek [Fri, 14 Oct 2016 12:06:28 +0000 (14:06 +0200)]
arm64: zynqmp: Decrease MALLOC size for SPL

Decrease malloc size to 1MB from 256MB. Huge malloc
space is adding huge delay in mem_malloc_init() because
we are enabling CONFIG_SYS_MALLOC_CLEAR_ON_INIT
which clear the whole malloc space.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoPrepare v2017.07--rc2 v2017.07-rc2
Tom Rini [Tue, 20 Jun 2017 00:41:27 +0000 (20:41 -0400)]
Prepare v2017.07--rc2

Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agoarm64: zynqmp: Fix psu_init* external functions
Michal Simek [Wed, 15 Feb 2017 09:13:22 +0000 (10:13 +0100)]
arm64: zynqmp: Fix psu_init* external functions

In older vivado version some psu_init* files didn't contain
mask*() which were missing.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: spl: use given boot_device instead of fetching it again
Jean-Francois Dagenais [Mon, 3 Apr 2017 01:44:35 +0000 (21:44 -0400)]
arm64: zynqmp: spl: use given boot_device instead of fetching it again

The boot_device argument to spl_boot_mode was massively added without
actually modifying the existing functions.

This commit actually makes use of the handed value, which is the same.

Signed-off-by: Jean-Francois Dagenais <jeff.dagenais@gmail.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: spl: fix dual SD controller support
Jean-Francois Dagenais [Mon, 3 Apr 2017 01:44:34 +0000 (21:44 -0400)]
arm64: zynqmp: spl: fix dual SD controller support

When enabling both SDHCI controllers, spl_mmc.c would actually choose
device sdhci0 even if booted from sdhci1 (boot_device). This is because
spl_mmc_get_device_index(boot_device) expects BOOT_DEVICE_MMC2[_2] in
order to return index 1 instead of 0.

The #if defined(...) statement is copied from board/xilinx/zynqmp/zynqmp.c

So the key to properly enabling both controllers as boot sources is
defining both CONFIG_ZYNQ_SDHCI0 and CONFIG_ZYNQ_SDHCI1 in your board's
include/configs/*.h.

Signed-off-by: Jean-Francois Dagenais <jeff.dagenais@gmail.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Wire SD1 level shifter mode to SPL
Michal Simek [Wed, 15 Feb 2017 08:10:16 +0000 (09:10 +0100)]
arm64: zynqmp: Wire SD1 level shifter mode to SPL

Add missing SD boot mode to SPL. zcu102-rev1.0 is supporting
this boot mode.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agomicroblaze: Build only DTBs for selected target
Michal Simek [Wed, 15 Feb 2017 07:22:04 +0000 (08:22 +0100)]
microblaze: Build only DTBs for selected target

Adding more targets to repository requires some additional
changes not simply just adding config file, defconfig and dts.
This patch makes this process easier by building only
particular DTB which is selected via defconfig
that Makefile doesn't need to contain all dts files in the repository.

Reported-by: Nathan Rossi <nathan@nathanrossi.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoarm64: zynqmp: Remove CPU_RELEASE_ADDR macro
Michal Simek [Wed, 3 May 2017 14:22:30 +0000 (16:22 +0200)]
arm64: zynqmp: Remove CPU_RELEASE_ADDR macro

CPU_RELEASE_ADDR is used only when CONFIG_ARMV8_MULTIENTRY
is enabled. ZynqMP is running ATF which takes care about this
that's why no need to have this macro.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
7 years agoconfigs: Resync defconfigs
Tom Rini [Mon, 19 Jun 2017 13:47:40 +0000 (09:47 -0400)]
configs: Resync defconfigs

Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agoMerge git://git.denx.de/u-boot-usb
Tom Rini [Mon, 19 Jun 2017 12:08:57 +0000 (08:08 -0400)]
Merge git://git.denx.de/u-boot-usb

7 years agoMerge branch 'master' of git://git.denx.de/u-boot-sunxi
Tom Rini [Mon, 19 Jun 2017 12:08:40 +0000 (08:08 -0400)]
Merge branch 'master' of git://git.denx.de/u-boot-sunxi

7 years agoodroid: remove CONFIG_DM_I2C_COMPAT config
Jaehoon Chung [Wed, 7 Jun 2017 03:08:40 +0000 (12:08 +0900)]
odroid: remove CONFIG_DM_I2C_COMPAT config

Remove the CONFIG_DM_I2C_COMPAT config.

Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
7 years agocmd: usb_mass_storage: Staticize do_usb_mass_storage()
Fabio Estevam [Sat, 28 May 2016 01:37:11 +0000 (22:37 -0300)]
cmd: usb_mass_storage: Staticize do_usb_mass_storage()

Make do_usb_mass_storage() static to fix the following sparse
warning:

cmd/usb_mass_storage.c:136:5: warning: symbol 'do_usb_mass_storage' was not declared. Should it be static?

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
7 years agocmd: usb_mass_storage: Use NULL for pointer
Fabio Estevam [Sat, 28 May 2016 01:37:10 +0000 (22:37 -0300)]
cmd: usb_mass_storage: Use NULL for pointer

Use NULL for pointer to fix the following sparse warning:
cmd/usb_mass_storage.c:47:15: warning: Using plain integer as NULL pointer

Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
7 years agoat91_udc.c: Fix unused variable warning
Tom Rini [Wed, 19 Apr 2017 02:23:51 +0000 (22:23 -0400)]
at91_udc.c: Fix unused variable warning

With gcc-6 and later we see warnings that at91sam9263_udc_caps and
at91rm9200_udc_caps are unused.

Fixes: 620197670a69 ("usb: gadget: at91_udc: add at91_udc into U-Boot")
Cc: Lukasz Majewski <lukma@denx.de>
Cc: Marek Vasut <marex@denx.de>
Signed-off-by: Tom Rini <trini@konsulko.com>
Acked-by: Heiko Schocher <hs@denx.de>
7 years agodrivers/usb/ehci: Use platform-specific accessors
Alexey Brodkin [Mon, 5 Jun 2017 19:31:51 +0000 (22:31 +0300)]
drivers/usb/ehci: Use platform-specific accessors

Current implementation doesn't allow utilization of platform-specific
reads and writes.

But some arches or platforms may want to use their accessors that do
some extra work like adding barriers for data serialization etc.

Interesting enough OHCI accessors already do that so just aligning
EHCI to it now.

This is a resend of http://patchwork.ozlabs.org/patch/726714/
Back in the day this patch broke some PPC and Sandbox boards
as they we missing inclusion of "asm/io.h". Those missing items were
fixed with:
 1) http://patchwork.ozlabs.org/patch/751397/
 2) http://patchwork.ozlabs.org/patch/771099/

So now it should be safe to apply this patch.
FWIW TravisCI builds everything with all 3 patches in place,
see https://travis-ci.org/abrodkin/u-boot/builds/239563813

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Mateusz Kulikowski <mateusz.kulikowski@gmail.com>
Acked-by: Marek Vasut <marex@denx.de>
7 years agousb: dwc2: force to host mode if not support HNP/SRP
Meng Dongyang [Thu, 8 Jun 2017 07:34:20 +0000 (15:34 +0800)]
usb: dwc2: force to host mode if not support HNP/SRP

In current code, after running the command of "usb start", the controller
will keep in otg mode and can't switch to host mode if not support
SNP/SRP capability. So add the property of "hnp-srp-disable" in the DTS
to config the contrller work in force mode of host.

Signed-off-by: Meng Dongyang <daniel.meng@rock-chips.com>
7 years agopowerpc, 5xx: remove some "5xx" remains
Heiko Schocher [Wed, 14 Jun 2017 03:49:42 +0000 (05:49 +0200)]
powerpc, 5xx: remove some "5xx" remains

we removed 5xx support. So delete some forgotten remains.

Signed-off-by: Heiko Schocher <hs@denx.de>
7 years agopowerpc, 82xx: remove some missed mpc82xx remains
Heiko Schocher [Wed, 14 Jun 2017 03:49:41 +0000 (05:49 +0200)]
powerpc, 82xx: remove some missed mpc82xx remains

we removed 82xx support. Missed some 82xx remains,
remove them now.

Signed-off-by: Heiko Schocher <hs@denx.de>
7 years agopowerpc, 5xxx, 512x: remove support for mpc5xxx and mpc512x
Heiko Schocher [Wed, 14 Jun 2017 03:49:40 +0000 (05:49 +0200)]
powerpc, 5xxx, 512x: remove support for mpc5xxx and mpc512x

There was for long time no activity in the mpx5xxx area.
We need to go further and convert to Kconfig, but it
turned out, nobody is interested anymore in mpc5xxx,
so remove it.

Signed-off-by: Heiko Schocher <hs@denx.de>
7 years agoarm: mach-omap2: Generate MLO file from SD boot capable targets
Andrew F. Davis [Tue, 13 Jun 2017 20:04:59 +0000 (15:04 -0500)]
arm: mach-omap2: Generate MLO file from SD boot capable targets

Secure boot targets that can be loaded from an SD card FAT partition
need to be called "MLO" on the filesystem, make a copy with this name
to clarify the correct image for SD card booting.

Signed-off-by: Andrew F. Davis <afd@ti.com>
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
7 years agocmd: ethsw: Fix out-of-bounds error
York Sun [Tue, 13 Jun 2017 16:50:41 +0000 (09:50 -0700)]
cmd: ethsw: Fix out-of-bounds error

The for loop in cmd_keywords_opt_check() seems to use wrong array to
set boundary, reported by Coverity analysis.

Signed-off-by: York Sun <york.sun@nxp.com>
CC: Joe Hershberger <joe.hershberger@ni.com>
CC: Tom Rini <trini@konsulko.com>
CC: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Reported-by: Coverity (CID: 163251)
Reviewed-by: Tom Rini <trini@konsulko.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
7 years agoarm: ti: Add missing guards to headers
Semen Protsenko [Tue, 13 Jun 2017 15:15:58 +0000 (18:15 +0300)]
arm: ti: Add missing guards to headers

To prevent possible double inclusions in future.

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoboard/BuR/brxre1: refactor default environment
Hannes Schmelzer [Tue, 13 Jun 2017 12:11:33 +0000 (14:11 +0200)]
board/BuR/brxre1: refactor default environment

For better readability and setup-handling we refactor the default
environment.

Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoboard/BuR/brxre1: cosmetic cleanup of config-header
Hannes Schmelzer [Tue, 13 Jun 2017 12:11:32 +0000 (14:11 +0200)]
board/BuR/brxre1: cosmetic cleanup of config-header

Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoboard/BuR/brxre1: drop obsolete CONFIG_LCD_NOSTDOUT from config-header
Hannes Schmelzer [Tue, 13 Jun 2017 12:11:31 +0000 (14:11 +0200)]
board/BuR/brxre1: drop obsolete CONFIG_LCD_NOSTDOUT from config-header

This define isn't used anymore in the u-boot source tree, so we drop it
from the config header.

We drop it also from the 'config_whitelist.txt'

Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoboard/BuR/brxre1: drop bootlogo feature
Hannes Schmelzer [Tue, 13 Jun 2017 12:11:30 +0000 (14:11 +0200)]
board/BuR/brxre1: drop bootlogo feature

the BMP logo loading has become obsolete and is not used any more. So we
drop the support for it.

Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoboard/BuR/brxre1: fix MMC boot
Hannes Schmelzer [Tue, 13 Jun 2017 12:11:29 +0000 (14:11 +0200)]
board/BuR/brxre1: fix MMC boot

since commit

'd5abcf94c7123167725fc22ace342f0d455093c1' -
ti: boot: Register the MMC controllers in SPL in the same way as in u-boot

MMC boot on brxre1 board is broken, with this commit we make our board
working again.

Signed-off-by: Hannes Schmelzer <hannes.schmelzer@br-automation.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Jean-Jacques Hiblot <jjhiblot@ti.com>
7 years agotreewide: remove unneeded semicolons
Masahiro Yamada [Tue, 13 Jun 2017 06:17:28 +0000 (15:17 +0900)]
treewide: remove unneeded semicolons

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
7 years agodefconfig: arm335x_evm_usbspl: Update MUSB Ethernet
Michael Welling [Mon, 12 Jun 2017 23:02:39 +0000 (18:02 -0500)]
defconfig: arm335x_evm_usbspl: Update MUSB Ethernet

Adds CONFIG_SPL_USB_GADGET_SUPPORT and CONFIG_SPL_USBETH_SUPPORT.

Removes CONFIG_SPL_NAND_SUPPORT, CONFIG_SPL_MTD_SUPPORT, and CONFIG_SPL_YMODEM_SUPPORT
to free up space to fit in RAM.

Signed-off-by: Michael Welling <mwelling@ieee.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Marek Vasut <marex@denx.de>
7 years agoinclude: nand: move endif to end of file
Mugunthan V N [Mon, 23 Jan 2017 22:04:34 +0000 (16:04 -0600)]
include: nand: move endif to end of file

The terminator endif of ifdef _NAND_H_ should be at the
end of file as a fail safe.

Signed-off-by: Mugunthan V N <mugunthanvnm@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
7 years agoMerge git://git.denx.de/u-boot-mpc85xx
Tom Rini [Wed, 14 Jun 2017 22:53:03 +0000 (18:53 -0400)]
Merge git://git.denx.de/u-boot-mpc85xx

7 years agoMerge branch 'master' of git://git.denx.de/u-boot-tegra
Tom Rini [Wed, 14 Jun 2017 22:52:49 +0000 (18:52 -0400)]
Merge branch 'master' of git://git.denx.de/u-boot-tegra

7 years agosun50i: h5: Add initial NanoPi NEO2 support
Jagan Teki [Tue, 30 May 2017 17:41:15 +0000 (17:41 +0000)]
sun50i: h5: Add initial NanoPi NEO2 support

NanoPi NEO2 is designed and developed by FriendlyElec
using the Allwinner 64-bit H5 SOC.

NanoPi Neo2 key features
- Allwinner H5, Quad-core 64-bit Cortex-A53
- 512MB DDR3 RAM
- microSD slot
- 10/100/1000M Ethernet
- Serial Debug Port
- 5V 2A DC MicroUSB power-supply

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Tested-by: Jagan Teki <jagan@amarulasolutions.com>
7 years agoRevert "ARM: fixed relocation using proper alignment"
Tom Rini [Wed, 14 Jun 2017 13:13:21 +0000 (09:13 -0400)]
Revert "ARM: fixed relocation using proper alignment"

It turns out this change was not intended to be merged and as such,
revert it.

This reverts commit cdde7de0364ffa505d631b342f1a2fa729e8e67d.

Reported-by: Manfred Schlaegl <manfred.schlaegl@ginzinger.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
7 years agosun50i: a64: Add initial Orangepi Win/WinPlus support
Jagan Teki [Tue, 13 Jun 2017 09:44:42 +0000 (15:14 +0530)]
sun50i: a64: Add initial Orangepi Win/WinPlus support

Orangepi Win/WinPlus is an open-source single-board computer
using the Allwinner A64 SOC.

A64 Orangepi Win/WinPlus has
- A64 Quad-core Cortex-A53 64bit
- 1GB(Win)/2GB(Win Plus) DDR3 SDRAM
- Debug TTL UART
- Four USB 2.0
- HDMI
- LCD
- Audio and MIC
- Wifi + BT
- IR receiver
- 5V DC power supply

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Tested-by: Jagan Teki <jagan@amarulasolutions.com>
7 years agosun50i: h5: Add initial Orangepi Zero Plus 2 support
Jagan Teki [Thu, 25 May 2017 16:00:09 +0000 (16:00 +0000)]
sun50i: h5: Add initial Orangepi Zero Plus 2 support

Orangepi Zero Plus 2 is an open-source single-board computer
using the Allwinner h5 SOC.

H5 Orangepi Zero Plus 2 has
- Quad-core Cortex-A53
- 512MB DDR3
- micrSD slot and 8GB eMMC
- Debug TTL UART
- HDMI
- Wifi + BT
- OTG+power supply

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Tested-by: Jagan Teki <jagan@amarulasolutions.com>
7 years agoARM: tegra: remove Whistler support
Stephen Warren [Thu, 13 Apr 2017 18:57:21 +0000 (12:57 -0600)]
ARM: tegra: remove Whistler support

Whistler is an ancient Tegra 2 reference board. I may have been the only
person who ever used it with upstream software, and I've just recycled
the board hardware. Hence, it makes sense to remove support from software.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
7 years agopowerpc: fsl: Update maintainers
York Sun [Mon, 12 Jun 2017 16:05:05 +0000 (09:05 -0700)]
powerpc: fsl: Update maintainers

Update maintainers for B4860QDS, P1010RDB, P1_TWR, T104xRDB.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8569mds: Update config and maintainer
York Sun [Mon, 12 Jun 2017 17:01:11 +0000 (10:01 -0700)]
powerpc: mpc8569mds: Update config and maintainer

Enable DHCP command by default. Update maintainer info.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8568mds: Update board config
York Sun [Fri, 9 Jun 2017 19:50:26 +0000 (12:50 -0700)]
powerpc: mpc8568mds: Update board config

Enable DHCP command by default. Move environmental variable location
to before U-Boot image. Enlarge reserved mem for malloc. Update
maintainer.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8536ds: Update maintainer
York Sun [Thu, 8 Jun 2017 19:51:34 +0000 (12:51 -0700)]
powerpc: mpc8536ds: Update maintainer

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8544ds: Update maintainer
York Sun [Wed, 7 Jun 2017 17:13:39 +0000 (10:13 -0700)]
powerpc: mpc8544ds: Update maintainer

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8544ds: Fix environmental variable location
York Sun [Wed, 7 Jun 2017 17:12:56 +0000 (10:12 -0700)]
powerpc: mpc8544ds: Fix environmental variable location

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8544ds: Enable DHCP command by default
York Sun [Wed, 7 Jun 2017 17:12:20 +0000 (10:12 -0700)]
powerpc: mpc8544ds: Enable DHCP command by default

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8548cds: Enable DHCP command by default
York Sun [Tue, 6 Jun 2017 16:33:02 +0000 (09:33 -0700)]
powerpc: mpc8548cds: Enable DHCP command by default

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8548cds: Update maintainer
York Sun [Tue, 6 Jun 2017 16:32:42 +0000 (09:32 -0700)]
powerpc: mpc8548cds: Update maintainer

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agonet: phy: marvell: Fix init function for m88e1145
York Sun [Tue, 6 Jun 2017 16:22:40 +0000 (09:22 -0700)]
net: phy: marvell: Fix init function for m88e1145

Commit a058052c changed the generic phy_reset() to clear all bits in
BMCR. This inevitably clears the ANEG bit. m88e1145 requires any
change to ANEG bit to be followed by a software reset. This seems to
be different from other PHYs. Implement read-modify-write procedure
for this PHY init.

Signed-off-by: York Sun <york.sun@nxp.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
7 years agopowerpc: mpc8641hpcn: Enable DHCP command and DDR debug
York Sun [Fri, 26 May 2017 00:04:42 +0000 (17:04 -0700)]
powerpc: mpc8641hpcn: Enable DHCP command and DDR debug

Enable DHCP command to make TFTP easier.
Enable DDR interactive debug by default.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agodriver: ddr: fsl: Fix compiling error for DDR2
York Sun [Fri, 26 May 2017 00:03:23 +0000 (17:03 -0700)]
driver: ddr: fsl: Fix compiling error for DDR2

Fix compiling error of "no member named 'taamin_ps'" for DDR2.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8536ds: Enable DHCP command by default
York Sun [Thu, 25 May 2017 22:43:45 +0000 (15:43 -0700)]
powerpc: mpc8536ds: Enable DHCP command by default

Enable DHCP command to make TFTP easier. Users don't have to use
static IP address.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc86xx: Update maintainer for MPC8610HPCD and MPC8641HPCN
York Sun [Wed, 17 May 2017 17:21:47 +0000 (10:21 -0700)]
powerpc: mpc86xx: Update maintainer for MPC8610HPCD and MPC8641HPCN

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc85xx: Update maintainer for MPC8541CDS and MPC8555CDS
York Sun [Wed, 17 May 2017 17:21:24 +0000 (10:21 -0700)]
powerpc: mpc85xx: Update maintainer for MPC8541CDS and MPC8555CDS

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agopowerpc: mpc8540ads: mpc8560ads: Drop support for MPC8540/60ADS
York Sun [Wed, 17 May 2017 17:19:35 +0000 (10:19 -0700)]
powerpc: mpc8540ads: mpc8560ads: Drop support for MPC8540/60ADS

Drop support for these two legacy boards.

Signed-off-by: York Sun <york.sun@nxp.com>
7 years agoboard: ti: am335x: Fix scale_vcore for beaglebones
Lokesh Vutla [Sat, 10 Jun 2017 07:52:56 +0000 (13:22 +0530)]
board: ti: am335x: Fix scale_vcore for beaglebones

commit 0650798824 ("board: am335x: Introduce scale_vcores")
updated voltages of each board based on efuse. It updated
beagle bone specific voltages under the condition board_is_bone().
But this is true only for BeagleBoneWhite. Due to which voltages
are not configured for BBB, BBW as wrong device is being probed.

So create a common function board_is_beaglebonex() which includes
am335x based beagle family. Use this for updating voltages.

Also remove extra if condition for selecting voltages which is
done later using a switch case and match usb current limit as
before the commit 0650798824.

Fixes: 0650798824 ("board: am335x: Introduce scale_vcores")
Reported-by: Emmanuel Vadot <manu@bidouilliste.com>
Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com>
7 years agoarm: am57xx: Keep environment in eMMC
Semen Protsenko [Fri, 9 Jun 2017 14:12:56 +0000 (17:12 +0300)]
arm: am57xx: Keep environment in eMMC

Use eMMC (instead of SD card) to store U-Boot environment. Use
"reserved" partition for U-Boot environment.

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Lokesh Vutla <lokeshvuta@ti.com>
7 years agoarm: dra7: Increase "bootloader" partition size
Semen Protsenko [Fri, 9 Jun 2017 14:12:48 +0000 (17:12 +0300)]
arm: dra7: Increase "bootloader" partition size

Increase the size of u-boot.img, so that it conforms with new DFU
configuration (see commit [1]).

[1] 7a53a1a8115b ARM: ti: Update layout for MMC and eMMC (env and dfu)

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoarm: am57xx: Increase "bootloader" partition size
Semen Protsenko [Fri, 9 Jun 2017 14:12:41 +0000 (17:12 +0300)]
arm: am57xx: Increase "bootloader" partition size

Increase the size of u-boot.img, so that it conforms with new DFU
configuration (see commit [1]).

[1] 7a53a1a8115b ARM: ti: Update layout for MMC and eMMC (env and dfu)

Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
7 years agoMAINTAINERS: mpc83xx: Add new custodian
mario.six@gdsys.cc [Fri, 9 Jun 2017 12:50:47 +0000 (14:50 +0200)]
MAINTAINERS: mpc83xx: Add new custodian

Add myself as mpc83xx custodian.

7 years agotools/tbot: update README
Heiko Schocher [Fri, 9 Jun 2017 04:13:34 +0000 (06:13 +0200)]
tools/tbot: update README

refer in the README to tbots webpage, and delete
the README in tools/tbot, as the latest documentation
for tbot is on this webpage.

Signed-off-by: Heiko Schocher <hs@denx.de>
7 years agoARM: dts: keystone-k2hk-evm: Add U-boot specific dtsi file
Cooper Jr., Franklin [Thu, 8 Jun 2017 17:33:21 +0000 (12:33 -0500)]
ARM: dts: keystone-k2hk-evm: Add U-boot specific dtsi file

With Davinci I2C switching to device model, K2HK requires U-boot specific
device tree entries. This is only required for I2C 1 which is needed
extremely early during the boot process.

Fixes: 1743d040b1df ("ARM: keystone: Enable DM_I2C by default")
Reported-by: Yan Liu <yan-liu@ti.com>
Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
7 years agocmd: nvedit: bring error message in sync with condition under which it appears
Lothar Waßmann [Thu, 8 Jun 2017 12:16:14 +0000 (14:16 +0200)]
cmd: nvedit: bring error message in sync with condition under which it appears

The list of symbols listed in the error message for missing Kconfig
symbols is out of sync with the symbols actually tested.
Add the missing symbols and reorder their appearance to be in sync
with the #if statement for easier checking.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agocmd: mtdparts: fix uninitialized variable warning
Lothar Waßmann [Thu, 8 Jun 2017 12:04:03 +0000 (14:04 +0200)]
cmd: mtdparts: fix uninitialized variable warning

commit 06a040a31bcf ("cmd: mtdparts: fix null pointer dereference in parse_mtdparts")
removed the initialization of a pointer variable, which is
subsequently used in a debug() call. This produces an uninitialized
variable warning, when compiling with DEBUG defined.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agodm: bcm6345_gpio: Set proper output level in bcm6345_gpio_direction_output
Axel Lin [Thu, 8 Jun 2017 09:20:12 +0000 (17:20 +0800)]
dm: bcm6345_gpio: Set proper output level in bcm6345_gpio_direction_output

Current code does not set output level in bcm6345_gpio_direction_output,
fix it.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
7 years agoboard_f: fix calculation of reloc_off
Lothar Waßmann [Thu, 8 Jun 2017 08:18:25 +0000 (10:18 +0200)]
board_f: fix calculation of reloc_off

relocate_code() calculates the relocation offset wrt. the symbol
__image_copy_start which happens to have the same value as
CONFIG_TEXT_BASE on most systems.
When creating an i.MX boot image with an integrated IVT it is
convenient to have CONFIG_TEXT_BASE point to the start of the IVT
that is prepended to the actual code. Thus CONFIG_TEXT_BASE will
differ from __image_copy_start, while the calculation
'gd->relocaddr - __image_copy_start' still gives the right relocation
offset.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agoARM: provide a valid exception stack address for startup code
Lothar Waßmann [Thu, 8 Jun 2017 08:16:36 +0000 (10:16 +0200)]
ARM: provide a valid exception stack address for startup code

Create exception stack in IRAM if available to facilitate debugging of
pre-relocation code by catching exceptions rather than stopping dead.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agoarm: adjust PC displayed in exception handlers to point to the failing instruction
Lothar Waßmann [Thu, 8 Jun 2017 07:52:33 +0000 (09:52 +0200)]
arm: adjust PC displayed in exception handlers to point to the failing instruction

Adjust the program counter register to point to the failing
instruction depending on the exeption type.
This makes it easier to localize the offending instruction leading to
a fatal exception.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agoARM: remove bogus cp_delay() function
Lothar Waßmann [Thu, 8 Jun 2017 07:48:41 +0000 (09:48 +0200)]
ARM: remove bogus cp_delay() function

The cp_delay() function was introduced because of a missing 'volatile'
attribute to the 'asm' statement in get_cr() which led to the 'mrc'
instruction in get_cr() being optimised out eventually.
This has been fixed in commit 53fd4b8c22bb ("arm: mmu: Add missing volatile for reading SCTLR register")
but the bogus cp_delay() function which was introduced as a workaround
for the malfunctioning get_cr() was never removed.

Signed-off-by: Lothar Waßmann <LW@KARO-electronics.de>
7 years agoserial: stm32x7: simplify baud rate register calculation
Patrice Chotard [Thu, 8 Jun 2017 07:26:55 +0000 (09:26 +0200)]
serial: stm32x7: simplify baud rate register calculation

Simplify baud rate register formula and use the oversampling
uart feature.
This code is aligned with what is implemented in kernel driver
drivers/tty/serial/stm32-usart.c since kernel v4.9.

Signed-off-by: Patrice Chotard <patrice.chotard@st.com>
Reviewed-by: Christophe KERELLO <christophe.kerello@st.com>
Reviewed-by: Patrick DELAUNAY <patrick.delaunay@st.com>
Acked-by: Vikas MANOCHA <vikas.manocha@st.com>
7 years agoserial: stm32x7: align compatible with kernel one
Patrice Chotard [Thu, 8 Jun 2017 07:26:54 +0000 (09:26 +0200)]
serial: stm32x7: align compatible with kernel one

stm32x7.c driver is dedicated for STM32F7.
In kernel, "st,stm32-usart" and "st,stm32-uart" compatible
strings are dedicated for STM32F4.

To keep U-boot and kernel aligned, replace the serial compatible
string from "st,stm32-usart", "st,stm32-uart" to
"st,stm32f7-usart", "st,stm32f7-uart" specific for STM32F7.

Signed-off-by: Patrice Chotard <patrice.chotard@st.com>
Reviewed-by: Christophe KERELLO <christophe.kerello@st.com>
Reviewed-by: Patrick DELAUNAY <patrick.delaunay@st.com>
Acked-by: Vikas MANOCHA <vikas.manocha@st.com>
7 years agorkcommon.c: Drop pointless assignment
Simon Glass [Wed, 7 Jun 2017 16:28:47 +0000 (10:28 -0600)]
rkcommon.c: Drop pointless assignment

Assigning a variable to itself is not necessary. Drop this and also add a
check for malloc() failure.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Coverity (CID: 161418)
Fixes: 111bcc4 (rockchip: mkimage: pad the header to 8-bytes (using a 'nop') for RK3399)
7 years agoboard_f: Use IS_ENABLED instead of #ifdef in initf_bootstage()
Simon Glass [Wed, 7 Jun 2017 16:28:46 +0000 (10:28 -0600)]
board_f: Use IS_ENABLED instead of #ifdef in initf_bootstage()

The current implementation makes it look like the 'if (from_spl)' part is
dead code because these features are not enabled for sandbox. We could
enable it for sandbox_spl, but this is not done yet (it requires sharing
memory between SPL and U-Boot proper which is in fact supported).

It is probably nicer to avoid #ifdef anyway. Change it.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Coverity (CID: 163244)
Fixes: 824bb1b (bootstage: Support SPL)