Re: [U-Boot] [PATCH 2/2] fs-test.sh: Update expected results

2016-11-05 Thread Stephen Warren
On 11/05/2016 06:32 PM, Stefan Brüns wrote: After the latest changes, ext4 no longer has any fails. Acked-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/2] ext4: Allow reading files with non-zero offset, clamp read len

2016-11-05 Thread Stephen Warren
On 11/05/2016 06:32 PM, Stefan Brüns wrote: Support was already implemented, but not hooked up. This fixes several fails in the test cases. diff --git a/fs/ext4/ext4fs.c b/fs/ext4/ext4fs.c @@ -217,21 +217,16 @@ int ext4_read_file(const char *filename, void *buf, loff_t offset, loff_t len,

Re: [U-Boot] [PATCH 2/3] test/py: Allow to pass u_boot_log instead of console for run_and_log

2016-11-05 Thread Stephen Warren
On 11/05/2016 10:45 AM, Stefan Brüns wrote: The runner actually has no console dependency, only on the log provided by the console. Accept both u_boot_console or a multiplexed_log. diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py -def run_and_log(u_boot_console, cmd,

Re: [U-Boot] [PATCH 1/3] test/py: expose config and log as session scoped fixture

2016-11-05 Thread Stephen Warren
On 11/05/2016 10:45 AM, Stefan Brüns wrote: If a test uses a fixture which is expensive to setup, the fixture can possibly created with session or module scope. As u_boot_console has function scope, it can not be used in this case. Acked-by: Stephen Warren

Re: [U-Boot] [PATCH v3 3/3] bcm2835: Reserve the spin table in efi memory map

2016-11-05 Thread Stephen Warren
On 11/02/2016 03:36 AM, Alexander Graf wrote: Firmware provides a spin table on the raspberry pi. This table shouldn't get overwritten by payloads, so we need to mark it as reserved. This is probably fine for now so, Acked-by: Stephen Warren However in the long term I

Re: [U-Boot] [PATCH v3 2/3] bcm2835 video: Map frame buffer as 32bpp

2016-11-05 Thread Stephen Warren
On 11/02/2016 03:36 AM, Alexander Graf wrote: To enable working efifb support, let's map the frame buffer as 32bpp instead of 16bpp. Hmmm. I feel like there was some good reason why I picked 16bpp when I first wrote this, especially since I had to add 16bpp support into the kernel's simplefb

Re: [U-Boot] [PATCH v3 1/3] ARM: bcm283x: Implement EFI RTS reset_system

2016-11-05 Thread Stephen Warren
On 11/02/2016 03:36 AM, Alexander Graf wrote: The rpi has a pretty simple way of resetting the whole system. All it takes is to poke a few registers at a well defined location in MMIO space. This patch adds support for the EFI loader implementation to allow an OS to reset and power off the

Re: [U-Boot] [PATCH 2/2] rpi: expose the firmware provided FDT blob in ${fw_fdt_addr}

2016-11-05 Thread Stephen Warren
On 11/02/2016 12:06 PM, Cédric Schieli wrote: If the fw_boot_param saved at an early stage points to a valid FDT blob, let's expose it in ${fw_fdt_addr}. I'd suggest naming the variable dtb_addr not fw_fdt_addr. Both names are just as easy to use from custom U-Boot scripts, however certain

Re: [U-Boot] [PATCH 1/2] rpi: save firmware provided boot param for later use

2016-11-05 Thread Stephen Warren
On 11/02/2016 12:06 PM, Cédric Schieli wrote: At U-Boot entry point, the r2 register holds the address of the firmware provided boot param. Let's save it for further processing. diff --git a/board/raspberrypi/rpi/lowlevel_init.S b/board/raspberrypi/rpi/lowlevel_init.S +.global

Re: [U-Boot] [PATCH 1/2] rpi: save firmware provided boot param for later use

2016-11-05 Thread Jonathan Liu
Hi Cédric, On 3 November 2016 at 05:06, Cédric Schieli wrote: > At U-Boot entry point, the r2 register holds the address of the > firmware provided boot param. Let's save it for further processing. > > Signed-off-by: Cédric Schieli > --- > >

[U-Boot] [PATCH 2/2] fs-test.sh: Update expected results

2016-11-05 Thread Stefan Brüns
After the latest changes, ext4 no longer has any fails. Signed-off-by: Stefan Brüns --- test/fs/fs-test.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/fs/fs-test.sh b/test/fs/fs-test.sh index 6e71b61..b194864 100755 ---

[U-Boot] [PATCH 1/2] ext4: Allow reading files with non-zero offset, clamp read len

2016-11-05 Thread Stefan Brüns
Support was already implemented, but not hooked up. This fixes several fails in the test cases. Signed-off-by: Stefan Brüns --- fs/ext4/ext4fs.c | 15 +-- include/ext4fs.h | 2 +- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git

[U-Boot] [PATCH 5/5] Makefile: preserve output for images that can contain HAB Blocks

2016-11-05 Thread Sven Ebenfeld
To being able to sign created binaries, we need to know the HAB Blocks for that image. Especially for the imximage type the HAB Blocks are only available during creation of the image. We want to preserve the information until we get to sign the files. In the verbose case we still get them printed

[U-Boot] [PATCH 3/5] tools: mkimage: add firmware-ivt image type for HAB verification

2016-11-05 Thread Sven Ebenfeld
When we want to use Secure Boot with HAB from SPL over U-Boot.img, we need to append the IVT to the image and leave space for the CSF. Images generated as firmware_ivt can directly be signed using the Freescale code signing tool. For creation of a CSF, mkimage outputs the correct HAB Blocks for

[U-Boot] [PATCH 2/5] arm: imx: add HAB authentication of image to SPL boot

2016-11-05 Thread Sven Ebenfeld
When using HAB as secure boot mechanism on Wandboard, the chain of trust breaks immediately after the SPL. As this is not checking the authenticity of the loaded image before jumping to it. The HAB status output will not be implemented in SPL as it adds a lot of strings that are only required in

[U-Boot] [PATCH 4/5] doc: imx6: add section for secure boot with SPL

2016-11-05 Thread Sven Ebenfeld
Signed-off-by: Sven Ebenfeld --- doc/README.imx6 | 48 1 file changed, 48 insertions(+) diff --git a/doc/README.imx6 b/doc/README.imx6 index 73b8b0b..add1d80 100644 --- a/doc/README.imx6 +++ b/doc/README.imx6 @@ -138,3

[U-Boot] [PATCH 0/5] arm: imx6: Enable Secure Boot (HAB) with SPL Builds

2016-11-05 Thread Sven Ebenfeld
When trying to build U-Boot for Wandboard with Secure Boot, the build fails because it tries to compile the dekblob commands within the SPL. The dekblob command depends on the CAAM driver, which is also not required in the SPL. Additionally, this blows the SPL up to a size beyond the limit of

[U-Boot] [PATCH 1/5] arm: imx: remove bmode , hdmidet and dek commands from SPL

2016-11-05 Thread Sven Ebenfeld
These files are blowing up the SPL and should not be required there as the SPL delivers no command console. Signed-off-by: Sven Ebenfeld --- arch/arm/imx-common/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm/imx-common/Makefile

[U-Boot] cubieboard sunxi u-boot help me

2016-11-05 Thread 崔超
Hi , I' from China. I am studying u-boot on Cubieboard and I have a problem. https://github.com/OSer916/u-boot.git . My code is on the "cubieboard" branch. When I do "make sunny_defconfig && make CROSS_COMPILE=arm-linux-gnueabihf-", the trouble comes out last: arm-linux-gnueabihf-ld.bfd: BFD

[U-Boot] Issues coupling am335x with VSC8601 PHY

2016-11-05 Thread Alex
Hi, I'm trying to enable ethernet on a custom board with an am335x soc, and a VSC8601 connected through RGMII. The plan is to use u-boot as a first stage loader to load a proprietary application (called "the app"). The app already has its own ethernet stack and drivers, and can successfully

[U-Boot] [PATCH] travis-ci: Try harder to build all ARM targets

2016-11-05 Thread Tom Rini
The way that we have things broken down currently allows for some combinations of vendor or CPU to not be built. To fix this, create a new catch-all job that excludes everything we've built elsewhere. For the sake of simplicity we are allowing for the possibility of some overlap between the

[U-Boot] [PATCH] api: Use hashtable function for API_env_enum

2016-11-05 Thread Emmanuel Vadot
Since the env is an hashtable, use the hashtable function for the API_ENV_ENUM api call. Signed-off-by: Emmanuel Vadot --- api/api.c | 60 +++- 1 file changed, 31 insertions(+), 29 deletions(-) diff --git

[U-Boot] [PATCH] ext4: Fix handling of sparse files

2016-11-05 Thread Stefan Brüns
A sparse file may have regions not mapped by any extents, at the start or at the end of the file, or anywhere between, thus not finding a matching extent region is never an error. Found by python filesystem tests. Signed-off-by: Stefan Brüns ---

[U-Boot] [PATCH] sunxi: mmc: Set CONFIG_SYS_MMC_MAX_DEVICE

2016-11-05 Thread Emmanuel Vadot
Set CONFIG_SYS_MMC_MAX_DEVICE to 4 for sunxi SoC. This define is needed in the API code. Signed-off-by: Emmanuel Vadot --- include/configs/sunxi-common.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h

Re: [U-Boot] [PATCH v5 1/2] spl: Convert CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR to Kconfig

2016-11-05 Thread Tom Rini
On Thu, Nov 03, 2016 at 09:47:48PM +0200, Sam Protsenko wrote: > Signed-off-by: Sam Protsenko [snip] > diff --git a/configs/Chuwi_V7_CW0825_defconfig > b/configs/Chuwi_V7_CW0825_defconfig > index a67038b..8507f73 100644 > --- a/configs/Chuwi_V7_CW0825_defconfig > +++

Re: [U-Boot] [PATCH] mx6sabresd: Make Ethernet functional again

2016-11-05 Thread Stefano Babic
On 05/11/2016 17:35, Fabio Estevam wrote: > Hi Stefano, > > On Mon, Oct 24, 2016 at 10:22 AM, Fabio Estevam wrote: >> From: Fabio Estevam >> >> Since commit ce412b79e7255770 ("drivers: net: phy: atheros: add separate >> config for AR8031") ethernet

Re: [U-Boot] [PATCH] tools: Makefile: improve cross_tools target usability

2016-11-05 Thread Masahiro Yamada
Hi. 2016-10-31 22:15 GMT+09:00 Stefan Müller-Klieser : > When building the cross_tools target, HOSTCFLAGS and HOSTLDFLAGS will > propagate to the target build. This should not happen and is easy to > prevent. > > Signed-off-by: Stefan Müller-Klieser

[U-Boot] [PATCH 1/3] test/py: expose config and log as session scoped fixture

2016-11-05 Thread Stefan Brüns
If a test uses a fixture which is expensive to setup, the fixture can possibly created with session or module scope. As u_boot_console has function scope, it can not be used in this case. Signed-off-by: Stefan Brüns --- test/py/conftest.py | 26

[U-Boot] [PATCH 2/3] test/py: Allow to pass u_boot_log instead of console for run_and_log

2016-11-05 Thread Stefan Brüns
The runner actually has no console dependency, only on the log provided by the console. Accept both u_boot_console or a multiplexed_log. Signed-off-by: Stefan Brüns --- test/py/u_boot_utils.py | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff

[U-Boot] [PATCH 3/3] test/py: Create tests for ext4 and fat testing on sandbox

2016-11-05 Thread Stefan Brüns
The following checks are currently implemented: 1. listing a directory 2. verifying size of a file 3. veryfying md5sum for a file region 4. reading the beginning of a file Signed-off-by: Stefan Brüns --- test/py/tests/test_fs.py | 298

[U-Boot] [PATCH 0/3] Create python filesystem tests

2016-11-05 Thread Stefan Brüns
This is a first attempt to create filesystem tests in python. It is currently targetting sandbox, although it should be possible to make it work with e.g. the qemu based testruns. First two patches are prep work, first patch is copied verbatim from a mail from Stephen Warren. The actual tests

Re: [U-Boot] [PATCH] mx6sabresd: Make Ethernet functional again

2016-11-05 Thread Fabio Estevam
Hi Stefano, On Mon, Oct 24, 2016 at 10:22 AM, Fabio Estevam wrote: > From: Fabio Estevam > > Since commit ce412b79e7255770 ("drivers: net: phy: atheros: add separate > config for AR8031") ethernet does not work on mx6sabresd. > > This commit correctly

Re: [U-Boot] [PATCH v3] gpio: dwapb: Add support for port B

2016-11-05 Thread Simon Glass
On 3 November 2016 at 05:05, Phil Edworthy wrote: > The IP supports two ports, A and B, each providing up to 32 gpios. > The driver already creates a 2nd gpio bank by reading the 2nd node > from DT, so this is quite a simple change to support the 2nd bank. > >

Re: [U-Boot] [PATCH] mkimage: Allow including a ramdisk in FIT auto mode

2016-11-05 Thread Simon Glass
On 4 November 2016 at 07:22, Tomeu Vizoso wrote: > Adds -i option that allows specifying a ramdisk file to be added to the > FIT image when we are using the automatic FIT mode (no ITS file). > > This makes adding Depthcharge support to LAVA much more convenient, as >

Re: [U-Boot] [RFC PATCH 02/10] Makefile: use "arm64" architecture for U-Boot image files

2016-11-05 Thread Simon Glass
On 2 November 2016 at 19:36, Andre Przywara wrote: > At the moment we use the arch/arm directory for arm64 boards as well, > so the Makefile will pick up the "arm" name for the architecture to use > for tagging binaries in U-Boot image files. > Differentiate between the

Re: [U-Boot] [PATCH v9 19/21] cmd: Add mtd command support

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > cmd/mtd.c is a generic command to access all low level > MTD devices, like SPI-NOR, Parallel NOR and NAND. > > This is implemented based on u-boot driver model, so any > new driver added for using this command must follow dm

Re: [U-Boot] [RFC PATCH 09/10] sunxi: introduce RMR switch to enter payloads in 64-bit mode

2016-11-05 Thread Simon Glass
Hi Andre, On 2 November 2016 at 19:36, Andre Przywara wrote: > The ARMv8 capable Allwinner A64 SoC comes out of reset in AArch32 mode. > To run AArch64 code, we have to trigger a warm reset via the RMR register, > which proceeds with code execution at the address stored

Re: [U-Boot] [RFC PATCH 08/10] SPL: read and store arch property from U-Boot image

2016-11-05 Thread Simon Glass
Hi Andre, On 2 November 2016 at 19:36, Andre Przywara wrote: > Read the specified "arch" value from a legacy or FIT U-Boot image and > store it in our SPL data structure. > This allows loaders to take the target architecture in account for > custom loading procedures. >

Re: [U-Boot] [PATCH v5 21/21] sf: dataflash: Minor cleanups

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > - fix single line comments > - remove unneeded spaces > - ascending order of include files > - rename SPI DATAFLASH to dataflash > - rename SPI DataFlash to dataflash > - return NULL replaced with error code > > Cc: Bin Meng

Re: [U-Boot] [PATCH v9 17/21] dm: mtd: Add uclass_driver.flags

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Add flags for mtd-uclass driver. > > Signed-off-by: Jagan Teki > --- > drivers/mtd/mtd-uclass.c | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 18/21] dm: mtd: Add post_bind

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Add .post_bind on mtd-uclass driver > > Signed-off-by: Jagan Teki > --- > drivers/mtd/mtd-uclass.c | 3 +++ > 1 file changed, 3 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH] buildman: Fix building based on 'options' field

2016-11-05 Thread Simon Glass
On 4 November 2016 at 20:59, Tom Rini wrote: > The README for buildman says that we can use any field in boards.cfg to > decide what to build. However, we were not saving the options field > correctly. > > Cc: Simon Glass > Signed-off-by: Tom Rini

Re: [U-Boot] [PATCH v9 13/21] mtd: spi-nor: Kconfig: Add MTD_M25P80 entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Add CONFIG_MTD_M25P80 kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 17 + > 1 file changed, 17 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 02/21] mtd: Add SPI-NOR core support

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 12:23, Jagan Teki wrote: > Some of the SPI device drivers at drivers/spi not a real > spi controllers, Unlike normal/generic SPI controllers they > operates only with SPI-NOR flash devices. these were technically > termed as SPI-NOR

Re: [U-Boot] [PATCH v9 12/21] mtd: spi-nor: Add m25p80 driver

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 12:23, Jagan Teki wrote: > This is MTD SPI-NOR driver for ST M25Pxx (and similar) > serial flash chips which is written as MTD_UCLASS. > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Makefile | 3 + >

Re: [U-Boot] [PATCH v9 11/21] spi: Add spi_write_then_read

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Add support for SPI synchronous write followed by read, > this is common interface call from spi-nor to spi drivers. > > Signed-off-by: Jagan Teki > --- > drivers/spi/spi-uclass.c | 24

Re: [U-Boot] [PATCH v9 06/21] mtd: spi-nor: Kconfig: Add SPI_NOR_MACRONIX entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_MACRONIX kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 5 + > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 07/21] mtd: spi-nor: Kconfig: Add SPI_NOR_SPANSION entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_SPANSION kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 5 + > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 08/21] mtd: spi-nor: Kconfig: Add SPI_NOR_STMICRO entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_STMICRO kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 5 + > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 10/21] mtd: spi-nor: Kconfig: Add SPI_NOR_WINBOND entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_WINBOND kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 5 + > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 09/21] mtd: spi-nor: Kconfig: Add SPI_NOR_SST entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_SST kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 5 + > 1 file changed, 5 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 05/21] mtd: spi-nor: Kconfig: Add SPI_NOR_MISC entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_SPI_NOR_MISC kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 6 ++ > 1 file changed, 6 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v9 03/21] mtd: spi-nor: Kconfig: Add MTD_SPI_NOR entry

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_MTD_SPI_NOR kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/Kconfig | 2 ++ > drivers/mtd/spi-nor/Kconfig | 14 ++ > 2 files changed, 16 insertions(+) >

Re: [U-Boot] [PATCH v9 04/21] mtd: spi-nor: Kconfig: Add MTD_SPI_NOR_USE_4K_SECTORS

2016-11-05 Thread Simon Glass
On 30 October 2016 at 12:23, Jagan Teki wrote: > Added CONFIG_MTD_SPI_NOR_USE_4K_SECTORS kconfig entry > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi-nor/Kconfig | 18 ++ > 1 file changed, 18 insertions(+) Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH v5 20/21] sf: dataflash: Fix add_dataflash return logic

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > This patch fixed the add_dataflash return logic, > so-that it can handle both jedec and older chips > same as Linux. > > Cc: Bin Meng > Cc: Simon Glass > Cc: York Sun >

Re: [U-Boot] [PATCH v5 18/21] sf: dataflash: Remove unneeded spi data

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > dataflash doesn't require options, memory_map from spi. > > Cc: Bin Meng > Cc: Simon Glass > Cc: York Sun > Signed-off-by: Jagan Teki >

Re: [U-Boot] [PATCH v5 19/21] sf: dataflash: Move flash id detection into jedec_probe

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Flash id detection should be the first step to enumerate > the connected flash on the board, once ie done checking > with respective id codes locally in the driver all this > should be part of jedec_probe instead of id detection

Re: [U-Boot] [PATCH v5 17/21] spi: Remove dual flash code

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > Dual flash code in spi are usually take the spi controller > to work with dual connected flash devices. Usually these > dual connection operation's are referred to flash controller > protocol rather with spi

Re: [U-Boot] [PATCH v5 15/21] sf: ids: Use small letter in ext_jedec

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Use small 'd' in s25s512s ext_jedec > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N > Cc:

Re: [U-Boot] [PATCH v5 14/21] sf: ids: Use small letter's with flash name

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > For readability use small letter's with flash name. > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N

Re: [U-Boot] [PATCH v5 16/21] sf: Rename few local functions

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > spi_flash_write_bar-> write_bar > spi_flash_write_bar -> read_bar > spi_flash_cmd_wait_ready -> spi_flash_wait_till_ready nit: how about wait_until_ready or wait_ready? 'till' is an abreviation > > Cc: Simon Glass

Re: [U-Boot] [PATCH v5 09/21] sf: Add INFO6 flash_info macro

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > INFO6 is for tabulating 6 byte flash parts, Ex: S25FS256S_64K > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N

Re: [U-Boot] [PATCH v5 12/21] sf: Remove non-meaningful comments

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N > Cc: Michal Simek >

Re: [U-Boot] [PATCH v5 13/21] sf: Rename sf_params.c to spi_flash_ids

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > spi_flash_ids.c is more meaningful name as the flash_info > table structure spi_flash_info has spi_flash_ids instance. > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun

Re: [U-Boot] [PATCH v5 10/21] sf: params: Add S25FS256S_64K spi flash support

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Add Spansion S25FS256S_64K spi flash to the list of spi_flash_ids. > > In spansion S25FS-S family the physical sectors are grouped as > normal and parameter sectors. Parameter sectors are 4kB in size > with 8 set located at the

Re: [U-Boot] [PATCH v5 11/21] sf: Remove legacy idcode detection code

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Since flash detection code is more mature to > detect even with 6 bytes id length devices > removed old code and related references. > > Cc: Yunhui Cui > Cc: Simon Glass > Cc: Bin Meng

Re: [U-Boot] [PATCH v5 08/21] sf: Increase max id length by 1 byte

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > So, now SPI_FLASH_ID_MAX_LEN is 6 bytes useful for > few spansion flash families S25FS-S > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R

Re: [U-Boot] [PATCH v5 06/21] sf: nr_sectors -> n_sectors

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Rename nr_sectors as n_sectors to sync with Linux. > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N

Re: [U-Boot] [PATCH v5 07/21] sf: Add SPI_FLASH_MAX_ID_LEN

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Add id length of 5 bytes numerical value to macro. > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R > Cc: Mugunthan V N

Re: [U-Boot] [PATCH v5 04/21] sf: Cleanup spi_flash_info{}

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > - Proper tabs spaces > - Removed unnecessary > - Added meaningful comments > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R

Re: [U-Boot] [PATCH v5 05/21] sf: Cleanup sf_params

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > - Move headers froms sf_params to common header file > - Removed unnecessary comment > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R

Re: [U-Boot] [PATCH v9 00/21] dm: Generic MTD Subsystem/SPI-NOR

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 12:23, Jagan Teki wrote: > The previous series [1] [2] are added SPI-NOR on top of > mtd/spi where it bypassing DM_SPI_FLASH and use the existing > mtd core (which is non-dm), I feel this is moving in a reverse > direction where adding new

Re: [U-Boot] [PATCH v5 03/21] sf: sandbox: Use JEDEC_MFR|ID in id exctract

2016-11-05 Thread Simon Glass
On 30 October 2016 at 11:46, Jagan Teki wrote: > Instead of extracting id's separately better > to use JEDEC_MFR|ID for code simplicity. > > Cc: Simon Glass > Cc: Bin Meng > Signed-off-by: Jagan Teki > --- >

Re: [U-Boot] [PATCH v5 02/21] sf: Simplify lock ops detection code

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 11:46, Jagan Teki wrote: > Simplify the flash_lock ops detection code and added > meaningful comment. > > Cc: Simon Glass > Cc: Bin Meng > Cc: York Sun > Cc: Vignesh R

Re: [U-Boot] [PATCH v9 01/21] dm: mtd: Add dm mtd core ops

2016-11-05 Thread Simon Glass
Hi Jagan, On 30 October 2016 at 12:23, Jagan Teki wrote: > - Add generic dm_mtd operations > - Add mtd_read|erase|write_dm > - Add add_mtd_device_dm > > The respetive MTD_UCLASS drivers must install the hooks to these > dm_mtd_ops and other core ops are act as a interface b/w

Re: [U-Boot] engicam: icorem6: Fix config files

2016-11-05 Thread Tom Rini
On Sun, Oct 30, 2016 at 01:14:31AM +0530, Jagan Teki wrote: > From: Jagan Teki > > Config file names on MAINTAINERS and README in > board/engicam/icorem6 seems to be wrong, hence fixed the same. > > Cc: Stefano Babic > Cc: Michael Trimarchi

Re: [U-Boot] [PATCH] engicam: icorem6: Fix config files

2016-11-05 Thread Tom Rini
On Sat, Nov 05, 2016 at 05:14:18PM +0530, Jagan Teki wrote: > Hi Stefano, > > On Sun, Oct 30, 2016 at 1:14 AM, Jagan Teki wrote: > > From: Jagan Teki > > > > Config file names on MAINTAINERS and README in > > board/engicam/icorem6 seems to be

Re: [U-Boot] [PATCH] engicam: icorem6: Fix config files

2016-11-05 Thread Jagan Teki
Hi Stefano, On Sun, Oct 30, 2016 at 1:14 AM, Jagan Teki wrote: > From: Jagan Teki > > Config file names on MAINTAINERS and README in > board/engicam/icorem6 seems to be wrong, hence fixed the same. > > Cc: Stefano Babic > Cc:

Re: [U-Boot] [bug report] sunxi: booting from eMMC

2016-11-05 Thread Alexandr Bochkarev
Hi, I have tried it, no success >>U-Boot SPL 2016.11-rc3-1-g9fa5bf3-dirty (Nov 05 2016 - 16:18:14) >>DRAM: 1024 MiB >>CPU: 91200Hz, AXI/AHB/APB: 3/2/2 >>Trying to boot from MMC1 >>mmc_load_image_raw_sector: mmc block read error >>SPL: failed to boot from all boot devices >>### ERROR ###

Re: [U-Boot] [PATCH RESEND v2 2/2] spi: ti_qspi: Fix baudrate divider calculation

2016-11-05 Thread R, Vignesh
[...] On 11/4/2016 4:31 PM, Jagan Teki wrote: >>> >> diff --git a/drivers/spi/ti_qspi.c b/drivers/spi/ti_qspi.c >>> >> index 52520dff6325..b5de70bf40e3 100644 >>> >> --- a/drivers/spi/ti_qspi.c >>> >> +++ b/drivers/spi/ti_qspi.c >>> >> @@ -16,6 +16,7 @@ >>> >> #include >>> >> #include >>> >>

[U-Boot] [PATCH v3 1/2] ARM: dts: dra7xx: Update spi-max-frequency for qspi slave node

2016-11-05 Thread Vignesh R
Update the spi-max-frequency property of m25p80 flash slave to match that of TI QSPI controller node, so that QSPI operations happen at maximum supported frequency of 76.8MHz. Signed-off-by: Vignesh R Reviewed-by: Jagan Teki --- v3: No changes.

[U-Boot] [PATCH v3 2/2] spi: ti_qspi: Fix baudrate divider calculation

2016-11-05 Thread Vignesh R
Fix the divider calculation logic to choose a value so that the resulting baudrate is either equal to or closest possible baudrate less than the requested value. While at that, cleanup ti_spi_set_speed(). Signed-off-by: Vignesh R --- v3: Move SCLK disable and enable blocks

Re: [U-Boot] [bug report] sunxi: booting from eMMC

2016-11-05 Thread Hans de Goede
Hi, On 04-11-16 23:57, Alexandr Bochkarev wrote: I have found problem place, this patch with hotfix allows to boot Allwinner A20 from eMMC. Dont know how to full fix it. Maxime posted a proper fix for this issue yesterday: https://patchwork.ozlabs.org/patch/691284/ Can you give that one a