On Tue, Oct 10, 2017 at 04:48:01AM +1100, Jonathan Gray wrote:
> On Mon, Oct 09, 2017 at 01:06:26PM -0400, Rob Clark wrote:
> > On Mon, Oct 9, 2017 at 12:41 PM, Jonathan Gray wrote:
> > > On Mon, Oct 09, 2017 at 12:06:24PM -0400, Rob Clark wrote:
> > >> On Mon, Oct 9, 2017 at
On Mon, Nov 06, 2017 at 06:15:11PM -0500, Tom Rini wrote:
> We first introduce CONFIG_USE_BOOTCOMMAND, similar to
> CONFIG_USE_BOOTARGS. We then migrate CONFIG_BOOTCOMMAND for most
> CONFIG_DISTRO_DEFAULT users. In some cases platforms have a complex
> scheme around this usage, and these have
On 11/17/2017 03:07 PM, Simon Glass wrote:
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
To avoid duplicate coding provide a helper function that
initializes an EFI object and adds it to the EFI object
list.
efi_exit() is the only place where we dereference a
On 11/17/2017 03:06 PM, Simon Glass wrote:
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
Use efi_add_protocol to install protocols.
Signed-off-by: Heinrich Schuchardt
---
v2
no change
---
lib/efi_loader/efi_disk.c | 39
On 11/17/2017 12:59 PM, York Sun wrote:
> On 11/17/2017 11:04 AM, Marek Vasut wrote:
>> On 11/17/2017 08:02 PM, Marek Vasut wrote:
>>> On 11/17/2017 05:43 PM, York Sun wrote:
On 09/12/2017 10:09 AM, Marek Vasut wrote:
> The status register is optional in the AMD command sets, but it's
On Fri, Nov 17, 2017 at 08:19:07PM +0100, Marek Vasut wrote:
> On 11/14/2017 07:32 PM, Otavio Salvador wrote:
> > This upgrades the U-Boot from 2017.09 to 2017.11 release.
> >
> > Signed-off-by: Otavio Salvador
> > ---
> >
> > .../u-boot/{u-boot-common_2017.09.inc =>
On 11/17/2017 11:04 AM, Marek Vasut wrote:
> On 11/17/2017 08:02 PM, Marek Vasut wrote:
>> On 11/17/2017 05:43 PM, York Sun wrote:
>>> On 09/12/2017 10:09 AM, Marek Vasut wrote:
The status register is optional in the AMD command sets, but it's
presence can be checked by reading out CFI
On Fri, Nov 17, 2017 at 12:46 PM, Heinrich Schuchardt
wrote:
> On 11/17/2017 03:04 PM, Rob Clark wrote:
>>
>> On Sun, Nov 12, 2017 at 9:02 AM, Heinrich Schuchardt
>> wrote:
>>>
>>> Assignments should not be made in the middle of nowhere.
>>>
>>>
Thanks for the review.
On Friday, November 17, 2017, Marek Vasut wrote:
> > +#ifndef RZA_USB
>
> This should be turned into Kconfig entry, some CONFIG_...
So you want RZA_USB in Kconfig, but is it OK if it is automatically
selected when an RZ/A device is selected?
For example:
config
On Nov 17, 2017, at 11:04, Marek Vasut
> wrote:
On 11/17/2017 08:02 PM, Marek Vasut wrote:
On 11/17/2017 05:43 PM, York Sun wrote:
On 09/12/2017 10:09 AM, Marek Vasut wrote:
The status register is optional in the AMD command sets, but it's
On 11/14/2017 07:32 PM, Otavio Salvador wrote:
> This upgrades the U-Boot from 2017.09 to 2017.11 release.
>
> Signed-off-by: Otavio Salvador
> ---
>
> .../u-boot/{u-boot-common_2017.09.inc => u-boot-common_2017.11.inc} | 2
> +-
>
On 11/13/2017 10:21 PM, Chris Brandt wrote:
> While the USB HW in the RZ/A is basically the same, there are some
> differences from the original versions that were in the SH4 SoCs.
>
> Signed-off-by: Chris Brandt
> ---
> drivers/usb/host/r8a66597-hcd.c | 46
>
On Sat, Oct 28, 2017 at 12:22:27AM -0400, tobet...@gmail.com wrote:
> From: Dongjin Kim
>
> Since CONFIG_DEFAULT_CONSOLE is already started with "console=",
> the console argument in CONFIG_EXTRA_ENV_SETTINGS is expanded to
> "console=console=ttySAC1,115200n8" and this
On Fri, Oct 20, 2017 at 10:45:50AM +0800, Zhiqiang Hou wrote:
> From: Minghuan Lian
>
> This patch is to change U-Boot PCI bus assignement compliant with Linux.
> It means each PCIe controller's bus number is 0, not the current maximum
> PCI bus number, when start to scan
On Thu, Jul 27, 2017 at 10:07:17PM -0700, Gan, Yau Wai wrote:
> Tag CPU with dm-pre-reloc to enable driver before
> relocation.
>
> Signed-off-by: Gan, Yau Wai
> Cc: Thomas Chou
> Reviewed-by: Marek Vasut
Applied to u-boot/master,
On 11/17/2017 02:26 PM, Alexey Brodkin wrote:
> Commit 9000eddbae0d ("drivers/usb/ehci: Use platform-specific accessors")
> broke USB 2.0 on big-endian platforms because for them writel/readl()
> does automatic conversion of BE data to LE.
>
> Proper implementation requires to use "raw" variant
On 11/17/2017 08:02 PM, Marek Vasut wrote:
> On 11/17/2017 05:43 PM, York Sun wrote:
>> On 09/12/2017 10:09 AM, Marek Vasut wrote:
>>> The status register is optional in the AMD command sets, but it's
>>> presence can be checked by reading out CFI table entry 0xc bit 0.
>>> If the register is
On 11/17/2017 05:43 PM, York Sun wrote:
> On 09/12/2017 10:09 AM, Marek Vasut wrote:
>> The status register is optional in the AMD command sets, but it's
>> presence can be checked by reading out CFI table entry 0xc bit 0.
>> If the register is present, prefer using it's bit 7 to determine
>> if
On 11/17/2017 03:06 PM, Simon Glass wrote:
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
Provide a test for the EFI_DEVICE_PATH_TO_TEXT_PROTOCOL protocol.
Signed-off-by: Heinrich Schuchardt
---
v2
no change
---
On 11/17/2017 03:06 PM, Simon Glass wrote:
Hi Heinrich,
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
The current implementation of efi_locate_device_path does not match
the UEFI specification. It completely ignores the protocol
parameters.
Signed-off-by:
On 11/17/2017 03:04 PM, Rob Clark wrote:
On Sun, Nov 12, 2017 at 9:02 AM, Heinrich Schuchardt wrote:
Assignments should not be made in the middle of nowhere.
Signed-off-by: Heinrich Schuchardt
---
v2
Call efi_dp_str in debug mode only.
---
On 11/17/2017 08:47 AM, Stefan Roese wrote:
When trying to load an image from a non-existent USB key, U-Boot v2017.11
crashes on my x86 platform:
=> load usb 0:1 0300 abc
General Protection
EIP: 0010:[<7b59030d>] EFLAGS: 00010286
Original EIP :[]
...
This used to work in v2017.09. Testing
On 11/09/2017 02:39 AM, Prabhakar Kushwaha wrote:
> mac help does not reflect correct descriptions and parameter.
>
> So update mac help command.
>
> Signed-off-by: Prabhakar Kushwaha
> ---
Minor change to commit message.
Applied to fsl-qoriq master. Thanks.
York
On 11/09/2017 01:26 AM, Prabhakar Kushwaha wrote:
> Value provided in MC_MEM_SIZE_ENV_VAR is in hex.
>
> So provide 16 as base in simple_strtoul.
>
> Signed-off-by: Prabhakar Kushwaha
> ---
Minor change in commit message.
Applied to fsl-qoriq master. Thanks.
York
On 11/09/2017 02:16 AM, Shengzhou Liu wrote:
> Signed-off-by: Shengzhou Liu
> ---
This set is applied to fsl-qoriq master. Thanks.
York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
On 11/05/2017 11:48 PM, Ashish Kumar wrote:
> Signed-off-by: Ashish Kumar
> Signed-off-by: Prabhakar Kushwaha
> Signed-off-by: Raghav Dogra
> ---
> v6:
> Sorting defconfig with help make savedefconfig
>
Applied to
On 11/05/2017 11:49 PM, Ashish Kumar wrote:
> Distro boot support gives flexibility to run distro RFS like Ubuntu's
> being deployed from SD card or SATA drive. If it fails
> to detect external storage, fall back to qspi/sd boot.
>
> Enable this by default in RDB's defconfig by selecting
>
On 11/05/2017 11:48 PM, Ashish Kumar wrote:
> Signed-off-by: Prabhakar Kushwaha
> Signed-off-by: Ashish Kumar
> Signed-off-by: Raghav Dogra
> ---
> v6:
> Incorporation of review comments in README wrt to line wrap.
>
On 11/01/2017 09:20 PM, Ashish Kumar wrote:
> Signed-off-by: Ashish Kumar
> Signed-off-by: Amrita Kumari
> ---
> v2: Rebase to top
> v3: Consolidate defines in common file
> v4: Protect define using CONFIG_SCSI
Squashed with patch #1.
Applied to
On 10/30/2017 02:58 PM, Scott Wood wrote:
> On Mon, 2017-10-30 at 18:50 +, York Sun wrote:
>> On 10/20/2017 03:43 AM, Kurt Kanzenbach wrote:
>>> Currently the chipselect used to identify the corresponding NAND chip is
>>> stored
>>> at the controller and only set during fsl_ifc_chip_init().
On 10/29/2017 11:24 PM, Ashish Kumar wrote:
> Signed-off-by: Ashish Kumar
> ---
Applied to fsl-qoriq master. Thanks.
York
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
On 10/26/2017 03:33 AM, Ashish Kumar wrote:
> Correcting error handing for qbman_swp_acquire. The return value is zero is
> an error condition since number of buffer copied is zero meaning
> there are no free buffers for allocation.
>
> Signed-off-by: Ashish Kumar
>
On Fri, Nov 17, 2017 at 03:40:51PM +0100, Lukasz Majewski wrote:
> Hi Maxime,
>
> > Hi Lukasz, Tom,
> >
> > On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote:
> > > On Fri, 17 Nov 2017 08:41:05 -0500
> > > Tom Rini wrote:
> > >
> > > > On Fri, Nov 17, 2017
On 09/12/2017 10:09 AM, Marek Vasut wrote:
> The status register is optional in the AMD command sets, but it's
> presence can be checked by reading out CFI table entry 0xc bit 0.
> If the register is present, prefer using it's bit 7 to determine
> if the flash is busy over reading the flash ; this
On 2017년 11월 17일 18:38, Guillaume Gardet wrote:
>
>
> Le 17/11/2017 à 10:36, Lukasz Majewski a écrit :
>> On Thu, 16 Nov 2017 13:29:56 +0100
>> Guillaume Gardet wrote:
>>
>>> I found a workaround. If I disable MMC_MODE_HS_52MHz, then it is
>>> working fine.
>>>
>>> I
Hi,
as I'm quite new to U-Boot and its patch workflow: what do I have to do to send
a reviewed-by or tested-by tag for a patch? I sent it as a reply to this list
but I can't see them in patchwork.
Did I miss something here? Or could it be a problem in the email format used?
Thanks,
Simon
On Fri, Nov 03, 2017 at 10:10:27AM +0800, Kever Yang wrote:
> We need to update gd in assamble code after relocate,
> this is a fix to:
> adc421e arm: move gd handling outside of C code
>
> Signed-off-by: Kever Yang
> Reviewed-by: Philipp Tomsich
On Mon, Oct 23, 2017 at 10:57:33AM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Allow to get and enable MMC related clock
>
> Signed-off-by: Patrice Chotard
Applied to u-boot/master, thanks!
--
Tom
signature.asc
On Wed, Oct 18, 2017 at 10:02:11AM +0200, Neil Armstrong wrote:
> On Amlogic Meson GXL/GXM, supplementary ethernet configuration registers
> were added to configure the internal RMII PHY interface.
>
> Signed-off-by: Neil Armstrong
Applied to u-boot/master, thanks!
--
On Tue, Oct 17, 2017 at 10:30:18PM +0900, Masahiro Yamada wrote:
> Prior to this commit, tools/Makefile pulls all libfdt files from
> lib/libfdt.
>
> lib/libfdt/ and scripts/dtc/libfdt have the same copies for the
> followings 6 files:
> fdt.c fdt_addresses.c fdt_empty_tree.c fdt_overlay.c
On Mon, Oct 23, 2017 at 10:57:34AM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Add .getcd callback to check is MMC card is present
>
> Signed-off-by: Patrice Chotard
Applied to u-boot/master, thanks!
--
Tom
On Tue, Oct 17, 2017 at 10:30:20PM +0900, Masahiro Yamada wrote:
> lib/libfdt/ and scripts/dtc/libfdt have the same copies for the
> followings 6 files:
> fdt.c fdt_addresses.c fdt_empty_tree.c fdt_overlay.c fdt_strerr.c
> fdt_sw.c
>
> Make them a wrapper of scripts/dtc/libfdt/*. This is
On Thu, Oct 26, 2017 at 01:23:19PM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Fix clock division factor initialization for RCC_PLLCFGR
> registers.
>
> PLLR bits (bit 31-28) in RCC_PLLCFGR must not be cleared,
> it's a forbidden value. So update
On Mon, Nov 06, 2017 at 02:16:38PM +0100, Jorge Ramirez-Ortiz wrote:
> Save the environment data at the end of the boot partition on emmc
>
> Signed-off-by: Jorge Ramirez-Ortiz
Applied to u-boot/master, thanks!
--
Tom
signature.asc
Description: PGP signature
On Tue, Oct 17, 2017 at 01:42:43PM +0900, Masahiro Yamada wrote:
> The pylibfdt is used by dtoc (and, indirectly by binman), but there
> is no reason why it must be generated in the tools/ directory.
>
> Recently, U-Boot switched over to the bundled DTC, and the directory
> structure under
On Thu, Oct 12, 2017 at 10:18:45AM +0530, Keerthy wrote:
> All regulators are hooked to EN_Pin at reset so that EN Pin controls
> their state. Hook the LDO1 regulator to EN pin which at reset is not
> hooked. This applies only to LP8733.
>
> Signed-off-by: Keerthy
>
On Fri, Oct 27, 2017 at 11:04:08AM -0600, Stephen Warren wrote:
> From: Stephen Warren
>
> It can be useful to record how long tests take; this can help debug slow
> running test systems or track changes in performance over time. Enhance
> the test system to record
On Fri, Oct 20, 2017 at 04:55:51PM -0400, Tom Rini wrote:
> This option provides better performance and should really always be
> enabled. Make this be default y.
>
> Signed-off-by: Tom Rini
> Acked-by: Jeroen Hofstee
Applied to u-boot/master,
On Tue, Oct 17, 2017 at 10:30:17PM +0900, Masahiro Yamada wrote:
> Fortunately, U-Boot did not modify libfdt_internal.h locally.
>
> Change it to a wrapper of scripts/dtc/libfdt/fdt.h, which will be
> periodically synced with the upstream DTC (or kernel).
>
> Signed-off-by: Masahiro Yamada
On Thu, Oct 12, 2017 at 03:50:30PM +0200, Neil Armstrong wrote:
> Synchronize the Amlogic ARM64 dts from mainline Linux 4.13.5
>
> In the preparation of the support of the Amlogic P212 board,
> import the corresponding meson-gxl-s905x-p212.dts file.
>
> Signed-off-by: Neil Armstrong
On Mon, Oct 23, 2017 at 10:57:31AM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Convert this driver to driver model.
> This driver is also used by VEXPRESS platforms which doesn't
> use driver model.
>
> Tested on STM32F746 and STM32F769 platforms.
>
On Tue, Oct 17, 2017 at 01:42:41PM +0900, Masahiro Yamada wrote:
> From: Rob Herring
>
> libfdt has gained some new files. We need to include them in the
> kernel's copy.
>
> Reported-by: Kyle Yan
> Signed-off-by: Rob Herring
>
On Wed, Oct 18, 2017 at 10:02:10AM +0200, Neil Armstrong wrote:
> The Amlogic Meson GXL/GXM families embeds an internal RMII Ethernet PHY.
>
> The PHY acts as a generic PHY but needs a slight configuration right
> before it's configuration.
>
> Signed-off-by: Neil Armstrong
On Thu, Oct 12, 2017 at 03:50:31PM +0200, Neil Armstrong wrote:
> Add the Amlogic Meson GXL pinctrl support based on the GXBB driver and
> the synchronized DTS from Linux 4.13.5
>
> Signed-off-by: Neil Armstrong
> Reviewed-by: Beniamino Galvani
Hi.
On 17 November 2017 at 08:41, wrote:
>
> On 3 October 2017 at 09:55, André Draszik wrote:
>
> Please add a commit message. Also where is this used? Is it dead code?
>
> Acked-by: Simon Glass
>
I applied this with changes:
- added a
On Thu, Nov 02, 2017 at 11:39:51PM -0700, Wilson Lee wrote:
> In U-boot, serial_tstc was use to determine is there have a character in
> serial console that pending for read. If there is no "pending" function
> implemented in serial driver, the serial-uclass will return "true(1)"
> to indicate
On Tue, Oct 17, 2017 at 01:42:44PM +0900, Masahiro Yamada wrote:
> Currently, pylibfdt is always compiled if swig is installed on your
> machine. It is really annoying because most of targets (excepts
> x86, sunxi, rockchip) do not use dtoc or binman.
>
> "checkbinman" and "checkdtoc" are
On Tue, Oct 17, 2017 at 10:30:16PM +0900, Masahiro Yamada wrote:
> Fortunately, U-Boot did not modify fdt.h locally.
>
> Change it to a wrapper of scripts/dtc/libfdt/fdt.h, which will be
> periodically synced with the upstream DTC (or kernel).
>
> Signed-off-by: Masahiro Yamada
On Mon, Nov 06, 2017 at 02:16:37PM +0100, Jorge Ramirez-Ortiz wrote:
> Allow the platform to define a partition by name at the end of which
> the environment data will be located.
>
> Signed-off-by: Jorge Ramirez-Ortiz
Applied to u-boot/master, thanks!
--
Tom
On Tue, Oct 17, 2017 at 01:42:42PM +0900, Masahiro Yamada wrote:
> This adds the following commits from upstream:
>
> b1a6003 tests: Add a test for overlays syntactic sugar
> 737b2df overlay: Add syntactic sugar version of overlays
> 497432f checks: Use proper format modifier for size_t
>
On Thu, Oct 12, 2017 at 03:50:32PM +0200, Neil Armstrong wrote:
> This adds platform code for the Amlogic P212 reference board based on a
> Meson GXL (S905X) SoC with the Meson GXL configuration.
>
> This initial submission only supports UART and MMC/SDCard, support for the
> internal Ethernet
We borrow the macros for these functions from ARM and remove references
to '__raw_'.
Signed-off-by: Tom Rini
---
arch/sandbox/include/asm/io.h | 57 +++
1 file changed, 57 insertions(+)
Applied to u-boot-dm thanks!
On Mon, Oct 23, 2017 at 10:57:32AM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Allow to get "bus-width" property from device tree
>
> Signed-off-by: Patrice Chotard
Applied to u-boot/master, thanks!
--
Tom
signature.asc
On Mon, Oct 23, 2017 at 10:57:30AM +0200, patrice.chot...@st.com wrote:
> From: Patrice Chotard
>
> Update arm_pl180_mmci_init() prototype by adding struct mmc**
> param. This is needed before converting this driver to driver model
> in order to use arm_pl180_mmci_init()
On Wed, Oct 18, 2017 at 10:02:12AM +0200, Neil Armstrong wrote:
> This patch adds support for the Internal RMII Ethernet PHY on the
> Amlogic P212 Reference Board.
>
> Signed-off-by: Neil Armstrong
Applied to u-boot/master, thanks!
--
Tom
signature.asc
Description:
On 3 October 2017 at 09:55, André Draszik wrote:
> From: André Draszik
>
> The offset of the permanent flags structure is in a different
> place in the response compared to what the code is doing,
> which gives us a completely useless result.
>
> Fix by
Hi Andre,
On 1 November 2017 at 03:09, André Draszik wrote:
>
> Hi,
>
>
> On Tue, 2017-10-03 at 16:55 +0100, André Draszik wrote:
> > From: André Draszik
> >
> > The maximum length of the name of the image is
> > obviously not sizeof(), which is just the
>
On Mon, Nov 06, 2017 at 07:57:30PM -0600, Adam Ford wrote:
> To keep the board file smaller and clean, let's move the pinmux to the header
> file.
>
> Signed-off-by: Adam Ford
>
> diff --git a/board/logicpd/omap3som/omap3logic.c
> b/board/logicpd/omap3som/omap3logic.c
>
On Tue, Oct 17, 2017 at 10:30:19PM +0900, Masahiro Yamada wrote:
> Add this typedef in the same place as in Linux. This is necessary
> to refactor libfdt inclusion.
>
> U-Boot also defines it in include/compiler.h. Of course it should
> not do that, but I do not want to open a can of worms.
>
On Wed, Oct 11, 2017 at 06:42:23PM +0200, Felix Brack wrote:
> For the DM TPS65910 driver I'm working on, querying the MPU voltage
> should return a value in uV. This value can then be used by the
> regulator's standard function set_value to set the MPU voltage.
>
> Signed-off-by: Felix Brack
On Thu, Nov 02, 2017 at 06:11:27PM -0600, Stephen Warren wrote:
> From: Stephen Warren
>
> This is required in the case where U-Boot is typically loaded and run at
> a particular address, but for some reason the RAM at that location is not
> available, e.g. due to memory
On 28 September 2017 at 03:29, Bernhard Messerklinger
wrote:
> Since the compiler is free to place a char array to any address in
> memory (in this case the stack), also to a non word aligned address the
> function "fdt_prop_parse" runs into troubles upon
On Sun, Oct 01, 2017 at 12:31:21AM +0200, Dr. Philipp Tomsich wrote:
>
> > On 30 Sep 2017, at 22:32, Tom Rini wrote:
> >
> > On Sat, Sep 30, 2017 at 06:08:02PM +0200, Dr. Philipp Tomsich wrote:
> >> Simon,
> >>
> >>> On 28 Sep 2017, at 18:14, Dr. Philipp Tomsich
> >>>
On 3 October 2017 at 09:55, André Draszik wrote:
> From: André Draszik
>
> While commit 36d35345b1f6 ("tpm: add missing va_end") added
> some missing calls to va_end(), it missed a few places.
>
> Signed-off-by: André Draszik
> ---
>
On 3 October 2017 at 09:55, André Draszik wrote:
Please add a commit message. Also where is this used? Is it dead code?
Acked-by: Simon Glass
> From: André Draszik
>
> Signed-off-by: André Draszik
> ---
>
On 9 November 2017 at 04:44, Baruch Siach wrote:
> Fixes: 4984de2baaa ("dm: core: Add ofnode to represent device tree nodes")
> Cc: Simon Glass
> Signed-off-by: Baruch Siach
> ---
> include/dm/ofnode.h | 2 +-
> 1 file changed, 1
On 3 October 2017 at 09:52, André Draszik wrote:
> From: André Draszik
>
> TPM_NV_PER_PPREAD: physical presence needed for reading
> TPM_NV_PER_WRITEDEFINE: persistent write lock by writing size 0
> TPM_NV_PER_WRITEALL: write in one go
>
> Signed-off-by:
On 11/17/2017 03:28 PM, Dirk Behme wrote:
> Its a valid use case to call ehci_submit_async() with a NULL buffer
> with length 0. E.g. from usb_set_configuration().
>
> As invalidate_dcache_range() isn't able to judge if the address
> NULL is valid or not (depending on the SoC hardware
Hi Maxime,
> Hi Lukasz, Tom,
>
> On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote:
> > On Fri, 17 Nov 2017 08:41:05 -0500
> > Tom Rini wrote:
> >
> > > On Fri, Nov 17, 2017 at 10:24:37AM +0100, Lukasz Majewski wrote:
> > > > Hi Maxime,
> > > >
> > >
Its a valid use case to call ehci_submit_async() with a NULL buffer
with length 0. E.g. from usb_set_configuration().
As invalidate_dcache_range() isn't able to judge if the address
NULL is valid or not (depending on the SoC hardware configuration it
might be valid) do the check in
On 17.11.17 14:04, Maxime Ripard wrote:
> On Fri, Nov 17, 2017 at 12:21:49PM +, Andre Przywara wrote:
>> Hi,
>>
>> On 17/11/17 08:27, Maxime Ripard wrote:
>>>
>>> I guess I also have a side question here. How do the installers deal
>>> with the ESP partition? Would they create a new
On 18 October 2017 at 10:13, Heinrich Schuchardt wrote:
> Environment variable efi_selftest is passed as load options
> to the selftest application. It is used to select a single
> test to be executed.
>
> The load options are an UTF8 string. Yet I decided to keep
> the name
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Check that helloworld.efi returns EFI_SUCCESS.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> test/py/tests/test_efi_loader.py | 2 ++
> 1 file changed, 2
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> The handle of a loaded image is the value of the handle
> member of the loaded image info object and not the
> address of the loaded image info.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
>
Hi Lukasz, Tom,
On Fri, Nov 17, 2017 at 03:00:15PM +0100, Lukasz Majewski wrote:
> On Fri, 17 Nov 2017 08:41:05 -0500
> Tom Rini wrote:
>
> > On Fri, Nov 17, 2017 at 10:24:37AM +0100, Lukasz Majewski wrote:
> > > Hi Maxime,
> > >
> > > > Now that we have everything in
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> As the selftest is not compiled as an EFI binary we do not
> need special compiler flags.
>
> This avoids the checkarmreloc error on vexpress_ca15_tc2.
>
> Signed-off-by: Heinrich Schuchardt
> ---
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> We need to test if we pass a valid image handle when loading
> and EFI application. This cannot be done in efi_selftest as
> it is not loaded as an image.
>
> So let's enhance helloworld to write the load options to
On 18 October 2017 at 10:13, Heinrich Schuchardt wrote:
> After executing bootefi selftest
> * restore GD
> * unlink the load image handle
> * return 0 or 1 and not a truncated efi_status_t.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v4
> no
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Add a missing line feed for an error message.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> new patch
> ---
> lib/efi_selftest/efi_selftest.c | 2 +-
> 1 file changed, 1
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> include/efi_loader.h | 6 ++-
> lib/efi_loader/efi_boottime.c | 107
>
On 10 November 2017 at 11:54, Heinrich Schuchardt wrote:
> Coccinelle is a program for static code analysis.
> For details on Coccinelle see
>
> http://coccinelle.lip6.fr/
>
> Add scripts/coccicheck and Documentation/dev-tools/coccinelle.rst
> copied from Linux kernel
On 3 October 2017 at 09:52, André Draszik wrote:
> From: André Draszik
>
> TPM_NV_PER_PPREAD: physical presence needed for reading
> TPM_NV_PER_WRITEDEFINE: persistent write lock by writing size 0
> TPM_NV_PER_WRITEALL: write in one go
>
> Signed-off-by:
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Use efi_add_protocol to add protocols.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> lib/efi_loader/efi_net.c | 34 --
> 1 file
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Use function efi_search_protocol.
Use efi_search_protocol().
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> lib/efi_loader/efi_device_path.c | 43
>
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Provide a test for the EFI_DEVICE_PATH_TO_TEXT_PROTOCOL protocol.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> lib/efi_selftest/Makefile | 3 +
>
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> Use efi_add_protocol to install protocols.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> lib/efi_loader/efi_disk.c | 39 +++
> 1
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> In the format specifier we want to specify the maximum width
> in case an ending \0 is missing.
>
> So slen must be used as precision and not as field width.
>
> Signed-off-by: Heinrich Schuchardt
On 6 November 2017 at 13:17, Heinrich Schuchardt wrote:
> Consistenly use efi_uintn_t wherever the UEFI spec uses
> UINTN in boot services interfaces.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2
> no change
> ---
> include/efi_api.h
On 10 November 2017 at 13:06, Heinrich Schuchardt wrote:
> x86_vendor_name is defined as
>
> static const char *const x86_vendor_name[]
>
> So its elements should not be compared to 0.
>
> Remove superfluous paranthesis.
>
> Problem identified with Coccinelle.
>
>
On 12 November 2017 at 07:02, Heinrich Schuchardt wrote:
> To avoid duplicate coding provide a helper function that
> initializes an EFI object and adds it to the EFI object
> list.
>
> efi_exit() is the only place where we dereference a handle
de-reference?
> to obtain a
1 - 100 of 150 matches
Mail list logo