On 23/03/18 18:14, Jagan Teki wrote:
> On Wed, Mar 14, 2018 at 7:27 AM, Andre Przywara
> wrote:
>> Update the .dts files for the various boards with an Allwinner A64 SoC.
>> This is as of v4.15-rc9, exactly Linux commit:
>>
>> {
>> pinctrl-names =
Signed-off-by: Petr Vorel
---
README| 6 --
configs/am335x_igep003x_defconfig | 1 +
configs/igep0032_defconfig| 1 +
configs/igep00x0_defconfig| 1 +
drivers/mtd/ubi/Kconfig | 7 +++
env/Kconfig
Introduce another difference from upstream (kernel) source in
fs/ubifs/super.c: adding preprocessor condition as y variable in
mount_ubifs() depends on CONFIG_UBIFS_SILENCE_MSG:
fs/ubifs/super.c:1337:15: error: variable ?y? set but not used
[-Werror=unused-but-set-variable]
long long x, y;
Not
Use of CONFIG_UBIFS_SILENCE_MSG was added in
147162dac6 ("ubi: ubifs: Turn off verbose prints")
Then it was removed in
ff94bc40af ("mtd, ubi, ubifs: resync with Linux-3.14")
Cc: Joe Hershberger
Cc: Heiko Schocher
Signed-off-by: Petr Vorel
Hello Heiko,
thanks for your review. I fixed issues:
https://travis-ci.org/pevik/u-boot/builds/357634133
Sorry, next time I'll test it in travis before posting to ML.
Changes v2->v3:
* Fix build error for sun5i, see
https://travis-ci.org/pevik/u-boot/builds/357634133
* Drop wrong dependency on
On 03/23/2018 03:29 PM, Simon Glass wrote:
> Hi Heinrich,
>
> On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
>> All initialization routines should return a status code instead of
>> a boolean.
>>
>> Signed-off-by: Heinrich Schuchardt
>> ---
>>
According to the documentation the EFI log category is called LOGC_EFI.
All other categories start with LOGC_. So let's fix it.
Fixes: 1973b381a1b3 ("log: add category LOGC_EFI")
Signed-off-by: Heinrich Schuchardt
---
include/log.h | 2 +-
1 file changed, 1 insertion(+), 1
On 03/23/2018 03:30 PM, Simon Glass wrote:
> Hi Heinrich,
>
> On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
>> The unit test checks if a device tree is installed. It requires that the
>> 'compatible' property of the root node exists. If available it prints the
>>
Hello Petr,
Am 19.03.2018 um 11:08 schrieb Petr Vorel:
Signed-off-by: Petr Vorel
Cc: Joe Hershberger
Cc: Heiko Schocher
---
README| 6 --
configs/am335x_igep003x_defconfig | 1 +
On 03/23/2018 08:01 PM, Heinrich Schuchardt wrote:
>>From 689ada7663efae5ef13d021f3266e081d1d53293 Mon Sep 17 00:00:00 2001
> From: Patrick Wildt
> Date: Fri, 23 Mar 2018 15:38:48 +0100
> Subject: [PATCH 2/2] efi_loader: set the dhcp ack received flag
>
> The PXE object
Hello Tom,
please pull from u-boot-i2c master:
The following changes since commit b0af10443afcb9e0dcfe18a7b8a013b230df9e39:
Merge git://git.denx.de/u-boot-net (2018-03-22 16:35:43 -0400)
are available in the Git repository at:
git://git.denx.de/u-boot-i2c.git master
for you to fetch
Hello Heinrich,
Am 18.03.2018 um 11:14 schrieb Heinrich Schuchardt:
In
lpi2c_status_t result = A;
result = B;
the first assignment has no effect. Let's remove it.
Signed-off-by: Heinrich Schuchardt
---
drivers/i2c/imx_lpi2c.c | 10 +-
1 file
On 03/23/2018 07:54 PM, Heinrich Schuchardt wrote:
>>From c38a011f699cf5a41b48c6fc547f3d0dde3d7cf9 Mon Sep 17 00:00:00 2001
> From: Patrick Wildt
> Date: Fri, 23 Mar 2018 15:36:58 +0100
> Subject: [PATCH 1/2] efi_loader: initialize ethernet device path fully
>
> Since the
On Wed, Mar 14, 2018 at 7:27 AM, Andre Przywara wrote:
> Update the .dts files for the various boards with an Allwinner A64 SoC.
> This is as of v4.15-rc9, exactly Linux commit:
> commit bdfe4cebea11476d278b1b98dd0f7cdac8269d62
> Author: Icenowy Zheng
>
On Mon, 19 Mar 2018 21:24:50 + (UTC)
Duncan Hare wrote:
Joe
My approach with the bootp options was a submenu in the net/Kconfig
file. coupled with imply directives as suggested in
defaults_distro_config.
However:
The "if NET" directive in net/kconfig precludes
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Also fix bad accents in my name.
>
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/phy/bcm6318-usbh-phy.c | 10 +++---
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
Reviewed-by: Daniel Schwierzeck
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Also fix bad accents in my name.
>
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/phy/bcm6358-usbh-phy.c | 10 +++---
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
Reviewed-by: Daniel Schwierzeck
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Also fix bad accents in my name.
>
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/phy/bcm6368-usbh-phy.c | 9 +++--
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
Reviewed-by: Daniel Schwierzeck
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Also fix bad accents in my name.
not sure why but some of the downloaded patches from Patchwork seemed to
be messed up. I also had to fix the author field in the commit message
when I applied your patches.
>
> Signed-off-by: Álvaro
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/cpu/bmips_cpu.c | 12 +++-
> 1 file changed, 3 insertions(+), 9 deletions(-)
>
Reviewed-by: Daniel Schwierzeck
--
- Daniel
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
> drivers/ram/bmips_ram.c |
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
>
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck and Simon Glass:
> - Use generic dev_remap_addr function.
> - Fix
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
>
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
> drivers/led/led_bcm6328.c
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
> drivers/led/led_bcm6358.c | 44
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr_index function.
>
>
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
>
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
> drivers/clk/clk_bcm6345.c
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> Reviewed-by: Simon Glass
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
>
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> ---
> v2: Introduce changes suggested by Daniel Schwierzeck:
> - Use generic dev_remap_addr function.
>
> drivers/watchdog/bcm6345_wdt.c | 8 ++--
> 1 file changed, 2
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/core/fdtaddr.c | 12
> drivers/core/read.c| 12
> include/dm/fdtaddr.h | 22 ++
> include/dm/read.h | 32
On 23.03.2018 18:15, Álvaro Fernández Rojas wrote:
> Hi Daniel,
>
>
> El 23/03/2018 a las 15:34, Daniel Schwierzeck escribió:
>>
>> On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
>>> Signed-off-by: Álvaro Fernández Rojas
>>> ---
>>> drivers/core/fdtaddr.c | 12
On Fri, Mar 23, 2018 at 3:25 AM, Sjoerd Simons
wrote:
> The ti,pindir-d0-out-d1-in property is not expected to have a value
> according to the device-tree binding, so treat it as a boolean not a
> uint property.
>
> Signed-off-by: Sjoerd Simons
Hi Daniel,
El 23/03/2018 a las 15:34, Daniel Schwierzeck escribió:
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
Signed-off-by: Álvaro Fernández Rojas
---
drivers/core/fdtaddr.c | 12
drivers/core/read.c| 12
include/dm/fdtaddr.h |
On Thu, Mar 22, 2018 at 9:46 AM, Bin Meng wrote:
> Hi,
>
> On Fri, Feb 2, 2018 at 9:53 PM, Stefan Mavrodiev wrote:
>> CONFIG_PHY_ADDR is used for old-style configuration. This makes
>> impossible changing the PHY address, if multiple boards share a same
>>
Hi Masahiro,
On 23 March 2018 at 09:57, Masahiro Yamada
wrote:
> Hi Simon,
>
>
> 2018-03-23 23:31 GMT+09:00 Simon Glass :
>> Hi Masahiro,
>>
>> On 21 March 2018 at 03:03, Masahiro Yamada
>> wrote:
>>> fdt_region
Hi Eugeniy,
Please populate commit message with brief explanation of what
kind of functionality is added by that change and why it is
needed in the first place.
On Fri, 2018-03-23 at 15:35 +0300, Eugeniy Paltsev wrote:
> Signed-off-by: Eugeniy Paltsev
> ---
>
Hi,
I finally managed to track down the parallel make issue in helloworld.efi:
(previous posting at
https://www.mail-archive.com/u-boot@lists.denx.de/msg268726.html):
make[2]: *** No rule to make target 'lib/efi_loader/helloworld.efi', needed by
'__build'. Stop.
The problematic make rule
On Fri, 23 Mar 2018 16:30:56 +0100
Stefan Agner wrote:
> On 03.03.2018 23:02, Lukasz Majewski wrote:
> > Hi Stefan,
> >
> >> On 02.03.2018 17:33, Lukasz Majewski wrote:
> >> > Hi Stefan,
> >> >
> >> >> From: Stefan Agner
> >> >>
> >> >> Make
Hi Simon,
2018-03-23 23:31 GMT+09:00 Simon Glass :
> Hi Masahiro,
>
> On 21 March 2018 at 03:03, Masahiro Yamada
> wrote:
>> fdt_region APIs are not part of libfdt. They are U-Boot extension
>> for the verified boot. Split the declartions
On 03.03.2018 23:02, Lukasz Majewski wrote:
> Hi Stefan,
>
>> On 02.03.2018 17:33, Lukasz Majewski wrote:
>> > Hi Stefan,
>> >
>> >> From: Stefan Agner
>> >>
>> >> Make sure we use objcopy from the cross compiler toolchain.
>> >>
>> >> Signed-off-by: Stefan Agner
Hi Miquel,
On 21 March 2018 at 07:49, Tom Rini wrote:
> On Tue, Mar 20, 2018 at 03:51:22PM +0100, Miquel Raynal wrote:
>> Hi Tom,
>>
>> On Tue, 20 Mar 2018 10:04:55 -0400, Tom Rini wrote:
>>
>> > On Tue, Mar 20, 2018 at 02:36:56PM +0100, Miquel Raynal
On 22.03.2018 19:39, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas
> ---
> drivers/core/fdtaddr.c | 12
> drivers/core/read.c| 12
> include/dm/fdtaddr.h | 22 ++
> include/dm/read.h | 32
Hi Tom,
please pull these changes to your tree.
Travis looks good.
https://travis-ci.org/michalsimek/u-boot/builds/357294420
And buildman output too.
Thanks,
Michal
The following changes since commit b0af10443afcb9e0dcfe18a7b8a013b230df9e39:
Merge git://git.denx.de/u-boot-net (2018-03-22
On 21 March 2018 at 03:03, Masahiro Yamada
wrote:
> This header needs to know 'fdt_region' is a struct for the
> fit_region_make_list() prototype.
>
> Signed-off-by: Masahiro Yamada
> ---
>
> include/image.h | 1 +
> 1 file changed,
Hi Masahiro,
On 21 March 2018 at 03:03, Masahiro Yamada
wrote:
> fdt_region APIs are not part of libfdt. They are U-Boot extension
> for the verified boot. Split the declartions related to fdt_region
> out ot . This allows to become a
> simple wrapper file,
Hi Masahiro,
On 21 March 2018 at 03:03, Masahiro Yamada
wrote:
> My goal is to sync lib/libfdt/ with scripts/dtc/libfdt/, that is,
> make lib/libfdt/ contain only wrapper files.
>
> fdt_region.c was written only for U-Boot to implement the verified
> boot. So,
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> The second argument of the bootefi command should always be usable to
> specify a device tree. This was missing for bootefi selftest and
> bootefi hello.
>
> Proper error handling is added.
>
> Signed-off-by: Heinrich
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> efi_init_obj_list() should be executed only once.
>
> Rather than having the caller check this variable and the callee set it,
> move all access to the variable inside the function. This reduces the
> logic needed to
On 20 March 2018 at 20:40, Chris Packham wrote:
> As of commit 1a1fa2406689 ("rtc: Set valid date after reset") the
> command "date reset" will set the date/time to 2000-01-01 0:00:00 after
> calling rtc_reset(). This means that the ds1307 implementation of
> rtc_reset()
Hi Heinrich,
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> The unit test checks if a device tree is installed. It requires that the
> 'compatible' property of the root node exists. If available it prints the
> 'serial-number' property.
>
> The serial-number
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> efi_reset_system_init provides the architecture or board specific
> initialization of the EFI subsystem. Errors should be caught and
> signalled by a return code.
>
> Signed-off-by: Heinrich Schuchardt
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> efi_watchdog_register() should always return a status code and not
> a boolean value.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> include/efi_loader.h |
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> Up to now errors in the initialization of the EFI subsystems was not
> checked.
>
> If any initialization fails, leave the bootefi command.
>
> We do not retry initialization because this would require to undo all
On 21 March 2018 at 03:03, Masahiro Yamada
wrote:
> This macro is locally referenced in common/image-fdt.c
>
> Signed-off-by: Masahiro Yamada
> ---
>
> common/image-fdt.c | 3 +++
> include/linux/libfdt.h | 3 ---
> 2 files
On 21 March 2018 at 03:03, Masahiro Yamada
wrote:
> fdt_region.c does not depend on anything in libfdt_internal.h
>
> Signed-off-by: Masahiro Yamada
> ---
>
> lib/libfdt/fdt_region.c | 2 --
> 1 file changed, 2 deletions(-)
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> All EFI initialization functions should return a status code.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> include/efi_loader.h | 2 +-
>
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> The return type of do_bootefi_exec() is efi_status_t. So in case
> of an error we should always return an EFI status code.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change,
Hi,
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> Consistently return status codes form efi_net_register().
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> include/efi_loader.h | 2 +-
>
Hi Heinrich,
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> All initialization routines should return a status code instead of
> a boolean.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> include/efi_loader.h
On 15 February 2018 at 00:31, Heinrich Schuchardt wrote:
> Errors may occur inside efi_smbios_register().
>
> - Return a status code.
> - Remove unused variables.
> - Use constants where applicable.
>
> Suggested-by: Simon Glass
> Signed-off-by: Heinrich
On Fri, Mar 23, 2018 at 01:38:48PM +, Alexey Brodkin wrote:
> Hi Tom,
>
> On Fri, 2018-03-23 at 09:33 -0400, Tom Rini wrote:
> > On Fri, Mar 23, 2018 at 01:23:53PM +, Alexey Brodkin wrote:
> >
> > > Hi Tom,
> > >
> > > This is the first batch of ARC changes for the next release.
> > >
Hi,
On 12 February 2018 at 00:05, Mario Six wrote:
> From: Mario Six
>
> To debug device tree issues involving 32- and 64-bit platforms, it is useful
> to
> have a generic 64-bit platform available.
>
> Add a version of the sandbox that uses 64-bit integers
On Wed, 21 Mar 2018 10:19:16 +0100
Stefan Roese wrote:
> I currently have no overview of the (ongoing ?) PHY support for
> A37xx in Linux. I know that we don't match the Linux upstream dts
> files in U-Boot currently. But we should strive to move to the
> "official" DT properties
Hi Tom,
On Fri, 2018-03-23 at 09:33 -0400, Tom Rini wrote:
> On Fri, Mar 23, 2018 at 01:23:53PM +, Alexey Brodkin wrote:
>
> > Hi Tom,
> >
> > This is the first batch of ARC changes for the next release.
> >
> > Here we:
> > 1. Significantly rework cache-related functionality.
> > In
On Fri, Mar 23, 2018 at 01:23:53PM +, Alexey Brodkin wrote:
> Hi Tom,
>
> This is the first batch of ARC changes for the next release.
>
> Here we:
> 1. Significantly rework cache-related functionality.
> In particular that fixes coherency problems in some corner-cases,
> allows us
Hi Tom,
This is the first batch of ARC changes for the next release.
Here we:
1. Significantly rework cache-related functionality.
In particular that fixes coherency problems in some corner-cases,
allows us to enable and disable caches in run-time and still
have properly running
On Fri, Mar 23, 2018 at 8:08 AM, Rasmus Villemoes
wrote:
> The u-boot-ivt.img.log file contains 0x prefixes in the HAB Blocks line,
> while the SPL.log does not. For consistency, and to make it easier to
> extract and put into a .csf file for use with NXP's code
[Adding Stefano and Tom]
On Fri, Mar 23, 2018 at 8:08 AM, Rasmus Villemoes
wrote:
> The current makefile logic disables creation of the
> SPL.log/u-boot-ivt.img.log etc. files when V=1 is given on the command
> line, the rationale presumably being that the user wants
Signed-off-by: Eugeniy Paltsev
---
arch/arc/dts/hsdk.dts | 56 +++
board/synopsys/hsdk/MAINTAINERS |4 +-
board/synopsys/hsdk/Makefile|2 +
board/synopsys/hsdk/clk-lib.c | 75 +++
board/synopsys/hsdk/clk-lib.h | 38 ++
Refactor GO and PREP subcommands implementation for a simpler
override in the boards platform code.
Signed-off-by: Eugeniy Paltsev
---
arch/arc/lib/bootm.c | 51 --
board/synopsys/axs10x/axs10x.c | 12 ++
Eugeniy Paltsev (2):
ARC: bootm: refactor GO and PREP subcommands implementation.
ARC: HSDK: add platform-specific commands
arch/arc/dts/hsdk.dts | 56 +++
arch/arc/lib/bootm.c| 51 +-
board/synopsys/axs10x/axs10x.c | 12 +
board/synopsys/hsdk/MAINTAINERS |4
The u-boot-ivt.img.log file contains 0x prefixes in the HAB Blocks line,
while the SPL.log does not. For consistency, and to make it easier to
extract and put into a .csf file for use with NXP's code signing tool,
add 0x prefixes here.
Signed-off-by: Rasmus Villemoes
The current makefile logic disables creation of the
SPL.log/u-boot-ivt.img.log etc. files when V=1 is given on the command
line, the rationale presumably being that the user wants and gets the
information on the console.
However, from general principles, I don't think a higher V= level
should
Hi,
Thanks for the report.
I am answering from my private email. At the moment I can't find any issue
regarding my patch. It should make no difference since dram_init_banksize in
quark/dram.c should set the dram bank. I will continue my investigation and
contact you if I find out something.
Hi,
On Fri, Feb 2, 2018 at 9:53 PM, Stefan Mavrodiev wrote:
> CONFIG_PHY_ADDR is used for old-style configuration. This makes
> impossible changing the PHY address, if multiple boards share a same
> config header file (for example include/configs/sunxi-common.h).
>
> Moving
于 2018年3月23日 GMT+08:00 下午5:39:25, Maxime Ripard 写到:
>On Fri, Mar 23, 2018 at 04:18:53PM +0800, Icenowy Zheng wrote:
>> Allwinner 64-bit SoCs can use 4GiB DRAM chip, however their memory
>map
>> has only allocated 3GiB for DRAM, so only 3GiB of the DRAM is
>>
On Fri, Mar 23, 2018 at 04:18:55PM +0800, Icenowy Zheng wrote:
> On newer Allwinner SoCs with the BROM start at 0x0 and the DRAM space at
> <0x4000 0xc000>, some parts of DRAM will be inaccessible when
> 4GiB module is used.
>
> Restrict the ram_size written to global_data in SPL.
>
>
On Fri, Mar 23, 2018 at 5:39 PM, Maxime Ripard
wrote:
> On Fri, Mar 23, 2018 at 04:18:53PM +0800, Icenowy Zheng wrote:
>> Allwinner 64-bit SoCs can use 4GiB DRAM chip, however their memory map
>> has only allocated 3GiB for DRAM, so only 3GiB of the DRAM is
>>
于 2018年3月23日 GMT+08:00 下午5:40:41, Maxime Ripard 写到:
>On Fri, Mar 23, 2018 at 04:18:56PM +0800, Icenowy Zheng wrote:
>> The get_ram_size() function in U-Boot can only deal with memory size
>> smaller than 2GiB. To enable the support of 3GiB DRAM on newer 64-bit
>>
On Fri, Mar 23, 2018 at 04:18:56PM +0800, Icenowy Zheng wrote:
> The get_ram_size() function in U-Boot can only deal with memory size
> smaller than 2GiB. To enable the support of 3GiB DRAM on newer 64-bit
> SoCs, an alternative way to detect DRAM size is needed.
Why not just fixing get_ram_size
On Fri, Mar 23, 2018 at 04:18:53PM +0800, Icenowy Zheng wrote:
> Allwinner 64-bit SoCs can use 4GiB DRAM chip, however their memory map
> has only allocated 3GiB for DRAM, so only 3GiB of the DRAM is
> accessible.
How would that work in Linux?
> Add a Kconfig option for the maximum accessible
Number of ECC status registers i.e. (ECCSTATx) has been increased in IFC
version 2.0.0 due to increase in SRAM size. This is causing eccstat
array to over flow.
So, replace eccstat array with u32 variable to make it fail-safe and
independent of number of ECC status registers or SRAM size.
The Allwinner A64 and H5 SoCs are both capable of use 3GiB capacity of
4GiB modules, and they both use the common DesignWare DRAM driver code.
Add size recalculation code for the driver.
Signed-off-by: Icenowy Zheng
---
New patch in v2.
arch/arm/mach-sunxi/Kconfig |
The get_ram_size() function in U-Boot can only deal with memory size
smaller than 2GiB. To enable the support of 3GiB DRAM on newer 64-bit
SoCs, an alternative way to detect DRAM size is needed.
Add the possibility to use some DRAM size recalculating code in DRAM
driver.
Signed-off-by: Icenowy
On newer Allwinner SoCs with the BROM start at 0x0 and the DRAM space at
<0x4000 0xc000>, some parts of DRAM will be inaccessible when
4GiB module is used.
Restrict the ram_size written to global_data in SPL.
Signed-off-by: Icenowy Zheng
---
No changes in v2.
As 4GiB capacity is above the range of 32-bit unsigned integer, raise
the return type of sunxi_dram_init() to unsigned long long, thus it can
hold 4GiB capacity (or maybe more on A80).
Some controllers that are possible to use 4GiB+ memory module are
also changed to calculate its memory capacity
Allwinner 64-bit SoCs can use 4GiB DRAM chip, however their memory map
has only allocated 3GiB for DRAM, so only 3GiB of the DRAM is
accessible.
Add a Kconfig option for the maximum accessible DRAM.
For A80 it should be a much higher value (8GiB), but as I have no A80
device to test and
Allwinner 64-bit SoCs have allocated 3GiB space in the memory map for
DRAM. If memory bigger than 3GiB is installed (as memory usually come as
pow of 2 and they are not known to support 3GiB LPDDR3 modules, it means
4GiB memory is installed), the whole 3GiB space can be all used.
However, in many
All Allwinner 64-bit SoCs now are known to be able to access 3GiB of
external DRAM, however the size of DRAM part in the MMU translation
table is still 2GiB.
Change the size of DRAM part in MMU table to 3GiB.
Signed-off-by: Icenowy Zheng
---
No changes in v2.
Hello Alexander,
Am 23.03.2018 um 08:50 schrieb Alexander Kochetkov:
23 марта 2018 г., в 10:35, Heiko Schocher написал(а):
Just triggered an travis build (not finsihed yet), and your patch drops
errors for arm926ejs boards:
> 23 марта 2018 г., в 10:35, Heiko Schocher написал(а):
>
> Just triggered an travis build (not finsihed yet), and your patch drops
> errors for arm926ejs boards:
>
> https://travis-ci.org/hsdenx/u-boot-i2c/jobs/357258790
>
> So, can you do a full travis build for your patch
Hello Alexander,
Am 23.03.2018 um 07:50 schrieb Heiko Schocher:
Hello Alexander,
Am 02.03.2018 um 14:37 schrieb Alexander Kochetkov:
The commit extract gpio description from device tree,
setup pins and toggle them until I2C slave device
release SDA.
Any comments? Ideas?
Could someone review
Hello Alexander,
Am 02.03.2018 um 14:37 schrieb Alexander Kochetkov:
The commit extract gpio description from device tree,
setup pins and toggle them until I2C slave device
release SDA.
Any comments? Ideas?
Could someone review the patch and tell that should
I do with it in order to bring the
95 matches
Mail list logo