January 2016 - U-Boot

[U-Boot] [PATCH 1/2] arm: add a weak function reset_cpu
by Peng Fan 07 Jan '16

07 Jan '16

This commit 4bdcbe60a142b08eefccb0e326a37ba81d3389e8 removes reset_cpu which breaks SPL build when DEBUG macro defined. Introudce a weak function reset_cpu to avoid build failure when building SPL for i.MX with DEBUG enabled. " arch/arm/lib/built-in.o: In function `do_reset': ~/uboot/arch/arm/lib/reset.c:45: undefined reference to `reset_cpu' scripts/Makefile.spl:244: recipe for target 'spl/u-boot-spl' failed " Signed-off-by: Peng Fan <peng.fan(a)nxp.com> Cc: Albert Aribaud <albert.u.boot(a)aribaud.net> Cc: Stefano Babic <sbabic(a)denx.de> Cc: Fabio Estevam <fabio.estevam(a)freescale.com> --- arch/arm/lib/reset.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/arch/arm/lib/reset.c b/arch/arm/lib/reset.c index 9a95f08..4dd0131 100644 --- a/arch/arm/lib/reset.c +++ b/arch/arm/lib/reset.c @@ -27,6 +27,10 @@ __weak void reset_misc(void) { } +__weak void reset_cpu(ulong addr) +{ +} + int do_reset(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { puts ("resetting ...\n"); -- 2.6.2

2 5

[U-Boot] [PATCH v2 1/3] phy: atheros: Use ar8035_config for AR8031
by Fabio Estevam 07 Jan '16

07 Jan '16

From: Fabio Estevam <fabio.estevam(a)nxp.com> Commit 08ad9b068afb88 (" ar8031: modify the config func of ar8031 to ar8021_config") selected 'ar8021_config' as the configuration function for AR8031. The correct would be to use 'ar8035_config' instead as AR8031/AR8035 have the same programming model and even share the same phy driver in the linux kernel: drivers/net/phy/at803x.c. Tested on a mx6qsabresd and wandboard, which now can work without any PHY setup code in the board files. Signed-off-by: Fabio Estevam <fabio.estevam(a)nxp.com> --- This is 2016.04 material. Zhao Qiang, could you please test this on your platform? Changes since v1: - None drivers/net/phy/atheros.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/atheros.c b/drivers/net/phy/atheros.c index d509e30..ba57b1a 100644 --- a/drivers/net/phy/atheros.c +++ b/drivers/net/phy/atheros.c @@ -51,7 +51,7 @@ static struct phy_driver AR8031_driver = { .uid = 0x4dd074, .mask = 0xffffffef, .features = PHY_GBIT_FEATURES, - .config = ar8021_config, + .config = ar8035_config, .startup = genphy_startup, .shutdown = genphy_shutdown, }; -- 1.9.1

4 12

[U-Boot] [PATCH] pfuze: Fix unsigned variable for less-than-zero comparison
by Peng Fan 07 Jan '16

07 Jan '16

From: "Ye.Li" <B37916(a)freescale.com> According to the Coverity result, a unsigned int variable is used fo less- than-zero comparison, the result is never true. Need to fix the variable type to signed int. Signed-off-by: Ye.Li <B37916(a)freescale.com> Signed-off-by: Peng Fan <peng.fan(a)nxp.com> Cc: Przemyslaw Marczak <p.marczak(a)samsung.com> Cc: Stefano Babic <sbabic(a)denx.de> --- board/freescale/common/pfuze.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/board/freescale/common/pfuze.c b/board/freescale/common/pfuze.c index 783c46d..69afa83 100644 --- a/board/freescale/common/pfuze.c +++ b/board/freescale/common/pfuze.c @@ -13,7 +13,8 @@ int pfuze_mode_init(struct pmic *p, u32 mode) { unsigned char offset, i, switch_num; - u32 id, ret; + u32 id; + int ret; pmic_reg_read(p, PFUZE100_DEVICEID, &id); id = id & 0xf; -- 2.6.2

2 3

[U-Boot] [PATCH 1/3] spi: fsl_qspi: fix an error of using GENMASK
by Gong Qianyu 07 Jan '16

07 Jan '16

This commit fixes the change of bad490a24212c068c5b718b9189f47ea4075d078. Signed-off-by: Gong Qianyu <Qianyu.Gong(a)freescale.com> --- drivers/spi/fsl_qspi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/spi/fsl_qspi.c b/drivers/spi/fsl_qspi.c index ed39114..d8d37cd 100644 --- a/drivers/spi/fsl_qspi.c +++ b/drivers/spi/fsl_qspi.c @@ -24,7 +24,7 @@ DECLARE_GLOBAL_DATA_PTR; #define TX_BUFFER_SIZE 0x40 #endif -#define OFFSET_BITS_MASK GENMASK(24, 0) +#define OFFSET_BITS_MASK GENMASK(23, 0) #define FLASH_STATUS_WEL 0x02 -- 2.1.0.27.g96db324

3 4

[U-Boot] [PATCH] spi: spi_flash: Fix the arguments of stm_is_locked_sr()
by Fabio Estevam 07 Jan '16

07 Jan '16

From: Fabio Estevam <fabio.estevam(a)nxp.com> stm_is_locked_sr() takes the status register (SR) value as the last parameter, not the second. Based on a patch from Brian Norris for the linux kernel: http://git.infradead.org/linux-mtd.git/commitdiff/a32d5b726ff8cf32bf491522b… Signed-off-by: Fabio Estevam <fabio.estevam(a)nxp.com> --- drivers/mtd/spi/spi_flash.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 7ffa136..b7b7f00 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -767,8 +767,8 @@ int stm_unlock(struct spi_flash *flash, u32 ofs, size_t len) return ret; /* Cannot unlock; would unlock larger region than requested */ - if (stm_is_locked_sr(flash, status_old, ofs - flash->erase_size, - flash->erase_size)) + if (stm_is_locked_sr(flash, ofs - flash->erase_size, flash->erase_size, + status_old)) return -EINVAL; /* * Need largest pow such that: -- 1.9.1

2 5

[U-Boot] [PATCH] power: pmic.h: Drop include <spi.h>
by Jagan Teki 07 Jan '16

07 Jan '16

include/power/pmic.h never used any generic spi code from include/spi.h, but this has been added in below commit. "dm: pmic: add implementation of driver model pmic uclass" (sha1: 4d9057e82be11a862db411c4867e859fe0d4ca2a) Adding functionalities in include/spi.h will trigger a build issue as this been added used in include/power/pmic.h Build issue on trats2 with adding BIT macro on spi.h: ---------------------------------------------------- CC lib/asm-offsets.s In file included from include/power/pmic.h:15:0, from include/power/max77686_pmic.h:11, from include/configs/trats2.h:212, from include/config.h:5, from include/common.h:18, from lib/asm-offsets.c:15: include/spi.h: In function 'spi_w8r8': include/spi.h:327:2: warning: implicit declaration of function 'BIT' [-Wimplicit-function-declaration] Cc: Simon Glass <sjg(a)chromium.org> Cc: Przemyslaw Marczak <p.marczak(a)samsung.com> Reported-by: Tom Rini <trini(a)konsulko.com> Signed-off-by: Jagan Teki <jteki(a)openedev.com> --- include/power/pmic.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/power/pmic.h b/include/power/pmic.h index 6ba4b6e..e0b2e12 100644 --- a/include/power/pmic.h +++ b/include/power/pmic.h @@ -12,7 +12,6 @@ #define __CORE_PMIC_H_ #include <i2c.h> -#include <spi.h> #include <linux/list.h> #include <power/power_chrg.h> -- 1.9.1

3 4

[U-Boot] u-boot building
by Flavio Leonel 07 Jan '16

07 Jan '16

Good afternoon friends! I am new to the u-boot I have a unifi ubiquiti AC outdoor this with a bootloader version CEF I wanted to put the u-boot again It would be possible without having to rewrite the flash?

1 0

[U-Boot] [PATCH 0/3] dm: add dev_get_reg() for getting device node's reg
by Przemyslaw Marczak 07 Jan '16

07 Jan '16

commit: dm: core: Enable optional use of fdt_translate_address() enables device's bus/child address translation method, depending on bus 'ranges' property and including child 'reg' property. This change makes impossible to decode the 'reg' for node with '#size-cells' equal to 0. Such case is possible by the specification and is also used in U-Boot, e.g. by I2C uclass or S5P GPIO - the last one is broken at present. For this purpose this patch set introduces new core function: fdt_addr_t dev_get_reg(struct udevice *dev) which returns the 'reg' value in the same way as previously dev_get_addr(). This fixes s5p gpio driver and booting issue on few Exynos based boards: - Trats2 - Odroid U3/X2 As an example of use, this patch set also modifies i2c uclass driver by using the new function for getting chip address. Przemyslaw Marczak (3): dm: core: extend API by new function: dev_get_reg() gpio: s5p: use dev_get_reg() instead of dev_get_addr() dm: i2c: get chip address with dev_get_reg() drivers/core/device.c | 17 +++++++++++++---- drivers/gpio/s5p_gpio.c | 2 +- drivers/i2c/i2c-uclass.c | 17 ++++++++--------- include/dm/device.h | 23 +++++++++++++++++++++++ include/i2c.h | 11 ++++------- 5 files changed, 49 insertions(+), 21 deletions(-) -- 1.9.1

3 19

[U-Boot] [PATCH v5 0/8] x86: qemu: add fw_cfg interface support for qemu-x86 targets
by Miao Yan 07 Jan '16

07 Jan '16

The fw_cfg interface provided by QEMU allow guests to retrieve various information about the system, e.g. cpu number, variaous firmware data, kernel setup, etc. The fw_cfg interface can be accessed through 3 IO ports (on x86), using x86 in/out instructions. - 0x510: select configuration items to access - 0x511: reading this port will return data selected in 0x510 - 0x514: this can be used to detect if DMA interface is available If fw_cfg DMA interface is available, it can be used to accelerate accesses. This patchset adds the following supports for qemu-x86 targets: + the fw_cfg driver itself + add a U-Boot command 'fw' to support direct accessing kernel informtion from fw_cfg interface, this saves the time of loading them from hard disk or network again, through emulated devices. + use fw_cfg to get cpu number at runtime, so smp boot no longer relies on the cpu node hard-coded in dts files. Changes in v5: + change 'fw' to 'qfw' + dynamically bind cpu device instead of fixing device tree blob + various cleanups Miao Yan (8): x86: adjust ramdisk load address x86: qemu: add fw_cfg support x86: qemu: add a cpu uclass driver for qemu target x86: fix a typo in function name x86: use actual CPU number for allocating memory x86: qemu: fix cpu device in smp boot x86: qemu: remove cpu node in device tree x86: qemu: add documentaion for the fw_cfg interface arch/x86/cpu/mp_init.c | 85 +++++++++++- arch/x86/cpu/qemu/Makefile | 2 +- arch/x86/cpu/qemu/cpu.c | 52 +++++++ arch/x86/cpu/qemu/fw_cfg.c | 286 +++++++++++++++++++++++++++++++++++++++ arch/x86/cpu/qemu/qemu.c | 3 + arch/x86/dts/qemu-x86_i440fx.dts | 9 +- arch/x86/dts/qemu-x86_q35.dts | 9 +- arch/x86/include/asm/fw_cfg.h | 95 +++++++++++++ doc/README.x86 | 34 ++++- include/configs/x86-common.h | 3 +- 10 files changed, 551 insertions(+), 27 deletions(-) create mode 100644 arch/x86/cpu/qemu/cpu.c create mode 100644 arch/x86/cpu/qemu/fw_cfg.c create mode 100644 arch/x86/include/asm/fw_cfg.h -- 1.9.1

3 14

[U-Boot] [PATCH] net: bootp: Ignore packets whose yiaddr is 0
by Peng Fan 07 Jan '16

07 Jan '16

When doing `dhcp`, there is a bad dhcp server in my network which always reply dhcp request with yiaddr 0, which cause uboot can not successfully get ipaddr from the good dhcp server. But the Linux PC can get the ip address even if there is a bad dhcp server. This patch is to fix that even if there is a bad dhcp server, uboot can still get ipaddr and tftp work ok. The way is to ignore the packets from the bad dhcp server by filtering out the yiaddr whose value is 0. Signed-off-by: Peng Fan <peng.fan(a)nxp.com> Cc: Joe Hershberger <joe.hershberger(a)ni.com> --- net/bootp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/net/bootp.c b/net/bootp.c index 8aeddb0..ff2ebbe 100644 --- a/net/bootp.c +++ b/net/bootp.c @@ -992,6 +992,9 @@ static void dhcp_handler(uchar *pkt, unsigned dest, struct in_addr sip, if (check_reply_packet(pkt, dest, src, len)) return; + if (net_read_ip(&bp->bp_yiaddr).s_addr == 0) + return; + debug("DHCPHandler: got DHCP packet: (src=%d, dst=%d, len=%d) state: " "%d\n", src, dest, len, dhcp_state); -- 2.6.2

2 2