Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Lothar Waßmann
Hi, On Fri, 22 Dec 2017 01:43:38 + Chee, Tien Fong wrote: > On Kha, 2017-12-21 at 16:08 +0100, Lothar Waßmann wrote: > > Hi, > > > > On Thu, 21 Dec 2017 12:48:53 + Chee, Tien Fong wrote: > > > > > > On Kha, 2017-12-21 at 12:53 +0100, Lothar Waßmann wrote: > > > > > > > > Hi, > > > > >

Re: [U-Boot] [PATCH 1/3] ddr: fsl: Do not allow for interactive mode in SPL

2017-12-21 Thread York Sun
Why not? I used it before. York Sent from my iPhone > On Dec 21, 2017, at 19:13, Tom Rini wrote: > > While the code in question here is used in SPL, it cannot be used in > interactive mode in SPL. We need a compile-time guard here in order to > be able to not include

Re: [U-Boot] [PATCH 1/2] disk: part: scan the disk if the part_type is unknow

2017-12-21 Thread Kever Yang
Hi Simon, On 12/19/2017 11:41 PM, Simon Glass wrote: Hi Kever, On 13 December 2017 at 23:39, Kever Yang wrote: We can get the new part table when we write a new partition table to a blank disk with this patch, or else we have to reset the board to get new

[U-Boot] [PATCH 3/3] SPL: Do not include cmd lists

2017-12-21 Thread Tom Rini
There are no CMDs to be run in SPL and no need for any of their lists. CC: Stefano Babic Cc: Fabio Estevam Cc: Jagan Teki Cc: Maxime Ripard Cc: "Andreas Bießmann" Cc: Michal

[U-Boot] [PATCH 1/3] ddr: fsl: Do not allow for interactive mode in SPL

2017-12-21 Thread Tom Rini
While the code in question here is used in SPL, it cannot be used in interactive mode in SPL. We need a compile-time guard here in order to be able to not include further bits of code. Cc: York Sun Signed-off-by: Tom Rini --- drivers/ddr/fsl/main.c | 2 +-

[U-Boot] [PATCH 2/3] build: Drop CONFIG_SPL_BUILD guards in some cases

2017-12-21 Thread Tom Rini
Given gcc-6.1 and later we can now safely have strings discarded when the functions are unused. This lets us drop certain cases of not building something so that we don't have the strings brought in when the code was discarded. Simplify the code now by dropping guards we don't need now. Cc:

Re: [U-Boot] Dragonboard410c and Dragonboard820c patchset

2017-12-21 Thread Peter Robinson
On Thu, Dec 21, 2017 at 8:12 AM, Jorge Ramirez wrote: > On 12/14/2017 02:26 PM, Jorge Ramirez-Ortiz wrote: >> >> The following version fixes some nasty compiler warnings present in >> the previous version. Sorry about it. >> > > just to follow up on the db820c and

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Chee, Tien Fong
On Kha, 2017-12-21 at 16:08 +0100, Lothar Waßmann wrote: > Hi, > > On Thu, 21 Dec 2017 12:48:53 + Chee, Tien Fong wrote: > > > > On Kha, 2017-12-21 at 12:53 +0100, Lothar Waßmann wrote: > > > > > > Hi, > > > > > > On Thu, 21 Dec 2017 09:36:41 + Chee, Tien Fong wrote: > > > > > > > >

Re: [U-Boot] [PATCH] crypto/fsl: fix BLOB encapsulation and decapsulation

2017-12-21 Thread Clemens Gruber
Hi Breno, On Thu, Dec 21, 2017 at 07:18:44PM -0200, Breno Matheus Lima wrote: > Hi Clemens, > > 2017-12-20 20:08 GMT-02:00 Clemens Gruber : > > The blob_encap and blob_decap functions were not flushing the dcache > > before passing data to CAAM/DMA and not

Re: [U-Boot] [PATCH 11/11] sunxi: Add limit with the MMC environment

2017-12-21 Thread André Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The MMC environment offset is getting very close to the end of the U-Boot > binary now. Since we want to make sure this will not overflow, add a size > limit in the board for arm64. arm32 has already that limit enforced in our > custom image

Re: [U-Boot] [PATCH] crypto/fsl: fix BLOB encapsulation and decapsulation

2017-12-21 Thread Breno Matheus Lima
Hi Clemens, 2017-12-20 20:08 GMT-02:00 Clemens Gruber : > The blob_encap and blob_decap functions were not flushing the dcache > before passing data to CAAM/DMA and not invalidating the dcache when > getting data back. > Therefore, blob encapsulation and decapsulation

Re: [U-Boot] [PATCH 07/11] video: bpp16: Disable by default on sunXi

2017-12-21 Thread Anatolij Gustschin
On Thu, 21 Dec 2017 13:40:26 +0100 Maxime Ripard maxime.rip...@free-electrons.com wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable

Re: [U-Boot] [PATCH 08/11] net: regex: Disable by default on sunXi

2017-12-21 Thread Andre Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable this command that is not critical until we

Re: [U-Boot] [PATCH 06/11] video: bpp8: Disable by default on sunXi

2017-12-21 Thread Anatolij Gustschin
On Thu, 21 Dec 2017 13:40:25 +0100 Maxime Ripard maxime.rip...@free-electrons.com wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable

Re: [U-Boot] [PATCH v1 06/10] mmc: add a Kconfig option to enable the support for MMC write operations

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 04:33:10PM +0100, Jean-Jacques Hiblot wrote: > > > On 21/12/2017 15:52, Tom Rini wrote: > >On Thu, Dec 21, 2017 at 09:46:36AM -0500, Tom Rini wrote: > >>On Thu, Dec 21, 2017 at 12:53:52PM +0100, Jean-Jacques Hiblot wrote: > >> > >>>This allows using

Re: [U-Boot] [PATCH v1 01/10] common: do not compile common fastboot code when building the SPL

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 04:28:40PM +0100, Jean-Jacques Hiblot wrote: > > > On 21/12/2017 15:46, Tom Rini wrote: > >On Thu, Dec 21, 2017 at 12:53:47PM +0100, Jean-Jacques Hiblot wrote: > > > >>This is not required as fastboot can't be started from SPL. > >> > >>Signed-off-by: Jean-Jacques Hiblot

Re: [U-Boot] [PATCH 1/2] mmc: sunxi: Support cd-inverted DT property

2017-12-21 Thread Maxime Ripard
On Thu, Dec 21, 2017 at 04:09:46PM +0200, Tuomas Tynkkynen wrote: > On Thu, 2017-12-21 at 14:09 +0100, Maxime Ripard wrote: > > > > sunxi_gpio_set_pull(cd_pin, SUNXI_GPIO_PULL_UP); > > > } > > > > > > So: if the DT specifies GPIO_ACTIVE_LOW, dm_gpio_get_value() > > > does its own

Re: [U-Boot] [PATCH v1 06/10] mmc: add a Kconfig option to enable the support for MMC write operations

2017-12-21 Thread Jean-Jacques Hiblot
On 21/12/2017 15:52, Tom Rini wrote: On Thu, Dec 21, 2017 at 09:46:36AM -0500, Tom Rini wrote: On Thu, Dec 21, 2017 at 12:53:52PM +0100, Jean-Jacques Hiblot wrote: This allows using CONFIG_IS_ENABLED(MMC_WRITE) to compile out code needed only if write support is required. The option is

Re: [U-Boot] [PATCH v1 01/10] common: do not compile common fastboot code when building the SPL

2017-12-21 Thread Jean-Jacques Hiblot
On 21/12/2017 15:46, Tom Rini wrote: On Thu, Dec 21, 2017 at 12:53:47PM +0100, Jean-Jacques Hiblot wrote: This is not required as fastboot can't be started from SPL. Signed-off-by: Jean-Jacques Hiblot --- common/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Lothar Waßmann
Hi, On Thu, 21 Dec 2017 12:48:53 + Chee, Tien Fong wrote: > On Kha, 2017-12-21 at 12:53 +0100, Lothar Waßmann wrote: > > Hi, > > > > On Thu, 21 Dec 2017 09:36:41 + Chee, Tien Fong wrote: > > > > > > On Kha, 2017-12-21 at 09:48 +0100, Lothar Waßmann wrote: > > > > > > > > Hi, > > > > >

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

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 09:24:48AM +0800, Bin Meng wrote: > Hi Tom, > > The following changes since commit b55c89ce0207d3a504238c1b8f268c56035656a3: > > Merge git://git.denx.de/u-boot-spi (2017-12-19 07:57:40 -0500) > > are available in the git repository at: > >

Re: [U-Boot] [PATCH v1 06/10] mmc: add a Kconfig option to enable the support for MMC write operations

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 09:46:36AM -0500, Tom Rini wrote: > On Thu, Dec 21, 2017 at 12:53:52PM +0100, Jean-Jacques Hiblot wrote: > > > This allows using CONFIG_IS_ENABLED(MMC_WRITE) to compile out code > > needed only if write support is required. > > The option is added for u-boot and for the

Re: [U-Boot] [PATCH 05/11] cmd: misc: Disable by default on sunXi

2017-12-21 Thread Andre Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable this command that is not critical until we

Re: [U-Boot] [PATCH 01/11] cmd: crc32: Disable by default on sunXi

2017-12-21 Thread Andre Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable this command that is not critical until we

Re: [U-Boot] [PATCH v1 06/10] mmc: add a Kconfig option to enable the support for MMC write operations

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 12:53:52PM +0100, Jean-Jacques Hiblot wrote: > This allows using CONFIG_IS_ENABLED(MMC_WRITE) to compile out code > needed only if write support is required. > The option is added for u-boot and for the SPL "for SPL". [snip] > +config SPL_MMC_WRITE > + bool

Re: [U-Boot] [PATCH v1 01/10] common: do not compile common fastboot code when building the SPL

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 12:53:47PM +0100, Jean-Jacques Hiblot wrote: > This is not required as fastboot can't be started from SPL. > > Signed-off-by: Jean-Jacques Hiblot > --- > > common/Makefile | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/common/Makefile

Re: [U-Boot] [PATCH 07/11] video: bpp16: Disable by default on sunXi

2017-12-21 Thread Andre Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable this command that is not critical until we

Re: [U-Boot] [PATCH 06/11] video: bpp8: Disable by default on sunXi

2017-12-21 Thread Andre Przywara
Hi, On 21/12/17 12:40, Maxime Ripard wrote: > The sunXi arm64 build has overflown, leading to the main U-boot binary > overwriting the environment when flashing the new image, or even worse, > overwriting itself when we're calling saveenv. > > Disable this command that is not critical until we

Re: [U-Boot] [PATCH 1/2] mmc: sunxi: Support cd-inverted DT property

2017-12-21 Thread Tuomas Tynkkynen
On Thu, 2017-12-21 at 14:09 +0100, Maxime Ripard wrote: > > sunxi_gpio_set_pull(cd_pin, SUNXI_GPIO_PULL_UP); > > } > > > > So: if the DT specifies GPIO_ACTIVE_LOW, dm_gpio_get_value() > > does its own inversion of the GPIO level. Then, on top of that > > we do another inversion

Re: [U-Boot] [GIT] Pull request: u-boot-dfu (20.12.2017)

2017-12-21 Thread Marek Vasut
On 12/21/2017 02:15 PM, Lukasz Majewski wrote: > Hi Marek, Hi, >> On 12/20/2017 01:52 PM, Lukasz Majewski wrote: >>> Dear Marek, >> >> Hi, >> >> I presume this is for -next , right ? > > Yes. It is supposed for the -next. > > I shall have made it more clear. My bad, sorry. Can you

Re: [U-Boot] [GIT] Pull request: u-boot-dfu (20.12.2017)

2017-12-21 Thread Lukasz Majewski
Hi Marek, > On 12/20/2017 01:52 PM, Lukasz Majewski wrote: > > Dear Marek, > > Hi, > > I presume this is for -next , right ? Yes. It is supposed for the -next. I shall have made it more clear. My bad, sorry. > > > The following changes since commit > >

Re: [U-Boot] [PATCH 1/2] mmc: sunxi: Support cd-inverted DT property

2017-12-21 Thread Maxime Ripard
Hi Tuomas, On Wed, Dec 20, 2017 at 03:34:47PM +0200, Tuomas Tynkkynen wrote: > > > @@ -606,8 +607,15 @@ static int sunxi_mmc_probe(struct udevice > > > *dev) > > > if (!gpio_request_by_name(dev, "cd-gpios", 0, > > > >cd_gpio, > > > GPIOD_IS_IN)) { > > >

Re: [U-Boot] [PATCH 2/2] ARM: sunxi: Enable DM MMC+SATA for the PcDuino3 Nano board

2017-12-21 Thread Maxime Ripard
On Wed, Dec 20, 2017 at 03:37:13PM +0200, Tuomas Tynkkynen wrote: > On Wed, 2017-12-20 at 13:26 +0100, Maxime Ripard wrote: > > On Mon, Dec 18, 2017 at 10:09:42PM +0200, Tuomas Tynkkynen wrote: > > > After the previous commit which adds support for the cd-inverted > > > property to the sunxi MMC

Re: [U-Boot] [PATCH] sunxi: maintainers: Add myself for the TBS A711

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 01:55:52PM +0100, Maxime Ripard wrote: > Support for that board got introduced recently without the maintainers > part. Let's fix that. > > Signed-off-by: Maxime Ripard Reviewed-by: Tom Rini Thanks! -- Tom

Re: [U-Boot] [PATCH 00/11] sunxi: arm64 binary size fixes

2017-12-21 Thread Tom Rini
On Thu, Dec 21, 2017 at 01:40:19PM +0100, Maxime Ripard wrote: > Hi, > > As we discussed already, this is the list of options that we will need > to disable by default. > > I also added to the mix a size check, which even though it's not > really optimal, is temporary and will be removed in a

[U-Boot] [PATCH] sunxi: maintainers: Add myself for the TBS A711

2017-12-21 Thread Maxime Ripard
Support for that board got introduced recently without the maintainers part. Let's fix that. Signed-off-by: Maxime Ripard --- board/sunxi/MAINTAINERS | 5 + 1 file changed, 5 insertions(+) diff --git a/board/sunxi/MAINTAINERS b/board/sunxi/MAINTAINERS

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Chee, Tien Fong
On Kha, 2017-12-21 at 12:53 +0100, Lothar Waßmann wrote: > Hi, > > On Thu, 21 Dec 2017 09:36:41 + Chee, Tien Fong wrote: > > > > On Kha, 2017-12-21 at 09:48 +0100, Lothar Waßmann wrote: > > > > > > Hi, > > > > > > On Thu, 21 Dec 2017 15:25:29 +0800 tien.fong.c...@intel.com > > > wrote: > >

[U-Boot] [PATCH 08/11] net: regex: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 11/11] sunxi: Add limit with the MMC environment

2017-12-21 Thread Maxime Ripard
The MMC environment offset is getting very close to the end of the U-Boot binary now. Since we want to make sure this will not overflow, add a size limit in the board for arm64. arm32 has already that limit enforced in our custom image generation. Signed-off-by: Maxime Ripard

[U-Boot] [PATCH 03/11] cmd: loadb: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 04/11] cmd: loads: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 07/11] video: bpp16: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 05/11] cmd: misc: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 10/11] Makefile: Add size check to the u-boot.itb make target

2017-12-21 Thread Maxime Ripard
The make macro to check if the binary exceeds the board size limit is not called. Make sure that is the case. Signed-off-by: Maxime Ripard --- Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile b/Makefile index c573ddba95ff..dd26795bb1f3

[U-Boot] [PATCH 09/11] sunxi: imply CONFIG_OF_LIBFDT_OVERLAY

2017-12-21 Thread Maxime Ripard
From: Andre Heider fdt overlay support is useful for all sunxi boards, enable per default and remove it from sunxi defconfigs. Signed-off-by: Andre Heider Signed-off-by: Maxime Ripard --- arch/arm/Kconfig | 1

[U-Boot] [PATCH 06/11] video: bpp8: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 02/11] cmd: unzip: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 01/11] cmd: crc32: Disable by default on sunXi

2017-12-21 Thread Maxime Ripard
The sunXi arm64 build has overflown, leading to the main U-boot binary overwriting the environment when flashing the new image, or even worse, overwriting itself when we're calling saveenv. Disable this command that is not critical until we can adress the issue properly. Signed-off-by: Maxime

[U-Boot] [PATCH 00/11] sunxi: arm64 binary size fixes

2017-12-21 Thread Maxime Ripard
Hi, As we discussed already, this is the list of options that we will need to disable by default. I also added to the mix a size check, which even though it's not really optimal, is temporary and will be removed in a future (and hopefully close) release. Tom, is this still something we can

Re: [U-Boot] [PATCH] am335x_hs_evm: Trim options in SPL to reduce binary size

2017-12-21 Thread Jean-Jacques Hiblot
On 20/12/2017 14:16, Jean-Jacques Hiblot wrote: On 19/12/2017 16:00, Tom Rini wrote: On Tue, Dec 19, 2017 at 08:54:25AM -0600, Andrew F. Davis wrote: On 12/16/2017 10:04 PM, Tom Rini wrote: The am335x_hs_evm runs into size constraint problems at times with various toolchains as changes

[U-Boot] [PATCH 3/3][v2] sf: parse Serial Flash Discoverable Parameters (SFDP) tables

2017-12-21 Thread Prabhakar Kushwaha
This patch adds support to the JESD216 rev B standard and parses the SFDP tables to dynamically initialize the 'struct spi_nor_flash_parameter'. It has been ported from Linux commit "mtd: spi-nor: parse Serial Flash Discoverable Parameters (SFDP) tables". It Also ports all modifications done on

[U-Boot] [PATCH 2/3][v2] sf: add method to support memory size above 128Mib

2017-12-21 Thread Prabhakar Kushwaha
This patch add support of memories with size above 128Mib. It has been ported from Linux commit "mtd: spi-nor: add a stateless method to support memory size above 128Mib". It convert 3byte opcode into 4byte opcode based upon OPCODES_4B or Spansion flash. Also the commands are malloc'ed at run

[U-Boot] [PATCH 1/3][v2] sf: Add support of 1-2-2, 1-4-4 IO READ protocols

2017-12-21 Thread Prabhakar Kushwaha
IO READ protocols transfers both address and data on multiple data bits. 1-2-2(DUAL IO), 1-4-4(QUAD IO) transfer address on 2 data bits or 4 bits per rising edge of SCK respectively. This patch update spi_nor_flash_parameter->spi_nor_read_command array based on DUAL or QUAD IO flag enabled in

[U-Boot] [PATCH 0/3][v2] sf: Update spi-nor framework

2017-12-21 Thread Prabhakar Kushwaha
SPI-NOR framework currently supports- - (1-1-1, 1-1-2, 1-1-4) read protocols - read latency(dummy bytes) are hardcoded with the assumption that the flash would support it. - No support of mode bits. - No support of flash size above 128Mib This patch set add support of 1-2-2, 1-4-4 read

[U-Boot] [PATCH 0/3][v2] sf: Update spi-nor framework

2017-12-21 Thread Prabhakar Kushwaha
SPI-NOR framework currently supports- - (1-1-1, 1-1-2, 1-1-4) read protocols - read latency(dummy bytes) are hardcoded with the assumption that the flash would support it. - No support of mode bits. - No support of flash size above 128Mib This patch set add support of 1-2-2, 1-4-4 read

[U-Boot] [PATCH v1 08/10] mmc: compile out erase and write mmc commands if write operations are not enabled

2017-12-21 Thread Jean-Jacques Hiblot
Also remove erase_grp_size and write_bl_len from struct mmc as they are not used anymore. On ARM, removing them saves about 100 bytes of code space in SPL. Signed-off-by: Jean-Jacques Hiblot --- cmd/mmc.c | 8 drivers/mmc/mmc.c | 16 ++--

[U-Boot] [PATCH v1 00/10] reduce the size of the mmc core

2017-12-21 Thread Jean-Jacques Hiblot
This series applies on u-boot/next It aims at reducing the size taken by the mmc core in the SPL. Recent changes (for which I'm to blame) have bloated the mmc core and have broken platforms that were already tight on code space. This is achieved mostly by compiling out parts of the

[U-Boot] [PATCH v1 09/10] mmc: don't read the size of eMMC enhanced user data area in SPL

2017-12-21 Thread Jean-Jacques Hiblot
This information is only used by the "mmc info" command. On ARM removing this information from SPL saves about 140 of code space. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 2 ++ include/mmc.h | 2 ++ 2 files changed, 4 insertions(+) diff --git

[U-Boot] [PATCH v1 05/10] mmc: reworked version lookup in mmc_startup_v4

2017-12-21 Thread Jean-Jacques Hiblot
Using a table versus a switch() structure saves a bit of space Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 42 +- 1 file changed, 17 insertions(+), 25 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index

[U-Boot] [PATCH v1 07/10] mmc: read ssr only if MMC write support is enabled

2017-12-21 Thread Jean-Jacques Hiblot
The content of ssr is useful only for erase operations. on ARM, removing sd_read_ssr() saves around 300 bytes. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 25 ++--- include/mmc.h | 2 ++ 2 files changed, 16 insertions(+), 11 deletions(-)

[U-Boot] [PATCH v1 02/10] mmc: atmel: when sending a data command, use the provided block size

2017-12-21 Thread Jean-Jacques Hiblot
struct mmc_data contains the block size to use for the data transfer. Use this information instead of relying on read_bl_len and write_bl_len stored in the mmc structure. Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/gen_atmel_mci.c | 3 +-- 1 file changed, 1

[U-Boot] [PATCH v1 03/10] mmc: remove unneeded verification in mmc_set_card_speed()

2017-12-21 Thread Jean-Jacques Hiblot
mmc_set_card_speed() reads the ext csd to check if switch has been OK. But it does it only for MMC_HS and MMC_HS_52. Moreover this check is not really required as there will be a ext csd reading later in the initialization process to make sure that it succeeded. So remove this partial verification

[U-Boot] [PATCH v1 06/10] mmc: add a Kconfig option to enable the support for MMC write operations

2017-12-21 Thread Jean-Jacques Hiblot
This allows using CONFIG_IS_ENABLED(MMC_WRITE) to compile out code needed only if write support is required. The option is added for u-boot and for the SPL Signed-off-by: Jean-Jacques Hiblot --- cmd/mvebu/bubt.c | 2 +- common/spl/Kconfig| 8

[U-Boot] [PATCH v1 10/10] mmc: remove hc_wp_grp_size from struct mmc if not needed

2017-12-21 Thread Jean-Jacques Hiblot
hc_wp_grp_size is needed only if hardware partitionning is used. On ARM removing it saves about 30 bytes of code space. Signed-off-by: Jean-Jacques Hiblot --- cmd/mmc.c | 2 ++ drivers/mmc/mmc.c | 2 ++ include/mmc.h | 2 ++ 3 files changed, 6 insertions(+) diff

[U-Boot] [PATCH v1 04/10] mmc: compile out more code if support for UHS and HS200 is not enabled

2017-12-21 Thread Jean-Jacques Hiblot
Signed-off-by: Jean-Jacques Hiblot --- drivers/mmc/mmc.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 7a92718..d006893 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -796,9 +796,11 @@ static

[U-Boot] [PATCH v1 01/10] common: do not compile common fastboot code when building the SPL

2017-12-21 Thread Jean-Jacques Hiblot
This is not required as fastboot can't be started from SPL. Signed-off-by: Jean-Jacques Hiblot --- common/Makefile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/common/Makefile b/common/Makefile index 1416620..c7bde23 100644 --- a/common/Makefile +++ b/common/Makefile

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Lothar Waßmann
Hi, On Thu, 21 Dec 2017 09:36:41 + Chee, Tien Fong wrote: > On Kha, 2017-12-21 at 09:48 +0100, Lothar Waßmann wrote: > > Hi, > > > > On Thu, 21 Dec 2017 15:25:29 +0800 tien.fong.c...@intel.com wrote: > > > > > > From: Tien Fong Chee > > > > > > This is file

[U-Boot] [PATCH] fat write: Fixed a problem with the case of file names when writing files

2017-12-21 Thread Jean-Jacques Hiblot
commit 21a24c3bf35b ("fs/fat: fix case for FAT shortnames") made it possible that get_name() returns file names with some upper cases. find_directory_entry() must be updated to take this account, and use case-insensitive functions to compare file names. Signed-off-by: Jean-Jacques Hiblot

Re: [U-Boot] [EXT] Re: arm64: a37xx: add distro boot compability

2017-12-21 Thread Kostya Porotchkin
> -Original Message- > From: Andre Heider [mailto:a.hei...@gmail.com] > Sent: Thursday, December 21, 2017 13:35 > To: Kostya Porotchkin; Stefan Roese > Cc: u-boot@lists.denx.de; Nadav Haklai; Igal Liberman > Subject: Re: [EXT] Re: arm64: a37xx: add distro boot compability > > On

Re: [U-Boot] [PATCH 3/3] sf: parse Serial Flash Discoverable Parameters (SFDP) tables

2017-12-21 Thread Prabhakar Kushwaha
Thanks Cyrille for taking time and reviewing this patch. Please find reply in-lined. > -Original Message- > From: Cyrille Pitchen [mailto:cyrille.pitc...@wedev4u.fr] > Sent: Wednesday, December 20, 2017 7:36 PM > To: Prabhakar Kushwaha ; u- >

Re: [U-Boot] [EXT] Re: arm64: a37xx: add distro boot compability

2017-12-21 Thread Andre Heider
On 20/12/17 17:08, Kostya Porotchkin wrote: [...] But instead I was offering the same patches against the Marvell tree, without the need for any syncing. They're now in a pull request I just created on Marvell's public github: https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/pull/8

Re: [U-Boot] [U-Boot, v2, 16/18] clk: rockchip: Add rk322x gamc clock support

2017-12-21 Thread David.Wu
Hi Philipp, 在 2017/11/26 22:50, Philipp Tomsich 写道: On Thu, 9 Nov 2017, David Wu wrote: Assuming mac_clk is fed by an external clock, set clk_rmii_src clock select control register from IO for rgmii interface. Signed-off-by: David Wu Acked-by: Philipp Tomsich

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Chee, Tien Fong
On Kha, 2017-12-21 at 17:36 +0800, Chee, Tien Fong wrote: > On Kha, 2017-12-21 at 09:48 +0100, Lothar Waßmann wrote: > > > > Hi, > > > > On Thu, 21 Dec 2017 15:25:29 +0800 tien.fong.c...@intel.com wrote: > > > > > > > > > From: Tien Fong Chee > > > > > > This is

[U-Boot] [PATCH] rockchip: rk3399-puma: reduce env size to 8kiB

2017-12-21 Thread Jakob Unterwurzacher
...even when the environment is saved to SD card. With the default of 32kiB, the environment overwrites the SPL stage which lives at 16kiB. Signed-off-by: Jakob Unterwurzacher --- board/theobroma-systems/puma_rk3399/Kconfig | 2 +- 1 file changed, 1

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Chee, Tien Fong
On Kha, 2017-12-21 at 09:48 +0100, Lothar Waßmann wrote: > Hi, > > On Thu, 21 Dec 2017 15:25:29 +0800 tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee > > > > This is file system generic loader which can be used to load > > the file image from the storage

Re: [U-Boot] [PATCH 2/3] sf: add method to support memory size above 128Mib

2017-12-21 Thread Prabhakar Kushwaha
Thanks Cyrille for taking time and reviewing this patch. Please find reply in-lined. > -Original Message- > From: Cyrille Pitchen [mailto:cyrille.pitc...@wedev4u.fr] > Sent: Wednesday, December 20, 2017 6:43 PM > To: Prabhakar Kushwaha ; u- >

Re: [U-Boot] [PATCH] common/memsize.c: restore content of the base address

2017-12-21 Thread Wolfgang Denk
Dear Patrick, In message <94c2391ea4e943fb9f65e7b0943c9...@sfhdag6node3.st.com> you wrote: > > example for physical size limited to 128MB = bit 0 to 26 mapped to the > memory, bit 27 used > access to 0x000 => physical access to 0x > acesss to 0x7FFF => physical access to

Re: [U-Boot] [PATCH v5 2/2] common: Generic firmware loader for file system

2017-12-21 Thread Lothar Waßmann
Hi, On Thu, 21 Dec 2017 15:25:29 +0800 tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > This is file system generic loader which can be used to load > the file image from the storage into target such as memory. > The consumer driver would then use this loader

Re: [U-Boot] Dragonboard410c and Dragonboard820c patchset

2017-12-21 Thread Jorge Ramirez
On 12/14/2017 02:26 PM, Jorge Ramirez-Ortiz wrote: The following version fixes some nasty compiler warnings present in the previous version. Sorry about it. just to follow up on the db820c and db410c patchset that I posted a couple of weeks ago (v1 and v2) any feedback will be greatly