> -Original Message-
> From: Lim, Elly Siew Chin
> Sent: Tuesday, November 10, 2020 2:45 PM
> To: u-boot@lists.denx.de
> Cc: Marek Vasut ; Tan, Ley Foon
> ; See, Chin Liang ;
> Simon Goldschmidt ; Chee, Tien Fong
> ; Westergreen, Dalon
> ; Simon Glass ; Gan,
> Yau Wai ; Lim, Elly Siew
> -Original Message-
> From: Lim, Elly Siew Chin
> Sent: Tuesday, November 10, 2020 2:44 PM
> To: u-boot@lists.denx.de
> Cc: Marek Vasut ; Tan, Ley Foon
> ; See, Chin Liang ;
> Simon Goldschmidt ; Chee, Tien Fong
> ; Westergreen, Dalon
> ; Simon Glass ; Gan,
> Yau Wai ; Lim, Elly Siew
On 02/11/2020 19.14, Moshe, Yaniv wrote:
> Hi Simon,
> Hope you're doing good.
>
> My name is Yaniv, and I have a question regarding an old commit in U-boot:
> commit 60e2809a848bccd3a8090d3f2237964670f2780c
> Author: Simon Glass mailto:s...@chromium.org>>
> Date: Tue Feb 17 15:29:35 2015 -0700
> -Original Message-
> From: Sean Anderson
> Sent: Thursday, November 19, 2020 9:01 AM
> To: Vabhav Sharma (OSS) ;
> s...@chromium.org; s...@denx.de
> Cc: u-boot@lists.denx.de; Varun Sethi ;
> andre.przyw...@arm.com; Vabhav Sharma
> Subject: Re: [PATCH v4 2/2] drivers: serial: probe
Am Sa., 21. Nov. 2020 um 13:22 Uhr schrieb Heinrich Schuchardt
:
>
> 0x05 is used as replacement letter for 0xe5 at the first position of short
> file names. We must not skip over directory entries starting with 0x05.
>
> Cf. Microsoft FAT Specification, August 30 2005
>
> Fixes: 39606d462c97
Hi Pragnesh,
On Mon, Nov 23, 2020 at 06:19:06AM +, Pragnesh Patel wrote:
> Hi Leo,
>
> >-Original Message-
> >From: Leo Liang
> >Sent: 23 November 2020 11:28
> >To: Pragnesh Patel
> >Cc: Rick Chen ; U-Boot Mailing List >b...@lists.denx.de>; Atish Patra ;
>
Hi Leo,
>-Original Message-
>From: Leo Liang
>Sent: 23 November 2020 11:28
>To: Pragnesh Patel
>Cc: Rick Chen ; U-Boot Mailing List b...@lists.denx.de>; Atish Patra ;
>palmerdabb...@google.com; Bin Meng ; Paul Walmsley
>( Sifive) ; Anup Patel ; Sagar
>Kadam ; Sean Anderson ;
>rick ;
On Tue, Nov 17, 2020 at 08:30:21AM +, Pragnesh Patel wrote:
Hi Pragnesh,
> Hi Rick,
>
> >-Original Message-
> >From: Rick Chen
> >Sent: 13 November 2020 13:37
> >To: Pragnesh Patel
> >Cc: U-Boot Mailing List ; Atish Patra
> >; palmerdabb...@google.com; Bin Meng
> >; Paul Walmsley
Hello Joel,
Am 22.11.20 um 02:18 schrieb Joel Peshkin:
> Adds an optional SALT value to AUTOBOOT_STOP_STR_SHA256. If a string
> followed by a ":" is prepended to the sha256, the portion to the left
> of the colon will be used as a salt and the password will be appended
> to the salt before the
Hello Andy,
Am 19.11.20 um 21:21 schrieb Andy Shevchenko:
> Update Tom's address
>
> On Thu, Nov 19, 2020 at 9:26 PM Andy Shevchenko
> wrote:
>>
>> It seems nobody tested the debug() option in spin_lock_irqsave().
>> Currently, when #define DEBUG, it spoils the compiler with
>>
>> In file
On 11/20/20 1:10 PM, Nishanth Menon wrote:
> power/regulator will not be built if just CONFIG_SPL_DM_REGULATOR is
> enabled. It needs CONFIG_SPL_POWER_SUPPORT to be enabled as well.
>
> For example, if we just need a GPIO regulator in SPL:
> CONFIG_DM_REGULATOR=y
> CONFIG_SPL_DM_REGULATOR=y
>
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to pci_init and use it directly in the post-relocation
> init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> common/board_r.c | 18
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to post_output_backlog and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> common/board_r.c | 10
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to pci_ep_init and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> common/board_r.c
Hi Yaniv,
On Mon, 2 Nov 2020 at 11:14, Moshe, Yaniv wrote:
>
> Hi Simon,
>
> Hope you’re doing good.
>
>
>
> My name is Yaniv, and I have a question regarding an old commit in U-boot:
>
> commit 60e2809a848bccd3a8090d3f2237964670f2780c
>
> Author: Simon Glass
>
> Date: Tue Feb 17 15:29:35
Hi Ilias,
On Sat, 21 Nov 2020 at 10:47, Ilias Apalodimas
wrote:
>
> The current tpm2_pcr_extend is hardcoded using SHA256.
> Let's make the actual command to the TPM2 configurable so we can support
> a wider range of algorithms and keep the current command line as-is i.e
> limited to SHA256 only
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to bb_miiphy_init and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> common/board_r.c
On Sat, 21 Nov 2020 at 05:22, Heinrich Schuchardt wrote:
>
> 0x05 is used as replacement letter for 0xe5 at the first position of short
> file names. We must not skip over directory entries starting with 0x05.
>
> Cf. Microsoft FAT Specification, August 30 2005
>
> Fixes: 39606d462c97 ("fs: fat:
On Sun, 22 Nov 2020 at 03:18, Heinrich Schuchardt wrote:
>
> Using constants instead of anonymous numbers increases code readability.
>
> Fixes: 704df6aa0a28 ("fs: fat: refactor write interface for a file offset")
> Signed-off-by: Heinrich Schuchardt
> ---
> fs/fat/fat_write.c | 3 ++-
> 1 file
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to api_init and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> api/api.c | 6 --
>
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to jumptable_init and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
> common/board_r.c | 8
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to noncached_init and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
>
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Currently, the following #ifdef construct is used to check whether to run
> console_record_init() during pre-relocation init:
> defined(CONFIG_CONSOLE_RECORD) && CONFIG_VAL(SYS_MALLOC_F_LEN)
>
> Introduce CONFIG_CONSOLE_RECORD_INIT_F Kconfig
On Sun, 22 Nov 2020 at 01:37, Ovidiu Panait wrote:
>
> Add a return value to cpu_secondary_init_r and use it directly in the
> post-relocation init sequence, rather than using a wrapper stub.
>
> Signed-off-by: Ovidiu Panait
> ---
> v2 updates:
> - add function comment
>
>
On Sat, 21 Nov 2020 at 18:19, Joel Peshkin wrote:
>
> Adds an optional SALT value to AUTOBOOT_STOP_STR_SHA256. If a string
> followed by a ":" is prepended to the sha256, the portion to the left
> of the colon will be used as a salt and the password will be appended
> to the salt before the
Em dom, 22 de nov de 2020 18:02, Campbell Suter
escreveu:
> SquashFS encodes directories with more than 256 entries, with xattrs or
> a few other things with a different type of inode than most directories.
>
> This type of directory entry has an indexing feature to increase the
> performance of
On 11/22/20 5:11 PM, Dario Binacchi wrote:
The __of_translate_address routine translates an address from the
device tree into a CPU physical address. A note in the description of
the routine explains that the crossing of any level with
#size-cells = <0> is to be considered an error not by
The series was born from the need to manage the PWM backlight of the
display connected to my beaglebone board. To hit the target, I had to
develop drivers for PWM management which in turn relied on drivers for
managing timers and clocks, all developed according to the driver model.
My intention
Check whether user has explicitly defined device and partition where
environment file will be located. Voids the need to set such partition
as bootable to work with the 'dev:auto'
Signed-off-by: Manuel Reis
Cc: Patrick Delaunay
Cc: Patrice Chotard
---
board/st/stm32mp1/stm32mp1.c | 5 +
The schedule for deprecating the features of the pre-driver-model puts
2019.17 as the deadline for the video subsystem. Furthermore, the latest
patches applied to the am335x-fb.c module have decreased the amount of
code shared with the pre-driver-model implementation. Splitting the two
With support for other clock drivers, the potentially supported CDCE913
device can no longer be probed without specifying its DT node name.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
board/ti/am335x/board.c | 2 +-
board/ti/am43xx/board.c | 2 +-
2 files changed, 2
Add drivers/video/ti/ folder and move all TI's code in this folder for
better maintenance.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
drivers/video/Kconfig | 5 +
drivers/video/Makefile| 4 +---
drivers/video/ti/Kconfig | 8
The previous version of am335x-fb.c contained the functionalities of two
drivers that this patch has split. It was a video type driver that used
the same registration compatible string that now registers a panel type
driver. The proof of this is that two compatible strings were referred
to within
The TI PWMSS driver is a simple bus driver for providing clock and power
management for the PWM peripherals on TI AM33xx SoCs, namely eCAP,
eHRPWM and eQEP.
For DT binding details see Linux doc:
- Documentation/devicetree/bindings/pwm/pwm-tipwmss.txt
Signed-off-by: Dario Binacchi
---
(no
Enhanced high resolution PWM module (EHRPWM) hardware can be used to
generate PWM output over 2 channels. This commit adds PWM driver support
for EHRPWM device present on AM33XX SOC.
The code is based on the drivers/pwm/pwm-tiehrpwm.c driver of the Linux
kernel version 5.9-rc7.
For DT binding
The patch adds a function to get display timings from the device tree
node attached to the device.
Signed-off-by: Dario Binacchi
Reviewed-by: Simon Glass
---
(no changes since v1)
arch/sandbox/dts/test.dts | 46 ++
drivers/core/read.c | 6 +++
include/dm/read.h
The patch configures the display DPLL using the functions provided by
the driver model API for the clock. The device tree contains everything
needed to get the DPLL clock. The round rate function developed for
calculating the DPLL multiplier and divisor and the platform routines
for accessing the
Enabling the domain clock is performed by the sysc interconnect target
module driver during the video device probing.
Signed-off-by: Dario Binacchi
---
(no changes since v3)
Changes in v3:
- Remove clock domain enabling/disabling.
- Update the commit message.
Adding the 'simple-bus' compatible string to the scm_clocks node will
allow its automatic binding.
Signed-off-by: Dario Binacchi
---
Changes in v6:
- Remove the 'am3-scm' driver.
- Add the 'simple-bus' compatible string to the scm_clocks node.
Changes in v4:
- Include device_compat.h header
The prescaler (PTV) setting must be taken into account even when the
timer input clock frequency has been set.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
drivers/timer/omap-timer.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
Dear Simon,
In message
you wrote:
>
> > "env import -t" does that, you know?
>
> Not to derail the discussion, but I mean as a way of specifying the
> default environment.
That was a build-time option, then. Sorry, I now remeber.
> OK I'll let you figure this out with Rasmus.
Hm...
Best
The __of_translate_address routine translates an address from the
device tree into a CPU physical address. A note in the description of
the routine explains that the crossing of any level with
#size-cells = <0> is to be considered an error not by specification but
since inherited from IBM. This
The patch moves the clk-ti-sci.c file to the 'ti' directory along with
all the other TI's drivers, and renames it clk-sci.c.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
drivers/clk/Kconfig| 8
drivers/clk/Makefile | 1 -
Adding the 'simple-bus' compatible string to the prcm_clocks node will
allow its automatic binding.
Signed-off-by: Dario Binacchi
---
Changes in v6:
- Remove the 'am3-prcm' driver.
- Add the 'simple-bus' compatible string to the prcm_clocks node.
Changes in v4:
- Include device_compat.h
Until now the clkctrl clocks have been enabled/disabled through platform
routines. Thanks to this patch they can be enabled and configured directly
by the probed devices that need to use them.
For DT binding details see Linux doc:
- Documentation/devicetree/bindings/clock/ti-clkctrl.txt
This minimal driver is only used to bind child devices.
For DT binding details see Linux doc:
- Documentation/devicetree/bindings/arm/omap/prcm.txt
Signed-off-by: Dario Binacchi
---
(no changes since v3)
Changes in v3:
- doc/device-tree-bindings/arm/omap,prcm.txt.
- Add to commit message the
Up till this commit passing NULL as input parameter was allowed, but not
handled properly. When a NULL parameter was passed to the function a data
abort was raised.
Signed-off-by: Dario Binacchi
Reviewed-by: Simon Glass
---
(no changes since v1)
arch/arm/mach-omap2/am33xx/clock.c | 10
Include the SoC U-boot DTS in each am335x--u-boot.dtsi.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
arch/arm/dts/am335x-chiliboard-u-boot.dtsi | 2 ++
arch/arm/dts/am335x-evm-u-boot.dtsi| 2 ++
arch/arm/dts/am335x-evmsk-u-boot.dtsi | 2 ++
There have been several changes to the am33xx.dtsi, so this patch
re-syncs it with Linux.
Let's add proper interconnect hierarchy for l4 interconnect instances
with the related ti-sysc interconnect module data as documented in
Documentation/devicetree/bindings/bus/ti-sysc.txt of the Linux kernel.
The patch adds support for TI divider clock binding. The driver uses
routines provided by the common clock framework (ccf).
The code is based on the drivers/clk/ti/divider.c driver of the Linux
kernel version 5.9-rc7.
For DT binding details see:
-
The patch adds support for TI gate clock binding. The code is based on
the drivers/clk/ti/gate.c driver of the Linux kernel version 5.9-rc7.
For DT binding details see:
- Documentation/devicetree/bindings/clock/ti/gate.txt
Signed-off-by: Dario Binacchi
---
(no changes since v5)
Changes in v5:
The digital phase-locked loop (DPLL) provides all interface clocks and
functional clocks to the processor of the AM33xx device. The AM33xx
device integrates five different DPLLs:
* Core DPLL
* Per DPLL
* LCD DPLL
* DDR DPLL
* MPU DPLL
The patch adds support for the compatible strings:
*
The driver manages a register-mapped multiplexer with multiple input
clock signals or parents, one of which can be selected as output. It
uses routines provided by the common clock framework (ccf).
The code is based on the drivers/clk/ti/mux.c driver of the Linux
kernel version 5.9-rc7.
For DT
Add missing DPLL_EN_FAST_RELOCK_BYPASS macro. Used to put the DPLL in
idle bypass fast relock mode.
Signed-off-by: Dario Binacchi
---
(no changes since v1)
arch/arm/include/asm/arch-am33xx/clock.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/include/asm/arch-am33xx/clock.h
It returns the rate which will be set if you ask clk_set_rate() to set
that rate. It provides a way to query exactly what rate you'll get if
you call clk_set_rate() with that same argument.
So essentially, clk_round_rate() and clk_set_rate() are equivalent
except the former does not modify the
We can handle the sysc interconnect target module in a generic way for
many TI SoCs. Initially let's just enable domain clocks before the
children are probed.
The code is loosely based on the drivers/bus/ti-sysc.c of the Linux
kernel version 5.9-rc7.
For DT binding details see:
-
Add support for PRUSS SYSC type:
The PRUSS module has a SYSCFG which is unique. The SYSCFG has two
additional unique fields called STANDBY_INIT and SUB_MWAIT in addition
to regular IDLE_MODE and STANDBY_MODE fields. Add the bindings for this
new sysc type.
Add support for MCAN on dra76x:
The
Export routines that can be used by other drivers avoiding duplicating
code.
Signed-off-by: Dario Binacchi
Reviewed-by: Simon Glass
---
(no changes since v2)
Changes in v2:
- Add the clk_ prefix to the divider functions.
- Add kernel-doc comments to the exported functions.
The following changes since commit 12e396303c487c9f0fdf8d36d31a97cd2dada643:
Merge tag 'efi-2021-01-rc3-2' of
https://gitlab.denx.de/u-boot/custodians/u-boot-efi (2020-11-21 08:04:39
-0500)
are available in the Git repository at:
git://git.denx.de/u-boot-usb.git master
for you to fetch
The following changes since commit 12e396303c487c9f0fdf8d36d31a97cd2dada643:
Merge tag 'efi-2021-01-rc3-2' of
https://gitlab.denx.de/u-boot/custodians/u-boot-efi (2020-11-21 08:04:39
-0500)
are available in the Git repository at:
git://git.denx.de/u-boot-sh.git master
for you to fetch
On 22.11.20 00:08, Simon Glass wrote:
> Hi Jan,
>
> On Sat, 21 Nov 2020 at 07:28, Jan Kiszka wrote:
>>
>> Hi,
>>
>> I stumbled over README.entries claiming
>>
>> Entry: u-boot-dtb: U-Boot device tree
>> -
>>
>> Properties / Entry arguments:
>> - filename:
On 11/22/20 2:10 PM, Ilias Apalodimas wrote:
commit 1fabfeef506c ("efi_loader: parameter check in GetNextVariableName()")
introduces a check using u16_strnlen(). This code is used on EFI
runtime variables as well, so unless we mark it as runtime, the kernel
will crash trying to access it.
commit 1fabfeef506c ("efi_loader: parameter check in GetNextVariableName()")
introduces a check using u16_strnlen(). This code is used on EFI
runtime variables as well, so unless we mark it as runtime, the kernel
will crash trying to access it.
Fixes: 1fabfeef506c ("efi_loader: parameter check in
SPI flash on this machine is located on bus 1, default to using bus 1
for SPI flash and stop aliasing it to bus 0. Formerly the alias spi1
pointed to , use an alias spi5 for this instead.
Signed-off-by: Hugh Cole-Baker
Suggested-by: Kever Yang
---
Changes in v2:
New patch.
SPI flash on this board is located on bus 1, default to using bus 1 for
SPI flash on both rk3399-roc-pc and -mezzanine, and stop aliasing it to
bus 0.
Signed-off-by: Hugh Cole-Baker
Suggested-by: Simon Glass
Fixes: c4cea2bb ("rockchip: Enable building a SPI ROM image on bob")
---
(no changes
SPI flash on this machine is located on bus 1, default to using bus 1
for SPI flash and stop aliasing it to bus 0.
Signed-off-by: Hugh Cole-Baker
Suggested-by: Simon Glass
Fixes: c4cea2bb ("rockchip: Enable building a SPI ROM image on bob")
---
(no changes since v1)
Commit c4cea2bbf995 ("rockchip: Enable building a SPI ROM image on bob")
added an alias spi1 referring to spi@ff1d, however in several boards
there was already an alias spi0 referring to the same node, and having
both aliases present broke booting U-Boot from SPI flash.
This series removes
There should be no need to insert non-printable characters
and this prevents line editing getting confused.
Signed-off-by: Steve Bennett
---
common/cli_readline.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/common/cli_readline.c b/common/cli_readline.c
index
Using constants instead of anonymous numbers increases code readability.
Fixes: 704df6aa0a28 ("fs: fat: refactor write interface for a file offset")
Signed-off-by: Heinrich Schuchardt
---
fs/fat/fat_write.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/fs/fat/fat_write.c
Add a return value to api_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
api/api.c | 6 --
api/api_private.h | 2 +-
common/board_r.c | 11 +--
CONFIG_HANDOFF is used in u-boot proper to locate handoff info from SPL
during pre-relocation init (in setup_spl_handoff). Add explicit dependency
on CONFIG_SPL, to fix the following build error when CONFIG_HANDOFF &&
!CONFIG_SPL:
common/board_f.c: In function ‘setup_spl_handoff’:
spl_handoff should only be enabled when CONFIG_HANDOFF is set. Drop the
nested ifdefs and check for CONFIG_HANDOFF instead.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
include/asm-generic/global_data.h | 4 ++--
1 file changed, 2 insertions(+),
In order to remove the arch-specific ifdefs around initr_trap, introduce
arch_initr_trap weak initcall. Implementations for ppc/m68k/mips have
been moved to arch//lib/traps.c
Default implementation is a nop stub.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add
Add a return value to bb_miiphy_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
common/board_r.c | 10 +-
drivers/net/phy/miiphybb.c | 4 +++-
Add a return value to noncached_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
arch/arm/include/asm/system.h | 13 -
arch/arm/lib/cache.c | 4 +++-
Add a return value to pci_ep_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
common/board_r.c | 11 +--
drivers/pci_endpoint/pci_ep-uclass.c | 4
Add a return value to xen_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
common/board_r.c | 10 +-
drivers/xen/hypervisor.c | 4 +++-
Add a return value to jumptable_init and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
common/board_r.c | 8 +---
common/exports.c | 4 +++-
include/exports.h | 10 --
Drop initf_console_record wrapper and call console_record_init directly.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- check defined(CONFIG_CONSOLE_RECORD_INIT_F) in ifdef condition
common/board_f.c | 12 +++-
1 file changed, 3 insertions(+), 9 deletions(-)
diff --git
Add a return value to post_output_backlog and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
common/board_r.c | 10 +-
include/post.h | 11 ++-
post/post.c | 4
Add a return value to pci_init and use it directly in the post-relocation
init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
common/board_r.c | 18 --
drivers/pci/pci-uclass.c | 4 +++-
drivers/pci/pci.c
Add a return value to cpu_secondary_init_r and use it directly in the
post-relocation init sequence, rather than using a wrapper stub.
Signed-off-by: Ovidiu Panait
---
v2 updates:
- add function comment
arch/powerpc/cpu/mpc85xx/cpu_init.c | 4 +++-
common/board_r.c| 17
Use IS_ENABLED(CONFIG_OF_EMBED) in instead of #ifdefs in reserve_fdt,
reloc_fdt functions.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
common/board_f.c | 41 +
1 file changed, 21 insertions(+), 20
Drop initr_console_record wrapper and call console_record_init directly.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
common/board_r.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/common/board_r.c
Use IS_ENABLED(CONFIG_TIMER_EARLY) instead of #ifdef in initf_dm. Also,
move timer code to the main ifdef, so that ret is defined.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
common/board_f.c | 11 ++-
1 file changed, 6 insertions(+), 5
setup_bdinfo is used to populate various bdinfo fields, so move
setup_machine code there, as all it does is setting
gd->bd->bi_arch_number.
Signed-off-by: Ovidiu Panait
Reviewed-by: Simon Glass
---
v2 updates:
- add reviewed-by tag
common/board_f.c | 13 -
1 file changed, 4
Currently, the following #ifdef construct is used to check whether to run
console_record_init() during pre-relocation init:
defined(CONFIG_CONSOLE_RECORD) && CONFIG_VAL(SYS_MALLOC_F_LEN)
Introduce CONFIG_CONSOLE_RECORD_INIT_F Kconfig option to get rid of the
complex ifdef check. Also, use
v2:
- Introduce CONFIG_CONSOLE_RECORD_INIT_F Kconfig to eliminate complex ifdef
around console_record_init in board_f.c
- Add function comments to all routines that get their signatures changed
- Add reviewed-by tags
* Use IS_ENABLED() instead of #ifdef where possible
* Add int return values to
88 matches
Mail list logo
