June 2017 - U-Boot

Re: [U-Boot] [PATCH 2/2] mx6cuboxi: Add support for sata
by Fabio Estevam 29 Jun '17

29 Jun '17

[Adding the list on Cc] On Thu, Jun 29, 2017 at 11:33 AM, Peter Robinson <pbrobinson(a)gmail.com> wrote: > So I can just drop that check? Yes, doing like this would be enough: +#ifdef CONFIG_CMD_SATA + setup_sata(); +#endif

1 0

[U-Boot] [PATCH] GE Bx50v3 boards: fix fdt file variable
by Peter Robinson 29 Jun '17

29 Jun '17

The CONFIG_DEFAULT_FDT_FILE expects just the file not a full path as that might vary depending on the variant of Linux distro you might be using. Cc: Martin Donnelly <martin.donnelly(a)ge.com> Signed-off-by: Peter Robinson <pbrobinson(a)gmail.com> --- configs/ge_b650v3_defconfig | 2 +- configs/ge_b850v3_defconfig | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configs/ge_b650v3_defconfig b/configs/ge_b650v3_defconfig index dd44663dde..78187051eb 100644 --- a/configs/ge_b650v3_defconfig +++ b/configs/ge_b650v3_defconfig @@ -5,7 +5,7 @@ CONFIG_TARGET_GE_B650V3=y CONFIG_BOOTDELAY=1 CONFIG_SYS_CONSOLE_IS_IN_ENV=y CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y -CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b650v3.dtb" +CONFIG_DEFAULT_FDT_FILE="imx6q-b650v3.dtb" CONFIG_BOARD_EARLY_INIT_F=y CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y diff --git a/configs/ge_b850v3_defconfig b/configs/ge_b850v3_defconfig index 45e5fb1ff4..967ddba13a 100644 --- a/configs/ge_b850v3_defconfig +++ b/configs/ge_b850v3_defconfig @@ -5,7 +5,7 @@ CONFIG_TARGET_GE_B850V3=y CONFIG_BOOTDELAY=1 CONFIG_SYS_CONSOLE_IS_IN_ENV=y CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y -CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b850v3.dtb" +CONFIG_DEFAULT_FDT_FILE="imx6q-b850v3.dtb" CONFIG_BOARD_EARLY_INIT_F=y CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y -- 2.13.0

2 2

[U-Boot] [PATCH] drivers:spi:sf: Add entry for micron, MT35XU512ABA1G12, flash
by Yogesh Gaur 29 Jun '17

29 Jun '17

Add entry for MT35XU512ABA1G12 flash in spi_flash_ids[] table. This flash supports subsector erase with 4KB granularity, have support of FSR(flag status register) and flash size is 64MB. Support of this flash is required for the upcoming NXP FlexSPI controller. Signed-off-by: Yogesh Gaur <yogeshnarayan.gaur(a)nxp.com> --- drivers/mtd/spi/spi_flash_ids.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/spi/spi_flash_ids.c b/drivers/mtd/spi/spi_flash_ids.c index edca94e..f144cc3 100644 --- a/drivers/mtd/spi/spi_flash_ids.c +++ b/drivers/mtd/spi/spi_flash_ids.c @@ -133,6 +133,7 @@ const struct spi_flash_info spi_flash_ids[] = { {"n25q1024a", INFO(0x20bb21, 0x0, 64 * 1024, 2048, RD_FULL | WR_QPP | E_FSR | SECT_4K) }, {"mt25qu02g", INFO(0x20bb22, 0x0, 64 * 1024, 4096, RD_FULL | WR_QPP | E_FSR | SECT_4K) }, {"mt25ql02g", INFO(0x20ba22, 0x0, 64 * 1024, 4096, RD_FULL | WR_QPP | E_FSR | SECT_4K) }, + {"mt35xu512g", INFO6(0x2c5b1a, 0x104100, 128 * 1024, 512, E_FSR | SECT_4K) }, #endif #ifdef CONFIG_SPI_FLASH_SST /* SST */ {"sst25vf040b", INFO(0xbf258d, 0x0, 64 * 1024, 8, SECT_4K | SST_WR) }, -- 1.9.1

1 0

[U-Boot] [PATCH v2 1/2] regulator: lp87565: Fix the return value of buck_get_enable function
by Keerthy 29 Jun '17

29 Jun '17

The function wrongly returned an integer while it is supposed to return boolean. Fix that. Fixes: 2dd9dc02a3("power: regulator: lp87565: add regulator support") Reported-by:Nishanth Menon <nm(a)ti.com> Signed-off-by: Keerthy <j-keerthy(a)ti.com> --- drivers/power/regulator/lp87565_regulator.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/regulator/lp87565_regulator.c b/drivers/power/regulator/lp87565_regulator.c index 2a0b8ca..e5de578 100644 --- a/drivers/power/regulator/lp87565_regulator.c +++ b/drivers/power/regulator/lp87565_regulator.c @@ -174,7 +174,7 @@ static bool buck_get_enable(struct udevice *dev) ret = lp87565_buck_enable(dev, PMIC_OP_GET, &enable); if (ret) - return ret; + return false; return enable; } -- 1.9.1

2 7

[U-Boot] [PATCH] mmc: cadence: use fdt32_t for DT property value to fix sparse warning
by Masahiro Yamada 29 Jun '17

29 Jun '17

DTB is encoded in big endian. When we retrieve property values, we need to use fdt32_to_cpu (aka be32_to_cpu) for endian conversion. This is a bit error-prone, but sparse is useful to detect endian mismatch. We need to use (fdt32_t *) instead of (u32 *) for a pointer of a property value. Otherwise sparse warns "cast to restricted __be32". Signed-off-by: Masahiro Yamada <yamada.masahiro(a)socionext.com> --- drivers/mmc/sdhci-cadence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/sdhci-cadence.c b/drivers/mmc/sdhci-cadence.c index 4bd2623eaef9..f83c1d724108 100644 --- a/drivers/mmc/sdhci-cadence.c +++ b/drivers/mmc/sdhci-cadence.c @@ -92,7 +92,7 @@ static int sdhci_cdns_write_phy_reg(struct sdhci_cdns_plat *plat, static int sdhci_cdns_phy_init(struct sdhci_cdns_plat *plat, const void *fdt, int nodeoffset) { - const u32 *prop; + const fdt32_t *prop; int ret, i; for (i = 0; i < ARRAY_SIZE(sdhci_cdns_phy_cfgs); i++) { -- 2.7.4

2 1

[U-Boot] [PATCH] mmc: add static to spl_mmc_get_device_index()
by Masahiro Yamada 29 Jun '17

29 Jun '17

This function is only used in common/spl/spl_mmc.c[ Signed-off-by: Masahiro Yamada <yamada.masahiro(a)socionext.com> --- common/spl/spl_mmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c index 18c1b59b22cc..bb48cac1efac 100644 --- a/common/spl/spl_mmc.c +++ b/common/spl/spl_mmc.c @@ -96,7 +96,7 @@ end: return 0; } -int spl_mmc_get_device_index(u32 boot_device) +static int spl_mmc_get_device_index(u32 boot_device) { switch (boot_device) { case BOOT_DEVICE_MMC1: -- 2.7.4

3 3

[U-Boot] [PATCH 1/2] mmc: use new hwpart API when CONFIG_BLK enabled
by Kever Yang 29 Jun '17

29 Jun '17

When CONFIG_BLK is enabled, the hwpart id is different with legacy interface, update it to kame driver work with CONFIG_BLK. Signed-off-by: Kever Yang <kever.yang(a)rock-chips.com> --- cmd/mmc.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cmd/mmc.c b/cmd/mmc.c index 832eeb0..c0ff49a 100644 --- a/cmd/mmc.c +++ b/cmd/mmc.c @@ -258,7 +258,11 @@ static int do_mmcrpmb(cmd_tbl_t *cmdtp, int flag, return CMD_RET_FAILURE; } /* Switch to the RPMB partition */ +#ifndef CONFIG_BLK original_part = mmc->block_dev.hwpart; +#else + original_part = mmc_get_blk_desc(mmc)->hwpart; +#endif if (blk_select_hwpart_devnum(IF_TYPE_MMC, curr_device, MMC_PART_RPMB) != 0) return CMD_RET_FAILURE; -- 1.9.1

4 8

[U-Boot] [PATCH 00/12] regulator: Change get_enable return type to integer from bool
by Keerthy 29 Jun '17

29 Jun '17

Change get_enable return type to int so errors can be returned. The series converts the return type of get_enable hook in the dm_regulator_ops to integer from bool. This enables it to return any error values if any. Compile tested. Keerthy (12): regulator: Change get_enable return type to integer from bool power: regulator: fixed: get_enable should return integer power: regulator: act8846: get_enable should return integer power: regulator: max77686: get_enable should return integer power: regulator: palmas: get_enable should return integer power: regulator: pfuze100: get_enable should return integer power: regulator: tps65090: get_enable should return integer power: regulator: rk8xx: get_enable should return integer power: sandbox: fixed: get_enable should return integer power: regulator: s5m8767: get_enable should return integer power: regulator: lp873x: get_enable should return integer power: regulator: lp87565: get_enable should return integer drivers/power/regulator/act8846.c | 2 +- drivers/power/regulator/fixed.c | 2 +- drivers/power/regulator/lp873x_regulator.c | 4 ++-- drivers/power/regulator/lp87565_regulator.c | 2 +- drivers/power/regulator/max77686.c | 4 ++-- drivers/power/regulator/palmas_regulator.c | 4 ++-- drivers/power/regulator/pfuze100.c | 2 +- drivers/power/regulator/regulator-uclass.c | 2 +- drivers/power/regulator/rk8xx.c | 6 +++--- drivers/power/regulator/s5m8767.c | 4 ++-- drivers/power/regulator/sandbox.c | 4 ++-- drivers/power/regulator/tps65090_regulator.c | 2 +- include/power/regulator.h | 8 ++++---- 13 files changed, 23 insertions(+), 23 deletions(-) -- 1.9.1

3 27

[U-Boot] [PATCH] net: fec_mxc: fix PHY initialization bug with CONFIG_DM_ETH
by Lothar Waßmann 29 Jun '17

29 Jun '17

When CONFIG_DM_ETH is set, the FEC ethernet controller is reset after the PHY has been set up and initialzed. This breaks the communication with the PHY and results in an inoperable ethernet interface. Do the initialization with CONFIG_DM_ETH in the same order as with legacy ETH support to fix this. Signed-off-by: Lothar Waßmann <LW(a)KARO-electronics.de> --- drivers/net/fec_mxc.c | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c index 910879b..f61268c 100644 --- a/drivers/net/fec_mxc.c +++ b/drivers/net/fec_mxc.c @@ -1216,17 +1216,6 @@ static int fecmxc_probe(struct udevice *dev) if (ret) return ret; - bus = fec_get_miibus((uint32_t)priv->eth, dev_id); - if (!bus) - goto err_mii; - - priv->bus = bus; - priv->xcv_type = CONFIG_FEC_XCV_TYPE; - priv->interface = pdata->phy_interface; - ret = fec_phy_init(priv, dev); - if (ret) - goto err_phy; - /* Reset chip. */ writel(readl(&priv->eth->ecntrl) | FEC_ECNTRL_RESET, &priv->eth->ecntrl); @@ -1242,6 +1231,19 @@ static int fecmxc_probe(struct udevice *dev) fec_reg_setup(priv); priv->dev_id = (dev_id == -1) ? 0 : dev_id; + bus = fec_get_miibus(dev, dev_id); + if (!bus) { + ret = -ENOMEM; + goto err_mii; + } + + priv->bus = bus; + priv->xcv_type = CONFIG_FEC_XCV_TYPE; + priv->interface = pdata->phy_interface; + ret = fec_phy_init(priv, dev); + if (ret) + goto err_phy; + return 0; err_timeout: -- 2.1.4

3 2

[U-Boot] [PATCH] icorem6_rqs: Rename icorem6_rqs config file
by Jagan Teki 29 Jun '17

29 Jun '17

From: Jagan Teki <jagan(a)amarulasolutions.com> imx6qdl_icore_mmc_defconfig => imx6qdl_icore_rqs_defconfig Since icorem6_rqs support MMC/eMMC boot, so doesn't need to name it explicitly. Signed-off-by: Jagan Teki <jagan(a)amarulasolutions.com> --- board/engicam/icorem6_rqs/MAINTAINERS | 2 +- .../{imx6qdl_icore_rqs_mmc_defconfig => imx6qdl_icore_rqs_defconfig} | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename configs/{imx6qdl_icore_rqs_mmc_defconfig => imx6qdl_icore_rqs_defconfig} (100%) diff --git a/board/engicam/icorem6_rqs/MAINTAINERS b/board/engicam/icorem6_rqs/MAINTAINERS index 74470ba..6205acb 100644 --- a/board/engicam/icorem6_rqs/MAINTAINERS +++ b/board/engicam/icorem6_rqs/MAINTAINERS @@ -3,7 +3,7 @@ M: Jagan Teki <jagan(a)amarulasolutions.com> S: Maintained F: board/engicam/icorem6_rqs F: include/configs/imx6qdl_icore_rqs.h -F: configs/imx6qdl_icore_rqs_mmc_defconfig +F: configs/imx6qdl_icore_rqs_defconfig F: arch/arm/dts/imx6qdl-icore-rqs.dtsi F: arch/arm/dts/imx6q-icore-rqs.dts F: arch/arm/dts/imx6dl-icore-rqs.dts diff --git a/configs/imx6qdl_icore_rqs_mmc_defconfig b/configs/imx6qdl_icore_rqs_defconfig similarity index 100% rename from configs/imx6qdl_icore_rqs_mmc_defconfig rename to configs/imx6qdl_icore_rqs_defconfig -- 2.7.4

2 1