Re: [U-Boot] [U-Boot, U-BOOT, TEST, 1/5] Cosmetic api: api_storage.c Align parenthesis

2016-12-04 Thread Tom Rini
On Wed, Nov 23, 2016 at 01:26:10AM -0500, Walt Feasel wrote: > Make checkpatch style modification for > CHECK: Alignment should match open parenthesis > > Signed-off-by: Walt Feasel Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature

Re: [U-Boot] defconfig: am43xx_evm: Enable DM_SPI and DM_SPI_FLASH

2016-12-04 Thread Tom Rini
On Tue, Nov 22, 2016 at 02:42:56PM +0530, Vignesh R wrote: > Commit 4c4e3b37750f3("ARM: AM43xx: Enable FIT") accidentally disabled > DM_SPI and DM_SPI_FLASH. Add back DM_SPI and DM_SPI_FLASH to > am43xx_evm_defconfig in order to make use of DM framework for QSPI. > > Signed-off-by: Vignesh R

Re: [U-Boot] common: image: Remove FIT header update from image post-processing

2016-12-04 Thread Tom Rini
On Mon, Nov 21, 2016 at 02:37:09PM -0600, Andrew F. Davis wrote: > After an image is selected out of a FIT blob for further processing we > run an optional, platform specific, post-processing function on this > component. This post-processing may modify the position and size of the > image, so

Re: [U-Boot] [U-Boot, 12/12] usb: gadget: remove unused shortname variable

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:16AM +, Andre Przywara wrote: > The shortname variable isn't referenced anywhere in the code, so just > remove it. > > Pointed out by a GCC 6.2 default warning option. > > Signed-off-by: Andre Przywara > Reviewed-by: Marek Vasut

Re: [U-Boot] [U-Boot,08/12] davinci: da8xxevm: fix indentation

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:12AM +, Andre Przywara wrote: > Apparently the indentation is wrong in this case, as the second message > should be printed indepdently of the if statement. > > Fix this indentation to avoid both compiler warnings and puzzled readers. > > Pointed out by GCC

Re: [U-Boot] [U-Boot, 12/12] usb: gadget: remove unused shortname variable

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:16AM +, Andre Przywara wrote: > The shortname variable isn't referenced anywhere in the code, so just > remove it. > > Pointed out by a GCC 6.2 default warning option. > > Signed-off-by: Andre Przywara > Reviewed-by: Marek Vasut

Re: [U-Boot] [U-Boot,06/12] marvell: comphy_a3700: fix bitmask

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:10AM +, Andre Przywara wrote: > Obviously the mask for the rx and tx select field cannot be right, > as it would overlap in one and exceed the 32-bit register in the other > case. From looking at the neighbouring bits it looks like the mask > should be really 4

Re: [U-Boot] [U-Boot,07/12] usb: eth: r8152_fw: fix indentation

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:11AM +, Andre Przywara wrote: > Apparently the indentation is wrong here, fix this to avoid compiler > warnings and puzzled readers. > > Pointed out by GCC 6.2's -Wmisleading-indentation warning. > > Signed-off-by: Andre Przywara >

Re: [U-Boot] [U-Boot, 05/12] net: rtl8169: remove unneeded definition

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:09AM +, Andre Przywara wrote: > The rtl8169_intr_mask variable isn't used anywhere in the code, so > just remove it to avoid a GCC 6.2 compiler warning. > > Signed-off-by: Andre Przywara > Acked-by: Joe Hershberger

Re: [U-Boot] [U-Boot,03/12] net: e1000: fix indentation

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:07AM +, Andre Przywara wrote: > Apparently the indentation is off here, for the IGB model just want to > bail out early. > Fix this to avoid both compiler warnings and puzzled readers. > > Pointed out by GCC 6.2's -Wmisleading-indentation warning. > >

Re: [U-Boot] [U-Boot,02/12] mtd: cfi_flash: fix indentation

2016-12-04 Thread Tom Rini
On Wed, Nov 16, 2016 at 12:50:06AM +, Andre Przywara wrote: > The indentation is misleading here and suggests that the write command > will be only executed in the else clause. > It seems like this is not intended, so fix the indentation to avoid > both compiler warnings and puzzled readers.

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

2016-12-04 Thread Tom Rini
On Sun, Dec 04, 2016 at 05:02:19PM +, york sun wrote: > Tom, > > The following changes since commit 6b29a395b62965eef6b5065d3a526a8588a92038: > >Merge git://git.denx.de/u-boot-mpc85xx (2016-11-29 19:42:48 -0500) > > are available in the git repository at: > >

Re: [U-Boot] Please pull u-boot-dm (take 2)

2016-12-04 Thread Tom Rini
On Sat, Dec 03, 2016 at 11:26:13AM -0700, Simon Glass wrote: > Hi Tom, > > (new version with more patches) > > Here are some buildman enhancements, a new TPM driver, the beginnings > of the USB gadget DM conversion along with a few fixes and improvements. > > The following changes since commit

[U-Boot] [PATCH] omap4_sdp4430: Disable SPL_OS_BOOT

2016-12-04 Thread Tom Rini
We are tight on space on this board so drop SPL_OS_BOOT Signed-off-by: Tom Rini --- configs/omap4_sdp4430_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/configs/omap4_sdp4430_defconfig b/configs/omap4_sdp4430_defconfig index 57dcf32e224a..de339ca2c9c3 100644

Re: [U-Boot] [PATCH 1/1] rpi: set serial port clock to 48 MHz

2016-12-04 Thread Mirza Krak
2016-12-01 10:34 GMT+01:00 Mirza Krak : > 2016-11-30 19:07 GMT+01:00 Stephen Warren : >> On 11/29/2016 11:47 PM, Mirza Krak wrote: >>> >>> 2016-11-30 5:05 GMT+01:00 Stephen Warren : On 11/27/2016 11:05 PM, Mirza Krak

[U-Boot] check_config.sh produces false positive error?

2016-12-04 Thread Mirza Krak
Hi. Recently a check was added to if new ad-hoc CONFIG options where added and produced an error. The exact commit is [1]. While building (2016.11) with some out of tree files in include folder I get the following error: Build error: | Error: You must add new CONFIG options using Kconfig | The

[U-Boot] Please pull u-boot-mpc85xx master

2016-12-04 Thread york sun
Tom, The following changes since commit 6b29a395b62965eef6b5065d3a526a8588a92038: Merge git://git.denx.de/u-boot-mpc85xx (2016-11-29 19:42:48 -0500) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx.git for you to fetch changes up to

[U-Boot] [PATCH 2/3] arm64: mvebu: Enable PCIe support in Armada-7040 configuration

2016-12-04 Thread kostap
From: Konstantin Porotchkin Enable PCIe bus support in Armada-7040 DB default configuration Signed-off-by: Konstantin Porotchkin Cc: Stefan Roese Cc: Nadav Haklai Cc: Neta Zur Hershkovits Cc: Omri

[U-Boot] [PATCH 1/3] arm64: mvebu: Add L3 cache flush functionality to A8K family

2016-12-04 Thread kostap
From: Konstantin Porotchkin Add missing L3 cache flush functionality which absence prevents Linux kernel from normal boot in case the L3 cache is enabled by ATF. The L3 cache is named the "last level" cache in order to keep the terminology similar to the ATF code. This cache

[U-Boot] [PATCH 3/3] arm64: mvebu: Enable hush parser in A8K default configuration

2016-12-04 Thread kostap
From: Konstantin Porotchkin Enable hush parser in Armada-7040 and Armada-8040 DB default configurations. Signed-off-by: Konstantin Porotchkin Cc: Stefan Roese Cc: Nadav Haklai Cc: Neta Zur Hershkovits

[U-Boot] [PATCH 0/3] arm64: mvebu: Linux kernel boot support

2016-12-04 Thread kostap
From: Konstantin Porotchkin This set of patches adds features for allowing Linux kernel boot and root on NFS for Marvell A8K family boards. These changes were tested when applied on top of previously sent series of patches named: "arm64: mvebu: Armada-8K family patches"

[U-Boot] [PATCH v2 3/6] arm64: mvebu: pinctrl: Add pin control driver for A8K family

2016-12-04 Thread kostap
From: Konstantin Porotchkin Add a DM port of Marvell pin control driver. The A8K SoC family contains several silicone dies interconnected in a single package. Every die is normally equipped with its own pin controller unit. There are 2 pin controllers in A70x0 SoC and 3 in

[U-Boot] [PATCH v2 6/6] arm64: mvebu: Enable pin control support in A8K default config

2016-12-04 Thread kostap
From: Konstantin Porotchkin Enable mvebu pin control support in the default configuration files for Armada-7040 and Armada-8040 development boards Changes for v2: - Remove Gerrit Change ID Signed-off-by: Konstantin Porotchkin Cc: Stefan Roese

[U-Boot] [PATCH v2 4/6] arm64: mvebu: Add pin control nodes to A8K family DTS files

2016-12-04 Thread kostap
From: Konstantin Porotchkin Add pin control nodes to APN806, CP-master, CP-slave and Armada-7040 and Armada-8040 boards DTS files Changes for v2: - Remove Gerrit Change ID - Fix comments - Fix wrong function values - Add missing cpm_i2c0 node to armada-7040-db.dts

[U-Boot] [PATCH v2 5/6] arm64: mvebu: Enable BUBT command support in A8K default config

2016-12-04 Thread kostap
From: Konstantin Porotchkin Enable mvebu "bubt" command support in the default configuration file for Armada-7040 and Armada-8040 development boards Changes for v2: - Remove Gerrit Change ID Signed-off-by: Konstantin Porotchkin Cc: Stefan Roese

[U-Boot] [PATCH v2 2/6] arm64: mvebu: Add bubt command for flash image burn

2016-12-04 Thread kostap
From: Konstantin Porotchkin Add support for mvebu bubt command for flash image load, check and burn on boot device. Changes for v2: - Add "bubt" documentation - Fix code syntax Signed-off-by: Konstantin Porotchkin Cc: Stefan Roese Cc:

[U-Boot] [PATCH v2 0/6] arm64: mvebu: Armada-8K family patches

2016-12-04 Thread kostap
From: Konstantin Porotchkin This set of patches is adding more features for bards based on new Marvell MoChi platforms - Armada-70x0 and Armada-80x0. The patches were applied on top of Stefan's mvebu-related patch set v2 with the last patch named "dm: core: Add

[U-Boot] [PATCH v2 1/6] arm64: mvebu: Modify the A8K SPI and I2C config in DTS

2016-12-04 Thread kostap
From: Konstantin Porotchkin Align the Armada-8040-db and Armada-7040-db SPI and I2C DTS settings with latest DB settings: - 8040-db: disable i2c0 and spi0 on AP (MPPs are reserved for SDIO) - 8040-db: disable cps_i2c0 on CP1 - 8040-db: enable spi1 on CP1 (the new location of

Re: [U-Boot] [PATCH 1/5] arm: am57xx: cl-som-am57x: add initial board support

2016-12-04 Thread Tom Rini
On Sun, Dec 04, 2016 at 09:58:42AM +0200, Uri Mashiach wrote: > Hello Tom, > > On 12/01/2016 05:36 PM, Tom Rini wrote: > > On Thu, Dec 01, 2016 at 10:28:12AM +0200, Uri Mashiach wrote: > > > >> From: Dmitry Lifshitz > >> > >> Features supported : > >> > >> * Serial

[U-Boot] [PATCH 4/6] mmc: move CONFIG_SDHCI to Kconfig, renaming to CONFIG_MMC_SDHCI

2016-12-04 Thread Masahiro Yamada
Move CONFIG_SDHCI to Kconfig and rename it to CONFIG_MMC_SDHCI. My motivation for the rename is, ultimately, to make all the MMC options prefixed with MMC_ and SDHCI options with MMC_SDHCI_, like Linux. This commit was created as follows: [1] Rename the config option with the following command:

[U-Boot] [PATCH 5/6] mmc: move some SDHCI related options to Kconfig

2016-12-04 Thread Masahiro Yamada
While I moved the options, I also renamed them so that they are all prefixed with MMC_SDHCI_. This commit was created in the following steps. [1] Rename with the following command find . -name .git -prune -o ! -path ./scripts/config_whitelist.txt \ -type f -print | xargs sed -i -e '

[U-Boot] [PATCH 0/6] mmc: move some config options to Kconfig

2016-12-04 Thread Masahiro Yamada
I wrote a new SDHCI driver for my new SoCs, but before posting it, I just want to make a clean-base for my driver entry. Of course, I could enable some needed options in my header file (for options in the "white-list"), but I just thought it is a good habit to contribute to Kconfig moves in the

[U-Boot] [PATCH 1/6] Sync defconfig files by savedefconfig

2016-12-04 Thread Masahiro Yamada
Generated by "tools/moveconfig -s". This will make config moves easier. Signed-off-by: Masahiro Yamada --- configs/A10-OLinuXino-Lime_defconfig | 2 +- configs/A20-OLinuXino-Lime2_defconfig | 2 +-

[U-Boot] [PATCH 6/6] mmc: move MMC_SDHCI_IO_ACCESSORS to Kconfig

2016-12-04 Thread Masahiro Yamada
This is an user-unconfigurable option that is selected by the drivers that need to overwrite SDHCI IO memory accessors. (BCM2835 SDHCI seems the only driver that needs to do so.) Signed-off-by: Masahiro Yamada --- drivers/mmc/Kconfig | 8

[U-Boot] [PATCH 3/6] mmc: make MMC driver entries dependent on MMC

2016-12-04 Thread Masahiro Yamada
Currently, CONFIG_MMC is not related to any other options by "depends on" or "select". One of big advantages of using Kconfig is automatic dependency tracking, but the current state is lacking it. As the first step, make the existing MMC driver entries depend on MMC. This commit was created by

<    1   2