Re: [U-Boot] [PATCH 02/13][v4] imx: iomux-v3: add imx7d support for iomuxc

Hi Fabio, On 02/08/2015 17:48, Fabio Estevam wrote: > Hi Stefano, > > On Sun, Aug 2, 2015 at 6:40 AM, Stefano Babic wrote: > >> I have a main question. For i.MX6, we could factorize (mainly thanks >> Tim's work) the major SOC variants, getting some important goals, as >> having a single u-boot

Re: [U-Boot] [PATCH RESEND] tools/imximage: set DCD pointer to NULL when its length is 0

Hi Baruch, On 02/08/2015 20:45, Baruch Siach wrote: > Hi Stefano, > > On Wed, Jul 15, 2015 at 09:13:33AM +0200, Stefano Babic wrote: >> On 14/07/2015 21:05, Baruch Siach wrote: >>> On Thu, Jul 09, 2015 at 06:04:17PM +0200, Stefano Babic wrote: On 09/07/2015 17:19, Baruch Siach wrote: > W

[U-Boot] [PATCH 3/5] Allow arch-specific setting of global_data in board_init_f_mem()

At present we have a simple assignment to gd. With some archs this is implemented as a register or through some other means; a simple assignment does not suit in all cases. Change this to a function and add documentation to describe how this all works. Signed-off-by: Simon Glass --- common/boa

[U-Boot] [PATCH 0/5] x86: Move x86 to use board_init_f_mem() instead of assembler

Much of the early-init assembler on x86 can be removed if we use the new board_init_f_mem() function. At present only PowerPC uses it, but we should try to move more archs over. This has been contemplated for a while but it is time to take the plunge. Simon Glass (5): x86: Remove init_gd() fun

[U-Boot] [PATCH v2 1/3] common: Display >=4GiB memory bank size

bd->bi_dram[] has both start address and size defined as 32-bit, which is not the case on some platforms where >=4GiB memory bank is used. Change them to support such memory banks. Signed-off-by: Bin Meng --- Changes in v2: - Drop patches which are already applied - Change start to phys_addr_t

[U-Boot] [PATCH v2 3/3] x86: Document how to write PIRQ information in the device tree

Document the development flow on figuring out PIRQ information during the U-Boot porting. Signed-off-by: Bin Meng --- Changes in v2: - New patch to document how to write PIRQ information in the device tree doc/README.x86 | 41 + 1 file changed, 41 inser

[U-Boot] [PATCH v2 2/3] pci: Remove DEBUG from pci_compat.c

Remove DEBUG in drivers/pci/pci_compat.c. Signed-off-by: Bin Meng --- Changes in v2: - New patch to remove DEBUG from pci_compat.c drivers/pci/pci_compat.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/pci/pci_compat.c b/drivers/pci/pci_compat.c index 05c3510..712c48f 100644 ---

[U-Boot] [PATCH v2] armv8: fsl-lsch3: Rewrite MMU translation table entries

This patch rewrites MMU translation table entries to achieve: a) Start with all table entries as "invalid". b) Rewrite the table entries as "device-ngnrne" for cache-inhibit access. c) Rewrite the table entries as "normal" for cache-enabled access. Signed-off-by: Alison Wang Signed-off-by: York S

[U-Boot] target TQM834x giving build error with gcc v4.9

Hi, I am getting below error with gcc v4.9 building for powerpc target. Looks like GCCv4.9 is doing more optimization than earlier version. build 31-Jul-2015 02:49:45 powerpc: + TQM834x build 31-Jul-2015 02:49:45 +(TQM834x) (((base + size - 1) >> CSBNDS_EA_SHIFT) & build 31-

Re: [U-Boot] [PATCH] arm:ls1021a: Reserve secure code into RAM instead of OCRAM

Acked-by: Alison Wang Tested-by: Alison Wang Best Regards, Alison Wang > -Original Message- > From: Zhang Zhuoyu-B46552 > Sent: Friday, July 31, 2015 2:02 PM > To: Sun York-R58495 > Cc: u-boot@lists.denx.de; Jin Zhengxiong-R64188; Wang Huan-B18965; Wang > Dongsheng-B40534 > Subject: R

Re: [U-Boot] [PATCH] armv8: fsl-lsch3: Rewrite MMU translation table entries

York, Ok, I will rebase. Best Regards, Alison Wang > -Original Message- > From: Sun York-R58495 > Sent: Saturday, August 01, 2015 1:16 AM > To: Wang Huan-B18965; u-boot@lists.denx.de > Cc: Wang Huan-B18965 > Subject: Re: [PATCH] armv8: fsl-lsch3: Rewrite MMU translation table > e

Re: [U-Boot] board_pci_pre_scan() and board_pci_post_scan()

Hi Simon, On Mon, Aug 3, 2015 at 6:31 AM, Simon Glass wrote: > Hi Bin, > > On 27 July 2015 at 01:45, Bin Meng wrote: >> Hi Simon, >> >> With driver model pci conversion on x86 boards, we now missed the call >> to board_pci_pre_scan() and board_pci_post_scan() which are called in >> the non-dm pa

Re: [U-Boot] [PATCH 4/5] x86: Explicitly trigger pci bus configuration with driver model

Hi Simon, On Mon, Aug 3, 2015 at 6:31 AM, Simon Glass wrote: > Hi Bin, > > On 27 July 2015 at 01:33, Bin Meng wrote: >> With driver model, probing pci bus is all done on a lazy basis, >> as needed. On x86, pci bus is the fundamental device that needs >> to work before any other peripherals. In o

Re: [U-Boot] [PATCH 3/5] x86: queensbay: Support pre-relocation dm pci

Hi Simon, On Mon, Aug 3, 2015 at 6:31 AM, Simon Glass wrote: > Hi Bin, > > On 27 July 2015 at 01:33, Bin Meng wrote: >> Increase CONFIG_MALLOC_F_LEN so that dm pci does not fail with -ENOMEM >> before relocation. This makes pci uart work again on Intel Crown Bay. >> >> Signed-off-by: Bin Meng >

Re: [U-Boot] [PATCH 2/5] x86: fsp: Reserve different malloc size before relocation

Hi Simon, On Mon, Aug 3, 2015 at 6:31 AM, Simon Glass wrote: > Hi Bin, > > On 27 July 2015 at 01:33, Bin Meng wrote: >> Currently Intel queensbay fsp initializes a CAR size with only 4KiB, >> which is not enough when dm pci is enabled before relocation. Change >> arch/x86/start.s to reserve a sm

Re: [U-Boot] [PATCH 4/5] power: regulator: add pfuze100 support

Hi Simon, On Sun, Aug 02, 2015 at 04:30:52PM -0600, Simon Glass wrote: >Hi Peng, > >On 28 July 2015 at 08:48, Peng Fan wrote: >> 1. Add new regulator driver pfuze100. >>* Introduce struct pfuze100_regulator_desc for mataining info for >> regulator. >> 2. Add new Kconfig entry DM_REGULATOR_PFU

Re: [U-Boot] [PATCH 2/5] power: regulator use node name when no regulator-name

Hi Simon, On Sun, Aug 02, 2015 at 04:31:06PM -0600, Simon Glass wrote: >Hi Peng, > >On 28 July 2015 at 08:48, Peng Fan wrote: >> If there is no property named 'regulator-name' for regulators, >> choose node name instead, but not directly return failure value. >> >> Signed-off-by: Peng Fan >> Cc:

[U-Boot] [PATCH v2 2/3] x86: Convert minnowmax to use CONFIG_DM_USB

Move to driver model for USB on minnowmax. Signed-off-by: Simon Glass --- Changes in v2: None configs/minnowmax_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/minnowmax_defconfig b/configs/minnowmax_defconfig index 6ba22d6..4c13968 100644 --- a/configs/minnowmax_defcon

[U-Boot] [PATCH v2 1/3] x86: Convert minnowmax to use CONFIG_DM_NET

Move to driver model for networking on minnowmax. Signed-off-by: Simon Glass --- Changes in v2: None board/intel/minnowmax/minnowmax.c | 6 -- configs/minnowmax_defconfig | 1 + 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/board/intel/minnowmax/minnowmax.c b/board/i

Re: [U-Boot] [PATCH 1/2] gpio: Add DW APB GPIO driver

On Monday, August 03, 2015 at 01:38:28 AM, Simon Glass wrote: > Hi Marek, Hi Simon, [...] > >> > + if (!fdtdec_get_bool(blob, node, "gpio-controller")) > >> > + continue; > >> > + > >> > + plat = NULL; > >> > + plat = calloc(1, size

[U-Boot] [PATCH v2 3/3] x86: Show the un-relocated IP address in exceptions

When trying to figure out where an exception has occured, the relocated address is not a lot of help. Its value depends on various factors. Show the un-relocated IP as well. This can be looked up in System.map directly. Signed-off-by: Simon Glass --- Changes in v2: - Tweak the dump_regs() output

[U-Boot] [PATCH] Revert "fdt: Fix fdtdec_get_addr_size() for 64-bit"

This reverts commit 5b34436035fc862b5e8d0d2c3eab74ba36f1a7f4. This function has a few problems. It calls fdt_parent_offset() which as mentioned in code review is very slow. https://patchwork.ozlabs.org/patch/499482/ https://patchwork.ozlabs.org/patch/452604/ It also happens to break SPI flash on

[U-Boot] [PATCH 4/5] x86: Move the GDT into global_data

Rather than keeping track of the Global Descriptor Table in its own memory we may as well put it in global_data with everything else. As a first step, stop using the separately allocated GDT. Signed-off-by: Simon Glass --- arch/x86/cpu/cpu.c | 9 + arch/x86/include/asm/g

[U-Boot] [PATCH 2/5] Align global_data to a 16-byte boundary

Some archs like to have larger alignment for their global data. Use 16 bytes which suits all current archs. Signed-off-by: Simon Glass --- common/board_f.c | 2 +- include/asm-generic/global_data.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/common/bo

[U-Boot] [PATCH 1/5] x86: Remove init_gd() function

This is declared but no-longer exists. Drop it. Signed-off-by: Simon Glass --- arch/x86/include/asm/u-boot-x86.h | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/x86/include/asm/u-boot-x86.h b/arch/x86/include/asm/u-boot-x86.h index 4dae365..1c459d5 100644 --- a/arch/x86/include/asm/u-b

[U-Boot] [PATCH 5/5] x86: Switch to using generic global_data setup

There is quite a bit of assembler code that can be removed if we use the generic global_data setup. Less arch-specific code makes it easier to add new features and maintain the start-up code. Drop the unneeded code and adjust the hooks in board_f.c to cope. Signed-off-by: Simon Glass --- arch/

[U-Boot] [PATCH v2] x86: Enable debug UART for Minnowmax

Enable the debug UART and emit a single 'a' early in the init sequence to show that it is working. Unfortunately the debug UART implementation needs a stack to work. I cannot seem to remove this limitation as the absolute 'jmp %eax' instruction goes off into the weeds. So this means that the char

Re: [U-Boot] [PATCH] dm: core: Add Kconfig for simple bus driver

Hi Marek, On 2 August 2015 at 17:55, Marek Vasut wrote: > On Monday, August 03, 2015 at 01:52:03 AM, Simon Glass wrote: >> On 2 August 2015 at 17:15, Marek Vasut wrote: >> > Add Kconfig entries for the simple-bus driver, both for U-Boot >> > and for SPL. The simple-bus is enabled by default in U

Re: [U-Boot] [PATCH] dm: core: Add Kconfig for simple bus driver

On Monday, August 03, 2015 at 01:52:03 AM, Simon Glass wrote: > On 2 August 2015 at 17:15, Marek Vasut wrote: > > Add Kconfig entries for the simple-bus driver, both for U-Boot > > and for SPL. The simple-bus is enabled by default in U-Boot and > > disabled by default in SPL to preserve the origin

Re: [U-Boot] [PATCH] dm: core: Add Kconfig for simple bus driver

On 2 August 2015 at 17:15, Marek Vasut wrote: > Add Kconfig entries for the simple-bus driver, both for U-Boot > and for SPL. The simple-bus is enabled by default in U-Boot and > disabled by default in SPL to preserve the original behavior. > > Signed-off-by: Marek Vasut > Cc: Simon Glass > ---

Re: [U-Boot] [PATCH 15/15] x86: Update README to explain booting Ubuntu on Minnowmax

On 28 July 2015 at 01:50, Bin Meng wrote: > Hi Simon, > > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> The steps required to boot a Linux distribution from U-Boot on x86 are not >> very complicated, but it is a good idea to have these written down in an >> accessible place. >> >> Docume

Re: [U-Boot] [PATCH 1/2] gpio: Add DW APB GPIO driver

Hi Marek, On 2 August 2015 at 16:19, Marek Vasut wrote: > On Sunday, August 02, 2015 at 11:28:13 PM, Simon Glass wrote: >> Hi Marek, > > Hi, > >> On 27 July 2015 at 14:44, Marek Vasut wrote: >> > Add driver for the DesignWare APB GPIO IP block. >> > This driver is DM capable and probes from DT.

Re: [U-Boot] [PATCH 10/15] x86: dts: Fix typo in intel, irq-router.txt

On 28 July 2015 at 01:48, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> Fix a small typo in this binding file. >> >> Signed-off-by: Simon Glass >> --- >> >> doc/device-tree-bindings/misc/intel,irq-router.txt | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-)

Re: [U-Boot] [PATCH 13/15] x86: minnowmax: Drop the old PCI settings

On 28 July 2015 at 01:49, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> These are now in the device tree so we don't need to use the CONFIG options. >> Drop them. >> >> Signed-off-by: Simon Glass >> --- >> >> include/configs/minnowmax.h | 12 >> 1 file ch

Re: [U-Boot] [PATCH 09/15] x86: Drop CONFIG_DCACHE_RAM_MRC_VAR_SIZE from header file

On 28 July 2015 at 01:48, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> This is provided in Kconfig so we don't need it here. >> >> Signed-off-by: Simon Glass >> --- >> >> include/configs/x86-chromebook.h | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/incl

Re: [U-Boot] [PATCH 07/15] x86: Move CONFIG_X86_SERIAL to Kconfig

On 28 July 2015 at 01:48, Bin Meng wrote: > Hi Simon, > > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> Move this config option to Kconfig and tidy up. >> >> Signed-off-by: Simon Glass >> --- > > Reviewed-by: Bin Meng > > But please see nits below. Nits fixed and Applied to u-boot-x8

Re: [U-Boot] [PATCH 02/15] pci: Indicate prefetchable memory allocate when debugging

On 28 July 2015 at 01:47, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> At present the PCI output displays 'Mem' when it allocates memory for a PCI >> device, whether it is prefetchable or not. There is a distinction since the >> memory comes from separate pools. Use 'P

Re: [U-Boot] [PATCH 01/15] pci: Add a constant for an invalid interrupt

On 28 July 2015 at 01:46, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> Rather than using 0xff in the code, add a constant. >> >> Signed-off-by: Simon Glass >> --- >> >> drivers/pci/pci.c | 3 ++- >> include/pci.h | 2 ++ >> 2 files changed, 4 insertions(+), 1 del

Re: [U-Boot] [PATCH 03/15] dm: Add a return value comment to device_get_child()

On 28 July 2015 at 01:47, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> At present this function does not specify its return value. Fix it. >> >> Signed-off-by: Simon Glass >> --- >> > > Reviewed-by: Bin Meng > > But please see comments below. > >> include/dm/device.

Re: [U-Boot] [PATCH 06/15] x86: Move Chrome OS options to defconfig

On 28 July 2015 at 01:48, Bin Meng wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass wrote: >> Drop these from the header file and use Kconfig instead. >> >> Signed-off-by: Simon Glass >> --- >> >> configs/chromebook_link_defconfig | 3 +++ >> configs/chromebox_panther_defconfig | 3 +++ >

Re: [U-Boot] [PATCH v2 20/47] efi: Avoid using non-existent text base

On 2 August 2015 at 09:04, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> From: Ben Stoltz >> >> When U-Boot runs as an EFI application is does not have a definition of >> CONFIG_SYS_TEXT_BASE. U-Boot is a relocatable application and the relocation >> is done by EFI. U

Re: [U-Boot] [PATCH v2 19/47] efi: Add start-up library code

On 2 August 2015 at 09:00, Bin Meng wrote: > Hi Simon, > > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> When running as an EFI application, U-Boot must request memory from EFI, >> and provide access to the boot services U-Boot needs. >> >> Add library code to perform these tasks. This

Re: [U-Boot] [PATCH v2 21/47] efi: Display the correct initcall pre-relocation values

On 31 July 2015 at 09:31, Simon Glass wrote: > With EFI the start address of U-Boot is specified differently. We could > consider just setting GD_FLG_RELOC and then setting up reloc_off. But that > flag has other implementations and we are not able to use U-Boot relocation > which this flag implie

Re: [U-Boot] [PATCH v2 18/47] x86: Allow use of global_data with EFI

On 2 August 2015 at 08:32, Bin Meng wrote: > Hi Simon, > > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> On x86 the global_data pointer is provided through a somewhat-bizarre and >> x86-specific mechanism: the F segment register is set to a pointer to the >> start of global_data, so tha

Re: [U-Boot] [PATCH v2 17/47] pci: Fix up code for CONFIG_PCI_ENUM_ONLY

On 2 August 2015 at 07:22, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> This option is not used by any board but appears to still be useful, at least >> for testing. With recent commits it does not build, so fix it. >> >> Signed-off-by: Simon Glass >> --- >> >> Chang

Re: [U-Boot] [PATCH v2 11/47] x86: Tidy up global_data flags

On 2 August 2015 at 07:17, Bin Meng wrote: > Hi Simon, > > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> These flags now overlap some global ones. Adjust the x86-specific flags to >> avoid this. Since this requires a change to the start.S code, add a way for >> tools to find the 32-bit

Re: [U-Boot] [PATCH v2 15/47] x86: Tidy up a few minor issues with interrupts

On 2 August 2015 at 07:19, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> Fix a typo, remove an unused field and make sure to use existing #define >> constants instead of open-coded values. >> >> Signed-off-by: Simon Glass >> --- >> >> Changes in v2: None >> >> arch/x

Re: [U-Boot] [PATCH v2 12/47] x86: Drop unused copy_fdt_to_ram()

On 31 July 2015 at 09:31, Simon Glass wrote: > This is now handled by generic U-Boot code so we do not need an x86 version. > It is no-longer called, so remove it. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > arch/x86/include/asm/relocate.h | 1 - >

Re: [U-Boot] [PATCH v2 10/47] Reserve the top 16 flag bits for architecture-specific use

On 2 August 2015 at 07:09, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> Add a convention that the generic global_data only occupy the bottom 16 bits >> of the flags word, so that there is less chance of a conflict. At present the >> x86 flags conflict. >> >> Signed-of

Re: [U-Boot] [PATCH v2 16/47] pci: Use debug() instead of DEBUGF() in pci_auto.c

On 31 July 2015 at 09:31, Simon Glass wrote: > This file does not need its own way of doing debug(). Clean it up to use the > new way. > > Signed-off-by: Simon Glass > Acked-by: Pavel Machek > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > drivers/pci/pci_auto.c | 48 +

Re: [U-Boot] [PATCH v2 13/47] x86: Tidy up the 64-bit calling code

On 31 July 2015 at 09:31, Simon Glass wrote: > The GDT works but technically the length is incorrect. Fix this and add a > comment. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > arch/x86/cpu/call64.S | 4 ++-- > 1 file changed, 2 insertions(+), 2 delet

Re: [U-Boot] [PATCH v2 14/47] x86: Add some missing global_data declarations in files that use gd

On 31 July 2015 at 09:31, Simon Glass wrote: > Some files use global_data but don't declare it. Fix this. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > arch/x86/cpu/mp_init.c | 2 ++ > arch/x86/lib/bootm.c | 2 ++ > arch/x86/lib/fsp/f

Re: [U-Boot] [PATCH v2 09/47] x86: Use CR0 constants in CPU init

On 31 July 2015 at 09:31, Simon Glass wrote: > We should use these constants where possible. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > arch/x86/cpu/cpu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Applied to u-boot-x86. __

Re: [U-Boot] [PATCH v2 07/47] Bring in __aligned_u64 and friends to linux/types.h

On 2 August 2015 at 07:07, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> These will be used for efi.h both for U-Boot running as an EFI application >> and as a payload. They come from Linux 4.1. >> >> Signed-off-by: Simon Glass >> --- >> >> Changes in v2: >> - Fix _al

Re: [U-Boot] [PATCH v2 08/47] x86: Add various minor tidy-ups to the 32-bit startup code

On 31 July 2015 at 09:31, Simon Glass wrote: > Fix a typo, improve some comments and add a little more detail in some > cases. > > Signed-off-by: Simon Glass > Reviewed-by: Bin Meng > --- > > Changes in v2: None > > arch/x86/cpu/start.S | 46 ++ > 1 f

Re: [U-Boot] [PATCH v2 04/47] elf: Add a few definitions for 64-bit relocation

On 2 August 2015 at 07:00, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> Provide the types necessary to relocate 64-bit images. >> >> Signed-off-by: Simon Glass >> >> --- >> >> Changes in v2: >> - Follow existing file conventions for type definitions >> >> include/el

Re: [U-Boot] [PATCH v2 06/47] Allow objcopy to work without filling gaps with 0xff

On 31 July 2015 at 09:31, Simon Glass wrote: > This is currently done for all targets, since 0xff is the default erased > value for most flash devices. In some cases this is not what we want (e.g. > for EFI images) so provide a command to do a vanilla objcopy. > > Signed-off-by: Simon Glass > Rev

Re: [U-Boot] [PATCH 2/2] common: Print nothing in the __weak checkboard()

On 2 August 2015 at 15:26, Simon Glass wrote: > On 31 July 2015 at 01:24, Bin Meng wrote: >> Do not print confusing "Board: Unknown" during boot. >> >> Signed-off-by: Bin Meng >> --- >> >> common/board_info.c | 1 - >> 1 file changed, 1 deletion(-) >> > > Acked-by: Simon Glass > >> diff --git

Re: [U-Boot] [PATCH 1/2] gitignore: Add defconfig and fdtgrep

On 2 August 2015 at 15:26, Simon Glass wrote: > On 31 July 2015 at 01:24, Bin Meng wrote: >> Ignore defconfig and tools/fdtgrep. >> >> Signed-off-by: Bin Meng >> --- >> >> .gitignore | 1 + >> tools/.gitignore | 1 + >> 2 files changed, 2 insertions(+) > > Acked-by: Simon Glass Applied

Re: [U-Boot] [PATCH v2 03/47] dm: serial: Correct logic in serial_find_console_or_panic()

On 31 July 2015 at 09:31, Simon Glass wrote: > In a final attempt to find a console UART this function uses the first > first available serial device. However the check for a valid device is > inverted. > > This code is only executed when there is in fact no serial UART, but at > present it can fa

Re: [U-Boot] [PATCH v2 01/47] Support removing default assembler flags

On 2 August 2015 at 09:55, Tom Rini wrote: > On Fri, Jul 31, 2015 at 09:31:18AM -0600, Simon Glass wrote: > >> The CFLAGS_REMOVE_ feature allows default C compiler flags to be >> removed for particular files. Add the same feature for assembler, using >> AFLAGS_REMOVE_. >> >> Signed-off-by: Simon G

Re: [U-Boot] [PATCH v2 02/47] dm: core: Check for empty list in uclass_find_device()

On 2 August 2015 at 06:57, Bin Meng wrote: > On Fri, Jul 31, 2015 at 11:31 PM, Simon Glass wrote: >> This function needs to check the list has entries before traversing it. >> Fix this bug. >> >> Signed-off-by: Simon Glass >> --- >> >> Changes in v2: >> - Correct the return code to avoid a test

Re: [U-Boot] [PATCH 7/7] x86: bayleybay: Configure PCI IRQ

On 2 August 2015 at 15:21, Simon Glass wrote: > Hi Bin, > > On 30 July 2015 at 22:01, Bin Meng wrote: >> Hi Simon, >> >> On Fri, Jul 31, 2015 at 3:53 AM, Simon Glass wrote: >>> Hi Bin, >>> >>> On 30 July 2015 at 04:49, Bin Meng wrote: Add PCI IRQ routing information in the board device tre

Re: [U-Boot] [PATCH] common: command add '\n' for debug msg

On 2 August 2015 at 15:25, Simon Glass wrote: > On 28 July 2015 at 08:45, Peng Fan wrote: >> Add '\n' for debug msg. >> >> Signed-off-by: Peng Fan >> Cc: Tom Rini >> Cc: Masahiro Yamada >> Cc: Simon Glass >> --- >> common/command.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > >

Re: [U-Boot] [PATCH] lib: fdt: fix indent of #ifdef..#endif conditional

On 2 August 2015 at 15:21, Simon Glass wrote: > On 1 August 2015 at 01:03, Masahiro Yamada > wrote: >> Match the depth of indentation between #ifdef and #endif >> for better readability. >> >> Signed-off-by: Masahiro Yamada >> --- >> >> lib/fdtdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1

Re: [U-Boot] [PATCH 2/2] x86: qemu: Add MP initialization

On 2 August 2015 at 15:23, Simon Glass wrote: > On 27 July 2015 at 05:16, Bin Meng wrote: >> Add a cpu1 node to the device tree and enable the MP initialization >> on QEMU targets (i440fx and q35). >> >> Signed-off-by: Bin Meng >> --- >> >> arch/x86/dts/qemu-x86_i440fx.dts | 7 +++ >> arch/

Re: [U-Boot] [PATCH 5/5] doc: dm: Update pci-info.txt for pci support

On 2 August 2015 at 15:24, Simon Glass wrote: > On 27 July 2015 at 01:33, Bin Meng wrote: >> Correct two typos and mention how pci bus will be probed. >> >> Signed-off-by: Bin Meng >> --- >> >> doc/driver-model/pci-info.txt | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) > > Acke

Re: [U-Boot] [PATCH 6/7] x86: Add Intel Bayley Bay board support

On 30 July 2015 at 13:53, Simon Glass wrote: > On 30 July 2015 at 04:49, Bin Meng wrote: >> Intel Bayley Bay board is a BayTrail based board. Add this board >> with existing baytrail fsp support. >> >> Signed-off-by: Bin Meng >> --- >> >> arch/x86/dts/Makefile | 3 +- >> arch/x86/

Re: [U-Boot] [PATCH 1/2] x86: Add a 'pause' instruction in __udelay() for QEMU target

On 2 August 2015 at 15:24, Simon Glass wrote: > On 27 July 2015 at 05:16, Bin Meng wrote: >> From: Miao Yan >> >> When running SMP configuration on QEMU (tcg mode, no kvm), there is >> a busy loop in start_aps(), calling udelay(), that waits for APs to >> show up online. However, there is a chan

Re: [U-Boot] [PATCH 5/7] x86: Add microcode for BayTrail-I B0 stepping

On 30 July 2015 at 13:53, Simon Glass wrote: > On 30 July 2015 at 04:49, Bin Meng wrote: >> This commit adds the microcode blob for BayTrail-I B0 stepping, >> CPUID signature 30671h. >> >> Signed-off-by: Bin Meng >> --- >> >> arch/x86/dts/microcode/m0230671117.dtsi | 4244 >> ++

Re: [U-Boot] [PATCH 4/7] x86: baytrail: Update UPD setting for FSP Gold4 release

On 30 July 2015 at 13:53, Simon Glass wrote: > On 30 July 2015 at 04:49, Bin Meng wrote: >> BayTrail FSP Gold4 release adds one UPD parameter to control IGD >> enable/disable. >> >> Signed-off-by: Bin Meng >> --- >> >> arch/x86/include/asm/arch-baytrail/fsp/fsp_vpd.h | 3 ++- >> 1 file changed,

Re: [U-Boot] [PATCH 3/7] x86: minnowmax: Remove smsc47x superio codes

On 30 July 2015 at 13:55, Simon Glass wrote: > On 30 July 2015 at 04:49, Bin Meng wrote: >> On Intel BayTrail SoC, there is a legacy UART (I/O 0x3f8) integrated >> into the SoC which is enabled by the FSP. Remove the smsc47x superio >> initialization codes. >> >> Signed-off-by: Bin Meng >> --- >

Re: [U-Boot] [PATCH 2/7] x86: Sanity test on vesa parameters before setting up kernel screen_info

On 30 July 2015 at 13:53, Simon Glass wrote: > On 30 July 2015 at 04:49, Bin Meng wrote: >> We should not set up kernel screen_info when the vesa parameters are >> insane, otherwise kernel will panic. >> >> Signed-off-by: Bin Meng >> --- >> >> drivers/pci/pci_rom.c | 4 >> 1 file changed,

Re: [U-Boot] [PATCH 1/5] dm: pci: Allow scan bridge child devices before relocation

On 2 August 2015 at 15:24, Simon Glass wrote: > On 27 July 2015 at 01:33, Bin Meng wrote: >> On some platforms pci devices behind bridge need to be probed (eg: >> a pci uart on recent x86 chipset) before relocation. Remove such >> limitation so that dm pci can be used before relocation. >> >> Sig

Re: [U-Boot] [PATCH 5/5] dra7xx: Add dra72_evm_defconfig using CONFIG_DM

Hi Tom. On 2 August 2015 at 17:29, Tom Rini wrote: > On Sun, Aug 02, 2015 at 05:22:30PM -0600, Simon Glass wrote: >> Hi Tom, >> >> On 2 August 2015 at 17:07, Tom Rini wrote: >> > On Sun, Aug 02, 2015 at 03:29:12PM -0600, Simon Glass wrote: >> >> Hi Tom, >> >> >> >> On 31 July 2015 at 17:55, Tom

[U-Boot] [PATCH] sf: Make 4K sector support configurable

Make the support for 4K subpage I/O on a SPI NOR flash configurable. A board which requires the SPI NOR to be accessed in larger 32KiB or 64KiB pages can disable the 4K subpage support, but by default, the support for 4K subpage I/O is enabled. The functionality of this option is the same as CONFIG

Re: [U-Boot] [PATCH 5/5] dra7xx: Add dra72_evm_defconfig using CONFIG_DM

On Sun, Aug 02, 2015 at 05:22:30PM -0600, Simon Glass wrote: > Hi Tom, > > On 2 August 2015 at 17:07, Tom Rini wrote: > > On Sun, Aug 02, 2015 at 03:29:12PM -0600, Simon Glass wrote: > >> Hi Tom, > >> > >> On 31 July 2015 at 17:55, Tom Rini wrote: > >> > - Import various DT files for DRA7 / DR72

Re: [U-Boot] [PATCH 5/5] dra7xx: Add dra72_evm_defconfig using CONFIG_DM

Hi Tom, On 2 August 2015 at 17:07, Tom Rini wrote: > On Sun, Aug 02, 2015 at 03:29:12PM -0600, Simon Glass wrote: >> Hi Tom, >> >> On 31 July 2015 at 17:55, Tom Rini wrote: >> > - Import various DT files for DRA7 / DR72x / dra72-evm from Linux Kernel >> > v4.1 >> > - Add config file for this b

[U-Boot] [PATCH 09/15] ddr: altera: sequencer: Pluck out IO_* macros from code

Actually convert the sequencer code to use socfpga_sdram_io_config instead of the IO_* macros. This is just an sed excercise here, no manual coding needed. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 201 - 1 file changed, 100 insertion

[U-Boot] [PATCH 15/15] ddr: altera: sequencer: Clean checkpatch issues

Fix most of the dangling checkpatch issues, no functional change. There are still 7 warnings, 1 checks , but those are left in place for the sake of readability of the code. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 159 +++-- 1 file chan

[U-Boot] [PATCH 04/15] ddr: altera: sequencer: Zap bogus redefinition of RW_MGR_MEM_NUMBER_OF_RANKS

This is defined in the QTS-generated headers, so it must not be re-defined in sequencer.h . Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/ddr/altera/sequencer.h b/drivers/ddr/altera/sequencer.h index 3ecd733..f621e14 1006

[U-Boot] [PATCH 05/15] ddr: altera: sequencer: Wrap ac_rom_init and inst_rom_init

Introduce two wrapper functions, socfpga_get_seq_ac_init() and socfpga_get_seq_inst_init() to avoid direct inclusion of the sequencer_auto_ac_init.h and sequencer_auto_inst_init.h QTS generated files. This reduces namespace polution again. Signed-off-by: Marek Vasut --- arch/arm/mach-socfpga/inc

[U-Boot] [PATCH 12/15] ddr: altera: sequencer: Zap SEQ_T(INIT|RESET)_CNTR._VAL

This is another macro used to obfuscate the real code. The T(INIT|RESET)_CNTR._VAL is always defined, so this indirection is unnecessary. Get rid of this. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 8 drivers/ddr/altera/sequencer.h | 45

[U-Boot] [PATCH 06/15] ddr: altera: sequencer: Wrap RW_MGR_* macros

Introduce structure socfpga_sdram_rw_mgr_config to wrap the RW manager configuration values in board file. Introduce a complementary function, socfpga_get_sdram_rwmgr_config(), which returns this the structure. This is another step toward wrapping the nasty QTS generated macros in board files and r

[U-Boot] [PATCH 14/15] ddr: altera: sequencer: Clean data types

Replace uintNN_t with uNN. No functional change. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 96 +- 1 file changed, 48 insertions(+), 48 deletions(-) diff --git a/drivers/ddr/altera/sequencer.c b/drivers/ddr/altera/sequencer.c index e6

[U-Boot] [PATCH 03/15] ddr: altera: sequencer: Zap unused params and macros

These parameters are not used in the code, zap them and the macros which are used by them as well. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 49 +- drivers/ddr/altera/sequencer.h | 31 -- 2 files changed, 5 ins

[U-Boot] [PATCH 11/15] ddr: altera: sequencer: Zap VFIFO_SIZE

Just use READ_VALID_FIFO_SIZE directly, no need for this macro obfuscation. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 8 drivers/ddr/altera/sequencer.h | 3 --- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/ddr/altera/sequencer.c b/drivers/

[U-Boot] [PATCH 10/15] ddr: altera: sequencer: Wrap misc remaining macros

Introduce structure socfpga_sdram_misc_config to wrap the remainging misc configuration values in board file. Again, introduce a function, socfpga_get_sdram_misc_config(), which returns this the structure. This is almost the final step toward wrapping the nasty QTS generated macros in board files a

[U-Boot] [PATCH 00/15] socfpga: sequencer.c cleanups

This entire series focuses solely on cleaning up the drivers/ddr/altera/sequencer.c file. After this series, this one file is totally checkpatch clean and does not pull in any weird qts-generated macros ; all that is wrapped in the board file. This micro-series applies on top of my previous middle

[U-Boot] [PATCH 13/15] ddr: altera: sequencer: Pluck out misc macros from code

Actually convert the sequencer code to use socfpga_sdram_misc_config instead of the various macros. This is just an sed excercise here, no manual coding needed. This patch actually removes the need to include any board-specific files in sequencer.c , so sequencer.c namespace is now no longer polut

[U-Boot] [PATCH 08/15] ddr: altera: sequencer: Wrap IO_* macros

Introduce structure socfpga_sdram_io_config to wrap the IO configuration values in board file. Introduce socfpga_get_sdram_io_config() function, which returns this the structure. This is another step toward wrapping the nasty QTS generated macros in board files and reducing the polution of the name

[U-Boot] [PATCH 07/15] ddr: altera: sequencer: Pluck out RW_MGR_* macros from code

Actually convert the sequencer code to use socfpga_sdram_rw_mgr_config instead of the RW_MGR_* macros. This is just an sed excercise here, no manual coding needed. Signed-off-by: Marek Vasut --- drivers/ddr/altera/sequencer.c | 292 - drivers/ddr/altera/se

[U-Boot] [PATCH 01/15] ddr: altera: sequencer: Move qts-generated files to board dir

Move the files generated by QTS into the board directory, they should not be part of the driver files at all. Signed-off-by: Marek Vasut --- .../altera => board/altera/socfpga/qts}/sequencer_auto.h| 0 .../altera/socfpga/qts}/sequencer_auto_ac_init.h| 0 .../altera/socfpga/qts}

[U-Boot] [PATCH 02/15] ddr: altera: sequencer: Clean up mach/sdram.h

Zap non-existent functions and place function prototypes at the beginning of the header file. Signed-off-by: Marek Vasut --- arch/arm/mach-socfpga/include/mach/sdram.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/arch/arm/mach-socfpga/include/mach/sdram.h b/arch/arm/ma

[U-Boot] [PATCH] dm: core: Add Kconfig for simple bus driver

Add Kconfig entries for the simple-bus driver, both for U-Boot and for SPL. The simple-bus is enabled by default in U-Boot and disabled by default in SPL to preserve the original behavior. Signed-off-by: Marek Vasut Cc: Simon Glass --- drivers/core/Kconfig | 15 +++ drivers/core/Ma

Re: [U-Boot] [PATCH] dm: core: Allow simple-bus driver in SPL

On Sunday, August 02, 2015 at 11:28:10 PM, Simon Glass wrote: > Hi Marek, > > On 27 July 2015 at 14:31, Marek Vasut wrote: > > There's no reason why simple-bus driver can not be used in SPL, > > in fact it is necessary to get SoCFPGA SPL probe the cadence > > SPI driver. So drop the restriction.

Re: [U-Boot] [PATCH 5/5] dra7xx: Add dra72_evm_defconfig using CONFIG_DM

On Sun, Aug 02, 2015 at 03:29:12PM -0600, Simon Glass wrote: > Hi Tom, > > On 31 July 2015 at 17:55, Tom Rini wrote: > > - Import various DT files for DRA7 / DR72x / dra72-evm from Linux Kernel > > v4.1 > > - Add config file for this board, enable DM and DM_GPIO > > > > Cc: Simon Glass > > Sig

Re: [U-Boot] [PATCH v2 9/9] dfu:tests: Modify dfu_gadget_test.sh to accept USB device major:minor number

Hi Lukasz, On 25 July 2015 at 02:11, Lukasz Majewski wrote: > In the dfu-util it is possible to set major:minor number by unsing -d flag using > (-d 0451:d022). > Such option is very handy when many DFU devices are connected to a single > host PC. This commit allows testing when above situation

Re: [U-Boot] [PATCH v2 8/9] config: bbb: Configs necessary for running update via TFTP on Beagle Bone Black

HI Lukasz, On 25 July 2015 at 02:11, Lukasz Majewski wrote: Commit message? E.g. 'Enable DFU via update on Beaglebone Black' > Signed-off-by: Lukasz Majewski > --- > Changes for v2: > - Do not enable CONFIG_UPDATE_TFTP since CONFIG_DFU_TFTP enables the common > code > - Do not enable CONFIG_C

Re: [U-Boot] [PATCH v2 5/9] dfu: tftp: update: Add dfu_write_from_mem_addr() function

Hi Lukasz, On 25 July 2015 at 02:11, Lukasz Majewski wrote: > This function allows writing via DFU data stored from fixed buffer address > (like e.g. loadaddr env variable). > > Such predefined buffers are used in the update_tftp() code. In fact this > function is a wrapper on the dfu_write() and

  1   2   3   >