Re: [U-Boot] [PATCH 2/2] armv8/ls1046a: RGMII PHY requires internal delay on Tx

2017-08-03 Thread Madalin-cristian Bucur
> -Original Message- > From: York Sun > Sent: Thursday, August 03, 2017 7:14 PM > To: Madalin-cristian Bucur ; u-boot@lists.denx.de; > joe.hershber...@ni.com > Cc: Mingkai Hu ; Shengzhou Liu > ; martin.blumensti...@googlemail.com > Subject: Re: [PATCH 2/2] armv8/ls1046a: RGMII PHY requires

[U-Boot] [PATCH] net: fman: add support RGMII_TXID to memac

2017-08-03 Thread Madalin Bucur
Signed-off-by: Madalin Bucur --- drivers/net/fm/memac.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/fm/memac.c b/drivers/net/fm/memac.c index 1b5779c..ea50ed3 100644 --- a/drivers/net/fm/memac.c +++ b/drivers/net/fm/memac.c @@ -84,6 +84,7 @@ static void mema

[U-Boot] [PATCH] x86: Remove dead ISA related codes

2017-08-03 Thread Bin Meng
Neither new design uses ISA bus, nor does any U-Boot codes use these codes. Remove them. Signed-off-by: Bin Meng --- arch/x86/include/asm/io.h | 74 +-- 1 file changed, 8 insertions(+), 66 deletions(-) diff --git a/arch/x86/include/asm/io.h b/arch/x8

Re: [U-Boot] [PATCH 1/5] arm: socfpga: Add checking function on searching boot device

2017-08-03 Thread Chee, Tien Fong
On Rab, 2017-08-02 at 23:32 +0200, Marek Vasut wrote: > On 08/02/2017 12:21 PM, Chee, Tien Fong wrote: > > > > On Isn, 2017-07-31 at 12:53 +0200, Marek Vasut wrote: > > > > > > On 07/31/2017 12:50 PM, tien.fong.c...@intel.com wrote: > > > > > > > > > > > > From: Tien Fong Chee > > > > > > > >

[U-Boot] [PATCH 3/4] video: Drop the sm501 driver

2017-08-03 Thread Bin Meng
This is not used in U-Boot. Signed-off-by: Bin Meng --- drivers/video/Makefile | 1 - drivers/video/sm501.c| 225 --- include/sm501.h | 35 --- scripts/config_whitelist.txt | 1 - 4 files changed, 262 deletions(-) dele

[U-Boot] [PATCH 4/4] video: Drop the ct69000 driver

2017-08-03 Thread Bin Meng
This is not used in U-Boot. Signed-off-by: Bin Meng --- drivers/video/Kconfig | 31 +- drivers/video/Makefile |1 - drivers/video/cfb_console.c | 10 - drivers/video/ct69000.c | 1168 --- 4 files changed, 2 insertions(+), 1208 delet

[U-Boot] [PATCH 2/4] video: Drop the sed156x driver

2017-08-03 Thread Bin Meng
This is not used in U-Boot. Signed-off-by: Bin Meng --- drivers/video/Makefile | 1 - drivers/video/sed156x.c | 546 include/sed156x.h | 26 --- 3 files changed, 573 deletions(-) delete mode 100644 drivers/video/sed156x.c delete mode

[U-Boot] [PATCH 1/4] video: Drop the l5f31188 driver

2017-08-03 Thread Bin Meng
This is not used in U-Boot. Signed-off-by: Bin Meng --- drivers/video/Makefile | 1 - drivers/video/l5f31188.c | 192 --- 2 files changed, 193 deletions(-) delete mode 100644 drivers/video/l5f31188.c diff --git a/drivers/video/Makefile b/drivers

Re: [U-Boot] [PATCH 2/2] sf: Preserve QE bit when clearing BP# bits for Macronix flash

2017-08-03 Thread Bin Meng
On Wed, Aug 2, 2017 at 6:26 AM, Bin Meng wrote: > Hi Jagan, > > On Wed, Aug 2, 2017 at 12:01 AM, Jagan Teki wrote: >> On Sun, Jul 23, 2017 at 8:14 PM, Bin Meng wrote: >>> On some flash (like Macronix), QE (quad enable) bit is in the same >>> status register as BP# bits, and we need preserve its

Re: [U-Boot] [PATCH] usb: kbd: don't fail with iomux

2017-08-03 Thread Bin Meng
On Fri, Aug 4, 2017 at 3:22 AM, Rob Clark wrote: > stdin might not be set, which would cause iomux_doenv() to fail > therefore causing probe_usb_keyboard() to fail. Furthermore if we do > have iomux enabled, the sensible thing (in terms of user experience) > would be to simply add ourselves to th

[U-Boot] [PATCH v3 6/6] board: atmel: Add SAMA5D27 SOM1 EK board

2017-08-03 Thread Wenyou Yang
From: Wenyou Yang The SAMA5D27-SiP (System in Package) integrates the SAMA5D2 with 1Gbit DDR2-SDRAM in a single package. The SAMA5D27 SOM1 embeds a 64Mbit QSPI flash, KSZ8081 Phy and Mac-address EEPROM. Signed-off-by: Wenyou Yang --- Changes in v3: - Rebase on the PATCH: Atmel PIT timer driv

[U-Boot] [PATCH v3 4/6] ARM: at91: mach: Add missing defines of MPDDRC

2017-08-03 Thread Wenyou Yang
Add missing defines of Multiport DDR-SDRAM Controller (MPDDRC). Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/mach-at91/include/mach/atmel_mpddrc.h | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/mach-at91/include/mac

[U-Boot] [PATCH v3 3/6] ARM: at91: spl: Add boot device for boot from QSPI

2017-08-03 Thread Wenyou Yang
Add the boot device for booting from the QSPI flash. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/mach-at91/spl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/mach-at91/spl.c b/arch/arm/mach-at91/spl.c index c2e2d90

[U-Boot] [PATCH v3 2/6] ARM: at91: spl: Add mck function to lower rate while switching

2017-08-03 Thread Wenyou Yang
Refer to the commit 70f8c8316ad(PMC: add new mck function to lower rate while switching) from AT91Bootstrap. While switching to a lower clock source, we must switch the clock source first instead of last. Otherwise, we could end up with too high frequency on internal bus and peripherals. This happ

[U-Boot] [PATCH v3 5/6] ARM: at91: Get the Chip ID of SAMA5D2 SiP

2017-08-03 Thread Wenyou Yang
From: Wenyou Yang The SAMA5D2 SiP(System in Package) has different Chip IDs in the CHIPID and CHIP_EXID registers. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/mach-at91/armv7/sama5d2_devices.c | 26 -- arch

[U-Boot] [PATCH v3 1/6] ARM: at91: spl: Adjust switching to oscillator for SAMA5D2

2017-08-03 Thread Wenyou Yang
As said in 29.5.7 section of SAMA5D2 datasheet, before switching to the crystal oscillator, a check must be carried out to ensure that the oscillator is present and that its freqency is valid. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v3: None Changes in v2: None arch

[U-Boot] [PATCH v3 0/6] board: atmel: Add new board SAMA5D27-SOM1-EK board.

2017-08-03 Thread Wenyou Yang
The SAMA5D27-SOM1-EK board embeds a SAMA5D27 SOM1 module, which includes a SAMA5D27-SiP chip and QSPI flash, Phy and MAC EEPROM. The SAMA5D27 SiP integrates the SAMA5D2 with 1Gbit DDR2-SDRAM in a single package. It is based on 1./ [PATCH v2 0/8] sf: improve support of (Q)SPI flash memories

[U-Boot] [PATCH 3/5] ARM: at91: Remove hardware.h included in configs

2017-08-03 Thread Wenyou Yang
From: Wenyou Yang As said in READRE.kconfig, include/configs/*.h will be removed after all options are switched to Kconfig. As the first step, remove the follow line from include/configs/*.h. #include Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/atmel_sfr.c | 1 + arch/arm/mach-

[U-Boot] [PATCH 4/5] ARM: at91: spl: Add macro CONFIG_XXXX_BOOT support

2017-08-03 Thread Wenyou Yang
Use the CONFIG__BOOT to indicate the boot media, instead of the CONFIG_SYS_USE_ option, which is declared by CONFIG_SYS_EXTRA_OPTIONS option. Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/spl.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/arc

[U-Boot] [PATCH 5/5] configs: at91: Remove CONFIG_SYS_EXTRA_OPTIONS assignment

2017-08-03 Thread Wenyou Yang
To remove the assignment of CONFIG_SYS_EXTRA_OPTIONS option, which is deprecated, use the CONFIG__BOOT options to indicate the boot media, and the SoC is selected by the board. Signed-off-by: Wenyou Yang --- board/atmel/at91sam9m10g45ek/at91sam9m10g45ek.c | 4 ++-- board/atmel/at91sam9n12e

[U-Boot] [PATCH 2/5] ARM: at91: Add the SoC options to Kconfig

2017-08-03 Thread Wenyou Yang
From: Wenyou Yang To prepare to remove the SoCs options such as SAMA5D2, SAMA5D3 and SAMA5D4 from the CONFIG_SYS_EXTRA_OPTIONS option which is deprecated, add the SoC options to Kconfig. Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/Kconfig | 108 +-

[U-Boot] [PATCH 1/5] ARM: at91: Move CONFIG_AT91FAMILY option to Kconfig

2017-08-03 Thread Wenyou Yang
From: Wenyou Yang Move the CONFIG_AT91FAMILY option from include/mach/.h header file to Kconfig. Signed-off-by: Wenyou Yang --- arch/arm/mach-at91/Kconfig| 3 +++ arch/arm/mach-at91/include/mach/at91rm9200.h | 1 - arch/arm/mach-at91/include/mach/at91sam9260.h | 5 -

[U-Boot] [PATCH 0/5] configs: at91: Remove value of CONFIG_SYS_EXTRA_OPTIONS option

2017-08-03 Thread Wenyou Yang
The CONFIG_SYS_EXTRA_OPTIONS option is deprecated, remove the value of this option from the board default config files. Wenyou Yang (5): ARM: at91: Move CONFIG_AT91FAMILY option to Kconfig ARM: at91: Add the SoC options to Kconfig ARM: at91: Remove hardware.h included in configs ARM: at91

[U-Boot] [PATCH] board: atmel: Enable to use new timer driver

2017-08-03 Thread Wenyou Yang
Use the Atmel PIT timer driver which supports the driver model and device tree. Signed-off-by: Wenyou Yang --- The patch is based on, [PATCH] timer: Add Atmel PIT timer driver https://lists.denx.de/pipermail/u-boot/2017-August/300788.html arch/arm/dts/sama5d2.dtsi

Re: [U-Boot] [PATCH 2/2] sandbox: Convert SANDBOX_BITS_PER_LONG to Kconfig

2017-08-03 Thread Bin Meng
Hi Simon, On Thu, Aug 3, 2017 at 11:24 PM, Simon Glass wrote: > Hi Bin, > > On 1 August 2017 at 17:33, Bin Meng wrote: >> Convert SANDBOX_BITS_PER_LONG to Kconfig and assign it a correct >> number depending on which host we are going to build and run. >> >> Signed-off-by: Bin Meng >> --- >> >>

Re: [U-Boot] [PATCH] usb: kbd: don't fail with iomux

2017-08-03 Thread Rob Clark
On Thu, Aug 3, 2017 at 3:37 PM, Simon Glass wrote: > Hi Rob, > > On 3 August 2017 at 13:22, Rob Clark wrote: >> stdin might not be set, which would cause iomux_doenv() to fail >> therefore causing probe_usb_keyboard() to fail. Furthermore if we do >> have iomux enabled, the sensible thing (in te

Re: [U-Boot] [PATCH] usb: kbd: don't fail with iomux

2017-08-03 Thread Simon Glass
Hi Rob, On 3 August 2017 at 13:22, Rob Clark wrote: > stdin might not be set, which would cause iomux_doenv() to fail > therefore causing probe_usb_keyboard() to fail. Furthermore if we do > have iomux enabled, the sensible thing (in terms of user experience) > would be to simply add ourselves t

Re: [U-Boot] [PATCH] fs: add fs_readdir()

2017-08-03 Thread Rob Clark
On Thu, Aug 3, 2017 at 3:10 PM, Brüns, Stefan wrote: > On Donnerstag, 3. August 2017 18:54:30 CEST Rob Clark wrote: >> Needed to support efi file protocol. The fallback.efi loader wants >> to be able to read the contents of the /EFI directory to find an OS >> to boot. >> >> Currently only impleme

[U-Boot] [PATCH] usb: kbd: don't fail with iomux

2017-08-03 Thread Rob Clark
stdin might not be set, which would cause iomux_doenv() to fail therefore causing probe_usb_keyboard() to fail. Furthermore if we do have iomux enabled, the sensible thing (in terms of user experience) would be to simply add ourselves to the list of stdin devices. This fixes an issue with usbkbd

Re: [U-Boot] [PATCH] fs: add fs_readdir()

2017-08-03 Thread Brüns , Stefan
On Donnerstag, 3. August 2017 18:54:30 CEST Rob Clark wrote: > Needed to support efi file protocol. The fallback.efi loader wants > to be able to read the contents of the /EFI directory to find an OS > to boot. > > Currently only implemented for FAT, but that is all that UEFI is > required to sup

Re: [U-Boot] [PATCH 1/2] usb: kbd: don't fail with iomux

2017-08-03 Thread Simon Glass
Hi Rob, On 3 August 2017 at 12:57, Rob Clark wrote: > On Thu, Aug 3, 2017 at 2:25 PM, Simon Glass wrote: >> Hi Rob, >> >> On 3 August 2017 at 07:17, Rob Clark wrote: >>> stdin might not be set, which would cause iomux_doenv() to fail >>> therefore causing probe_usb_keyboard() to fail. Furtherm

Re: [U-Boot] [PATCH] ahci-pci: Update call to ahci_probe_scsi_pci()

2017-08-03 Thread Tom Rini
On Wed, Aug 02, 2017 at 07:07:17PM -0700, Bin Meng wrote: > ahci_probe_scsi() now takes a 'base' argument, and there is an API > that prepares base address for us: ahci_probe_scsi_pci(). > > Reviewed-by: Simon Glass > Reported-by: Tom Rini > Signed-off-by: Bin Meng Applied to u-boot/master, t

Re: [U-Boot] [PATCH] PowerPC: mpc85xx: Update ft_verify_fdt

2017-08-03 Thread Tom Rini
On Thu, Aug 03, 2017 at 09:51:53AM -0400, Tom Rini wrote: > With the changes to fdt_get_base_address() we need to modify the logic > in ft_verify_fdt() for how we check the validity of the CCSR address. > > Tested-on: qemu-ppce500 -M mpc8544ds > Fixes: 336a44877af8 ("fdt: Correct fdt_get_base_add

Re: [U-Boot] Please pull u-boot-sh/rmobile branch

2017-08-03 Thread Tom Rini
On Thu, Aug 03, 2017 at 08:24:45AM +0900, Nobuhiro Iwamatsu wrote: > Hi Tom, > > The following changes since commit 07d778382200a05a8b86cc135f79ec48e386f25a: > > Merge git://git.denx.de/u-boot-x86 (2017-08-01 15:38:32 -0400) > > are available in the git repository at: > > git://git.denx.de

Re: [U-Boot] [PATCH] qemu-ppce500: Update get_phys_ccsrbar_addr_early()

2017-08-03 Thread Tom Rini
On Thu, Aug 03, 2017 at 09:16:36AM -0400, Tom Rini wrote: > The logic of what fdt_get_base_address() will search for and return has > changed. Rework get_phys_ccsrbar_addr_early() to perform the logic that > fdt_get_base_address used to perform. > > Fixes: 336a44877af8 ("fdt: Correct fdt_get_bas

Re: [U-Boot] Please pull u-boot-fsl-qoriq master

2017-08-03 Thread Tom Rini
On Tue, Aug 01, 2017 at 10:56:45PM +, York Sun wrote: > Tom, > > The following changes since commit f19955a01482d118eac0fdaf530ec6e5b08cf414: > >Merge git://git.denx.de/u-boot-uniphier (2017-07-26 11:29:25 -0400) > > are available in the git repository at: > >git://git.denx.de/u-bo

Re: [U-Boot] [GIT PULL] Xilinx fixes for v2017.09

2017-08-03 Thread Tom Rini
On Wed, Aug 02, 2017 at 01:15:09PM +0200, Michal Simek wrote: > Hi Tom, > > here are changes I have collected in my tree. > I have added also a patch which make reserve_mmu as weak function which > was reviewed by Simon. We need to for mini u-boot configurations which > will be sent later. > Trav

Re: [U-Boot] [PATCH 1/2] usb: kbd: don't fail with iomux

2017-08-03 Thread Rob Clark
On Thu, Aug 3, 2017 at 2:25 PM, Simon Glass wrote: > Hi Rob, > > On 3 August 2017 at 07:17, Rob Clark wrote: >> stdin might not be set, which would cause iomux_doenv() to fail >> therefore causing probe_usb_keyboard() to fail. Furthermore if we do >> have iomux enabled, the sensible thing (in te

[U-Boot] [PATCH v5 30/30] env: Adjust the load() method to return an error

2017-08-03 Thread Simon Glass
The load() methods have inconsistent behaviour on error. Some of them load an empty default environment. Some load an environment containing an error message. Others do nothing. As a step in the right direction, have the method return an error code. Then the caller could handle this itself in a co

Re: [U-Boot] [PATCH] ahci-pci: Update call to ahci_probe_scsi_pci()

2017-08-03 Thread Simon Glass
On 2 August 2017 at 20:07, Bin Meng wrote: > ahci_probe_scsi() now takes a 'base' argument, and there is an API > that prepares base address for us: ahci_probe_scsi_pci(). > > Reported-by: Tom Rini > Signed-off-by: Bin Meng > --- > > drivers/ata/ahci-pci.c | 2 +- > 1 file changed, 1 insertion(

[U-Boot] [PATCH v5 16/30] env: Drop the env_name_spec global

2017-08-03 Thread Simon Glass
Add a name to the driver and use that instead of the global variable declared by each driver. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None cmd/nvedit.c | 4 +++- env/dataflash.c | 3 +-- e

[U-Boot] [PATCH v5 28/30] env: Rename some other getenv()-related functions

2017-08-03 Thread Simon Glass
We are now using an env_ prefix for environment functions. Rename these other functions as well, for consistency: getenv_vlan() getenv_bootm_size() getenv_bootm_low() getenv_bootm_mapsize() env_get_default() Suggested-by: Wolfgang Denk Signed-off-by: Simon Glass --- Changes in v

Re: [U-Boot] [PATCH 1/2] usb: kbd: don't fail with iomux

2017-08-03 Thread Simon Glass
Hi Rob, On 3 August 2017 at 07:17, Rob Clark wrote: > stdin might not be set, which would cause iomux_doenv() to fail > therefore causing probe_usb_keyboard() to fail. Furthermore if we do > have iomux enabled, the sensible thing (in terms of user experience) > would be to simply add ourselves t

[U-Boot] [PATCH v5 29/30] env: Adjust the get_char() method to return an int

2017-08-03 Thread Simon Glass
In principle this can fail, e.g. if the index is out of range. Adjust the driver signature to allow returning an error code. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/dataflash.c | 2 +- env/e

[U-Boot] [PATCH v5 27/30] env: Rename eth_getenv_enetaddr() to eth_env_get_enetaddr()

2017-08-03 Thread Simon Glass
Rename this function for consistency with env_get(). Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: - Add new patch to rename eth_getenv_enetaddr() Changes in v3: None Changes in v2: None arch/arm/mach-davinci/misc.c| 2 +- arch/powerpc/cpu/mpc85xx/fdt.c | 2

[U-Boot] [PATCH v5 24/30] env: Rename eth_setenv_enetaddr() to eth_env_set_enetaddr()

2017-08-03 Thread Simon Glass
Rename this function for consistency with env_set(). Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: - Add new patch to rename eth_setenv_enetaddr() Changes in v3: None Changes in v2: None arch/arm/mach-davinci/misc.c | 2 +- arch/arm/mach-omap2/utils.c

[U-Boot] [PATCH v5 14/30] env: Switch over to use environment location drivers

2017-08-03 Thread Simon Glass
Move over to use a the master implementation of the location drivers, with each method calling out to the appropriate driver. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: - Adjust implementation of env_get_char_spec() to avoid recursion Changes in v3:

[U-Boot] [PATCH v5 17/30] env: Drop unused env_ptr variables

2017-08-03 Thread Simon Glass
This variable is declared as a global in most environment location drivers. But it is not used outside the drivers and most of the declarations are unnecessary. Also some drivers call free() on env_ptr which seems wrong since it is not in the heap. Drop the variable where possible, and all calls

[U-Boot] [PATCH v5 26/30] env: Rename getenv_hex(), getenv_yesno(), getenv_ulong()

2017-08-03 Thread Simon Glass
We are now using an env_ prefix for environment functions. Rename these for consistency. Also add function comments in common.h. Suggested-by: Wolfgang Denk Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to rename getenv

[U-Boot] [PATCH v5 21/30] env: Drop saveenv() in favour of env_save()

2017-08-03 Thread Simon Glass
Use the env_save() function directly now that there is only one implementation of saveenv(). Signed-off-by: Simon Glass Reviewed-by: Wolfgang Denk Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: - Add changes for edison also Changes in v3: None Changes in v2: None arch/arm/mach

[U-Boot] [PATCH v5 20/30] env: Drop env_relocate_spec() in favour of env_load()

2017-08-03 Thread Simon Glass
This is a strange name for a function that loads the environment. There is now only one implementation of this function, so use the new env_load() function directly instead. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes

[U-Boot] [PATCH v5 15/30] env: Drop common init() functions

2017-08-03 Thread Simon Glass
Most of the init() implementations just use the default environment. Adjust env_init_new() to do this automatically, and drop the redundant code. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: - Fix incorrect setting of env_valid in env_init_new() Chang

[U-Boot] [PATCH v5 19/30] env: Drop env_get_char_spec()

2017-08-03 Thread Simon Glass
We only have a single implementation of this function now and it is called env_get_char(). Drop the old function and the weak version. Reviewed-by: Tom Rini Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/common.c | 1

[U-Boot] [PATCH v5 18/30] env: Drop env_init_new()

2017-08-03 Thread Simon Glass
Now that env_init() is only defined once we can drop the env_init_new() name and just use env_init(). Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/env.c | 6 +- 1 file changed, 1 insertion(+), 5 de

[U-Boot] [PATCH v5 11/30] env: Create a location driver for each location

2017-08-03 Thread Simon Glass
Set up a location driver for each supported environment location. At present this just points to the global functions and is not used. A later patch will switch this over to use private functions in each driver. There are several special cases here in various drivers to handle peculiarities of cer

[U-Boot] [PATCH v5 09/30] env: Add an enum for environment state

2017-08-03 Thread Simon Glass
At present we have three states for the environment, numbered 0, 1 and 2. Add an enum to record this to avoid open-coded values. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None board/Arcturus/ucp1020/spl.c

[U-Boot] [PATCH v5 23/30] env: Rename common functions related to setenv()

2017-08-03 Thread Simon Glass
We are now using an env_ prefix for environment functions. Rename these commonly used functions, for consistency. Also add function comments in common.h. Suggested-by: Wolfgang Denk Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: - Split out changes to setenv_ulong(), etc. int

[U-Boot] [PATCH v5 10/30] env: Rename nand env_location to nand_env_location

2017-08-03 Thread Simon Glass
We want to use this name for all environment drivers. Update the nand driver to use a more specific name. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/nand.c | 6 +++--- 1 file changed, 3 insertions(+)

[U-Boot] [PATCH v5 13/30] env: Add a new implementation of environment access

2017-08-03 Thread Simon Glass
We plan to move to a environment access via drivers for each location where the environment can be stored. Add an implementation for this. So far it is not used, but will be pressed into service in a future patch. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in

[U-Boot] [PATCH v5 12/30] env: Convert CONFIG_ENV_IS_IN... to a choice

2017-08-03 Thread Simon Glass
At present we support multiple environment drivers but there is not way to select between them at run time. Also settings related to the position and size of the environment area are global (i.e. apply to all locations). Until these limitations are removed we cannot really support more than one en

[U-Boot] [PATCH v5 07/30] env: common: Drop env_get_char_init()

2017-08-03 Thread Simon Glass
This function does nothing but call env_get_char_spec(). Drop it and adjust its only caller. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/common.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletio

[U-Boot] [PATCH v5 05/30] env: common: Drop env_get_addr()

2017-08-03 Thread Simon Glass
This function is not used anywhere other than env_get_char(). Move the code into that function. Reviewed-by: Tom Rini Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/common.c | 11 +++ 1 file changed, 3 insertions(+),

[U-Boot] [PATCH v5 02/30] Move environment files from common/ to env/

2017-08-03 Thread Simon Glass
About a quarter of the files in common/ relate to the environment. It seems better to put these into their own subdirectory and remove the prefix. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: - Rebase to master (e.g. for ENV_FAT_DEVICE_AND_PART) Changes in v4: None Chang

[U-Boot] [PATCH v5 04/30] env: common: Make env_get_addr/get_char_memory() static

2017-08-03 Thread Simon Glass
These functions are not used outside this file. Make them static and order them to avoid forward declarations. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None env/common.c | 18 +- in

[U-Boot] [PATCH v5 03/30] env: Use tabs in ENV_IS_IN_FAT

2017-08-03 Thread Simon Glass
Tidy up the formatting of this option. Signed-off-by: Simon Glass --- Changes in v5: - Rewrite commit message to mention that it just changes tabs now Changes in v4: None Changes in v3: None Changes in v2: None env/Kconfig | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --g

[U-Boot] [PATCH v5 06/30] env: common: Factor out the common env_valid check

2017-08-03 Thread Simon Glass
The check for gd->env_valid is used in both the 'if' and 'else' part of env_get_char(). Move it into that function instead for simplicity. Drop that code from the two leaf functions. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None

[U-Boot] [PATCH v5 08/30] env: common: Drop env_get_char_memory()

2017-08-03 Thread Simon Glass
This function is the same as env_get_char_spec() apart from dropping the brackets. Drop the brackets from env_get_char_spec() and use that instead of env_get_char_memory(). Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes i

[U-Boot] [PATCH v5 00/30] env: Move environment code to use location drivers

2017-08-03 Thread Simon Glass
The environment code is ripe for improvement in various ways. It has lots of duplication and inconsistencies between how things work with different environment locations. It does not properly use Kconfig. Error checking and handling is patchy. This series makes a start at improving things: - move

[U-Boot] [PATCH v5 01/30] Makefile: Rename 'env' target to 'environ'

2017-08-03 Thread Simon Glass
This target stops us using 'env' as a subdirectory. It is not mentioned in the help so seems to be an internal target. Rename it. Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: None Makefile | 4 ++-- 1 file chan

Re: [U-Boot] U-boot 2017.07 not working on Raspberry Pi

2017-08-03 Thread Tom Rini
On Thu, Aug 03, 2017 at 09:42:13AM -0600, Stephen Warren wrote: > On 08/03/2017 07:45 AM, Simon Glass wrote: > ... > >I'm not sure if we have a Raspberry Pi in a test farm anywhere. I > >should be able to look next week if no one can figure these out > >beforehand. > > I thought that Tom had some

[U-Boot] [PATCH] fs: add fs_readdir()

2017-08-03 Thread Rob Clark
Needed to support efi file protocol. The fallback.efi loader wants to be able to read the contents of the /EFI directory to find an OS to boot. Currently only implemented for FAT, but that is all that UEFI is required to support. Signed-off-by: Rob Clark --- fs/fat/fat.c | 59

[U-Boot] [PATCH 5/6] db410c: config updates

2017-08-03 Thread Rob Clark
Signed-off-by: Rob Clark --- configs/dragonboard410c_defconfig | 11 +++ 1 file changed, 11 insertions(+) diff --git a/configs/dragonboard410c_defconfig b/configs/dragonboard410c_defconfig index c78b4a6fd5..d9cb269085 100644 --- a/configs/dragonboard410c_defconfig +++ b/configs/dragonbo

[U-Boot] [PATCH 6/6] db410c: enable r8152 usb eth

2017-08-03 Thread Rob Clark
Signed-off-by: Rob Clark --- include/configs/dragonboard410c.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/dragonboard410c.h b/include/configs/dragonboard410c.h index 626dff8dcd..bbc9685e0e 100644 --- a/include/configs/dragonboard410c.h +++ b/include/configs/dragonboard41

[U-Boot] [PATCH 4/6] db410c: on aarch64 the fdtfile is in per-vendor subdirectory

2017-08-03 Thread Rob Clark
Signed-off-by: Rob Clark --- include/configs/dragonboard410c.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/dragonboard410c.h b/include/configs/dragonboard410c.h index d9dc639aeb..626dff8dcd 100644 --- a/include/configs/dragonboard410c.h +++ b/include/confi

[U-Boot] [PATCH 3/6] db410c: add reserved-memory node to dts

2017-08-03 Thread Rob Clark
If lk lights up display and populates simple-framebuffer node, it will also setup a reserved-memory node (needed by simplefb on linux). But it isn't clever enough to cope when the reserved-memory node is not present. Signed-off-by: Rob Clark --- arch/arm/dts/dragonboard410c.dts | 7 ++- 1 f

[U-Boot] [PATCH 2/6] db410c: use fdt passed from lk

2017-08-03 Thread Rob Clark
lk patches the fdt to set some device's MAC addresses and more importantly to patch in the simple-framebuffer node that we want u-boot to see. Signed-off-by: Rob Clark --- board/qualcomm/dragonboard410c/Makefile | 1 + board/qualcomm/dragonboard410c/dragonboard410c.c | 8 ++ board

[U-Boot] [PATCH 1/6] fdtdec: allow board to provide fdt for CONFIG_OF_SEPARATE

2017-08-03 Thread Rob Clark
Similar to CONFIG_OF_BOARD, but in this case the fdt is still built by u-boot build. This allows the board to patch the fdt, etc. In the specific case of dragonboard 410c, we pass the u-boot generated fdt to the previous stage of bootloader (by embedding it in the u-boot.img that is loaded by lk/

[U-Boot] [PATCH 2/3] video: simplefb

2017-08-03 Thread Rob Clark
Not really qcom specific, but for now qcom/lk is the one firmware that is (afaiu) setting up the appropriate dt node for pre-configured display. Uses the generic simple-framebuffer DT bindings so this should be useful on other platforms. Signed-off-by: Rob Clark --- drivers/video/Kconfig| 1

[U-Boot] [PATCH 3/3] video: add config option to skip framebuffer clear

2017-08-03 Thread Rob Clark
The use-case is that the thing that loaded u-boot already put a splash image on screen. And we want to preserve that until grub boot menu takes over. Signed-off-by: Rob Clark --- drivers/video/Kconfig| 8 drivers/video/cfb_console.c | 3 ++- drivers/video/video-uclass.c | 4 ++

[U-Boot] [PATCH 1/3] dm: core: also parse chosen node

2017-08-03 Thread Rob Clark
This is the node that would contain, for example, the framebuffer setup by an earlier stage. Signed-off-by: Rob Clark --- drivers/core/root.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/drivers/core/root.c b/drivers/core/root.c index d691d6ff94..5e6

Re: [U-Boot] [PATCH 2/2] armv8/ls1046a: RGMII PHY requires internal delay on Tx

2017-08-03 Thread York Sun
On 08/02/2017 11:01 PM, Madalin-cristian Bucur wrote: >> -Original Message- >> From: York Sun >> Sent: Wednesday, August 02, 2017 11:36 PM >> To: Madalin-cristian Bucur ; u-boot@lists.denx.de; >> joe.hershber...@ni.com >> Cc: Mingkai Hu >> Subject: Re: [PATCH 2/2] armv8/ls1046a: RGMII PHY

Re: [U-Boot] U-boot 2017.07 not working on Raspberry Pi

2017-08-03 Thread Stephen Warren
On 08/03/2017 07:45 AM, Simon Glass wrote: ... I'm not sure if we have a Raspberry Pi in a test farm anywhere. I should be able to look next week if no one can figure these out beforehand. I thought that Tom had some Pis in his test farm? ___ U-Boot m

Re: [U-Boot] rockchip: rk3288: fix EMMC_DIV_MASK definition in header

2017-08-03 Thread Philipp Tomsich
> It shoulb be '<<' instead of '<' for _MASK definition, fix it. > > Signed-off-by: Ziyuan Xu > Signed-off-by: Kever Yang > --- > > arch/arm/include/asm/arch-rockchip/cru_rk3288.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Philipp Tomsich ___

Re: [U-Boot] [U-Boot, 1/3] rockchip: add u-boot specific dts for rk3368 based boards

2017-08-03 Thread Philipp Tomsich
> Device drivers like debug serial, dmc should be enabled before > relocation, so add u-boot.dtsi files to contain devices that > should be marked as dm-pre-reloc. > > Signed-off-by: Andy Yan > Reviewed-by: Philipp Tomsich > --- > > arch/arm/dts/rk3368-geekbox-u-boot.dtsi | 35 > +

Re: [U-Boot] rockchip: rk3288: fix EMMC_DIV_MASK definition in header

2017-08-03 Thread Philipp Tomsich
> It shoulb be '<<' instead of '<' for _MASK definition, fix it. > > Signed-off-by: Ziyuan Xu > Signed-off-by: Kever Yang > --- > > arch/arm/include/asm/arch-rockchip/cru_rk3288.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Philipp Tomsich

Re: [U-Boot] [PATCH] dm: core: don't fail to iterate if first one fails to probe

2017-08-03 Thread Rob Clark
On Thu, Aug 3, 2017 at 11:24 AM, Simon Glass wrote: > Hi Rob, > > On 19 July 2017 at 10:53, Rob Clark wrote: >> On Thu, Jul 13, 2017 at 3:10 PM, Simon Glass wrote: >>> Hi Rob, >>> >>> On 21 June 2017 at 04:52, Rob Clark wrote: On Wed, Jun 21, 2017 at 6:23 AM, Peter Robinson wrote: >

Re: [U-Boot] [PATCH 0/9] spl: dm: Make it possible for the SPL to pick its own DTB from a FIT

2017-08-03 Thread Simon Glass
(correcting Tom's address) Hi Jean-Jacques, I see that I have made comments on v1 but now v2 has been sent. Sorry I was so late. Can you please take a look at the comments on v1? - Simon On 3 August 2017 at 09:25, Simon Glass wrote: > Hi Jean-Jacques, > > On 20 July 2017 at 11:02, Jean-Jacqu

Re: [U-Boot] [U-Boot, 1/3] rockchip: add u-boot specific dts for rk3368 based boards

2017-08-03 Thread Philipp Tomsich
> Device drivers like debug serial, dmc should be enabled before > relocation, so add u-boot.dtsi files to contain devices that > should be marked as dm-pre-reloc. > > Signed-off-by: Andy Yan > --- > > arch/arm/dts/rk3368-geekbox-u-boot.dtsi | 35 > + > arch/arm

Re: [U-Boot] [PATCH] board: pm9261: Update to support DT and DM

2017-08-03 Thread Simon Glass
On 21 July 2017 at 03:04, Wenyou Yang wrote: > Add the dts files to support deivce tree, update the configuration > files to support the device tree and driver model. The peripheral > clock and pins configuration are handled by the clock and the pinctrl > drivers respectively. > > Signed-off-by: W

Re: [U-Boot] [PATCH 5/9] lzo: add a function to check the validity of the header

2017-08-03 Thread Simon Glass
On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > Signed-off-by: Jean-Jacques Hiblot > --- > include/linux/lzo.h| 3 +++ > lib/lzo/lzo1x_decompress.c | 21 + > 2 files changed, 20 insertions(+), 4 deletions(-) Reviewed-by: Simon Glass _

Re: [U-Boot] [PATCH 2/9] fit: use 'const' for the input of fdt_offset() and locate_dtb_in_fit()

2017-08-03 Thread Simon Glass
On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > Those 2 functions don't modify their input, we can mark it const. > This prevents compilation warnings when they are provided const input. > > Signed-off-by: Jean-Jacques Hiblot > --- > common/boot_fit.c | 4 ++-- > include/boot_fit.h | 4 +

Re: [U-Boot] [PATCH 9/9] omap: detect board before spl_early_init()

2017-08-03 Thread Simon Glass
On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > In order to be able to select the right DTB, we need to have identified the > board before spl_early_init() is called. > > Signed-off-by: Jean-Jacques Hiblot > --- > arch/arm/mach-omap2/hwinit-common.c | 4 +++- > 1 file changed, 3 insertion

Re: [U-Boot] [PATCH 7/9] lib: allow building lzo and gunzip for the SPL

2017-08-03 Thread Simon Glass
Hi Jean-Jacques, On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > Signed-off-by: Jean-Jacques Hiblot > --- > lib/Kconfig | 11 +++ > lib/Makefile | 6 +++--- > 2 files changed, 14 insertions(+), 3 deletions(-) > > diff --git a/lib/Kconfig b/lib/Kconfig > index 09670f0..434bf23 1

Re: [U-Boot] [PATCH 3/3] board: at91sam9260ek: Use SPI-flash-based AT45xxx DataFlash

2017-08-03 Thread Simon Glass
On 20 July 2017 at 23:40, Wenyou Yang wrote: > To support driver model and device tree, use the SPI-flash-based > AT45xxx DataFlash driver, DataFlash is a kind of SPI flash. > Instead of ATMEL_DATAFLASH_SPI DataFlash older driver that will > be removed in the future. > > Signed-off-by: Wenyou Yang

Re: [U-Boot] [PATCH 1/3] board: at91sam9263ek: Use SPI-flash-based AT45xxx DataFlash

2017-08-03 Thread Simon Glass
On 20 July 2017 at 23:40, Wenyou Yang wrote: > To support driver model and device tree, use the SPI-flash-based > AT45xxx DataFlash driver, DataFlash is a kind of SPI flash. > Instead of ATMEL_DATAFLASH_SPI DataFlash older driver that will > be removed in the future. > > Signed-off-by: Wenyou Yang

Re: [U-Boot] [PATCH 4/9] fit: If no matching config is found in fit_find_config_node(), use the default one

2017-08-03 Thread Simon Glass
Hi Jean-Jacuqes, On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > If board_fit_config_name_match() doesn't match any configuration node, > then use the default one (if provided). > > Signed-off-by: Jean-Jacques Hiblot > --- > common/common_fit.c | 20 > 1 file changed

Re: [U-Boot] [PATCH 2/3] board: at91sam9rlek: Use SPI-flash-based AT45xxx DataFlash

2017-08-03 Thread Simon Glass
On 20 July 2017 at 23:40, Wenyou Yang wrote: > To support driver model and device tree, use the SPI-flash-based > AT45xxx DataFlash driver, DataFlash is a kind of SPI flash. > Instead of ATMEL_DATAFLASH_SPI DataFlash older driver that will > be removed in the future. > > Signed-off-by: Wenyou Yang

Re: [U-Boot] [PATCH 0/9] spl: dm: Make it possible for the SPL to pick its own DTB from a FIT

2017-08-03 Thread Simon Glass
Hi Jean-Jacques, On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: > Following the RFC, here is the series implementing the mechanism in a cleaner > way. The idea is that the SPL may take advantage of selecting its DTB from a > pool of available DTBs. To do that several DTBs are embedded in a

Re: [U-Boot] [PATCH] mtd: spi: sf_dataflash: Add print message while erase error

2017-08-03 Thread Simon Glass
On 20 July 2017 at 23:26, Wenyou Yang wrote: > Add the print message to tell us why the erase operation doesn't work. > > Signed-off-by: Wenyou Yang > --- > > drivers/mtd/spi/sf_dataflash.c | 10 -- > 1 file changed, 8 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass In general

Re: [U-Boot] [PATCH 1/9] dts: renamed FIT_EMBED to MULTI_DTB_FIT and moved it to the dts Kconfig

2017-08-03 Thread Simon Glass
On 20 July 2017 at 11:02, Jean-Jacques Hiblot wrote: Commit message? Need to explain reason for the change. > Signed-off-by: Jean-Jacques Hiblot > --- > Makefile | 2 +- > board/ti/ks2_evm/board_k2e.c | 2 +- > board/ti/ks2_evm/board_k2g.c | 2 +- > board/ti/ks2_evm/

  1   2   >