On 09.08.18 07:15, AKASHI Takahiro wrote:
> The commit 21b3edfc964 ("efi_loader: check parameters of CreateEvent")
> enforces a strict parameter check at CreateEvent(). Unfortunately,
> however, EDK2's Shell.efi calls this function with notify_tpl == 0.
>
> The patch above does right thing and
Currently, image's image_base points to an address where the image was
temporarily uploaded for further loading. Since efi_loader relocates
the image to final destination, image_base and image_size should reflect
that.
This bug was detected in UEFI SCT, "Loaded Image Protocol Test - test 2,"
which
Currently, unload function in EFI_LOADED_IMAGE_PROTOCOL is never called
at UnloadImage Boot Service. This is not compliant to UEFI specification.
See chapter "9.1 EFI Loaded Image Protocol."
Signed-off-by: AKASHI Takahiro
---
include/efi_api.h | 4 +++-
lib/efi_loader/efi_boottime.c
Some SPI controller do not support full-duplex SPI transfers. This patch
changes the SPI transfer into 2 separate transfers - or 1, if no data is
to be transmitted.
With this change, only a small buffer for the command, address and
dummy bytes needs to be allocated. We use the TX and RX buffers th
Adding this info helps seeing, what really is being erased - especially
if no arguments are passed for offset and size. Now this is the
output:
=> mtd erase spi-nand0
Erasing 0x ... 0x07ff (1024 eraseblock(s))
nand: attempt to erase a bad/reserved block @600
nand: attempt to erase
When negative return codes are used in commands (do_foo()), the shell
prints these messages:
exit not allowed from main input shell.
Change the return codes in the new mtd commands to use only positive
values and these annoying warnings are gone.
Signed-off-by: Stefan Roese
Cc: Miquel Raynal
C
It was noticed, that the erase command (mtd erase spi-nand0) aborts upon
the first bad block. With this change, bad blocks are now skipped and
the erase operation will continue.
Signed-off-by: Stefan Roese
Cc: Miquel Raynal
Cc: Boris Brezillon
Cc: Jagan Teki
---
v3:
- Handle bad-block skipping
The commit 21b3edfc964 ("efi_loader: check parameters of CreateEvent")
enforces a strict parameter check at CreateEvent(). Unfortunately,
however, EDK2's Shell.efi calls this function with notify_tpl == 0.
The patch above does right thing and we'd better fix the issue on EDK2
side, and yet we mig
On 8.8.2018 22:25, Marek Vasut wrote:
> On 07/16/2018 02:56 PM, Marek Vasut wrote:
>> On 07/16/2018 02:49 PM, Michal Simek wrote:
>>> From: Vipul Kumar
>>>
>>> Update the DWC3 USB driver to support a live tree.
>>>
>>> Signed-off-by: Vipul Kumar
>>> Tested-by: Michal Simek
>>> Signed-off-by: Mic
Hi Miquel,
On 08.08.2018 11:58, Miquel Raynal wrote:
Hi Stefan,
Minor typo if you decide to send a v3.
Boris Brezillon wrote on Tue, 7 Aug 2018
15:28:02 +0200:
On Tue, 7 Aug 2018 14:16:52 +0200
Stefan Roese wrote:
Some SPI controller do not support full-duplex SPI transfers. This patch
Hi Miquel,
On 08.08.2018 10:56, Miquel Raynal wrote:
Boris Brezillon wrote on Tue, 7 Aug 2018
15:28:02 +0200:
On Tue, 7 Aug 2018 14:16:52 +0200
Stefan Roese wrote:
Some SPI controller do not support full-duplex SPI transfers. This patch
changes the SPI transfer into 2 separate transfers -
Hi Marek,
On Thu, Aug 9, 2018 at 8:36 AM, Marek Vasut wrote:
> On 08/09/2018 01:24 AM, Bin Meng wrote:
>> Hi Marek,
>>
>> On Thu, Aug 9, 2018 at 3:37 AM, Marek Vasut wrote:
>>> On 08/08/2018 05:32 PM, Bin Meng wrote:
Hi Marek,
On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote:
>
On 08/09/2018 01:24 AM, Bin Meng wrote:
> Hi Marek,
>
> On Thu, Aug 9, 2018 at 3:37 AM, Marek Vasut wrote:
>> On 08/08/2018 05:32 PM, Bin Meng wrote:
>>> Hi Marek,
>>>
>>> On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote:
On 08/08/2018 03:39 PM, Bin Meng wrote:
> Hi Marek,
>
>
On Wed, Aug 08, 2018 at 09:37:36AM +0900, Minkyu Kang wrote:
> Dear Tom,
>
> The following changes since commit 2547e91dc15e5203e15d4ebde9172174743b14a7:
>
> tegra: Indicate that binman makes all three output files (2018-07-26
> 15:49:40 -0400)
>
> are available in the git repository at:
>
Hi Marek,
On Thu, Aug 9, 2018 at 3:37 AM, Marek Vasut wrote:
> On 08/08/2018 05:32 PM, Bin Meng wrote:
>> Hi Marek,
>>
>> On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote:
>>> On 08/08/2018 03:39 PM, Bin Meng wrote:
Hi Marek,
On Wed, Aug 8, 2018 at 9:24 PM, Marek Vasut wrote:
>
On 07/16/2018 02:56 PM, Marek Vasut wrote:
> On 07/16/2018 02:49 PM, Michal Simek wrote:
>> From: Vipul Kumar
>>
>> Update the DWC3 USB driver to support a live tree.
>>
>> Signed-off-by: Vipul Kumar
>> Tested-by: Michal Simek
>> Signed-off-by: Michal Simek
>
> Applied, thanks
>
Dropped becau
On 08/08/2018 05:32 PM, Bin Meng wrote:
> Hi Marek,
>
> On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote:
>> On 08/08/2018 03:39 PM, Bin Meng wrote:
>>> Hi Marek,
>>>
>>> On Wed, Aug 8, 2018 at 9:24 PM, Marek Vasut wrote:
On 08/08/2018 03:14 PM, Bin Meng wrote:
> Hi Marek,
>
>
Hello Tom, Alexander,
On Wed, Aug 01, 2018 at 01:25:54PM +0200, Eugeniu Rosca wrote:
> Hello Tom, Simon, Alexander, Heinrich,
>
> On Wed, Jul 25, 2018 at 03:30:16PM +0200, Eugeniu Rosca wrote:
> > Hello Alexander,
> >
> > Heinrich was kind to have a look at [1] and already provided his
> > Revie
On Mon, Aug 6, 2018 at 5:22 PM Peter Howard wrote:
>
> Sorry a clarification - my main focus is the LCDK board - it's that one which
> has the 4K overrun.
>
I wanted the rest of the people on this thread to know the status.
Both Peter and I did separate drivers. His was more complete than
mine,
On 07/27/2018 01:39 PM, Tom Rini wrote:
> On Fri, Jul 27, 2018 at 11:59:32AM +0200, Lukasz Majewski wrote:
>> On Thu, 26 Jul 2018 23:14:01 +0200
>> Marek Vasut wrote:
>>
>>> On 07/26/2018 11:08 PM, Lukasz Majewski wrote:
On Thu, 26 Jul 2018 21:25:02 +0200
Marek Vasut wrote:
Add clock driver for the Arria10, which allows reading the clock
frequency from all the clock described in the DT. The driver also
allows enabling and disabling the clock. Reconfiguring frequency
is not supported thus far.
Since the DT bindings for the SoCFPGA clock are massively misdesigned
and t
Add support for fetching the clock frequency both using the legacy
method in case clock framework is disabled as well as via the clock
framework if it is enabled. This allows for migration to the clock
framework on platforms which supports it while not breaking legacy
platforms. That said, the lega
Add the pre-reloc DT markers to clock nodes needed in SPL and early
U-Boot stages. This is required to let the Arria10 clock driver start
early and provide clock information for UART and SDMMC.
Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Ley Foon Tan
---
arch/arm/dts/soc
The variables removed in this patch are never used, they are only ever
assigned and then waste precious memory. Drop both the assignment and
the variables.
Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Ley Foon Tan
---
arch/arm/mach-socfpga/clock_manager_arria10.c | 19 ++-
Bind fixed clock driver to the base clock instantiated in the handoff
DT and use DM clock framework to get their clock rate. This replaces
the ad-hoc DT parsing present thus far.
Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Ley Foon Tan
---
arch/arm/mach-socfpga/Kconfig
The L4SP and MMC clock precalculation is specific to Gen5, it is not
needed on Arria10/Stratix10. Isolate it to Gen5 until there is a proper
clock driver for Gen5, at which point this will go away completely.
Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Ley Foon Tan
---
a
Add trivial implementation of the clk dump in case DM is enabled.
This implementation just iterates over all the clock registered
with the CLK uclass and prints their rate.
Signed-off-by: Marek Vasut
Cc: Chin Liang See
Cc: Dinh Nguyen
Cc: Ley Foon Tan
Cc: Simon Glass
Cc: Tom Rini
---
cmd/cl
On Wed, Aug 8, 2018 at 6:54 PM Tom Rini wrote:
>
> On Wed, Aug 08, 2018 at 06:44:07PM +0200, Eugeniu Rosca wrote:
> > On Wed, Aug 8, 2018 at 6:18 PM Jean-Jacques Hiblot wrote:
> > > On 08/08/2018 15:08, Eugeniu Rosca wrote:
> > [snip]
> > > > I think having a warning-free build is a basic policy
Hi York,
> From: York Sun
> Sent: Wednesday, August 8, 2018 7:23 PM
>
> On 08/08/2018 02:05 AM, laurentiu.tu...@nxp.com wrote:
> > From: Laurentiu Tudor
> >
> > This patch series adds the required devices setup and device tree
> > fixups for SMMU enablement on NXP LS1046A chips. The approach tak
On Wed, Aug 08, 2018 at 06:44:07PM +0200, Eugeniu Rosca wrote:
> On Wed, Aug 8, 2018 at 6:18 PM Jean-Jacques Hiblot wrote:
> > On 08/08/2018 15:08, Eugeniu Rosca wrote:
> [snip]
> > > I think having a warning-free build is a basic policy everybody is
> > > expected to comply with.
> > I agree. I m
On Wed, Aug 8, 2018 at 6:18 PM Jean-Jacques Hiblot wrote:
> On 08/08/2018 15:08, Eugeniu Rosca wrote:
[snip]
> > I think having a warning-free build is a basic policy everybody is
> > expected to comply with.
> I agree. I meant that it could be fixed at the time of the commit by the
> maintainer.
On 08/08/2018 02:05 AM, laurentiu.tu...@nxp.com wrote:
> From: Laurentiu Tudor
>
> This patch series adds the required devices setup and device tree
> fixups for SMMU enablement on NXP LS1046A chips. The approach taken
> tries to mimic the implementation of PAMU LIODN setup on booke powerpc.
>
>
On 08/08/2018 15:08, Eugeniu Rosca wrote:
On Wed, Aug 8, 2018 at 1:36 PM Jean-Jacques Hiblot wrote:
Hi all,
On 08/08/2018 11:17, Eugeniu Rosca wrote:
Hello,
FWIW, patch "[7/7] cmd: Add bind/unbind commands to bind a device to a
driver from the command line" contributes with a compiler war
Hi Marek,
On Wed, Aug 8, 2018 at 10:33 PM, Marek Vasut wrote:
> On 08/08/2018 03:39 PM, Bin Meng wrote:
>> Hi Marek,
>>
>> On Wed, Aug 8, 2018 at 9:24 PM, Marek Vasut wrote:
>>> On 08/08/2018 03:14 PM, Bin Meng wrote:
Hi Marek,
On Wed, Aug 8, 2018 at 9:03 PM, Marek Vasut wrote:
>
Hi Tom,
I hope it's okay for post RC1, but I wanted to wait until you had the Kconfig
sorting stuff merged.
Travis CI: https://travis-ci.org/danielschwierzeck/u-boot/builds/413538994
The following changes since commit b1aad8dbe28415695b290726199681ced969c118:
zynqmp: Add avnet_ultra96_rev1_
On 08/08/2018 04:10 PM, Patrice Chotard wrote:
> Add vbus-supply regulator support.
> On some board vbus is not controlled by the phy but by
> an external regulator.
>
> Signed-off-by: Patrice Chotard
> ---
>
> drivers/usb/host/ehci-generic.c | 37 -
> 1 file
On 08/08/2018 03:39 PM, Bin Meng wrote:
> Hi Marek,
>
> On Wed, Aug 8, 2018 at 9:24 PM, Marek Vasut wrote:
>> On 08/08/2018 03:14 PM, Bin Meng wrote:
>>> Hi Marek,
>>>
>>> On Wed, Aug 8, 2018 at 9:03 PM, Marek Vasut wrote:
The PCI controller can have DT subnodes describing extra properties
Add vbus-supply regulator support.
On some board vbus is not controlled by the phy but by
an external regulator.
Signed-off-by: Patrice Chotard
---
drivers/usb/host/ehci-generic.c | 37 -
1 file changed, 36 insertions(+), 1 deletion(-)
diff --git a/drivers/u
Hi Marek,
On Wed, Aug 8, 2018 at 9:24 PM, Marek Vasut wrote:
> On 08/08/2018 03:14 PM, Bin Meng wrote:
>> Hi Marek,
>>
>> On Wed, Aug 8, 2018 at 9:03 PM, Marek Vasut wrote:
>>> The PCI controller can have DT subnodes describing extra properties
>>> of particular PCI devices, ie. a PHY attached t
Add a PHY driver for the R-Car Gen2 which allows configuring the mux
connected to the EHCI controllers and USBHS controller.
Signed-off-by: Marek Vasut
Cc: Nobuhiro Iwamatsu
---
drivers/phy/Kconfig | 8 ++
drivers/phy/Makefile| 1 +
drivers/phy/phy-rcar-gen2.c | 190
On 08/08/2018 03:14 PM, Bin Meng wrote:
> Hi Marek,
>
> On Wed, Aug 8, 2018 at 9:03 PM, Marek Vasut wrote:
>> The PCI controller can have DT subnodes describing extra properties
>> of particular PCI devices, ie. a PHY attached to an EHCI controller
>> on a PCI bus. This patch parses those DT subn
On Tue, Aug 7, 2018 at 1:29 AM Jagan Teki wrote:
>
> davinci_spi now support dt along with platform data,
> respective boards need to switch into dm for the same.
>
> Cc: Adam Ford
> Cc: Vitaly Andrianov
> Cc: Stefano Babic
> Cc: Peter Howard
> Cc: Tom Rini
> Signed-off-by: Jagan Teki
> ---
Hi Marek,
On Wed, Aug 8, 2018 at 9:03 PM, Marek Vasut wrote:
> The PCI controller can have DT subnodes describing extra properties
> of particular PCI devices, ie. a PHY attached to an EHCI controller
> on a PCI bus. This patch parses those DT subnodes and assigns a node
> to the PCI device insta
On Wed, Aug 8, 2018 at 1:36 PM Jean-Jacques Hiblot wrote:
>
> Hi all,
>
>
> On 08/08/2018 11:17, Eugeniu Rosca wrote:
> > Hello,
> >
> > FWIW, patch "[7/7] cmd: Add bind/unbind commands to bind a device to a
> > driver from the command line" contributes with a compiler warning, as
> > described in
The PCI controller can have DT subnodes describing extra properties
of particular PCI devices, ie. a PHY attached to an EHCI controller
on a PCI bus. This patch parses those DT subnodes and assigns a node
to the PCI device instance, so that the driver can extract details
from that node and ie. conf
Add support for operating a PHY attached to ehci-pci. There are
systems where the EHCI controller is internally wired to a PCI
bus and has a PHY connected to it as well, ie. the R-Car Gen2.
Signed-off-by: Marek Vasut
---
drivers/usb/host/ehci-pci.c | 32
1 file c
The code fails to copy the last PHY phandle argument, so it is
missing from the adjusted phandle args and the consumer cannot
use it to determine what the PHY should do.
Signed-off-by: Marek Vasut
Cc: Patrice Chotard
---
drivers/phy/phy-uclass.c | 2 +-
1 file changed, 1 insertion(+), 1 deletio
Pull out the EHCI PHY functions into the ehci-hcd.c to let other
EHCI drivers use them.
Signed-off-by: Marek Vasut
---
drivers/usb/host/ehci-generic.c | 56 +++-
drivers/usb/host/ehci-hcd.c | 57 +
drivers/usb/host/e
Hi York,
On Tue, Aug 07, 2018 at 09:52:46PM +, York Sun wrote:
> On 08/02/2018 01:03 AM, Kurt Kanzenbach wrote:
> > From: Darwin Dingel
> >
> > This is a fix made for the fsl_ifc_nand driver on linux kernel by
> > Pavel Machek and is applied to uboot. It is currently on applied on
> > linux-m
ls1088ardb-pb is another varinat of the ls1088ardb.
This board support two m.2 x2 gen 3 pcie card slot.
There is no support of sata, pcie standard slot,
miniPCIE slot and TDM in this board.
Signed-off-by: Pramod Kumar
---
arch/arm/Kconfig | 14 +++
arch/arm/cpu/armv8
Hi Johann,
I apologize, that typo is actually just in the email. My actual file has
"kernel-0" in both locations. It seems my fit image is formed correctly
based on the header information:
...$ mkimage -F sel4.itb
FIT description: SeL4
Created: Tue Aug 7 10:56:42 2018
Image 0 (kernel-0)
On 08.08.2018 11:09, Anson Huang wrote:
> Hi, Stefan
>
> Anson Huang
> Best Regards!
>
>
>> -Original Message-
>> From: Stefan Agner [mailto:ste...@agner.ch]
>> Sent: Wednesday, August 8, 2018 5:03 PM
>> To: Anson Huang
>> Cc: sba...@denx.de; Fabio Estevam ;
>> albert.u.b...@aribaud.net
Move fpga info to U_BOOT_CMD_MKENT subcommand.
Also use strtol instead of simple_strtoul. The reason is that if -1 is
passed (or fpga info without "fpga" variable) the list of all fpgas is
shown.
This functionality is in the fpga core but it couldn't be performed.
Signed-off-by: Michal Simek
Revi
Convert loadfs command to fpga subcommands.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 58 --
1 file changed, 28 insertions(+), 30 deletions(-)
diff --git a/cmd/fpga.c b
Convert loadmk command to fpga subcommands. Not all combinations are
working but they have never worked properly. This will be fixed later.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 238 ---
Use standard return command failure macro.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/cmd/fpga.c b/cmd/fpga.c
index de9d19dd8e91..89fae86b8253 10
Convert last loads command to fpga subcommands.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 148 +++--
1 file changed, 36 insertions(+), 112 deletions(-)
diff --git a/cmd
On 8.8.2018 11:56, Simon Glass wrote:
> Hi Michal,
>
> On 3 August 2018 at 00:13, Michal Simek wrote:
>> On 2.8.2018 22:37, Simon Glass wrote:
>>> On 26 July 2018 at 08:16, Michal Simek wrote:
Not all 4 fpga loadmk FIT image commands
>>>
>>> I don't understand this. Can you explain what you
There are ways how to call fpga loadmk
1. Full command
fpga loadmk [dev] [address]
2. Dev setup via variable
set fpga [dev]
fpga loadmk [address]
3. Address setup via variable
set fpgadata [address]
fpga loadmk [dev]
4. Dev and address setup via variables
set fpga [dev]
set fpgadata [address]
f
There is no reason to check parameters in separate switch. Check them
directly when they are read. Also there is no reason to check loadmk
case separately because fpga_data address must be non zero too.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1:
Parameter checking is dead code because all the time there must be all
params assigned. If they are not assigned there is no 9th parameters
passed and checking before return CMD_RET_USAGE.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpg
Create command wrapper to clean fpga subcommands.
The function logic is taken from cmd_dm.c
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 42 --
1 file changed, 40 insertions(+), 2 deletion
Separate dump, load, loadb, loadp and loadbp commands to separate
functions to make it clear how they are called and what parameters they
need.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
Maybe they can be still groupped together with one sw
FPGA subsystem requires special care that's why it should be maintained
via one tree.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
MAINTAINERS | 8
1 file changed, 8 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 58b61a
There is no reason to check parameters in separate switch before main
one. This patch is simplifying error path and checking parameters right
after assignment.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 44
Move fpga_get_op() to top of file to remove local function declaration
and also remove useless retyping.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 85 ++
1 file chan
Add support for info, load, loadp, loadb, loadbp, loadmk_legacy,
loadmk_legacy_gz, loadmk_fit, loadfs also with variable support.
There are probably missing failed tests.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1:
- Remove debug message checking
Incorrect command is already handled and FPGA_NONE should be used only
one. In case of error CMD_RET_USAGE can be returned directly without any
addition logic around.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 8 ++--
1 fi
The patch applied in 2010
"cmd_fpga: cleanup help and check parameters"
(sha1: a790b5b2326be9d7c9ad9e3d9b51a8bfabc62d07"
was adding this checking
+ if (dev == FPGA_INVALID_DEVICE) {
+ puts("FPGA device not specified\n");
+ op = FPGA_NONE;
+ }
which simply b
Clean fpga_get_op() error handling by moving checking/print to do_fpga.
Signed-off-by: Michal Simek
Reviewed-by: Simon Glass
---
Changes in v2: None
Changes in v1: None
cmd/fpga.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/cmd/fpga.c b/cmd/fpga.c
index 791fe5cb7
Hi,
We are using this framework for a while and by adding more and more
features it requires small redesigned how commands are handled.
This is something what I have put together to improve readability of
this code and also remove code which is bogus and completely untested.
The patch
https://lis
Hi all,
On 08/08/2018 11:17, Eugeniu Rosca wrote:
Hello,
FWIW, patch "[7/7] cmd: Add bind/unbind commands to bind a device to a
driver from the command line" contributes with a compiler warning, as
described in https://patchwork.ozlabs.org/patch/933310/#1952127 .
This looks like a false posit
Cache up to 4 kiB entries. 4 kiB is the default block size on ext4, yet
the underlying block layer devices usually report support for 512B . In
most cases, the 512B support is emulated (ie. SD cards, SSDs, USB sticks
etc.) and the real block size of those devices is much bigger.
To avoid performan
On Wed, Aug 8, 2018 at 11:58 AM Michal Simek wrote:
>
> On 8.8.2018 11:17, Eugeniu Rosca wrote:
> > Hello,
> >
> > FWIW, patch "[7/7] cmd: Add bind/unbind commands to bind a device to a
> > driver from the command line" contributes with a compiler warning, as
> > described in https://patchwork.ozl
Hi Jagan,
Jagan Teki wrote on Mon, 6 Aug 2018
23:20:19 +0530:
> Are you planning to send the next version?
I was not as we gave you a branch to pull patches from and test with
Travis if the warnings were gone [1].
Of course I can send the patches if you want them here also.
If Stefan sends a
On 3 August 2018 at 05:38, Patrick Delaunay wrote:
> Unify the fgetc function when MUX is activated or not:
> - always call tstc() : it is the normal behavior expected
> by serial uclass (call tstc then getc) and that avoids
> issue when SERIAL_RX_BUFFER is activated
> - reload WATCHDOG in the
Hi Michal,
On 3 August 2018 at 00:13, Michal Simek wrote:
> On 2.8.2018 22:37, Simon Glass wrote:
>> On 26 July 2018 at 08:16, Michal Simek wrote:
>>> Not all 4 fpga loadmk FIT image commands
>>
>> I don't understand this. Can you explain what you mean?
>
> hm not sure why I have written this sh
Hi Stefan,
Minor typo if you decide to send a v3.
Boris Brezillon wrote on Tue, 7 Aug 2018
15:28:02 +0200:
> On Tue, 7 Aug 2018 14:16:52 +0200
> Stefan Roese wrote:
>
> > Some SPI controller do not support full-duplex SPI transfers. This patch
> > changes the SPI transfer into 2 separate tra
Hi Patrice,
On 3 August 2018 at 09:01, Patrice Chotard wrote:
>
> This series :
> - Add new gpio ops get_alternate_function
> - Add get_function and get_alternate_function support to stm32 gpio driver
> - Add get_alternate_function test
>
> Changes in v2:
>- Rename get_function_number t
At present map_sysmem() maps an address into the sandbox RAM buffer,
return a pointer, while map_to_sysmem() goes the other way.
The mapping is currently just 1:1 since a case was not found where a more
flexible mapping was needed. PCI does have a separate and more complex
mapping, but uses its ow
On 3 August 2018 at 00:58, Bin Meng wrote:
> Imply CONFIG_BITREVERSE for Sandbox.
>
> Signed-off-by: Bin Meng
>
> ---
>
> Changes in v2:
> - Change to imply in the Kconfig
>
> arch/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
Reviewed-by: Simon Glass
__
On 3 August 2018 at 05:38, Patrick Delaunay wrote:
> Add test to avoid access to rx buffer when this buffer is empty.
> In this case directly call getc() function to avoid issue when tstc()
> is not called.
>
> Signed-off-by: Patrick Delaunay
> ---
>
> drivers/serial/serial-uclass.c | 3 +++
> 1
On 3 August 2018 at 07:07, Patrice Chotard wrote:
> Replace setparity by more generic setconfig ops
> to allow uart parity, bits word length and stop bits
> number change.
>
> Adds SERIAL_GET_PARITY/BITS/STOP macros.
>
> Signed-off-by: Patrick Delaunay
> Signed-off-by: Patrice Chotard
> ---
>
>
Hi Patrice,
+Stephen and Masahiro for comment on this vs. pinctrl.
On 3 August 2018 at 09:01, Patrice Chotard wrote:
> From: Patrick Delaunay
>
> When a pin is not configured as a GPIO, it could
> have several alternate function.
>
> To be able to identify the alternate function,
> add ops get_
On 5 August 2018 at 10:53, Miquel Raynal wrote:
> The second check on pcr_map in sandbox_tpm2_xfer() is wrong. It should
> check for pcr_map not being empty. Instead, it is a pure copy/paste of
> the first check which is redundant.
>
> This has been found thanks to a Coverity Scan report:
>
>
Enable this for sandbox since it passes now.
Signed-off-by: Simon Glass
---
Changes in v9: None
Changes in v8:
- Drop 'efi: Adjust memory handling to support sandbox'
- Drop 'efi: sandbox: Add relocation constants'
- Drop 'sandbox: smbios: Update to support sandbox'
- Expand series substantially
At present this function takes a pointer as its argument, then passes this
to efi_allocate_pages(), which actually takes an address. It uses casts,
which are not supported on sandbox.
Also the function calculates the FDT size rounded up to the neared EFI
page size, then its caller recalculates the
On 8.8.2018 11:17, Eugeniu Rosca wrote:
> Hello,
>
> FWIW, patch "[7/7] cmd: Add bind/unbind commands to bind a device to a
> driver from the command line" contributes with a compiler warning, as
> described in https://patchwork.ozlabs.org/patch/933310/#1952127 .
any other problem?
M
___
On 3 August 2018 at 09:01, Patrice Chotard wrote:
> In order to test get_alternate_function ops, add
> sandbox_gpio_set_function() and sandbox_gpio_set_alternate_function()
> to be able to configure a pin not as a GPIO and to select the
> alternate function index.
>
> Signed-off-by: Patrice Chotar
On 3 August 2018 at 07:07, Patrice Chotard wrote:
> setparity users has been updated to use new setconfig ops,
> so we can safely remove setparity ops
>
> Signed-off-by: Patrice Chotard
> ---
>
> Changes in v3:
> - Add patch to remove obsolete setparity ops
>
> Changes in v2: None
>
> include/
On 3 August 2018 at 07:07, Patrice Chotard wrote:
> Signed-off-by: Patrice Chotard
> ---
>
> Changes in v3:
> - Update serial test when wrong serial options are specified
> - Add SERIAL_CONFIG(par, bits, stop) macro to create serial mask
>
> Changes in v2:
> - Add sandbox serial test
>
> d
On 5 August 2018 at 10:53, Miquel Raynal wrote:
> The recv variable in sandbox_tpm2_fill_buf() is a pointer on a pointer
> of a char array. It means accessing *recv is the char array pointer
> itself while **recv is the first character of that array. There is no
> need for such indirection here, s
From: Alexander Graf
The fs_read() function wants to get an address rather than the
pointer to a buffer.
So let's convert the passed buffer from pointer back a the address
to make efi_loader on sandbox happier.
Signed-off-by: Alexander Graf
Reviewed-by: Simon Glass
Signed-off-by: Simon Glass
On 3 August 2018 at 07:07, Patrice Chotard wrote:
> Replace stm32_serial_setparity by stm32_serial_setconfig
> which allows to set serial bits number, parity and stop
> bits number.
> Only parity setting is implemented.
>
> Signed-off-by: Patrick Delaunay
> Signed-off-by: Patrice Chotard
> ---
>
On 3 August 2018 at 09:01, Patrice Chotard wrote:
> To avoid confusion with enum gpio_func_t GPIOF_OUTPUT defined in
> asm-generic/gpio.h, rename all sandbox flags GPIOF_(OUTPUT|HIGH|ODR)
> to SDBX_GPIO_(OUTPUT|HIGH|ODR)
>
> Signed-off-by: Patrice Chotard
> ---
>
> Changes in v2: None
>
> driver
Use a starting address of 256MB which should be available. This helps to
make sandbox RAM buffers pointers more recognisable.
Signed-off-by: Simon Glass
---
Changes in v9: None
Changes in v8: None
Changes in v7: None
Changes in v6: None
Changes in v5: None
Changes in v4: None
Changes in v3: None
Add some more verbose debugging when doing memory allocations. This might
help to find bugs later.
Signed-off-by: Simon Glass
---
Changes in v9: None
Changes in v8: None
Changes in v7: None
Changes in v6: None
Changes in v5: None
Changes in v4: None
Changes in v3: None
Changes in v2: None
lib/
On 3 August 2018 at 09:01, Patrice Chotard wrote:
> From: Patrick Delaunay
>
> Allow to display alternate function index using "gpio" command.
> Example, display all gpio state :
>
> gpio status -a
> GPIOD2: func: 12
> GPIOD3: func: 9
> GPIOE3: func: 9
> GPIOG6: func: 10
>
> It's useful t
This function can be used from do_bootefi_exec() so that we use mostly the
same code for a normal EFI application and an EFI test.
Rename the function and use it in both places.
Signed-off-by: Simon Glass
---
Changes in v9: None
Changes in v8: None
Changes in v7:
- Drop patch "efi: Init the 'ro
This function is useful to signal that the application needs to exit
immediate. It can be caught with a debugger (e.g. gdb). Add a stub for it
so that it can be called from within sandbox when an internal error
occurs.
Signed-off-by: Simon Glass
---
Changes in v9: None
Changes in v8: None
Change
1 - 100 of 126 matches
Mail list logo
