[U-Boot] [PATCH] cmd: nvedit: env_get_f must check for env_get_char error codes

2018-01-30 Thread Simon Goldschmidt
env_get_f calls env_get_char to load single characters from the environment. However, the return value of env_get_char was not checked for errors. Now if the env driver does not support the .get_char call, env_get_f did not notice this and looped over the whole size of the environment, calling env_

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread Simon Goldschmidt
On 31.01.2018 00:02, York Sun wrote: On 01/30/2018 12:16 PM, York Sun wrote: On 01/30/2018 11:40 AM, York Sun wrote: On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure

Re: [U-Boot] [PATCH v2] arm64: ls1012ardb: Add distro secure boot support

2018-01-30 Thread Sumit Garg
> -Original Message- > From: York Sun > Sent: Tuesday, January 30, 2018 2:57 AM > To: Sumit Garg ; u-boot@lists.denx.de > Cc: Ruchika Gupta ; Prabhakar Kushwaha > ; Vini Pillai > Subject: Re: [PATCH v2] arm64: ls1012ardb: Add distro secure boot support > > On 01/15/2018 09:34 AM, Sumit Ga

Re: [U-Boot] Convert CONFIG_SYS_OMAP24_I2C_SLAVE et al to Kconfig

2018-01-30 Thread Tom Rini
On Wed, Jan 24, 2018 at 03:21:21PM -0600, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_SYS_OMAP24_I2C_SLAVE >CONFIG_SYS_OMAP24_I2C_SPEED > > Signed-off-by: Adam Ford > Reviewed-by: Paul Kocialkowski Applied to u-boot/master, thanks! -- Tom signature.asc Descri

Re: [U-Boot] [U-Boot, V4] Convert CONFIG_SYS_DV_CLKMODE et al to Kconfig

2018-01-30 Thread Tom Rini
On Tue, Jan 23, 2018 at 04:04:28AM -0600, Adam Ford wrote: > This converts the following to Kconfig: >CONFIG_SYS_DV_CLKMODE >CONFIG_SYS_DA850_PLL0_POSTDIV >CONFIG_SYS_DA850_PLL0_PLLDIV1 >CONFIG_SYS_DA850_PLL0_PLLDIV2 >CONFIG_SYS_DA850_PLL0_PLLDIV3 >CONFIG_SYS_DA850_PLL0_PLL

Re: [U-Boot] [U-Boot, v2] Convert CONFIG_SYS_BOOT_GET_{CMDLINE, KBD} to Kconfig

2018-01-30 Thread Tom Rini
On Mon, Jan 22, 2018 at 05:17:10PM -0600, Derald D. Woods wrote: > This converts the following to Kconfig: > CONFIG_SYS_BOOT_GET_CMDLINE > CONFIG_SYS_BOOT_GET_KBD > > Signed-off-by: Derald D. Woods Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

[U-Boot] [RFC 1/1] dm: video: tegra124: incorrect logical condition

2018-01-30 Thread Heinrich Schuchardt
2 << 24 | A is always true. To use check against a bitmask we need &. Identified with cppcheck. Signed-off-by: Heinrich Schuchardt --- I do not have the hardware available. But the current coding is fishy. Please, clarify what should be coded here. --- drivers/video/tegra124/sor.c | 2 +- 1 fi

[U-Boot] [PATCH 1/1] atcspi200: avoid possible NULL dereference

2018-01-30 Thread Heinrich Schuchardt
Check if ns before and not after dereferencing it. Indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- drivers/spi/atcspi200_spi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/atcspi200_spi.c b/drivers/spi/atcspi200_spi.c index 3e29df03a4..5b2e9d

[U-Boot] [PATCH 1/1] i2c: mvtwsi.c: Avoid NULL dereference

2018-01-30 Thread Heinrich Schuchardt
For '#ifndef CONFIG_DM_I2C' twsi_i2c_init() passes NULL as 4th parameter to __twsi_i2c_init(). Identified with cppcheck. Signed-off-by: Heinrich Schuchardt --- drivers/i2c/mvtwsi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/mvtwsi.c b/drivers/i2c/mvtwsi.c

Re: [U-Boot] [PATCH 1/1] rockchip: gpio: remove dead code

2018-01-30 Thread Dr. Philipp Tomsich
> On 31 Jan 2018, at 00:45, Heinrich Schuchardt wrote: > > In the following statements > if (a) return a; if (a) return c; > the second return can never be executed. > > Identified by cppcheck. > > Signed-off-by: Heinrich Schuchardt Acked-by: Philipp Tomsich Reviewed-by: Philipp Tomsich _

[U-Boot] [PATCH 1/1] rockchip: gpio: remove dead code

2018-01-30 Thread Heinrich Schuchardt
In the following statements if (a) return a; if (a) return c; the second return can never be executed. Identified by cppcheck. Signed-off-by: Heinrich Schuchardt --- drivers/gpio/rk_gpio.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpio/rk_gpio.c b/drivers/gpio/rk_gpio.c ind

[U-Boot] [PATCH v2 1/1] efi_loader: efi_smbios_register should have a return value

2018-01-30 Thread Heinrich Schuchardt
Errors may occur inside efi_smbios_register(). - Return a status code. - Remove unused variables. - Use constants where applicable. Suggested-by: Simon Glass Signed-off-by: Heinrich Schuchardt --- v2 remove a change in unrelated code --- include/efi_loader.h| 2 +- lib/efi_loa

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 12:16 PM, York Sun wrote: > On 01/30/2018 11:40 AM, York Sun wrote: >> On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: >>> On 23.01.2018 21:16, Maxime Ripard wrote: Now that we have everything in place to support multiple environment, let's make sure the current code can us

[U-Boot] [PATCH v2] mx6sabresd: Add fastboot support

2018-01-30 Thread Fabio Estevam
From: Fabio Estevam fastboot tool is a convenient way to flash the eMMC, so add support for it. Examples of usages: On the mx6sabresd: => fastboot 0 On the Linux PC connected via USB: $ sudo fastboot getvar bootloader-version -i 0x0525 bootloader-version: U-Boot 2018.01-00550-g7517cfe finish

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 11:40 AM, York Sun wrote: > On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: >> On 23.01.2018 21:16, Maxime Ripard wrote: >>> Now that we have everything in place to support multiple environment, let's >>> make sure the current code can use it. >>> >>> The priority used between the va

[U-Boot] [PATCH 1/1] efi_loader: use constants in efi_allocate_pages()

2018-01-30 Thread Heinrich Schuchardt
Using the existing predefined constants is less error prone and makes the code easier to read. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_memory.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c in

[U-Boot] [PATCH 1/1] efi_loader: efi_smbios_register should have a return value

2018-01-30 Thread Heinrich Schuchardt
Errors may occur inside efi_smbios_register(). - Return a status code. - Remove unused variables. - Use constants where applicable. Suggested-by: Simon Glass Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h| 2 +- lib/efi_loader/efi_smbios.c | 18 -- lib/smb

Re: [U-Boot] [PATCH v3 09/15] env: Support multiple environments

2018-01-30 Thread York Sun
On 01/30/2018 12:19 AM, Simon Goldschmidt wrote: > On 23.01.2018 21:16, Maxime Ripard wrote: >> Now that we have everything in place to support multiple environment, let's >> make sure the current code can use it. >> >> The priority used between the various environment is the same one that was >> u

Re: [U-Boot] standalone example on mips/ath79/QCA9558/Tp-Link Archer-c7

2018-01-30 Thread Daniel Schwierzeck
Hi Arvid, On 27.01.2018 12:36, Arvid Picciani wrote: > HI, > > i'm trying the standalone hello_world.bin example on a > mips/ath79/QCA9558/Tp-Link Archer-c7 > > it just hangs for me: > > ap135> tftpboot 0x8020 > dup 1 speed 1000 > *** Warning: no boot file name; using '6F01A8C0.img' > Using

[U-Boot] [PATCH v2 1/2] efi_loader: split README.efi into two separate documents

2018-01-30 Thread Heinrich Schuchardt
README.efi describes two different concepts: * U-Boot exposing the UEFI API * U-Boot running on top of UEFI. This patch splits the document in two. Religious references are removed. The separation of the concepts makes sense before detailing the internals of U-Boot exposing the UEFI API in a futu

[U-Boot] [PATCH v2 0/2] efi_loader: update README.efi

2018-01-30 Thread Heinrich Schuchardt
Split README.efi into two separate documents. One for U-Boot on EFI and one for EFI on U-Boot. Provide information about - usage of the bootefi command - overview of UEFI - interaction between U-Boot and EFI drivers in README.efi --- v2 Provide new README.efi --- Heinrich Schuchardt (2):

[U-Boot] [PATCH v2 2/2] efi_loader: rewrite README.efi

2018-01-30 Thread Heinrich Schuchardt
Provide information about - usage of the bootefi command - overview of UEFI - interaction between U-Boot and EFI drivers Signed-off-by: Heinrich Schuchardt --- v2 new file --- doc/README.efi | 322 ++--- 1 file changed, 264 insertions(

[U-Boot] [PATCH 1/1] efi_loader: fix the online help for bootefi bootmgr

2018-01-30 Thread Heinrich Schuchardt
The bootefi command is missing in the online help for bootefi bootmgr. Signed-off-by: Heinrich Schuchardt --- cmd/bootefi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/bootefi.c b/cmd/bootefi.c index 4233d36b72..2106ed9c8c 100644 --- a/cmd/bootefi.c +++ b/cmd/bootefi.

Re: [U-Boot] [PATCH] fdt: Fixup only valid memory banks

2018-01-30 Thread Stephen Warren
On 01/30/2018 03:34 AM, Thierry Reding wrote: From: Thierry Reding Memory banks with address 0 and size 0 are empty and should not be passed to the OS via device tree. if (!banks) return 0; + for (i = 0; i < banks; i++) + if (start[i] == 0 && size[i]

Re: [U-Boot] Please pull u-boot-x86

2018-01-30 Thread Tom Rini
On Tue, Jan 30, 2018 at 10:42:32PM +0800, Bin Meng wrote: > Hi Tom, > > The following changes since commit eef11acebaa48e241e9187c717dc92d3e175c119: > > Prepare v2018.03-rc1 (2018-01-29 20:12:33 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-x86.git > > fo

Re: [U-Boot] MIPS relocation DT error

2018-01-30 Thread Daniel Schwierzeck
Hi Álvaro, On 26.01.2018 22:58, Álvaro Fernández Rojas wrote: > I just found out I was missing OF_EMBED, which confirms that it was > working because CFE was relocating u-boot and not u-boot itself... > > Sorry for the noise... > > El 26/01/2018 a las 22:53, Álvaro Fernández Rojas escribió: >> H

Re: [U-Boot] [PATCH 3/3] MIPS: add BMIPS Netgear DGND3700v2 board

2018-01-30 Thread Daniel Schwierzeck
On 26.01.2018 21:30, Álvaro Fernández Rojas wrote: > Signed-off-by: Álvaro Fernández Rojas > --- > arch/mips/dts/Makefile | 1 + > arch/mips/dts/netgear,dgnd3700v2.dts | 121 > +++ > arch/mips/mach-bmips/Kconfig | 12 +++ > board

Re: [U-Boot] [PATCH] Build system: Don't check for CONFIG_SYS_TEXT_BASE being set

2018-01-30 Thread Alexey Brodkin
Hi MAsahiro-san, Tom, On Wed, 2018-01-31 at 01:23 +0900, Masahiro Yamada wrote: > 2018-01-31 0:23 GMT+09:00 Alexey Brodkin : > > > Please feel free to include historical reason: > > This ifneq conditional was added for Blackfin because it did not use > CONFIG_SYS_TEXT_BASE. > See. > https://urld

Re: [U-Boot] [PATCH] Build system: Don't check for CONFIG_SYS_TEXT_BASE being set

2018-01-30 Thread Masahiro Yamada
2018-01-31 0:23 GMT+09:00 Alexey Brodkin : > CONFIG_SYS_TEXT_BASE must be set anyways and then it is used in many > places in the same Makefile without any checks so there's no point in > keeping this check araound just in one place. > > Signed-off-by: Alexey Brodkin > Cc: Tom Rini > --- > Makef

Re: [U-Boot] AM3703 does not boot from MicroSD card

2018-01-30 Thread Peter Robinson
On Tue, Jan 30, 2018 at 1:54 AM, phantom0 wrote: > Hi > I'm trying to upgrade u-boot from 2016.03 to 2017.09. When I boot it up, I vaguely remember something similar to this with some similar device, any reason you can't just got to 2018.01? > u-boot gives me this error. > What could be the prob

Re: [U-Boot] [PATCH v1] spl: eMMC/SD: Provide one __weak spl_boot_mode() function

2018-01-30 Thread Stefano Babic
On 27/01/2018 22:55, Lukasz Majewski wrote: > The goal of this patch is to clean up the code related to choosing spl > MMC boot mode. > > The spl_boot_mode() now is called only in spl_mmc_load_image() function, > which is only compiled in if CONFIG_SPL_MMC_SUPPORT is enabled. > > To achieve the g

Re: [U-Boot] [PATCH v1] spl: eMMC/SD: Provide one __weak spl_boot_mode() function

2018-01-30 Thread Michal Simek
On 27.1.2018 22:55, Lukasz Majewski wrote: > The goal of this patch is to clean up the code related to choosing spl > MMC boot mode. > > The spl_boot_mode() now is called only in spl_mmc_load_image() function, > which is only compiled in if CONFIG_SPL_MMC_SUPPORT is enabled. > > To achieve the go

[U-Boot] [PATCH] Build system: Don't check for CONFIG_SYS_TEXT_BASE being set

2018-01-30 Thread Alexey Brodkin
CONFIG_SYS_TEXT_BASE must be set anyways and then it is used in many places in the same Makefile without any checks so there's no point in keeping this check araound just in one place. Signed-off-by: Alexey Brodkin Cc: Tom Rini --- Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/M

Re: [U-Boot] [PATCH] env: sf: fix return value of env_sf_load

2018-01-30 Thread Tom Rini
On Tue, Jan 30, 2018 at 09:15:09AM +0100, Simon Goldschmidt wrote: > On 30.01.2018 08:55, Simon Goldschmidt wrote: > >With the recent changes to support multiple environments, I see a > >message "Failed (1)" when loading environment from sf. > > > >env_sf_load() returns the return value of env_impo

Re: [U-Boot] [PATCH] armv8: ls1088aqds: Add IFC-NOR as boot source for LS1088

2018-01-30 Thread York Sun
Ashish, Please use proper quotation ">" when you reply. On 01/30/2018 06:24 AM, Ashish Kumar wrote: > > > -Original Message- > From: York Sun > Sent: Tuesday, January 30, 2018 2:54 AM > To: Ashish Kumar ; u-boot@lists.denx.de > Subject: Re: [PATCH] armv8: ls1088aqds: Add IFC-NOR as boo

[U-Boot] [PATCH v3 09/24] mmc: omap_hsmmc: use mmc_of_parse to populate mmc_config

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I Use the mmc_of_parse library function to populate mmc_config instead of repeating the same code in host controller driver. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c | 24 +---

[U-Boot] [PATCH v3 23/24] ARM: DRA7x/AM57x: Add MMC/SD fixups for rev1.0 and rev 1.1

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I Since DRA7xx/AM57xx SR1.1 and SR1.0 has errata to limit the frequency of MMC1 to 96MHz and frequency of MMC2 to 48MHz for AM572x SR1.1, limit the frequency and disable higher speed modes for those revision. Also use the recommended IO delays (those tagged with "rev11"

[U-Boot] [PATCH v3 22/24] dts: am57xx-idk: disable HS200 support

2018-01-30 Thread Jean-Jacques Hiblot
HS200 cannot be supported on mmc2, because the IO lines of mmc2 are connected to 3.3v. Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None arch/arm/dts/am57xx-idk-common.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/dts/am57xx-idk-common.dtsi b/arch/arm/dts/am57xx

[U-Boot] [PATCH v3 15/24] mmc: omap_hsmmc: implement send_init_stream callback

2018-01-30 Thread Jean-Jacques Hiblot
This callback is used to send the 74 clock cycles after power up. Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/mmc/omap_hsmmc.c b/drivers/mmc/omap_hsmmc.c index b10d55e..71608d1 10

[U-Boot] [PATCH v3 20/24] ARM: dts: dra7: Add supported MMC/SD modes in MMC dt nodes

2018-01-30 Thread Jean-Jacques Hiblot
On DRA7 family SoCs, MMC1 controller supports SDR104, SDR50, DDR50, SDR25 and SDR12 UHS modes. MMC2 controller supports HS200 and DDR modes. MMC3 controller supports SDR12, SDR25 and SDR50 modes. MMC4 controller supports SDR12 and SDR25 modes. Add these supported modes in device-tree file. Sig

[U-Boot] [PATCH v3 19/24] ARM: dts: DRA7: use new dra7-specific compatible string

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I Use the new compatible string "ti,dra7-hsmmc" that was specifically added for dra7 and dra72. This is required since for dra7 and dra72 processors iodelay values has to be set unlike other processors. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Sekhar Nori

[U-Boot] [PATCH v3 10/24] ARM: OMAP5/DRA7: Enable iodelay recalibration to be done from uboot

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I Add a new API to perform iodelay recalibration without isolate io to be used in uboot. The data manual of J6/J6 Eco recommends to set different IODELAY values depending on the mode in which the MMC/SD is enumerated in order to ensure IO timings are met. The MMC drive

[U-Boot] [PATCH v3 07/24] mmc: omap_hsmmc: Workaround for errata id i802

2018-01-30 Thread Jean-Jacques Hiblot
According to errata i802, DCRC error interrupts (MMCHS_STAT[21] DCRC=0x1) can occur during the tuning procedure. The DCRC interrupt, occurs when the last tuning block fails (the last ratio tested). The delay from CRC check until the interrupt is asserted is bigger than the delay until assertion of

[U-Boot] [PATCH v3 21/24] dts: am57xx-beagle-x15: disable UHS and HS200 support

2018-01-30 Thread Jean-Jacques Hiblot
The UHS modes are not supported in beagle-x15 because it's not possible to switch the IO lines supply voltage to 1.8v. Also HS200 cannot be supported on mmc2, because the IO lines of mmc2 are connected to 3.3v. Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None arch/arm/dts/am57xx-beag

[U-Boot] [PATCH v3 17/24] mmc: omap_hsmmc: add signal voltage selection support

2018-01-30 Thread Jean-Jacques Hiblot
I/O data lines of UHS SD card operates at 1.8V when in UHS speed mode (same is true for eMMC in DDR and HS200 modes). Add support to switch signal voltage to 1.8V in order to support UHS cards and eMMC HS200 and DDR modes. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot

[U-Boot] [PATCH v3 18/24] ARM: OMAP5: set mmc clock frequency to 192MHz

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I Now that omap_hsmmc has support for hs200 mode, change the clock frequency to 192MHz. Also change the REFERENCE CLOCK frequency to 192MHz based on which the internal mmc clock divider is calculated. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques H

[U-Boot] [PATCH v3 12/24] mmc: omap_hsmmc: Add support to get pinctrl values and max frequency for different hw revisions

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I AM572x SR1.1 requires different IODelay values to be used than that used in AM572x SR2.0. These values are populated in device tree. Add capability in omap_hsmmc driver to extract IOdelay values for different silicon revision. The maximum frequency is also reduced whe

[U-Boot] [PATCH v3 14/24] mmc: omap_hsmmc: update mmc->clock with the actual bus speed

2018-01-30 Thread Jean-Jacques Hiblot
When the clock is applied, compute the actual value of the clock. It may be slightly different from the requested value (max freq, divisor threshold) Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) dif

[U-Boot] [PATCH v3 03/24] mmc: omap_hsmmc: add support to set default io voltage

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I "ti,dual-volt" is used in linux kernel to set the voltage capabilities. For host controller dt nodes that doesn't have "ti,dual-volt", it's assumed 1.8v is the io voltage. This is not always true (like in the case of beagle-x15 where the io lines are connected to 3.3v

[U-Boot] [PATCH v3 24/24] configs: dra7xx_evm/dra7xx_hs_evm: Enable MMC HS200 and SD UHS support

2018-01-30 Thread Jean-Jacques Hiblot
By default UHS and HS200 are not enabled. Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None configs/dra7xx_evm_defconfig| 3 +++ configs/dra7xx_hs_evm_defconfig | 3 +++ 2 files changed, 6 insertions(+) diff --git a/configs/dra7xx_evm_defconfig b/configs/dra7xx_evm_defconfig ind

[U-Boot] [PATCH v3 13/24] mmc: omap_hsmmc: allow the simple HS modes to use the default pinctrl

2018-01-30 Thread Jean-Jacques Hiblot
The default configuration is usually working fine for the the HS modes. Don't enforce the presence of a dedicated pinmux for the HS modes. Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c | 23 +-- 1 file changed, 13 insertions(+), 10 dele

[U-Boot] [PATCH v3 11/24] mmc: omap_hsmmc: Add support to set IODELAY values

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I The data manual of J6/J6 Eco recommends to set different IODELAY values depending on the mode in which the MMC/SD is enumerated in order to ensure IO timings are met. Add support to parse mux values and iodelay values from device tree and set these depending on the e

[U-Boot] [PATCH v3 04/24] mmc: omap_hsmmc: set MMC mode in the UHSMS bit field

2018-01-30 Thread Jean-Jacques Hiblot
Use the timing parameter set in the MMC core to set the mode in UHSMS bit field. This is in preparation for adding HS200 support in omap hsmmc driver. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: - prefer the usage of the BIT() macro over the con

[U-Boot] [PATCH v3 16/24] mmc: omap_hsmmc: allow mmc clock to be gated

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I mmc core has defined a new parameter *clk_disable* to gate the clock. Disable the clock here if *clk_disable* is set. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c | 6 ++ 1 file cha

[U-Boot] [PATCH v3 08/24] mmc: omap_hsmmc: Reduce the max timeout for reset controller fsm

2018-01-30 Thread Jean-Jacques Hiblot
From OMAP3 SoCs (OMAP3, OMAP4, OMAP5, AM572x, AM571x), the DAT/CMD lines reset procedure section in TRM suggests to first poll the SRD/SRC bit until it is set to 0x1. But looks like that bit is never set to 1 and there is an observable delay of 1sec everytime the driver tries to reset DAT/CMD. (The

[U-Boot] [PATCH v3 00/24] omap_hsmmc: Add support for HS200 and UHS modes

2018-01-30 Thread Jean-Jacques Hiblot
This series adds the missing bits to enable the UHS and HS200 modes for the TI platforms. Enabling support for high speed modes on omap5 requires implementing: * io signal voltage selection * tuning support * pin configuration (IO delays) The few last patches enable the high speed modes for t

[U-Boot] [PATCH v3 06/24] mmc: omap_hsmmc: Add tuning support

2018-01-30 Thread Jean-Jacques Hiblot
HS200/SDR104 requires tuning command to be sent to the card. Use the mmc_send_tuning library function to send the tuning command and configure the internal DLL. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: - prefer the usage of the BIT() macro ove

[U-Boot] [PATCH v3 01/24] mmc: omap_hsmmc: cleanup clock configuration

2018-01-30 Thread Jean-Jacques Hiblot
Add a separate function for starting the clock, stopping the clock and setting the clock. Starting the clock and stopping the clock can be used irrespective of setting the clock (For example during iodelay recalibration). Also set the clock only if there is a change in frequency. Signed-off-by: Ki

[U-Boot] [PATCH v3 05/24] mmc: omap_hsmmc: Enable DDR mode support

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I In order to enable DDR mode, Dual Data Rate mode bit has to be set in MMCHS_CON register. Set it here. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None arch/arm/include/asm/omap_mmc.h | 1 + drivers/mmc/omap_hsmmc.

[U-Boot] [PATCH v3 02/24] mmc: omap_hsmmc: cleanup omap_hsmmc_set_ios

2018-01-30 Thread Jean-Jacques Hiblot
From: Kishon Vijay Abraham I No functional change. Move bus width configuration setting to a separate function and invoke it only if there is a change in the bus width. Signed-off-by: Kishon Vijay Abraham I Signed-off-by: Jean-Jacques Hiblot --- Changes in v3: None drivers/mmc/omap_hsmmc.c

Re: [U-Boot] [PATCH] x86: kconfig: Remove meaningless 'select n'

2018-01-30 Thread Bin Meng
On Tue, Jan 30, 2018 at 9:04 PM, Bin Meng wrote: > On Tue, Jan 30, 2018 at 8:59 PM, Ulf Magnusson wrote: >> 'select n' selects a constant symbol, which is meaningless and has no >> effect. Maybe this was meant to be a 'default n', though bool and >> tristate symbols already implicitly default to

Re: [U-Boot] [PATCH 2/2] x86: qemu: qfw: Implement acpi_get_rsdp_addr()

2018-01-30 Thread Bin Meng
On Tue, Jan 30, 2018 at 9:05 PM, Andy Shevchenko wrote: > On Tue, 2018-01-30 at 05:01 -0800, Bin Meng wrote: >> U-Boot on QEMU does not build ACPI table by ourself, instead it uses >> the prebuilt ACPI table via the qfw interface. This implements the >> qfw version of acpi_get_rsdp_addr() for setu

Re: [U-Boot] [PATCH 1/2] x86: acpi: Use an API to get the ACPI RSDP table address

2018-01-30 Thread Bin Meng
On Tue, Jan 30, 2018 at 9:01 PM, Bin Meng wrote: > At present the acpi_rsdp_addr variable is directly referenced in > setup_zimage(). This changes to use an API for better encapsulation > and extension. > > Signed-off-by: Bin Meng > --- > > arch/x86/include/asm/acpi_table.h | 9 + > arc

[U-Boot] Please pull u-boot-x86

2018-01-30 Thread Bin Meng
Hi Tom, The following changes since commit eef11acebaa48e241e9187c717dc92d3e175c119: Prepare v2018.03-rc1 (2018-01-29 20:12:33 -0500) are available in the git repository at: git://git.denx.de/u-boot-x86.git for you to fetch changes up to e21b04fec465c84a51ca6fc6450263e0c0953fcb: x86: kc

Re: [U-Boot] [PATCH] armv8: ls1088aqds: Add IFC-NOR as boot source for LS1088

2018-01-30 Thread Ashish Kumar
-Original Message- From: York Sun Sent: Tuesday, January 30, 2018 2:54 AM To: Ashish Kumar ; u-boot@lists.denx.de Subject: Re: [PATCH] armv8: ls1088aqds: Add IFC-NOR as boot source for LS1088 On 01/01/2018 09:24 PM, Ashish Kumar wrote: > IFC-NOR and QSPI-NOR pins are muxed on SoC,so the

Re: [U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Ulf Magnusson
On Tue, Jan 30, 2018 at 2:58 PM, Michal Simek wrote: > On 30.1.2018 14:38, Ulf Magnusson wrote: >> On Tue, Jan 30, 2018 at 2:25 PM, Michal Simek >> wrote: >>> On 30.1.2018 14:02, Ulf Magnusson wrote: There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was probably intended. >>

Re: [U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Michal Simek
On 30.1.2018 14:38, Ulf Magnusson wrote: > On Tue, Jan 30, 2018 at 2:25 PM, Michal Simek wrote: >> On 30.1.2018 14:02, Ulf Magnusson wrote: >>> There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was >>> probably intended. >>> >>> No functional changes. Kconfig choices fall back on using

Re: [U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Ulf Magnusson
On Tue, Jan 30, 2018 at 2:38 PM, Ulf Magnusson wrote: > On Tue, Jan 30, 2018 at 2:25 PM, Michal Simek wrote: >> On 30.1.2018 14:02, Ulf Magnusson wrote: >>> There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was >>> probably intended. >>> >>> No functional changes. Kconfig choices fall

Re: [U-Boot] [PATCH v2 04/24] mmc: omap_hsmmc: set MMC mode in the UHSMS bit field

2018-01-30 Thread Jean-Jacques Hiblot
Hi Jaehoon, On 29/01/2018 05:33, Jaehoon Chung wrote: Hi, On 01/25/2018 07:51 PM, Jean-Jacques Hiblot wrote: From: Kishon Vijay Abraham I Use the timing parameter set in the MMC core to set the mode in UHSMS bit field. This is in preparation for adding HS200 support in omap hsmmc driver.

Re: [U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Ulf Magnusson
On Tue, Jan 30, 2018 at 2:25 PM, Michal Simek wrote: > On 30.1.2018 14:02, Ulf Magnusson wrote: >> There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was >> probably intended. >> >> No functional changes. Kconfig choices fall back on using the first >> (visible) symbol in the choice as t

Re: [U-Boot] [PATCH] serial: Make full device search optional

2018-01-30 Thread Alexander Graf
On 01/30/2018 02:09 PM, Derald Woods wrote: On Jan 30, 2018 3:17 AM, "Alexander Graf" > wrote: On 01/30/2018 12:41 AM, Derald D. Woods wrote: On Mon, Jan 29, 2018 at 07:46:09AM -0600, Derald Woods wrote: On Jan 29, 2018 6:57 AM, "Alexander Graf" ma

Re: [U-Boot] [PATCH v2 00/24] omap_hsmmc: Add support for HS200 and UHS modes

2018-01-30 Thread Jean-Jacques Hiblot
Hi Jaehoon, On 30/01/2018 06:17, Jaehoon Chung wrote: Hi JJ, On 01/25/2018 07:51 PM, Jean-Jacques Hiblot wrote: This series adds the missing bits to enable the UHS and HS200 modes for the TI platforms. Enabling support for high speed modes on omap5 requires implementing: * io signal voltag

Re: [U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Michal Simek
On 30.1.2018 14:02, Ulf Magnusson wrote: > There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was > probably intended. > > No functional changes. Kconfig choices fall back on using the first > (visible) symbol in the choice as the default if the default symbol is > not visible. > > Disc

Re: [U-Boot] [PATCH] serial: Make full device search optional

2018-01-30 Thread Derald Woods
On Jan 30, 2018 3:17 AM, "Alexander Graf" wrote: On 01/30/2018 12:41 AM, Derald D. Woods wrote: > On Mon, Jan 29, 2018 at 07:46:09AM -0600, Derald Woods wrote: > >> On Jan 29, 2018 6:57 AM, "Alexander Graf" wrote: >> >> Commit 608b0c4ad4e5ec0c ("serial: Use next serial device if probing >> fail

[U-Boot] [PATCH] usb: ulpi: kconfig: Remove meaningless choice default

2018-01-30 Thread Ulf Magnusson
'default' on a choice refers to the symbol selected by default, not to the choice mode, so 'default n' is meaningless. No functional changes. Optional choices implicitly default to n mode (and there is no way to make them default to another mode). Discovered in Kconfiglib (https://github.com/ulfa

Re: [U-Boot] [PATCH 2/2] x86: qemu: qfw: Implement acpi_get_rsdp_addr()

2018-01-30 Thread Andy Shevchenko
On Tue, 2018-01-30 at 05:01 -0800, Bin Meng wrote: > U-Boot on QEMU does not build ACPI table by ourself, instead it uses > the prebuilt ACPI table via the qfw interface. This implements the > qfw version of acpi_get_rsdp_addr() for setup_zimage(). > > Signed-off-by: Bin Meng > --- > > drivers/

[U-Boot] [PATCH] am335x, shc: kconfig: Fix misspelled choice default

2018-01-30 Thread Ulf Magnusson
There is no EMMC symbol in the "enable different boot versions for the shc board" choice. SHC_EMMC was probably intended. No functional changes. Kconfig choices fall back on using the first (visible) symbol in the choice as the default if the default symbol is not visible. Discovered in Kconfigli

Re: [U-Boot] [PATCH] x86: kconfig: Remove meaningless 'select n'

2018-01-30 Thread Bin Meng
On Tue, Jan 30, 2018 at 8:59 PM, Ulf Magnusson wrote: > 'select n' selects a constant symbol, which is meaningless and has no > effect. Maybe this was meant to be a 'default n', though bool and > tristate symbols already implicitly default to n. > > Discovered in Kconfiglib (https://github.com/ulf

[U-Boot] [PATCH] arm64: zynqmp: Fix misspelled choice default

2018-01-30 Thread Ulf Magnusson
There is no JTAG symbol in the "Boot mode" choice. JTAG_MODE was probably intended. No functional changes. Kconfig choices fall back on using the first (visible) symbol in the choice as the default if the default symbol is not visible. Discovered in Kconfiglib (https://github.com/ulfalizer/Kconfi

[U-Boot] [PATCH] x86: kconfig: Remove meaningless 'select n'

2018-01-30 Thread Ulf Magnusson
'select n' selects a constant symbol, which is meaningless and has no effect. Maybe this was meant to be a 'default n', though bool and tristate symbols already implicitly default to n. Discovered in Kconfiglib (https://github.com/ulfalizer/Kconfiglib), which does more strict checking here:

[U-Boot] [PATCH 2/2] x86: qemu: qfw: Implement acpi_get_rsdp_addr()

2018-01-30 Thread Bin Meng
U-Boot on QEMU does not build ACPI table by ourself, instead it uses the prebuilt ACPI table via the qfw interface. This implements the qfw version of acpi_get_rsdp_addr() for setup_zimage(). Signed-off-by: Bin Meng --- drivers/misc/qfw.c | 8 1 file changed, 8 insertions(+) diff --gi

[U-Boot] [PATCH 1/2] x86: acpi: Use an API to get the ACPI RSDP table address

2018-01-30 Thread Bin Meng
At present the acpi_rsdp_addr variable is directly referenced in setup_zimage(). This changes to use an API for better encapsulation and extension. Signed-off-by: Bin Meng --- arch/x86/include/asm/acpi_table.h | 9 + arch/x86/lib/acpi_table.c | 7 ++- arch/x86/lib/acpi_tab

Re: [U-Boot] [PATCH v1 2/2] x86: zImage: Propagate acpi_rsdp_addr to kernel via boot parameters

2018-01-30 Thread Bin Meng
On Mon, Jan 15, 2018 at 10:33 AM, Bin Meng wrote: > On Fri, Jan 12, 2018 at 9:01 PM, Andy Shevchenko > wrote: >> On Fri, 2018-01-12 at 17:00 +0800, Bin Meng wrote: >>> Hi Andy, >>> >>> On Thu, Jan 11, 2018 at 1:40 AM, Andy Shevchenko >>> wrote: >>> > New field acpi_rsdp_addr, which has been intr

Re: [U-Boot] [PATCH v7 3/3] common: Generic firmware loader for file system

2018-01-30 Thread Marek Vasut
On 01/30/2018 12:16 PM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > This is file system generic loader which can be used to load > the file image from the storage into target such as memory. > The consumer driver would then use this loader to program whatever, > ie. the FPGA device

Re: [U-Boot] [PATCH v7 2/3] cmd: ubifs: Move ubifs_initialized checking into cmd_ubifs_umount()

2018-01-30 Thread Marek Vasut
On 01/30/2018 12:16 PM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > cmd_ubifs_umount() function would be called directly instead of involving > whole command machinery in generic firmware loader, so checking on > ubifs_initialized status need to be done in cmd_ubifs_umount() withou

Re: [U-Boot] [PATCH] fdt: Fixup only valid memory banks

2018-01-30 Thread Thierry Reding
On Tue, Jan 30, 2018 at 12:41:15PM +0100, Lothar Waßmann wrote: > Hi, > > On Tue, 30 Jan 2018 11:34:17 +0100 Thierry Reding wrote: > > From: Thierry Reding > > > > Memory banks with address 0 and size 0 are empty and should not be > > passed to the OS via device tree. > > > > Signed-off-by: Thi

Re: [U-Boot] [PATCH] fdt: Fixup only valid memory banks

2018-01-30 Thread Lothar Waßmann
Hi, On Tue, 30 Jan 2018 11:34:17 +0100 Thierry Reding wrote: > From: Thierry Reding > > Memory banks with address 0 and size 0 are empty and should not be > passed to the OS via device tree. > > Signed-off-by: Thierry Reding > --- > common/fdt_support.c | 8 +++- > 1 file changed, 7 inser

Re: [U-Boot] Uboot as x86_64 EFI payload

2018-01-30 Thread Javier Santos Romo
FYI [cid:image001.gif@01D39927.A0D208D0] Javier Santos Romo Ingeniero de Telecomunicaciones / Telecommunications Engineer / GMV Juan de Herrera nº17 Boecillo E-47151 Valladolid Tel. +34 983 54 65 54 Fax +34 983 54 65 53 www.gmv.com [cid:imag

Re: [U-Boot] [PATCH v3 2/2] Enable test case with A20-OLinuXino-Lime2

2018-01-30 Thread Stefan Mavrodiev
On 01/22/2018 10:36 AM, Jagan Teki wrote: On Tue, Jan 2, 2018 at 4:31 PM, Stefan Mavrodiev wrote: On 12/26/2017 11:47 AM, Jagan Teki wrote: On Fri, Dec 22, 2017 at 3:30 PM, Stefan Mavrodiev wrote: Driver testing is done with A20-OLinuXino-Lime2. Testing requirements are: - Exposing spi0

[U-Boot] Raspberry Pi USB storage broken after 64 byte device descriptor patch

2018-01-30 Thread Kristian Amlie
I have a recent problem with USB storage on the Raspberry Pi 3 with U-Boot. Take a look at the output from v2017.01: -- USB0: Core Release: 2.80a scanning bus 0 for devices... 4 USB Device(s) found scanning usb for storage devices... 1 Storage Device(s) found scanning usb for e

[U-Boot] [PATCH] Fix FreeBSD endian checks

2018-01-30 Thread Justin Hibbits
FreeBSD, like OpenBSD, uses BIG_ENDIAN, LITTLE_ENDIAN, and BYTE_ORDER, whereas Linux and compatibles use __-prefixed names. Define the names the same as the OpenBSD block below it. --- include/compiler.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/compiler.h b/include/compiler.

[U-Boot] AM3703 does not boot from MicroSD card

2018-01-30 Thread phantom0
Hi I'm trying to upgrade u-boot from 2016.03 to 2017.09. When I boot it up, u-boot gives me this error. What could be the problem? Thanks. U-Boot SPL 2017.09-dirty (Jan 29 2018 - 10:56:46) Trying to boot from MMC1 MMC Device 0 not found spl: could not find mmc device. error: -19 SPL: failed to bo

[U-Boot] Uboot as x86_64 EFI payload

2018-01-30 Thread Juan Alfonso Reyes Ajenjo
Hi, I am Juan Alfonso Reyes, a firmware engineer in GMV. Currently we are developing new boards based in Apollo Lake CPU. We are trying to load uboot from UEFI. Using the default qemu-x86_efi_payload64_defconfig we are getting "U-Boot EFI Payload 2002 No memory map" error code. As I can see in

[U-Boot] [PATCH v7 3/3] common: Generic firmware loader for file system

2018-01-30 Thread tien . fong . chee
From: Tien Fong Chee This is file system generic loader which can be used to load the file image from the storage into target such as memory. The consumer driver would then use this loader to program whatever, ie. the FPGA device. Signed-off-by: Tien Fong Chee Reviewed-by: Lothar Waßmann ---

[U-Boot] [PATCH v7 2/3] cmd: ubifs: Move ubifs_initialized checking into cmd_ubifs_umount()

2018-01-30 Thread tien . fong . chee
From: Tien Fong Chee cmd_ubifs_umount() function would be called directly instead of involving whole command machinery in generic firmware loader, so checking on ubifs_initialized status need to be done in cmd_ubifs_umount() without breaking original functionality design. Signed-off-by: Tien Fon

[U-Boot] [PATCH v7 1/3] spl: Remove static declaration on spl_mmc_find_device function

2018-01-30 Thread tien . fong . chee
From: Tien Fong Chee This patch removes the static declation on spl_mmc_find_device_function so this function is accessible by the caller from other file. This patch is required for later patch. Signed-off-by: Tien Fong Chee Reviewed-by: Tom Rini --- common/spl/spl_mmc.c | 2 +- include/spl.h

[U-Boot] [PATCH v7 0/3] Generic firmware loader

2018-01-30 Thread tien . fong . chee
From: Tien Fong Chee This patchset contains generic firmware loader which is very close to Linux firmware loader but for U-Boot framework. Generic firmware loader can be used load whatever into target location, and then consumer driver would use it to program whatever, ie. the FPGA. This version

[U-Boot] [PATCH] fdt: Fixup only valid memory banks

2018-01-30 Thread Thierry Reding
From: Thierry Reding Memory banks with address 0 and size 0 are empty and should not be passed to the OS via device tree. Signed-off-by: Thierry Reding --- common/fdt_support.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/common/fdt_support.c b/common/fdt_support.

[U-Boot] [PATCH v3] armv8: Remove dependency of SERDES for LSCH2 and LSCH3

2018-01-30 Thread Sriram Dash
Remove dependency of SYS_HAS_SERDES for Layerscape Chasis 3/2. Signed-off-by: Sriram Dash --- Changes in v3: - Rebase to latest code. - Include changes for LSCH2. Changes in v2: - Remove ifdef when including fsl_serdes.h arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 14 ++ 1 fi

Re: [U-Boot] [PATCH 1/4] mtd: nand: mxs_nand: use self init

2018-01-30 Thread stefan
On 29.01.2018 17:01, Stefan Agner wrote: > From: Stefan Agner > > Instead of completing initialization via scan_bbt callback use > NAND self init to initialize the GPMI (MXS) NAND controller. > > Signed-off-by: Stefan Agner > --- > > drivers/mtd/nand/Kconfig| 1 + > drivers/mtd/nand/mxs_

  1   2   >