Re: [U-Boot] ** Can't read Driver Desriptor Block **

2017-09-03 Thread Heinrich Schuchardt
On 09/04/2017 05:59 AM, Bin Meng wrote: > Hi Heinrich, > > On Fri, Sep 1, 2017 at 1:24 AM, Heinrich Schuchardt > wrote: >> On 08/31/2017 02:55 PM, Bin Meng wrote: >>> Hi Simon, >>> >>> On Thu, Aug 31, 2017 at 8:52 PM, Simon Glass wrote: Hi Bin,

[U-Boot] [next PATCH v2 1/2] ARM: mvebu: Add SoC IDs for Marvell's integrated CPUs

2017-09-03 Thread Chris Packham
These SoCs are network packet processors (switch chips) with integrated ARMv7 cores. They share a great deal of commonality with the Armada-XP CPUs. Signed-off-by: Chris Packham --- There are actually a number of IDs for these chips, probably a dozen in total. I haven't

[U-Boot] [next PATCH v2 2/2] ARM: mvebu: add additional information to board_add_ram_info()

2017-09-03 Thread Chris Packham
From: Joshua Scott Display more information about the current RAM configuration. With these changes the output on a 88F6820 board is SoC: MV88F6820-A0 at 1600 MHz DRAM: 2 GiB (800 MHz, 32-bit, ECC not enabled) Signed-off-by: Joshua Scott

Re: [U-Boot] [PATCH v3 1/1] usb: gadget: g_dnl: Sync internal SN variable with env

2017-09-03 Thread Heiko Schocher
Hello Lukasz, Am 02.09.2017 um 13:08 schrieb Łukasz Majewski: Hi Heiko, Would you find some time and run this patch through your test setup? Thanks in advance. Of course, as it is automated ;-) Okay, I had to add an oneliner, to say tbot, which patchwork patch it has to download... for the

Re: [U-Boot] [PATCH 01/19] configs: Add FPGA loadfs config for Arria 10

2017-09-03 Thread Chee, Tien Fong
On Rab, 2017-08-30 at 10:45 +0200, Marek Vasut wrote: > On 08/30/2017 07:59 AM, Chee, Tien Fong wrote: > > > > On Sel, 2017-08-29 at 13:51 +0200, Marek Vasut wrote: > > > > > > On 08/29/2017 12:45 PM, tien.fong.c...@intel.com wrote: > > > > > > > > > > > > From: Tien Fong Chee

[U-Boot] [PATCH 3/4] block: ide: Don't bother to create BLK device if no CDROM inserted

2017-09-03 Thread Bin Meng
When there is no CDROM inserted, the block size is zero hence there is no need to create a BLK device for it. Signed-off-by: Bin Meng --- drivers/block/ide.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/block/ide.c b/drivers/block/ide.c index

[U-Boot] [PATCH 2/4] block: ide: Fix block read/write with driver model

2017-09-03 Thread Bin Meng
This converts the IDE driver to driver model so that block read and write are fully functional. Fixes: b7c6baef ("x86: Convert MMC to driver model") Reported-by: Heinrich Schuchardt Signed-off-by: Bin Meng --- drivers/block/ide.c| 68

Re: [U-Boot] ** Can't read Driver Desriptor Block **

2017-09-03 Thread Bin Meng
Hi Heinrich, On Fri, Sep 1, 2017 at 1:24 AM, Heinrich Schuchardt wrote: > On 08/31/2017 02:55 PM, Bin Meng wrote: >> Hi Simon, >> >> On Thu, Aug 31, 2017 at 8:52 PM, Simon Glass wrote: >>> Hi Bin, >>> >>> On 31 August 2017 at 10:53, Bin Meng

[U-Boot] [PATCH 4/4] cmd: ide: Make the first device the default one

2017-09-03 Thread Bin Meng
At present the IDE device number is initialized to -1, which means we cannot type "ide read" command before setting the device number via "ide device #". For convenience, let's set the first device as the default one. Signed-off-by: Bin Meng --- cmd/ide.c | 2 +- 1 file

[U-Boot] [PATCH 1/4] blk: Use macros for block device vendor/product/rev string size

2017-09-03 Thread Bin Meng
So far these are using magic numbers. Replace them with macros. Signed-off-by: Bin Meng --- include/blk.h | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/include/blk.h b/include/blk.h index a106f9c..fced138 100644 --- a/include/blk.h +++

[U-Boot] [PATCH 3/3] armv8: ls1088a: Enable PCIe in defconfigs

2017-09-03 Thread Zhiqiang Hou
From: Hou Zhiqiang Enabled PCIe support and PCI command feature. Signed-off-by: Hou Zhiqiang --- configs/ls1088aqds_qspi_defconfig| 4 configs/ls1088aqds_sdcard_qspi_defconfig | 4 configs/ls1088ardb_qspi_defconfig| 4

[U-Boot] [PATCH 1/3] armv8: ls1088a: fix the MMU table for pcie config space

2017-09-03 Thread Zhiqiang Hou
From: Hou Zhiqiang The pcie config space of ls1088a is different from ls2080a. Signed-off-by: Hou Zhiqiang --- arch/arm/include/asm/arch-fsl-layerscape/immap_lsch3.h | 6 ++ 1 file changed, 6 insertions(+) diff --git

[U-Boot] [PATCH 2/3] armv8: ls1088a: add PCIe dts node

2017-09-03 Thread Zhiqiang Hou
From: Hou Zhiqiang Signed-off-by: Hou Zhiqiang --- arch/arm/dts/fsl-ls1088a.dtsi | 48 +++ 1 file changed, 48 insertions(+) diff --git a/arch/arm/dts/fsl-ls1088a.dtsi b/arch/arm/dts/fsl-ls1088a.dtsi index

[U-Boot] [PATCH 0/3] armv8: ls1088a: add pcie support

2017-09-03 Thread Zhiqiang Hou
From: Hou Zhiqiang This patch set depends on ls1088a platform support patchs: http://patchwork.ozlabs.org/patch/800408/ http://patchwork.ozlabs.org/patch/800410/ http://patchwork.ozlabs.org/patch/800409/ http://patchwork.ozlabs.org/patch/803037/

[U-Boot] [PATCH 2/2] tools: .gitignore: Add libfdt related files

2017-09-03 Thread Bin Meng
Some files are generated during libfdt build. Ignore them. Signed-off-by: Bin Meng --- tools/.gitignore | 4 1 file changed, 4 insertions(+) diff --git a/tools/.gitignore b/tools/.gitignore index 6a487d2..5293d44 100644 --- a/tools/.gitignore +++ b/tools/.gitignore @@

[U-Boot] [PATCH 1/2] tools: .gitignore: Sort in alphabetical order

2017-09-03 Thread Bin Meng
These are currently out of alphabetical order. Signed-off-by: Bin Meng --- tools/.gitignore | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/tools/.gitignore b/tools/.gitignore index ac0c979..6a487d2 100644 --- a/tools/.gitignore +++

[U-Boot] [PATCH v2] part: mac: Suppress the error message after reading ddb

2017-09-03 Thread Bin Meng
Change to use 'debug' to output the error message if it fails to read the driver descriptor block. Signed-off-by: Bin Meng --- Changes in v2: - rebase on top of u-boot/master disk/part_mac.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git

Re: [U-Boot] [U-Boot, 2/2] Makefile: Suppress output of python libfdt build command

2017-09-03 Thread Tom Rini
On Sun, Sep 03, 2017 at 04:37:49AM -0700, Bin Meng wrote: > This should not be printed by default. Prefix it with $(Q). > > Fixes ee95d10b: ("fdt: Build the new python libfdt module") > Signed-off-by: Bin Meng Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, 3/3] nvme: Remove dead codes in nvme_setup_io_queues()

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:15:37AM -0700, Bin Meng wrote: > Execution cannot reach this statement: "nr_io_queues = result;" > > Reported-by: Coverity (CID: 166731) > Signed-off-by: Bin Meng > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! --

Re: [U-Boot] [U-Boot,3/3] block: Drop the ftide020 driver

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:43:54AM -0700, Bin Meng wrote: > This is not used in U-Boot. > > Signed-off-by: Bin Meng Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list

Re: [U-Boot] [U-Boot,1/2] Makefile: Quiesce libfdt build

2017-09-03 Thread Tom Rini
On Sun, Sep 03, 2017 at 04:37:48AM -0700, Bin Meng wrote: > Since commit 3809e302 "Makefile: honor PYTHON configuration properly", > the build commands of libfdt are printed while previously were not. > > This adds the missing '--quiet' back. > > Signed-off-by: Bin Meng

Re: [U-Boot] [U-Boot,1/1] disk: part: fix typo

2017-09-03 Thread Tom Rini
On Tue, Aug 29, 2017 at 06:36:59PM +0200, Heinrich Schuchardt wrote: > %s/Desriptor/Descriptor/g > > Fix lines over 80 characters with said typo. > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital

Re: [U-Boot] bch: don't use __BSD_VISIBLE to test for fls

2017-09-03 Thread Tom Rini
On Sun, Aug 27, 2017 at 08:45:14PM +1000, Jonathan Gray wrote: > Commit 4ecc988301bc8e981e6d7538c57cdb3aa82f7c1d assumes fls is in libc > if __BSD_VISIBLE is defined. This appears to only be true on FreeBSD > and DragonFlyBSD. OpenBSD defines __BSD_VISIBLE and does not have fls > in

Re: [U-Boot] [U-Boot, 2/3] nvme: Fix potential sign extension issue in nvme_blk_rw()

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:15:36AM -0700, Bin Meng wrote: > "lbas" with type "u16" (16 bits, unsigned) is promoted in > "lbas << ns->lba_shift" to type "int" (32 bits, signed), then > sign-extended to type "unsigned long long" (64 bits, unsigned). > If "lbas << ns->lba_shift" is greater than

Re: [U-Boot] fix: fw_env: Prevent writing error message on special files, which don't support fsync

2017-09-03 Thread Tom Rini
On Sun, Aug 27, 2017 at 01:46:22PM +0200, Lukasz Majewski wrote: > According to fsync specification [1] some special files (e.g., a pipe, FIFO, > or socket) don't support synchronization and return either EROFS or EINVAL. > > On the linux side the sys_fsync -> do_fsync() checks if the requested

Re: [U-Boot] [U-Boot, 3/8] include/config_fallbacks.h: change fallback for CONFIG_SYS_PBSIZE

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:01PM +0200, Thomas Petazzoni wrote: > Most of the platforms are using CONFIG_SYS_CBSIZE + > sizeof(CONFIG_SYS_PROMPT) + 16 as their value for CONFIG_SYS_PBSIZE, > so let's adopt this for the fallback value of CONFIG_SYS_PBSIZE. > > This will allow us to drop an

Re: [U-Boot] [U-Boot, 8/8] include/configs: remove numerous CONFIG_SYS_BARGSIZE definitions

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:06PM +0200, Thomas Petazzoni wrote: > This commit removes definitions of CONFIG_SYS_BARGSIZE defined to be > equal to CONFIG_SYS_CBSIZE in numerous configuration files. > > We remove such definitions in two situations: > Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] [U-Boot, 4/8] include/configs: drop default definitions of CONFIG_SYS_PBSIZE

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:02PM +0200, Thomas Petazzoni wrote: > Now that the fallback value of CONFIG_SYS_PBSIZE in > include/config_fallbacks.h has been adjusted, remove its definition > from a large number of board configuration files. > > Signed-off-by: Thomas Petazzoni

Re: [U-Boot] [U-Boot,2/3] block: ide: Drop CONFIG_IDE_LED

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:43:53AM -0700, Bin Meng wrote: > This is actually not used. Drop it. > > Signed-off-by: Bin Meng Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot

Re: [U-Boot] [U-Boot, 1/8] include/config_fallbacks.h: add default for CONFIG_SYS_CBSIZE

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:58:59PM +0200, Thomas Petazzoni wrote: > CONFIG_SYS_CBSIZE contains the buffer size for input for the > console. The vast majority of platforms define them to some reasonable > value (256, 512 or 1024 bytes), and it is quite annoying to repeat > this definition for all

Re: [U-Boot] [U-Boot, 1/3] block: ide: Drop CONFIG_IDE_INIT_POSTRESET

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:43:52AM -0700, Bin Meng wrote: > This is not referenced anywhere. Drop it. > > Signed-off-by: Bin Meng Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot

Re: [U-Boot] [U-Boot, v3] omap3: evm: Fixes for CONFIG_NAND, SPL_OS_BOOT, USB, and environment

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 05:43:05PM -0500, Derald D. Woods wrote: Applied to u-boot/master, thanks! -- Tom signature.asc Description: Digital signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [U-Boot, 5/8] include/config_fallbacks.h: add default for CONFIG_SYS_MAXARGS

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:03PM +0200, Thomas Petazzoni wrote: > CONFIG_SYS_MAXARGS contains the maximum number of arguments accepted > by U-Boot commands. Since the vast majority of the platforms define it > to 16, it makes sense to have a default definition to 16, which will > allow to

Re: [U-Boot] [U-Boot, 2/8] include/configs: remove CONFIG_SYS_CBSIZE when the default value is used

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:00PM +0200, Thomas Petazzoni wrote: > Now that include/config_fallbacks.h define a sane fallback for > CONFIG_SYS_CBSIZE, we can drop the definition of this constant in all > configurations that were using the default value. > > Signed-off-by: Thomas Petazzoni

Re: [U-Boot] [U-Boot,4/4] part: efi: Disable overlap check

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 04:01:33PM +0200, Maxime Ripard wrote: > The current code checks that no partitions overlap with the GPT partition > table using the offset of the first LBA usable for that partition. > > This works fine, unless you have a partition entry that is further away > than it

Re: [U-Boot] [U-Boot, 6/8] include/configs: drop default definitions of CONFIG_SYS_MAXARGS

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:04PM +0200, Thomas Petazzoni wrote: > Now that include/config_fallbacks.h define a sane fallback for > CONFIG_SYS_MAXARGS, we can drop the definition of this constant in all > configurations that were using the default value. > > Signed-off-by: Thomas Petazzoni

Re: [U-Boot] [U-Boot,1/3] nvme: Fix wrong ndev->queues memset

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 08:15:35AM -0700, Bin Meng wrote: > memset() was given a sizeof(NVME_Q_NUM * sizeof(struct nvme_queue *) > to clear, which is wrong. > > Reported-by: Coverity (CID: 166729) > Signed-off-by: Bin Meng > Reviewed-by: Tom Rini

Re: [U-Boot] [U-Boot, 7/8] include/configs: remove default values of CONFIG_SYS_BARGSIZE

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 10:59:05PM +0200, Thomas Petazzoni wrote: > CONFIG_SYS_BARGSIZE is already defined to 512 in common/image.c when > not defined. Therefore, there is no point in having board > configuration files define it to 512. > > Signed-off-by: Thomas Petazzoni

Re: [U-Boot] [U-Boot, 2/4] part: efi: rework the partition start and size in gpt_fill_pte

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 04:01:31PM +0200, Maxime Ripard wrote: > The start variable is only used inside a loop, and is never affected inside > it, so it's a purely local variable. > > In the same way the partition size is accessed several times, so we can > store it in a variable. > >

Re: [U-Boot] [U-Boot, 3/4] part: efi: make gpt_fill_pte take the device descriptor

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 04:01:32PM +0200, Maxime Ripard wrote: > The gpt_fill_pte will need to access the device block size. Let's pass the > device descriptor as an argument. > > Signed-off-by: Maxime Ripard > Reviewed-by: Tom Rini

Re: [U-Boot] [U-Boot,1/4] part: efi: Fix offset

2017-09-03 Thread Tom Rini
On Wed, Aug 23, 2017 at 04:01:30PM +0200, Maxime Ripard wrote: > Both the config option and the DT options specify the offset to set the GPT > at in bytes, yet the code treats those values as block numbers. > > Fix that. > > Signed-off-by: Maxime Ripard >

Re: [U-Boot] [PATCH 1/9] lib: Add CRC32-C

2017-09-03 Thread Marek Behun
Hi Łukasz, the Castagnoli CRC32 algorithm just uses different polynomial. Perhaps I should generalize the original CRC32. Marek On Sun, 3 Sep 2017 17:47:28 +0200 Łukasz Majewski wrote: > Hi Marek, > > > This is needed for BTRFS. > > > > Signed-off-by: Marek Behun

Re: [U-Boot] [PATCH] crypto/fsl: fix obj-yy in Makefile

2017-09-03 Thread Fabio Estevam
On Sun, Sep 3, 2017 at 2:17 PM, Clemens Gruber wrote: > When enabling CONFIG_CMD_BLOB and/or CONFIG_CMD_DEKBLOB, the build fails > with a linker error: > ... > LD u-boot > arch/arm/mach-imx/built-in.o: In function `blob_encap_dek': >

[U-Boot] [PATCH] crypto/fsl: fix obj-yy in Makefile

2017-09-03 Thread Clemens Gruber
When enabling CONFIG_CMD_BLOB and/or CONFIG_CMD_DEKBLOB, the build fails with a linker error: ... LD u-boot arch/arm/mach-imx/built-in.o: In function `blob_encap_dek': /home/clemens/dev/u-boot/arch/arm/mach-imx/cmd_dek.c:46: undefined reference to `blob_dek' This is due to an error in

Re: [U-Boot] [PATCH 1/1] scripts/Makefile.lib: remove overridden target $(obj)/helloworld.so:

2017-09-03 Thread Heinrich Schuchardt
On 09/03/2017 02:19 PM, Alexander Graf wrote: > > > On 03.09.17 08:17, Heinrich Schuchardt wrote: >> The target >> $(obj)/helloworld.so: >> exists twice in Makefile.lib. >> >> If you add an echo command to each of the two recipes you get >> warnings like: >> >> scripts/Makefile.lib:383: warning:

Re: [U-Boot] [PATCH v2 1/8] fs/fat: split out helper to init fsdata

2017-09-03 Thread Łukasz Majewski
On 09/03/2017 05:47 PM, Rob Clark wrote: On Sun, Sep 3, 2017 at 10:52 AM, Łukasz Majewski wrote: On 09/02/2017 06:37 PM, Rob Clark wrote: Want to re-use this in fat dirent iterator in next patch. Signed-off-by: Rob Clark Reviewed-by: Łukasz Majewski

[U-Boot] [PATCH] git: mailrc: Update e-mail address

2017-09-03 Thread Lukasz Majewski
Signed-off-by: Lukasz Majewski --- doc/git-mailrc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/git-mailrc b/doc/git-mailrc index af10a3d..556db0a 100644 --- a/doc/git-mailrc +++ b/doc/git-mailrc @@ -31,7 +31,7 @@ alias jhersh Joe Hershberger

Re: [U-Boot] [PATCH 1/9] lib: Add CRC32-C

2017-09-03 Thread Łukasz Majewski
Hi Marek, This is needed for BTRFS. Signed-off-by: Marek Behun create mode 100644 lib/crc32c.c Excuse me my ignorance, but in u-boot we already have: ./lib/crc32.c is the crc32c algorithm a different one from venerable crc32? diff --git a/include/u-boot/crc.h

Re: [U-Boot] [PATCH v2 1/8] fs/fat: split out helper to init fsdata

2017-09-03 Thread Rob Clark
On Sun, Sep 3, 2017 at 10:52 AM, Łukasz Majewski wrote: > On 09/02/2017 06:37 PM, Rob Clark wrote: >> >> Want to re-use this in fat dirent iterator in next patch. >> >> Signed-off-by: Rob Clark > > > Reviewed-by: Łukasz Majewski > thanks btw,

Re: [U-Boot] [PATCH 1/1] scripts/Makefile.lib: remove overridden target $(obj)/helloworld.so:

2017-09-03 Thread Heinrich Schuchardt
On 09/03/2017 02:19 PM, Alexander Graf wrote: > > > On 03.09.17 08:17, Heinrich Schuchardt wrote: >> The target >> $(obj)/helloworld.so: >> exists twice in Makefile.lib. >> >> If you add an echo command to each of the two recipes you get >> warnings like: >> >> scripts/Makefile.lib:383: warning:

Re: [U-Boot] [RFC] toradex: imx6: Move g_dnl_bind_fixup() into common SPL code

2017-09-03 Thread Łukasz Majewski
On 09/03/2017 04:56 PM, Fabio Estevam wrote: From: Fabio Estevam Instead of having every board file to add its own g_dnl_bind_fixup() implementation, move it to the common imx6 SPL code. Reviewed-by: Łukasz Majewski Signed-off-by: Fabio Estevam

Re: [U-Boot] [PATCH v2 8/8] fs/fat: fix case for FAT shortnames

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:38 PM, Rob Clark wrote: Noticed when comparing our output to linux. There are some lcase bits which control whether filename and/or extension should be downcase'd. Reviewed-by: Łukasz Majewski Signed-off-by: Rob Clark ---

Re: [U-Boot] [PATCH v2 7/8] fat/fs: move ls to generic implementation

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:38 PM, Rob Clark wrote: Add a generic implementation of 'ls' using opendir/readdir/closedir, and replace fat's custom implementation. Other filesystems should move to the generic implementation after they add opendir/readdir/closedir support. Reviewed-by: Łukasz Majewski

Re: [U-Boot] [PATCH v2 5/8] fs/fat: implement opendir/readdir/closedir

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:38 PM, Rob Clark wrote: Implement the readdir interface using the directory iterators. Reviewed-by: Łukasz Majewski Signed-off-by: Rob Clark --- fs/fat/fat.c | 56 1 file

Re: [U-Boot] [PATCH v2 4/8] fs: add fs_readdir()

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:37 PM, Rob Clark wrote: Needed to support efi file protocol. The fallback.efi loader wants to be able to read the contents of the /EFI directory to find an OS to boot. Modelled after POSIX opendir()/readdir()/closedir(). Unlike the other fs APIs, this is stateful (ie. state

Re: [U-Boot] [PATCH v2 3/8] fat/fs: convert to directory iterators

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:37 PM, Rob Clark wrote: And drop a whole lot of ugly code! +1 Reviewed-by: Łukasz Majewski Signed-off-by: Rob Clark --- fs/fat/fat.c | 723 ++ include/fat.h | 6 - 2 files

Re: [U-Boot] [PATCH v2 2/8] fs/fat: introduce new director iterators

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:37 PM, Rob Clark wrote: Untangle directory traversal into a simple iterator, to replace the existing multi-purpose do_fat_read_at() + get_dentfromdir(). Signed-off-by: Rob Clark Reviewed-by: Łukasz Majewski --- fs/fat/fat.c | 326

[U-Boot] [PATCH 6/9] fs: btrfs: Add single-device read-only BTRFS implementation

2017-09-03 Thread Marek Behún
This adds the proper implementation for the BTRFS filesystem. The implementation currently supports only read-only mode and the filesystem can be only on a single device. Checksums of data chunks is unimplemented. Compression is implemented (ZLIB + LZO). Signed-off-by: Marek Behun

[U-Boot] [PATCH 9/9] mvebu: turris_omnia: Add CONFIG_CMD_BTRFS to defconfig

2017-09-03 Thread Marek Behún
Signed-off-by: Marek Behun diff --git a/configs/turris_omnia_defconfig b/configs/turris_omnia_defconfig index a3834acb96..9a456e67aa 100644 --- a/configs/turris_omnia_defconfig +++ b/configs/turris_omnia_defconfig @@ -26,6 +26,7 @@ CONFIG_CMD_USB=y CONFIG_CMD_TFTPPUT=y

[U-Boot] [PATCH 8/9] cmd: Add the 'btrsubvol' command to list BTRFS subvolumes

2017-09-03 Thread Marek Behún
Signed-off-by: Marek Behun create mode 100644 cmd/btrfs.c diff --git a/cmd/Kconfig b/cmd/Kconfig index d6d130edfa..77623052c4 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -1311,6 +1311,16 @@ config CMD_CROS_EC endmenu menu "Filesystem commands" +config CMD_BTRFS +

[U-Boot] [PATCH 4/9] fs: btrfs: Add btrfs_tree.h and ctree.h from Linux (and modified)

2017-09-03 Thread Marek Behún
Add btrfs_tree.h and ctree.h from Linux which contains constants and structures for the BTRFS filesystem. Signed-off-by: Marek Behun create mode 100644 fs/btrfs/btrfs_tree.h create mode 100644 fs/btrfs/ctree.h diff --git a/fs/btrfs/btrfs_tree.h b/fs/btrfs/btrfs_tree.h new

[U-Boot] [PATCH 7/9] fs: btrfs: Add U-Boot fs handlers.

2017-09-03 Thread Marek Behún
Signed-off-by: Marek Behun create mode 100644 fs/btrfs/Kconfig create mode 100644 fs/btrfs/Makefile create mode 100644 fs/btrfs/btrfs.c create mode 100644 include/btrfs.h diff --git a/fs/Kconfig b/fs/Kconfig index e6803ac8cb..1cb9831be8 100644 --- a/fs/Kconfig +++

[U-Boot] [PATCH 1/9] lib: Add CRC32-C

2017-09-03 Thread Marek Behún
This is needed for BTRFS. Signed-off-by: Marek Behun create mode 100644 lib/crc32c.c diff --git a/include/u-boot/crc.h b/include/u-boot/crc.h index 6764d58bab..6d08f5df98 100644 --- a/include/u-boot/crc.h +++ b/include/u-boot/crc.h @@ -28,4 +28,8 @@ uint32_t crc32_no_comp

[U-Boot] [PATCH 2/9] fs: Create a common fs_devread for ext4/reiserfs/zfs

2017-09-03 Thread Marek Behún
The ext4, reiserfs and zfs filesystems all have their own implementation of the same function, *_devread. Generalize this function into fs_devread and put the code into fs/fs_internal.c. Signed-off-by: Marek Behun create mode 100644 fs/fs_internal.c create mode 100644

[U-Boot] [PATCH 5/9] fs: btrfs: Add disk-to-cpu and cpu-to-disk conversion functions

2017-09-03 Thread Marek Behún
BTRFS on disk structures are stored in Little Endian. Add functions to convert this structures to cpu and to disk format. On Little Endian hosts, these functions do nothing. On Big Endian the CALL_MACRO_FROM_EACH from variadic-macro.h is used to define all the members for each structure on which

[U-Boot] [PATCH 0/9] Add single-device read-only BTRFS support

2017-09-03 Thread Marek Behún
This is the first version of patches for adding a single-device read-only BTRFS support to U-Boot. Compression (zlib/lzo) is supported. Data checksumming is unimplemented. The code was tested on the Turris Omnia router, where BTRFS is used as the main filesystem from which kernel and device-tree

[U-Boot] [PATCH 3/9] include: Add a variadic macro to call a callback for all arguments

2017-09-03 Thread Marek Behún
Add a header variadic-macro.h which defines the CALL_MACRO_FOR_EACH marco. This macro can be used as follows: #define TEST(x) CALL_MACRO_FOR_EACH(TEST, a, b, c, d) This will expand to TEST(a) TEST(b) TEST(c) TEST(d) The nice thing is that CALL_MACRO_FOR_EACH is a variadic macro, thus the

[U-Boot] [RFC] toradex: imx6: Move g_dnl_bind_fixup() into common SPL code

2017-09-03 Thread Fabio Estevam
From: Fabio Estevam Instead of having every board file to add its own g_dnl_bind_fixup() implementation, move it to the common imx6 SPL code. Signed-off-by: Fabio Estevam --- Stefan, I don't have access to Toradex board to test it, hence marking

Re: [U-Boot] [PATCH v2 1/8] fs/fat: split out helper to init fsdata

2017-09-03 Thread Łukasz Majewski
On 09/02/2017 06:37 PM, Rob Clark wrote: Want to re-use this in fat dirent iterator in next patch. Signed-off-by: Rob Clark Reviewed-by: Łukasz Majewski --- fs/fat/fat.c | 73 +++ include/fat.h

[U-Boot] [PATCH] Configs: Rename MTDIDS_DEFAULT and MTDPARTS_DEFAULT

2017-09-03 Thread Adam Ford
There is already an entry in Kconfig with 'CONFIG_' prepended. This patch renames them to match CONFIG_MTDIDS_DEFAULT and CONFIG_MTDPARTS_DEFAULT to make the migration to Kconfig easier in the future. Signed-off-by: Adam Ford diff --git a/include/configs/BSC9131RDB.h

[U-Boot] [PATCH 2/2] Convert CONFIG_SYS_NAND_ECCSIZE et al to Kconfig

2017-09-03 Thread Adam Ford
This converts the following to Kconfig: CONFIG_SYS_NAND_ECCSIZE CONFIG_SYS_NAND_ECCBYTES CONFIG_SYS_NAND_ECCSTEPS CONFIG_SYS_NAND_MAX_ECCPOS CONFIG_SYS_NAND_MAX_OOBFREE Signed-off-by: Adam Ford --- README | 4

[U-Boot] [PATCH 1/2] Fix boards missing CONFIG_NAND after Kconfig migration

2017-09-03 Thread Adam Ford
Several boards need CONFIG_NAND in order to migrate additional settings to Kconfig. These configs include: SYS_NAND_ECCSIZE, SYS_NAND_ECCBYTES, SYS_NAND_ECCSTEPS, SYS_NAND_MAX_ECCPOS, and SYS_NAND_MAX_OOBFREE Fixes 5bbc265becbe ("Convert CONFIG_NAND to Kconfig") Signed-off-by: Adam Ford

[U-Boot] [PATCH] mvebu: turris_omnia: Only set eth?addr env if CONFIG_CMD_NET

2017-09-03 Thread Marek Behún
Otherwise the linking will fail since eth_env_set_enetaddr cannot be found. Signed-off-by: Marek Behun --- board/CZ.NIC/turris_omnia/turris_omnia.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c

Re: [U-Boot] [PATCH 1/1] scripts/Makefile.lib: remove overridden target $(obj)/helloworld.so:

2017-09-03 Thread Alexander Graf
On 03.09.17 08:17, Heinrich Schuchardt wrote: The target $(obj)/helloworld.so: exists twice in Makefile.lib. If you add an echo command to each of the two recipes you get warnings like: scripts/Makefile.lib:383: warning: overriding recipe for target 'drivers/power/battery/helloworld.so'

[U-Boot] [PATCH] efi_loader: Fix efi_exit gd clobbering

2017-09-03 Thread Alexander Graf
Commit f494950b (efi_loader: call __efi_exit_check in efi_exit) added a call to __efi_exit_check inside efi_exit to account for the fact that we're exiting the efi_exit function via a longjmp call. However, __efi_exit_check also swizzles gd to the application gd while the longjmp will put us back

[U-Boot] [PATCH 2/2] Makefile: Suppress output of python libfdt build command

2017-09-03 Thread Bin Meng
This should not be printed by default. Prefix it with $(Q). Fixes ee95d10b: ("fdt: Build the new python libfdt module") Signed-off-by: Bin Meng --- tools/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/Makefile b/tools/Makefile index

[U-Boot] [PATCH 1/2] Makefile: Quiesce libfdt build

2017-09-03 Thread Bin Meng
Since commit 3809e302 "Makefile: honor PYTHON configuration properly", the build commands of libfdt are printed while previously were not. This adds the missing '--quiet' back. Signed-off-by: Bin Meng --- tools/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [U-Boot] Please pull u-boot-x86

2017-09-03 Thread Tom Rini
On Sat, Sep 02, 2017 at 11:56:45PM +0800, Bin Meng wrote: > Hi Tom, > > These are some bug fixes for v2017.09. > > The following changes since commit 6ef2f90108e2cf101d931c71ea7904f2b6301641: > > Convert CONFIG_BCH to Kconfig (2017-09-01 20:45:26 -0400) > > are available in the git

Re: [U-Boot] [PATCH v1 2/2] x86: edison: Bring ACPI minimal support to the board

2017-09-03 Thread Bin Meng
On Wed, Aug 30, 2017 at 11:04 PM, Andy Shevchenko wrote: > This board is based on Intel Tangier SoC (Intel Merrifield platform) > and may utilize ACPI powerfulness. > > Bring minimum support by appending initial DSDT table for it. > > Note, the addresses for

Re: [U-Boot] [PATCH v1 1/2] x86: tangier: Enable ACPI support for Intel Tangier

2017-09-03 Thread Bin Meng
Hi Andy, On Wed, Aug 30, 2017 at 11:04 PM, Andy Shevchenko wrote: > Intel Tangier SoC is a part of Intel Merrifield platform which doesn't > utilize ACPI by default. Here is an attempt to unleash ACPI flexibility > power on Intel Merrifield based platforms. > >

[U-Boot] [PATCH 1/1] scripts/Makefile.lib: remove overridden target $(obj)/helloworld.so:

2017-09-03 Thread Heinrich Schuchardt
The target $(obj)/helloworld.so: exists twice in Makefile.lib. If you add an echo command to each of the two recipes you get warnings like: scripts/Makefile.lib:383: warning: overriding recipe for target 'drivers/power/battery/helloworld.so' scripts/Makefile.lib:379: warning: ignoring old recipe