Knowing that at least one section header follows the optional header we
only need to check for the length of the 64bit header which is longer than
the 32bit header.
Signed-off-by: Heinrich Schuchardt
---
lib/efi_loader/efi_image_loader.c | 15 ++-
1 file changed, 6 insertions(+), 9
Rework the description of function cmp_pe_section().
Signed-off-by: Heinrich Schuchardt
---
lib/efi_loader/efi_image_loader.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/lib/efi_loader/efi_image_loader.c
b/lib/efi_loader/efi_image_loader.c
index
We want to follow the Linux kernel style for function descriptions.
Add missing parentheses after function names.
Signed-off-by: Heinrich Schuchardt
---
lib/efi_loader/efi_image_loader.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git
Hello Tom,
please pull from u-boot-i2c master
The following changes since commit ab80137cc436e977ef91a154372ae5aeae3f4fb0:
Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell (2020-05-27
10:56:25 -0400)
are available in the Git repository at:
Hello Heinrich,
Am 09.05.2020 um 18:20 schrieb Heinrich Schuchardt:
When compiling with -Wtype-limits we see this error:
drivers/i2c/i2c-uclass.c: In function ‘i2c_deblock_gpio_loop’:
drivers/i2c/i2c-uclass.c:517:21: error: comparison of
unsigned expression >= 0 is always true
Hello Eugen,
Am 07.05.2020 um 10:53 schrieb Eugen Hristev:
Because of this commit :
5ae84860b0 ("misc: i2c_eeprom: verify that the chip is functional at probe()")
at probe time, each eeprom is tested for read at offset 0.
The Atmel AT24MAC402 eeprom has different mapping. One i2c slave address
Hello Simon,
Am 22.04.2020 um 18:13 schrieb Simon Glass:
From: Raul E Rangel
If the device doesn't return a version that means the device is
non-functional.
The dw_i2c_regs had invalid offsets for the version field. I got the
correct value from the DesignWare databook. It also matches what
Hello Simon,
Am 22.04.2020 um 18:13 schrieb Simon Glass:
At present we still have pre-driver-model code in this driver and it makes
things a bit confusing. In particular calc_bus_speed() is called with priv
as NULL if not using driver model.
This results in spk_cnt and comp_param1 being read
This doesn't affect the size of the image: with
arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola GNU/Linux
distribution, the text, data, bss and total sizes remain
unchanged.
Signed-off-by: Denis 'GNUtoo' Carikli
Reviewed-by: Soeren Moch
---
Changelog since v4:
---
- Added the
This documents the u-boot installation procedure and the
hardware in order to get started.
Signed-off-by: Denis 'GNUtoo' Carikli
---
Changelog since v4:
---
- Added dded information about SYSBOOT boot support as requested.
- Fixed the switch positions as suggested.
---
As that the resulting image is already very close to the
size limit, and that CONFIG_GZIP is not strictly required,
removing it shouldn't hurt.
With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola
GNU/Linux distribution we have the following size
reduction:
- text: 9752
- data: 0
- bss: 16
-
This is needed to enable distro_boot support later on.
Adding the necessary configuration to enable booting general
purpose distributions is typically done by enabling
CONFIG_DISTRO_DEFAULTS.
However the resulting image size is is already very close to
the size limit and PXE isn't strictly
As that the resulting image is already very close to the
size limit, and that CONFIG_DHCP is not strictly required,
removing it shouldn't hurt.
With arm-linux-gnueabi-gcc 9.3.0-1 from the Parabola
GNU/Linux distribution, we have the following size
differences:
- text: -1708 bytes
- data: -28
This keeps the compatibility with the old bootcmd.
The fdtfile environment variable also needed to be set to
imx6q-tbs2910.dtb to enable booting mainline kernels
otherwise with extlinux.conf it tries to load
mx6-tbs2910.dtb instead.
With arm-linux-gnueabi-gcc 9.2.0-1 from the Parabola
GNU/Linux
The loadb and loads commands are not needed for booting.
There are also more reliable and faster alternatives to
loadb and loads that can be used with the current configuration.
As that the resulting image is already very close to the
size limit, removing the loadb and loads commands shouldn't
The iMX7D RM 9.2.4.9.3 Power removal flow Table 9-11. Re-enabling power
explicitly says both the DDR controller and the PHY must be reset in the
correct sequence. Currently the code only resets the controller. This
leads to a misbehavior where the system brings the DRAM up after reboot,
but the
Hi,
Patch [0] added devm_gpiod_get_index_optional() which would return NULL
when when no GPIO was assigned to the requested function. This is
convenient for drivers that need to handle optional GPIOs.
We need to take a stance on who is responsible for the NULL check: the
driver or the GPIO core?
Prepare the way for a managed GPIO API by handling NULL pointers without
crashing or failing. validate_desc() comes from Linux with the prints
removed to reduce code size.
Signed-off-by: Jean-Jacques Hiblot
Signed-off-by: Pratyush Yadav
---
drivers/gpio/Kconfig | 9
RNG module works fine on RockPro64 so let's enable it by default.
=> rng
: 77 08 dd 04 2c 4d b8 cf 25 07 29 2d c0 ce 28 3b w...,M..%.)-..(;
0010: d4 5a 38 68 b1 40 97 59 b8 a5 7c 42 f8 4c 38 28 .Z8h.@.Y..|B.L8(
0020: 25 20 0b 4b f0 1a d5 c1 e6 a2 c2 34 5a 5e 64 26 %
Hi,
This is a re-submission of Jean-Jacques' earlier work in October last
year. It can be found at [0]. The goal is to facilitate porting drivers
from the linux kernel. Most of the series will be about adding managed
API to existing infrastructure (GPIO, reset, regmap (already
submitted)).
This
From: Jean-Jacques Hiblot
Add managed functions to get a gpio from the devce-tree, based on a
property name (minus the '-gpios' suffix) and optionally an index.
When the device is unbound, the GPIO is automatically released and the
data structure is freed.
Signed-off-by: Jean-Jacques Hiblot
From: Jean-Jacques Hiblot
Add a test to verify that GPIOs can be acquired/released using the managed
API. Also check that the GPIOs are released when the consumer device is
removed.
Signed-off-by: Jean-Jacques Hiblot
---
arch/sandbox/dts/test.dts | 10
test/dm/gpio.c| 102
at:
>
> https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic.git
> tags/u-boot-amlogic-20200529
>
> for you to fetch changes up to 3678a5f0597d1cd491e3bded19ea72e570b5c8b1:
>
> arm: dts: meson-gxl: fix USB gadget by adding missing nodes for U-Boot
> (2020-05
Hi Walter,
On Fri, 29 May 2020 at 13:21, Walter Lozano wrote:
>
> Hi Simon,
>
> On 29/5/20 16:00, Simon Glass wrote:
> > Hi Walter,
> >
> > On Fri, 29 May 2020 at 12:56, Walter Lozano
> > wrote:
> >>
> >> On 29/5/20 15:15, Walter Lozano wrote:
> >>> Currently when creating an U_BOOT_DEVICE
Hi Simon,
On 29/5/20 16:00, Simon Glass wrote:
Hi Walter,
On Fri, 29 May 2020 at 12:56, Walter Lozano wrote:
On 29/5/20 15:15, Walter Lozano wrote:
Currently when creating an U_BOOT_DEVICE entry a struct driver_info
is declared, which contains the data needed to instantiate the device.
Hi Jagan
On 29/5/20 15:25, Jagan Teki wrote:
Hi Walter,
On Fri, May 29, 2020 at 11:45 PM Walter Lozano
wrote:
When using OF_PLATDATA dtbs are converted to C structs in order to save
space as we can remove both dtbs and libraries from TPL/SPL binaries.
This patchset tries to improve its
Hi Kurt,
On Fri, 29 May 2020 at 06:42, Kurt Miller wrote:
>
> On Fri, 2020-05-29 at 09:27 +0100, Peter Robinson wrote:
> > On Thu, May 28, 2020 at 8:32 PM Kurt Miller
> > wrote:
> > >
> > >
> > > The cooling levels are tuned to the fan that comes with the rockpro64 NAS
> > > case. A
Hi Walter,
On Fri, 29 May 2020 at 12:56, Walter Lozano wrote:
>
>
> On 29/5/20 15:15, Walter Lozano wrote:
> > Currently when creating an U_BOOT_DEVICE entry a struct driver_info
> > is declared, which contains the data needed to instantiate the device.
> > However, the actual device is created
On 29/5/20 15:15, Walter Lozano wrote:
Currently when creating an U_BOOT_DEVICE entry a struct driver_info
is declared, which contains the data needed to instantiate the device.
However, the actual device is created at runtime and there is no proper
way to get the device based on its struct
Hi Walter,
On Fri, May 29, 2020 at 11:45 PM Walter Lozano
wrote:
>
> When using OF_PLATDATA dtbs are converted to C structs in order to save
> space as we can remove both dtbs and libraries from TPL/SPL binaries.
>
> This patchset tries to improve its support by overcoming some limitations
> in
Add a test for dtoc taking into account the cd-gpios property.
Signed-off-by: Walter Lozano
---
tools/dtoc/dtoc_test_phandle_cd_gpios.dts | 42 ++
tools/dtoc/test_dtoc.py | 67 +++
2 files changed, 109 insertions(+)
create mode 100644
After using a new approach to link nodes when OF_PLATDATA is enabled
the test cases need to be update.
This patch updates the tests based on this new implementation.
Signed-off-by: Walter Lozano
---
tools/dtoc/test_dtoc.py | 95 +++--
1 file changed, 54
Currently dtoc does not support the property cd-gpios used to declare
the gpios for card detect in mmc.
This patch adds support to cd-gpios property.
Signed-off-by: Walter Lozano
---
tools/dtoc/dtb_platdata.py | 13 -
tools/dtoc/test_dtoc.py| 2 +-
2 files changed, 9
Currently when creating an U_BOOT_DEVICE entry a struct driver_info
is declared, which contains the data needed to instantiate the device.
However, the actual device is created at runtime and there is no proper
way to get the device based on its struct driver_info.
This patch extends struct
In the current implementation, when dtoc parses a dtb to generate a struct
platdata it converts the information related to linked nodes as pointers
to struct platdata of destination nodes. By doing this, it makes
difficult to get pointer to udevices created based on these
information.
This patch
Update documentation to reflect the new phandle support when OF_PLATDATA
is used. Now phandles are implemented as pointers to U_BOOT_DEVICE,
which makes it possible to get a pointer to the actual device.
Signed-off-by: Walter Lozano
---
doc/driver-model/of-plat.rst | 24
In order to prepare for a new support of phandle when OF_PLATDATA is used
drop the const for struct driver_info as this struct will need to be
updated on runtime.
Signed-off-by: Walter Lozano
---
drivers/core/device.c| 2 +-
drivers/core/root.c | 2 +-
As dtoc now performs checks for valid driver names, when running dtoc
tests several warnings arise as these tests don't use valid driver
names.
This patch adds an option to disable those warning, which is only
intended for running tests.
Signed-off-by: Walter Lozano
---
Update the documentation with the support for driver aliases using
U_BOOT_DRIVER_ALIAS.
Signed-off-by: Walter Lozano
---
doc/driver-model/of-plat.rst | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/doc/driver-model/of-plat.rst b/doc/driver-model/of-plat.rst
Currently dtoc scans dtbs to convert them to struct platdata and
to generate U_BOOT_DEVICE entries. These entries need to be filled
with the driver name, but at this moment the information used is the
compatible name present in the dtb. This causes that only nodes with
a compatible name that
When using OF_PLATDATA dtbs are converted to C structs in order to save
space as we can remove both dtbs and libraries from TPL/SPL binaries.
This patchset tries to improve its support by overcoming some limitations
in the current implementation
First, the support for scan and check for valid
This option marks any U-Boot variable which does not have explicit 'w'
writeable flag set as read-only. This way the environment can be locked
down and only variables explicitly configured to be writeable can ever
be changed by either 'env import', 'env set' or loading user environment
from
Add configuration option which prevents the environment hash table to be
ever cleared and reloaded with different content. This is useful in case
the first environment loaded into the hash table contains e.g. sensitive
content which must not be dropped or reloaded.
Signed-off-by: Marek Vasut
---
In case the env storage driver marks environment as ENV_INVALID, we must
reset the $ret return value to -ENOENT to let the env init code reset the
environment to the default one a bit further down.
Signed-off-by: Marek Vasut
---
env/env.c | 3 +++
1 file changed, 3 insertions(+)
diff --git
Add .load callback for the 'nowhere' environment driver. This is useful
for when the 'nowhere' driver is used in combination with other drivers
and should be used to load the default environment.
Signed-off-by: Marek Vasut
---
env/nowhere.c | 7 +++
1 file changed, 7 insertions(+)
diff
Add another internal environment flag which indicates that the operation
is resetting the environment to the default one. This allows the env code
to discern between import of external environment and reset to default.
Signed-off-by: Marek Vasut
---
env/common.c | 3 ++-
include/search.h |
Add support for U-Boot DM.
Signed-off-by: Benedikt Spranger
Reviewed-by: Kurt Kanzenbach
---
arch/arm/mach-sunxi/pmic_bus.c | 20
1 file changed, 20 insertions(+)
diff --git a/arch/arm/mach-sunxi/pmic_bus.c b/arch/arm/mach-sunxi/pmic_bus.c
index 091c59331b..8ecd037528
Add the compatible string for the Allwinner A10 i2c controller.
Signed-off-by: Benedikt Spranger
Reviewed-by: Kurt Kanzenbach
---
drivers/i2c/mvtwsi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/i2c/mvtwsi.c b/drivers/i2c/mvtwsi.c
index d3cc9b9d83..57cd442eee 100644
---
The TestBox board is an open hardware enhancement for the Lamobo R1 router
board. The Testbox board is used in the CI-RT project to manage devices
under test (https://ci-rt.linutronix.de).
The hardware project is located at https://github.com/ci-rt/testbox-shield
Check if the hardware is present
The Testbox board is an open hardware enhancement for the Lamobo R1 router
board. The Testbox board is used in the CI-RT project to manage devices
under test (https://ci-rt.linutronix.de).
Add support for the LX TestBox.
Benedikt Spranger (4):
sunxi: pmic_bus: Refactor pmic_bus_read() and
Unify I2C calls to ease the driver model conversion.
Signed-off-by: Benedikt Spranger
Reviewed-by: Kurt Kanzenbach
---
arch/arm/mach-sunxi/pmic_bus.c | 28
1 file changed, 16 insertions(+), 12 deletions(-)
diff --git a/arch/arm/mach-sunxi/pmic_bus.c
Add Kconfig entry for the PFUZE PMIC, SPL variant.
Signed-off-by: Marek Vasut
Cc: Fabio Estevam
Cc: Jaehoon Chung
Cc: Peng Fan
Cc: Stefano Babic
---
drivers/power/pmic/Kconfig | 7 +++
1 file changed, 7 insertions(+)
diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
This patch update the current gpio hog implementation with
the new API gpio_hog_probe_all() actviated with CONFIG_GPIO_HOG.
Signed-off-by: Patrick Delaunay
---
board/dhelectronics/dh_stm32mp1/board.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git
abootimg creates images where all load addresses are 0.
Android Boot Image Info:
* file name = artifacts/fastboot.img
* image size = 31381504 bytes (29.93 MB)
page size = 2048 bytes
* Boot Name = ""
* kernel size = 9397406 bytes (8.96 MB)
ramdisk size = 21981144 bytes (20.96 MB)
From: Matthias Brugger
With the new RPi4 which has 8 GB of RAM, we can have up to four DRAM
banks. Bump up the configuration files to detect all the memory in
U-Boot.
Signed-off-by: Matthias Brugger
---
configs/rpi_4_32b_defconfig | 2 +-
configs/rpi_4_defconfig | 2 +-
Hi,
OK thank you. I got it booting once but I'm actually not sure how.
It looks like that patch is in rk/master, but this is what I am getting:
U-Boot TPL 2020.07-rc3-00062-gfb14231483 (May 29 2020 - 08:27:15)
Channel 0: DDR3, 800MHz
BW=32 Col=10 Bk=8 CS0 Row=15 CS1 Row=15 CS=2 Die BW=16
On Fri, May 29, 2020 at 06:06:16PM +0530, Jagan Teki wrote:
> On Fri, May 29, 2020 at 2:14 AM Tom Rini wrote:
> >
> > On Thu, May 28, 2020 at 02:32:39PM -0600, Simon Glass wrote:
> > > Hi,
> > >
> > > On Thu, 28 May 2020 at 14:15, Tom Rini wrote:
> > > >
> > > > On Thu, May 28, 2020 at
Thanks for the work.
On Fri, May 29, 2020 at 4:01 PM Bhargav Shah wrote:
>
> Dropped DM supprot for Kirkwood
> Enable DM_SPI on all boards using Kirkwood
>
> Signed-off-by: Bhargav Shah
> Reviewed-by: Stefan Roese
>
> ---
> Changes for v2:
> - Added more meaningful commit message
> -
On Fri, May 29, 2020 at 2:05 PM Pragnesh Patel
wrote:
>
> Set ethernet clock rate to 125 Mhz so that it will work with 1000Mbps,
> Earlier this is done by FSBL. With this change We can remove the
> ethernet clock rate code from FSBL.
>
> Signed-off-by: Pragnesh Patel
> ---
>
On Fri, May 29, 2020 at 2:45 PM Pragnesh Patel
wrote:
>
> Add L2 cache node to enable all cache ways from U-Boot proper.
>
> Signed-off-by: Pragnesh Patel
> Reviewed-by: Bin Meng
> ---
> arch/riscv/cpu/fu540/Makefile | 1 +
> arch/riscv/cpu/fu540/cache.c | 53
On Fri, 2020-05-29 at 09:27 +0100, Peter Robinson wrote:
> On Thu, May 28, 2020 at 8:32 PM Kurt Miller
> wrote:
> >
> >
> > The cooling levels are tuned to the fan that comes with the rockpro64 NAS
> > case. A gpu_thermal zone was not added because having two active cooling
> > maps control
On Wed, May 27, 2020 at 7:40 PM Lars Poeschel wrote:
>
> On Wed, May 27, 2020 at 06:26:34PM +0530, Jagan Teki wrote:
> > OF_CONTROL, DM_SPI and other driver model migration deadlines
> > are expired for this board.
> >
> > Drop it.
>
> Acked-by: Lars Poeschel
Slite changes on the commit message
On Thu, May 28, 2020 at 3:12 PM Gražvydas Ignotas wrote:
>
> I'm not aware of any remaining mainline u-boot users of this device (also
> asked around a bit on IRC), so:
> Acked-by: Grazvydas Ignotas
Slite changes on the commit message and
Applied to u-boot-spi/master
On Fri, May 29, 2020 at 2:14 AM Tom Rini wrote:
>
> On Thu, May 28, 2020 at 02:32:39PM -0600, Simon Glass wrote:
> > Hi,
> >
> > On Thu, 28 May 2020 at 14:15, Tom Rini wrote:
> > >
> > > On Thu, May 28, 2020 at 07:07:14AM +, Priyanka Jain wrote:
> > > > >-Original Message-
> > > >
>-Original Message-
>From: U-Boot On Behalf Of Claudiu Manoil
>Sent: Tuesday, May 5, 2020 8:43 PM
>To: u-boot@lists.denx.de
>Cc: joe.hershber...@ni.com; Alexandru Marginean
>; Vladimir Oltean
>
>Subject: [PATCH v4 0/6] Introduce DSA Ethernet switch class and Felix driver
>
Hi Joe,
How
mmit ab80137cc436e977ef91a154372ae5aeae3f4fb0:
Merge https://gitlab.denx.de/u-boot/custodians/u-boot-marvell (2020-05-27
10:56:25 -0400)
are available in the Git repository at:
https://gitlab.denx.de/u-boot/custodians/u-boot-amlogic.git
tags/u-boot-amlogic-20200529
for you to fetch change
On 28/05/2020 15:47, Neil Armstrong wrote:
> The khadas-vim, khadas-vim2, libretech-ac & libretech-*-pc boards were missing
> DT tweak to enable USB gadget.
> Add them to their -u-boot.dtsi files and include the right gxl-u-boot.dtsi.
>
> Fixes: a19e8a0f03 ("arm: dts: meson-gxl: Add USB Gadget
On 26. 05. 20 10:20, Mike Looijmans wrote:
> SPL boot is slow on the ZynqMP because of this delay. The initialization done
> in psu_init no longer requires an external delay, so this can be removed to
> speed up the SPL boot flow considerably.
>
> Signed-off-by: Mike Looijmans
> ---
>
Hi Kever,
On Fri, May 29, 2020 at 3:25 PM Kever Yang wrote:
>
> Hi Suniel,
>
> Any idea bout ChenYu's comment?
Yes, ChenYu's point is valid, right now we are in communication with
Libretech about the I/O differences to differentiate baseboard vs
expansion. We will come back with proper
On 5/29/20 8:41 AM, AKASHI Takahiro wrote:
> UEFI specification requires that we shall support three type of
> certificates of authenticode in PE image:
> WIN_CERT_TYPE_EFI_GUID with the guid, EFI_CERT_TYPE_PCKS7_GUID
> WIN_CERT_TYPE_PKCS_SIGNED_DATA
> WIN_CERT_TYPE_EFI_PKCS1_15
>
> As EDK2
Dropped DM supprot for Kirkwood
Enable DM_SPI on all boards using Kirkwood
Signed-off-by: Bhargav Shah
Reviewed-by: Stefan Roese
---
Changes for v2:
- Added more meaningful commit message
- Corrected typo from Krikwood to Kirkwood in commit message
---
diff --git
On 5/29/20 8:41 AM, AKASHI Takahiro wrote:
> The global variable, efi_guid_cert_type_pkcs7, will also be used in
> efi_image_loader.c in a succeeding patch so as to correctly handle
> a signature type of authenticode in signed image.
>
> Meanwhile, it is currently defined in efi_variable.c. Once
On 2020/5/26 上午11:32, Frank Wang wrote:
There are some sub-nodes under the grf DT, so add bind callback
function in rk3399 syscon driver to scan them recursively.
Signed-off-by: Frank Wang
Reviewed-by: Kever Yang
Thanks,
- Kever
---
arch/arm/mach-rockchip/rk3399/syscon_rk3399.c | 3
On 2020/5/27 上午12:18, Marcin Juszkiewicz wrote:
USB stack uses DM so DM_KEYBOARD is needed to get USB keyboard working.
Signed-off-by: Marcin Juszkiewicz
Reviewed-by: Kever Yang
Thanks,
- Kever
---
configs/rockpro64-rk3399_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git
On 2020/5/25 下午10:44, Marcin Juszkiewicz wrote:
U-Boot has video output enabled so time to get keyboard working.
=> usb reset;usb tree
resetting USB...
Bus usb@fe38: USB EHCI 1.00
Bus usb@fe3a: USB OHCI 1.0
Bus usb@fe3c: USB EHCI 1.00
Bus usb@fe3e: USB OHCI 1.0
Bus dwc3: usb
Hi Marcin,
The commit message is acceptable, but you will need to send a new
V2 patch for this instead of reply this by the comment.
Thanks,
- Kever
On 2020/5/25 下午4:52, Marcin Juszkiewicz wrote:
RNG module works fine on RockPro64 so let's enable it by default.
=> rng
: 77
Hi,
On 29. 05. 20 1:25, Brendan Simon (eTRIX) wrote:
> I came across the "Boot Count Limit" feature in the U-Boot wiki, and it
> looks like it will do exactly what I want.
>
> https://www.denx.de/wiki/view/DULG/UBootBootCountLimit
>
>
> However, it also states that the "feature is
Hi Suniel,
Any idea bout ChenYu's comment?
Thanks,
- Kever
On 2020/5/23 上午10:07, Chen-Yu Tsai wrote:
Hi,
On Sat, May 23, 2020 at 2:04 AM wrote:
From: Suniel Mahesh
This patch series adds runtime detection of add-on board(ROC-RK3399-PC
Mezzanine) by
enabling I2C in SPL.
This add on
On 2020/5/22 下午10:14, Walter Lozano wrote:
As documentation is being moved to doc/boards/rockchip create a warning
message and remove the redundant list of supported boards.
Signed-off-by: Walter Lozano
Reviewed-by: Kever Yang
Thanks,
- Kever
---
doc/README.rockchip | 72
On 2020/5/22 下午10:14, Walter Lozano wrote:
As an additional step to move documentation to doc/boards/rockchip
improve format of the supported board list to make it more readable.
Additionally, add the configuration files used to build them based on
doc/README.rockchip.
Signed-off-by: Walter
On 2020/5/22 下午10:14, Walter Lozano wrote:
Update the list of supported boards with the information available
on doc/README.rockchip.
Signed-off-by: Walter Lozano
Reviewed-by: Kever Yang
Thanks,
- Kever
---
doc/board/rockchip/rockchip.rst | 16
1 file changed, 16
On Thu, May 28, 2020 at 8:32 PM Kurt Miller wrote:
>
> The cooling levels are tuned to the fan that comes with the rockpro64 NAS
> case. A gpu_thermal zone was not added because having two active cooling
> maps control one physical fan causes them to compete for the fan speed
> which results in
Hi Simon,
>-Original Message-
>From: Simon Glass
>Sent: 28 May 2020 08:38
>To: Pragnesh Patel
>Cc: U-Boot Mailing List ; Atish Patra
>; Palmer Dabbelt ; Bin
>Meng ; Paul Walmsley ;
>Jagan Teki ; Anup Patel
>; Sagar Kadam ; rick
>; Tero Kristo ; Heiko Stuebner
>; Finley Xiao chips.com>;
Add L2 cache node to enable all cache ways from U-Boot proper.
Signed-off-by: Pragnesh Patel
Reviewed-by: Bin Meng
---
arch/riscv/cpu/fu540/Makefile | 1 +
arch/riscv/cpu/fu540/cache.c | 53 +++
arch/riscv/dts/fu540-c000-u-boot.dtsi | 4 ++
Patch 1: enable all cache ways from U-Boot proper
This patch enables all cache ways from U-Boot proper, earlier this was
done by FSBL.
(https://github.com/sifive/freedom-u540-c000-bootloader/blob/master/fsbl/main.c#L428)
L2 cache is of 2 MB (16 cache ways) and 1 cache way is of 128 KB.
FSBL is
This patch enables SiFive PWM driver for the SiFive
Unleashed board.
Signed-off-by: Pragnesh Patel
---
board/sifive/fu540/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/board/sifive/fu540/Kconfig b/board/sifive/fu540/Kconfig
index 86193d7668..683668d059 100644
---
Signature database (db or dbx) may have not only certificates that contain
a public key for RSA decryption, but also digests of signed images.
In this test case, if database has an image's digest (EFI_CERT_SHA256_GUID)
and if the value matches to a hash value calculated from image's binary,
In this test case, an image is signed multiple times with different
keys. If any of signatures contained is not verified, the whole
authentication check should fail.
Signed-off-by: AKASHI Takahiro
---
test/py/tests/test_efi_secboot/conftest.py| 3 ++
Revocation database (dbx) may have not only certificates, but also
message digests of certificates with revocation time
(EFI_CERT_X509_SHA256_GUILD).
In this test case, if the database has such a digest and if the value
matches to a certificate that created a given image's signature,
In the test case (1g) of test_authvar, "db" is mistakenly used,
and it ends up being the exact same as (1f).
So correct it as "dbx" test case.
Signed-off-by: AKASHI Takahiro
---
test/py/tests/test_efi_secboot/test_authvar.py | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
Currently, we don't use any regular expression in matching outputs from
U-Boot. Since its use is just redundant, we can remove all.
Signed-off-by: AKASHI Takahiro
---
.../py/tests/test_efi_secboot/test_authvar.py | 73 +--
test/py/tests/test_efi_secboot/test_signed.py | 34
Split the existing test case-1 into case1 and a new case-2:
case-1 for non-SecureBoot mode; case-2 for SecureBoot mode.
In addition, one corner case is added to case-2; a image is signed
but a corresponding certificate is not yet installed in "db."
Signed-off-by: AKASHI Takahiro
---
A signed image may have multiple signatures in
- each WIN_CERTIFICATE in authenticode, and/or
- each SignerInfo in pkcs7 SignedData (of WIN_CERTIFICATE)
In the initial implementation of efi_image_authenticate(), the criteria
of verification check for multiple signatures case is a bit
In case that a type of certificate in "db" or "dbx" is
EFI_CERT_X509_SHA256_GUID, it is actually not a certificate which contains
a public key for RSA decryption, but a digest of image to be loaded.
If the value matches to a value calculated from a given binary image, it is
granted for loading.
There are a couple of occurrences of hash calculations in which a new
efi_hash_regions will be commonly used.
Signed-off-by: AKASHI Takahiro
---
lib/efi_loader/efi_signature.c | 44 +-
1 file changed, 16 insertions(+), 28 deletions(-)
diff --git
Since the size check against an entry in efi_search_siglist() is
incorrect, this function will never find out a to-be-matched certificate
and its associated revocation time in signature list.
Signed-off-by: AKASHI Takahiro
---
lib/efi_loader/efi_signature.c | 5 +++--
1 file changed, 3
Since certificate table, which is indexed by IMAGE_DIRECTORY_ENTRY_SECURITY
and contains authenticode in PE image, doesn't always exist, we should make
sure that we will retrieve its pointer only if it exists.
Signed-off-by: AKASHI Takahiro
---
lib/efi_loader/efi_image_loader.c | 29
The global variable, efi_guid_cert_type_pkcs7, will also be used in
efi_image_loader.c in a succeeding patch so as to correctly handle
a signature type of authenticode in signed image.
Meanwhile, it is currently defined in efi_variable.c. Once some secure
storage solution for UEFI variables is
UEFI specification requires that we shall support three type of
certificates of authenticode in PE image:
WIN_CERT_TYPE_EFI_GUID with the guid, EFI_CERT_TYPE_PCKS7_GUID
WIN_CERT_TYPE_PKCS_SIGNED_DATA
WIN_CERT_TYPE_EFI_PKCS1_15
As EDK2 does, we will support the first two that are pkcs7
Summary
===
I'm currently working on reworking UEFI secure boot, aiming to add
"intermediate certificates" support. In this effort, I found a couple
of issues that should immediately be fixed or useful improvements even
without intermediate certificates support.
Each commit in this patch
1 - 100 of 121 matches
Mail list logo