Match the depth of indentation between #ifdef and #endif
for better readability.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
lib/fdtdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/fdtdec.c b/lib/fdtdec.c
index a954051..48667ef 100644
---
On Fri, Jul 31, 2015 at 09:36:45PM -0500, Scott Wood wrote:
On Sat, 2015-08-01 at 09:15 +0800, Peng Fan wrote:
On Fri, Jul 31, 2015 at 12:07:50PM -0500, Scott Wood wrote:
On Tue, 2015-07-21 at 16:15 +0800, Peng Fan wrote:
If ecc chunk data size is 512 and oobsize is bigger than 512, there is
As we discussed a couple of times, negative CONFIG options make our
life difficult; CONFIG_SYS_NO_FLASH, CONFIG_SYS_DCACHE_OFF, ...
and here is another one.
Now, there are actually two boards enabling OF_CONTROL on SPL:
- socfpga_arria5_defconfig
- socfpga_cyclone5_defconfig
But they were
We have flipped CONFIG_SPL_DISABLE_OF_CONTROL. We have cleansing
devices, $(SPL_) and CONFIG_IS_ENABLED(), so we are ready to clear
away the ugly logic in include/fdtdec.h:
#ifdef CONFIG_OF_CONTROL
# if defined(CONFIG_SPL_BUILD) !defined(SPL_OF_CONTROL)
# define OF_CONTROL 0
# else
#
Refer to Simon's question, too:
http://lists.denx.de/pipermail/u-boot/2015-July/219598.html
Since U-boot introduced SPL (not since Kconfig),
enabling features for U-boot and SPL independently is always a PITA.
- decide if each feature should be supported for SPL or not
- Add
Hi Marek,
On 28 July 2015 at 09:57, Marek Vasut ma...@denx.de wrote:
On Tuesday, July 28, 2015 at 05:53:10 PM, Masahiro Yamada wrote:
[...]
$ git describe
v2015.07-307-g2647394
$ make vpac270_ond_256_defconfig
#
# configuration written to .config
#
$ make -s
The current comments need a bit of tweaking since we now support stack
and global_data relocation in SPL. Also add a reference to the README.
For AArch64 this is not implemented, so leave a TODO for this.
Signed-off-by: Simon Glass s...@chromium.org
Reported-by: Tim Harvey thar...@gateworks.com
On Sat, Aug 01, 2015 at 10:14:45PM +0900, Masahiro Yamada wrote:
We have flipped CONFIG_SPL_DISABLE_OF_CONTROL. We have cleansing
devices, $(SPL_) and CONFIG_IS_ENABLED(), so we are ready to clear
away the ugly logic in include/fdtdec.h:
#ifdef CONFIG_OF_CONTROL
# if
Hi Masahiro,
On Sun, Jul 26, 2015 at 1:46 AM, Masahiro Yamada
yamada.masah...@socionext.com wrote:
The menuconfig for drivers are getting more and more cluttered
and unreadable because too many entries are displayed in a single
flat menu. Use hierarchic menu for each category.
Hi Hans,
On 25 July 2015 at 03:58, Hans de Goede hdego...@redhat.com wrote:
Currently the serial code assumes that there is always at least one serial
port (and panics / crashes due to null pointer dereferences when there is
none).
This makes it impossible to use u-boot on boards where there
On Fri, Jul 31, 2015 at 03:06:49AM +0200, Albert ARIBAUD wrote:
Hello,
The following changes since commit cc35734358540a1bbaf042fdf9f4cb2de17389ed:
Merge branch 'master' of git://git.denx.de/u-boot-tegra (2015-07-29
18:58:39 -0400)
are available in the git repository at:
On Sat, Aug 01, 2015 at 10:14:31PM +0900, Masahiro Yamada wrote:
If the target string matches CONFIG_, move the pointer p
forward. This saves several 7-chars adjustments.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
On Fri, Jul 31, 2015 at 07:35:23AM -0600, Simon Glass wrote:
Hi Tom,
A fix for a nasty problem where patman runs twice!
The following changes since commit cc35734358540a1bbaf042fdf9f4cb2de17389ed:
Merge branch 'master' of git://git.denx.de/u-boot-tegra (2015-07-29
18:58:39 -0400)
On Fri, Jul 31, 2015 at 09:00:43AM -0700, York Sun wrote:
Tom,
The following changes since commit 26473945ad6667183296e7edee2a65edf31bb6f7:
Merge branch 'master' of http://git.denx.de/u-boot-sunxi (2015-07-25
09:04:18
-0400)
are available in the git repository at:
On Saturday, August 01, 2015 at 06:08:56 PM, Simon Glass wrote:
Hi Marek,
Hi!
On 28 July 2015 at 09:57, Marek Vasut ma...@denx.de wrote:
On Tuesday, July 28, 2015 at 05:53:10 PM, Masahiro Yamada wrote:
[...]
$ git describe
v2015.07-307-g2647394
$ make
Just preparing for upcoming cleaning.
The board-specific linker script board/vpac270/u-boot-spl.lds
has been touched to avoid build error. It does not change the
size of spl/u-boot-spl.bin for this board, so it should be OK.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile| 2 +-
drivers/led/Kconfig | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index 1baecb4..3188a51 100644
--- a/drivers/Makefile
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index 22d316e..1baecb4 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -1,5
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile| 2 +-
drivers/ram/Kconfig | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index 8d4ac06..22d316e 100644
--- a/drivers/Makefile
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile| 2 +-
drivers/clk/Kconfig | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index dd57849..a1e24c1 100644
--- a/drivers/Makefile
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index a1e24c1..8d4ac06 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -1,8
We do not want to compile the DM remove code for SPL. Currently,
we undef it in include/config_uncmd_spl.h (for C files) and in
scripts/Makefile.uncmd_spl (for Makefiles). This is really ugly.
This commit demonstrates how we can deprecate those two files.
Use $(SPL_) for the entry in the
There is no case where defined(SPL_DISABLE_OF_CONTROL) is true.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
include/fdtdec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/fdtdec.h b/include/fdtdec.h
index 4b3f8d1..2681a64
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index 3188a51..4901e16 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -1,5
Hi Masahiro,
On Sat, Aug 1, 2015 at 9:27 PM, Masahiro Yamada
yamada.masah...@socionext.com wrote:
Hi Bin,
Looks like I missed some items..
Anyway, they do not have many drivers below them,
so menuconfig is not so messy.
Do you want me send v2,
or will you send a follow-up patch?
I can
On Sat, Aug 01, 2015 at 10:14:39PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:38PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:35PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:32PM +0900, Masahiro Yamada wrote:
Commit e02ee2548afe (kconfig: switch to single .config
configuration) made the configuration itself pretty simple,
instead, we lost the way to systematically enable/disable config
options for each image independently.
Our
On Sat, Aug 01, 2015 at 10:14:34PM +0900, Masahiro Yamada wrote:
Just preparing for upcoming cleaning.
The board-specific linker script board/vpac270/u-boot-spl.lds
has been touched to avoid build error. It does not change the
size of spl/u-boot-spl.bin for this board, so it should be OK.
On Sat, Aug 01, 2015 at 10:14:40PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:41PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:43PM +0900, Masahiro Yamada wrote:
There is no case where defined(SPL_DISABLE_OF_CONTROL) is true.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
On Sat, Aug 01, 2015 at 10:14:42PM +0900, Masahiro Yamada wrote:
We do not want to compile the DM remove code for SPL. Currently,
we undef it in include/config_uncmd_spl.h (for C files) and in
scripts/Makefile.uncmd_spl (for Makefiles). This is really ugly.
This commit demonstrates how we
On Sat, Aug 01, 2015 at 10:14:37PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
On Sat, Aug 01, 2015 at 10:14:36PM +0900, Masahiro Yamada wrote:
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
Reviewed-by: Tom Rini tr...@konsulko.com
--
Tom
signature.asc
Description: Digital signature
___
U-Boot mailing list
Hi Bin,
Looks like I missed some items..
Anyway, they do not have many drivers below them,
so menuconfig is not so messy.
Do you want me send v2,
or will you send a follow-up patch?
2015-08-01 20:11 GMT+09:00 Bin Meng bmeng...@gmail.com:
Hi Masahiro,
On Sun, Jul 26, 2015 at 1:46 AM,
On Sat, Aug 01, 2015 at 10:14:44PM +0900, Masahiro Yamada wrote:
As we discussed a couple of times, negative CONFIG options make our
life difficult; CONFIG_SYS_NO_FLASH, CONFIG_SYS_DCACHE_OFF, ...
and here is another one.
Now, there are actually two boards enabling OF_CONTROL on SPL:
-
Le samedi 01 août 2015 à 17:18 +0200, Marek Vasut a écrit :
On Saturday, August 01, 2015 at 11:08:10 AM, Paul Kocialkowski wrote:
Hi,
Le lundi 27 juillet 2015 à 14:33 +0200, Paul Kocialkowski a écrit :
Le dimanche 26 juillet 2015 à 02:46 +0900, Masahiro Yamada a écrit :
The
On Saturday, August 01, 2015 at 08:32:07 PM, Scott Wood wrote:
On Sat, 2015-08-01 at 17:18 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 07:56:39 AM, Peng Fan wrote:
On Fri, Jul 31, 2015 at 09:36:45PM -0500, Scott Wood wrote:
On Sat, 2015-08-01 at 09:15 +0800, Peng Fan wrote:
Do not add a bogus (pointing to a non existing serial port) stdout-path
alias to dts on boards without a serial port.
Signed-off-by: Hans de Goede hdego...@redhat.com
---
include/configs/sunxi-common.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/configs/sunxi-common.h
USB devices are not really designed to get the power bounced off and on
at them. Esp. USB powered harddisks do not like this.
Currently we power off the USB ports both on a usb reset and when
booting the kernel, causing the usb-power to bounce off and then back
on again.
This patch removes the
Now that we have code to check the id-pin and detect usb-host adapters
plugged into the otg port that way, enable it on the tablets which I own.
Signed-off-by: Hans de Goede hdego...@redhat.com
---
configs/Chuwi_V7_CW0825_defconfig | 1 +
configs/UTOO_P66_defconfig| 1 +
2 files changed,
On Sat, 2015-08-01 at 17:18 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 07:56:39 AM, Peng Fan wrote:
On Fri, Jul 31, 2015 at 09:36:45PM -0500, Scott Wood wrote:
On Sat, 2015-08-01 at 09:15 +0800, Peng Fan wrote:
On Fri, Jul 31, 2015 at 12:07:50PM -0500, Scott Wood wrote:
LS2085 targets supports following UART console
LS2085AQDS UART0
LS2085ARDB UART1
LS2085ASim UART0
LS2085AEmu UART0
So update the bootargs as per the default console present at the target
Signed-off-by: Prabhakar Kushwaha prabha...@freescale.com
---
On Sat, Aug 01, 2015 at 01:54:48PM -0500, Scott Wood wrote:
On Sat, 2015-08-01 at 20:38 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 08:32:07 PM, Scott Wood wrote:
On Sat, 2015-08-01 at 17:18 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 07:56:39 AM, Peng Fan wrote:
On Sat, 2015-08-01 at 20:38 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 08:32:07 PM, Scott Wood wrote:
On Sat, 2015-08-01 at 17:18 +0200, Marek Vasut wrote:
On Saturday, August 01, 2015 at 07:56:39 AM, Peng Fan wrote:
On Fri, Jul 31, 2015 at 09:36:45PM -0500, Scott Wood
Along with SOURCE_DATE_EPOCH SOURCE_DATE_TZ can be used to recreate a
build with a specific date timestamp. This allows the verification of
source supplied with a pre-compiled binary.
If SOURCE_DATE_EPOCH is supplied SOURCE_DATE_TZ can be used to specify
what will appear in the output of the
Hi,
On 23-07-15 14:33, Piotr Zierhoffer wrote:
This is a basic driver for the sunxi NAND controller for Allwinner A20.
It supports only SPL.
The driver uses DMA for data transfers. It does not support writing.
Thanks for your work on this.
I've merged patches 1 - 3 in my sunxi-wip branch,
When building with SOURCE_DATE_EPOCH the timezone is in UTC. When
building normally the timezone is taken from the build machine's locale
setting.
Signed-off-by: Chris Packham judge.pack...@gmail.com
Tested-by: Bin Meng bmeng...@gmail.com
Tested-by: Paul Kocialkowski cont...@paulk.fr
---
Changes
Hi Hans,
On Sat, 1 Aug 2015 14:15:53 +0200
Hans de Goede hdego...@redhat.com wrote:
Hi,
On 23-07-15 14:33, Piotr Zierhoffer wrote:
This is a basic driver for the sunxi NAND controller for Allwinner A20.
It supports only SPL.
The driver uses DMA for data transfers. It does not support
Commit e02ee2548afe (kconfig: switch to single .config
configuration) made the configuration itself pretty simple,
instead, we lost the way to systematically enable/disable config
options for each image independently.
Our current strategy is, put entries into Makefile.spl for options
we need
If the target string matches CONFIG_, move the pointer p
forward. This saves several 7-chars adjustments.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
scripts/basic/fixdep.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git
The previous commit introduced a useful macro used in makefiles,
in order to reference to different variables (CONFIG_... or
CONFIG_SPL_...) depending on the build context.
Per-image config option control is a PITA in C sources, too.
Here are some macros useful in C/CPP expressions.
Signed-off-by: Masahiro Yamada yamada.masah...@socionext.com
---
Changes in v2: None
drivers/Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/Makefile b/drivers/Makefile
index 2515aab..dd57849 100644
--- a/drivers/Makefile
+++ b/drivers/Makefile
@@ -1,7
On Saturday, August 01, 2015 at 11:08:10 AM, Paul Kocialkowski wrote:
Hi,
Le lundi 27 juillet 2015 à 14:33 +0200, Paul Kocialkowski a écrit :
Le dimanche 26 juillet 2015 à 02:46 +0900, Masahiro Yamada a écrit :
The menuconfig for drivers are getting more and more cluttered
and
On Saturday, August 01, 2015 at 07:56:39 AM, Peng Fan wrote:
On Fri, Jul 31, 2015 at 09:36:45PM -0500, Scott Wood wrote:
On Sat, 2015-08-01 at 09:15 +0800, Peng Fan wrote:
On Fri, Jul 31, 2015 at 12:07:50PM -0500, Scott Wood wrote:
On Tue, 2015-07-21 at 16:15 +0800, Peng Fan wrote:
If
On Sat, 2015-08-01 at 14:30 +0200, Boris Brezillon wrote:
Hi Hans,
On Sat, 1 Aug 2015 14:15:53 +0200
Hans de Goede hdego...@redhat.com wrote:
Hi,
On 23-07-15 14:33, Piotr Zierhoffer wrote:
This is a basic driver for the sunxi NAND controller for Allwinner A20.
It supports only
Clean the function up so that it's obvious what it is doing,
fix the formating strings in debug outputs, add kerneldoc.
Make the function return proper errno-compliant return values
and propagate this change throughout sdram.c
Signed-off-by: Marek Vasut ma...@denx.de
---
Zap an obscure unneeded cast and clean other minor nits in this function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index
Originally, both sdram_start and sdram_end were 64b values. The
sdram_start had no reason for being so, since our address space
is only 32b, so switching sdram_start to u32 is simple.
The sdram_end is a bit more complex, since it can actually be
set to (1 32) if someone really wanted to use an
Clean up coding style, mostly clean up comments, add kerneldoc.
Also, zap assignment of the cs variable, which is outright
dead code, so just remove it.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 35 ---
1 file changed, 12
Merge sdr_set_*() functions which are just setting registers among
the sea of register setting in sdram_mmr_init_full(). There is no
need to keep them separate this way, there is nothing special about
them.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 98
Introduce socfpga_sdram_get_config() function implement in a board file,
which returns the socfpga_sdram_config structure. This is the last step
in cleaning up the socfpga_mmr_init_full(), but not the last step which
allows removing the inclusion of sdram.h from drivers/ddr/altera/sdram.c
thus
Fix the return value so that standard errno return values can be used.
Signed-off-by: Marek Vasut ma...@denx.de
---
arch/arm/mach-socfpga/include/mach/sdram.h | 2 +-
drivers/ddr/altera/sdram.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git
Rework remaining two register setting functions such that they only
return the final register value. Move the register setting into the
block of register I/O in sdram_mmr_init_full().
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 21 +++--
1 file
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git
Introduce this seemingly massive structure, which holds required values
of all the registers of the SDRAM controller. The idea here is to avoid
including the sdram.h header file, which is full of ad-hoc macros that
polute the global namespace. Once the cleanup of sdram.c would be complete
and all
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 37 -
1 file changed, 16 insertions(+), 21
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 30 ++
1 file changed, 10 insertions(+), 20
Pluck out the remaining CONFIG_HPS_SDR_CTRLCFG_ and put it into
the socfpga_sdram_config structure.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 67 +++---
1 file changed, 39 insertions(+), 28 deletions(-)
diff --git
Pull out the block of register programming into a separate function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 31 +--
1 file changed, 21 insertions(+), 10 deletions(-)
diff --git a/drivers/ddr/altera/sdram.c
Add kerneldoc.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index 2377b45..295747b 100644
--- a/drivers/ddr/altera/sdram.c
+++
Zap all the ad-hoc readbacks from the registers and other useless
and broken debug output. This is really not useful and is only
confusing.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 44
1 file changed, 44 deletions(-)
Suck out all the CONFIG_HPS_SDR_CTRLCFG_* from sdram_mmr_init_full()
into the socfpga_sdram_config structure. There is still one ugly
macro left behind, but this will be taken care of in subsequent patch.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 150
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 37 -
1 file changed, 16 insertions(+), 21
Fix the data types and zap unnecessary type change.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index da7f27f..22cad88 100644
---
Pluck out all of the CONFIG_HPS_SDR_CTRLCFG_* macros. This change
makes sdram.c completely clear of these macros and allows removing
of the ugly include of sdram.h . The namespace is now a much nicer
place!
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 19
Add kerneldoc to functions which are missing it, but are already
cleaned up.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 12
1 file changed, 12 insertions(+)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index fe2e753..e16f116
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 24 +++-
1 file changed, 11 insertions(+), 13 deletions(-)
Use generic function instead of CPU-specific one.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
index 474df42..9e6acfe 100644
---
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Merge set_sdr_dram_timing{1,2,3,4,lowpwr}() into single function
set_sdr_dram_timing() , since there's no point in keeping all this
stuff separate anymore.
Clean up weird parenthesis and odd type casts from the function.
Fix comment style.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/ddr/altera/sdram.c b/drivers/ddr/altera/sdram.c
Remove all parameters of this function, since they are only constants
passed in from another function, so make them local. Also, rename the
function to get_errata_rows() as this is closer to what it does.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 35
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
The big plan here is to remove all the CONFIG_HPS_SDR_ macros, hide
them in QTS compatibility layer in board implementation and pass only
a small structure
This entire series focuses solely on cleaning up the drivers/ddr/altera/sdram.c
file. After this series, this one file is totally checkpatch clean and does not
pull in any weird qts-generated macros ; all that is wrapped in the board file.
This micro-series applies on top of my previous
Get rid of the constant clrsetbits_le32(), instead prepare the whole
content of the register once and write it at the end of the function.
Signed-off-by: Marek Vasut ma...@denx.de
---
drivers/ddr/altera/sdram.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git
Le samedi 01 août 2015 à 22:32 +1200, Chris Packham a écrit :
Along with SOURCE_DATE_EPOCH SOURCE_DATE_TZ can be used to recreate a
build with a specific date timestamp. This allows the verification of
source supplied with a pre-compiled binary.
If SOURCE_DATE_EPOCH is supplied
Since commit a26cd04920dc (arch: Make board selection choices
optional), Kconfig could create such an insane .config file that
no SoC/board is selected.
This is now a real problem for Buildroot, for example.
(http://lists.busybox.net/pipermail/buildroot/2015-July/135125.html)
This commit drops
One disadvantage of commit a26cd04920dc (arch: Make board selection
choices optional) is that Kconfig could create such an insane
.config file that no board is selected.
Rip off the optional again in favor of Zedboard as the default
target.
Signed-off-by: Masahiro Yamada
Hi Alexey,
2015-07-30 2:32 GMT+09:00 Alexey Brodkin alexey.brod...@synopsys.com:
Hi Joe, Masahiro-san,
On Wed, 2015-07-29 at 20:25 +0300, Alexey Brodkin wrote:
Hi Joe,
On Wed, 2015-05-13 at 10:11 +0900, Masahiro Yamada wrote:
Hi Joe,
2015-05-13 7:12 GMT+09:00 Tom Rini
On 07/28/2015 07:55 AM, Tom Rini wrote:
On Mon, Jul 27, 2015 at 09:10:32PM -0600, Stephen Warren wrote:
On 07/24/2015 07:44 AM, Tom Rini wrote:
On Thu, Jul 23, 2015 at 10:17:29PM -0600, Stephen Warren
wrote:
On 07/14/2015 09:44 AM, Simon Glass wrote:
Hi Stephen,
On 13 July 2015 at 22:52,
On 07/28/2015 09:44 AM, Simon Glass wrote:
On 28 July 2015 at 07:55, Tom Rini tr...@konsulko.com wrote:
...
Maybe examples like these are why we will need (and want) to keep
platform data as a possibility in our DM work. There's value in keeping
the DT that we use as minimal as possible (so
Hi,
Le lundi 27 juillet 2015 à 14:33 +0200, Paul Kocialkowski a écrit :
Le dimanche 26 juillet 2015 à 02:46 +0900, Masahiro Yamada a écrit :
The menuconfig for drivers are getting more and more cluttered
and unreadable because too many entries are displayed in a single
flat menu. Use
On Sat, Aug 1, 2015 at 12:06 AM, Simon Glass s...@chromium.org wrote:
Hi Chris,
On 30 July 2015 at 23:34, Chris Packham judge.pack...@gmail.com wrote:
Hi Simon,
On Fri, Jul 31, 2015 at 7:47 AM, Simon Glass s...@chromium.org wrote:
Commit 488d19c (patman: add distutils based installer) has
(note I had some patman config that erroneously included extra people
on the original Cc list, I've now dropped them)
On Sat, Aug 1, 2015 at 5:05 AM, Paul Kocialkowski cont...@paulk.fr wrote:
Le vendredi 31 juillet 2015 à 22:04 +1200, Chris Packham a écrit :
When building with SOURCE_DATE_EPOCH
96 matches
Mail list logo
