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,
> > > >
>
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
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
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
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 +-
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:
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
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:
> > > >
> > > >
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
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
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
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
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
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
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
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
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
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
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
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,
> > > >
>
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:
>
>
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
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
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
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
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
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
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
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
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
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
> >
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)) {
> > >
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
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
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
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
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:
> >
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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 ++--
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
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
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
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(-)
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
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
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
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
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
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
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
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
> -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
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-
>
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
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
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
...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
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
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-
>
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
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
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
79 matches
Mail list logo