On 01.08.21 05:28, Tom Rini wrote:
On Sat, Jul 31, 2021 at 12:04:01PM +0200, Stefan Roese wrote:
Hi Tom,
please pull the next batch of Marvell MVEBU related patches. Here the
summary log:
First off, I've applied the whole series to u-boot/master and pushed.
Second, I see from:
commit 5fce28
Simon,
On Sat, Jul 31, 2021 at 10:59:32AM -0600, Simon Glass wrote:
> Hi Takahiro,
>
> On Tue, 27 Jul 2021 at 03:12, AKASHI Takahiro
> wrote:
> >
> > This new configuration, which was derived from sandbox_defconfig, will be
> > used solely to run efi capsule authentication test as the test requi
During cold start, with some HDDs, mv_sata_identify() does not populate
the ID words on the 1st ATA ID command. In fact, the first ATA ID
command will only power up the drive, and then the ATA ID command
processing is lost in the process.
Tests with:
- Seagate ST9250320AS 250GB HDD and Seagate ST
On Sat, Jul 31, 2021 at 12:04:01PM +0200, Stefan Roese wrote:
> Hi Tom,
>
> please pull the next batch of Marvell MVEBU related patches. Here the
> summary log:
First off, I've applied the whole series to u-boot/master and pushed.
Second, I see from:
commit 5fce2875569d6e859443af7af3477c3aebfee
On Sun, Jul 18, 2021 at 02:14:22PM -0600, Simon Glass wrote:
> From: Joe Perches
>
> Allow prefixing typical strings with L for wide strings and u for
> unicode strings.
>
> Signed-off-by: Joe Perches
> Signed-off-by: Simon Glass
> ---
> This was found on LKML:
>
> https://lore.kernel.org/lk
On Sun, Jul 18, 2021 at 09:36:42PM -0400, Tom Rini wrote:
> On Mon, Jul 19, 2021 at 03:07:50AM +0200, Heinrich Schuchardt wrote:
> > On 7/19/21 3:01 AM, Tom Rini wrote:
> > > On Mon, Jul 19, 2021 at 02:41:46AM +0200, Heinrich Schuchardt wrote:
> > > > On 7/18/21 10:14 PM, Simon Glass wrote:
> > > >
Both am335x_boneblack_vboot and am335x_evm_spiboot require
SPL_OF_CONTROL to function but are currently missing this option. Add
it.
Signed-off-by: Tom Rini
---
configs/am335x_boneblack_vboot_defconfig | 1 +
configs/am335x_evm_spiboot_defconfig | 1 +
2 files changed, 2 insertions(+)
diff
On Sat, Jul 31, 2021 at 05:07:41PM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Mon, 26 Jul 2021 at 08:43, Tom Rini wrote:
> >
> > On Mon, Jul 26, 2021 at 07:45:27AM -0600, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Mon, 26 Jul 2021 at 06:09, Tom Rini wrote:
> > > >
> > > > On Sun, Jul 25, 20
Hi Tom,
On Mon, 26 Jul 2021 at 08:43, Tom Rini wrote:
>
> On Mon, Jul 26, 2021 at 07:45:27AM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On Mon, 26 Jul 2021 at 06:09, Tom Rini wrote:
> > >
> > > On Sun, Jul 25, 2021 at 09:57:25PM -0600, Simon Glass wrote:
> > > > Hi Tom,
> > > >
> > > > On Sun
The examples here are a bit messed up since the command does not match
the documentation. Use a different example instead.
Signed-off-by: Simon Glass
---
doc/develop/moveconfig.rst | 58 +++---
1 file changed, 22 insertions(+), 36 deletions(-)
Applied to u-boot-
An int array can hold a single int so we should not need to do anything
in the widening operation. However due to a quirk in the code, an int[3]
widened with an int produced an int[4]. Fix this and add a test.
Fix a comment typo while we are here.
Signed-off-by: Simon Glass
Reported-by: Tom Rini
Hi Heinrich,
On Wed, 21 Jul 2021 at 23:53, Heinrich Schuchardt wrote:
>
> On 7/22/21 5:35 AM, Simon Glass wrote:
> > This tool has nothing to do with testing. It is for refactoring code
> > automatically using a 'semantic patch' tool.
> >
> > Create a new section for 'refactoring' and move it int
Hi Heinrich,
On Wed, 21 Jul 2021 at 23:52, Heinrich Schuchardt wrote:
>
> On 7/22/21 5:35 AM, Simon Glass wrote:
> > At present this information is hidden away. Make it more visible by
> > putting it first, in an intro section.
> >
> > Signed-off-by: Simon Glass
> > ---
> >
> > doc/develop/ind
Move these docs into htmldocs so they can be read there.
Signed-off-by: Simon Glass
---
doc/develop/index.rst | 1 +
doc/develop/moveconfig.rst | 296 +
tools/moveconfig.py| 291 +---
3 files changed, 298 inserti
Some of the more advanced features of this tool don't work anymore since
kconfiglib was update. Update the code accordingly.
Signed-off-by: Simon Glass
---
tools/moveconfig.py | 24
1 file changed, 12 insertions(+), 12 deletions(-)
Applied to u-boot-dm, thanks!
The current name is confusing because the logic is actually backwards from
what you might expect. Rename it to needs_widening() and update the
comments.
Signed-off-by: Simon Glass
---
tools/dtoc/fdt.py | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
Applied to u-boot-dm,
Hi Simon,
On 7/24/21 11:14 PM, Simon Glass wrote:
> The autoboot tests are a recent addition to U-Boot, providing much-needed
> coverage in this area.
>
> A side effect of the keyed autoboot test is that this feature is enabled
> in sandbox always. This changes the autoboot prompt and confuses the
At present if we see 'ranges' property (with no value) we assume it is a
boolean, as per the devicetree spec.
But another node may define 'ranges' with a value, forcing us to widen it
to an int array. At present this is not supported and causes an error.
Fix this and add some test cases.
Signed-
Hi Stefan,
On Sat, Jul 31, 2021 at 4:50 AM Stefan Roese wrote:
>
> On 31.07.21 12:27, Stefan Roese wrote:
> > Hi Tony,
> >
> > (added Joe & Ramon as network custodians)
> >
> > On 31.07.21 11:55, Tony Dinh wrote:
> >> Hi Stefan,
> >>
> >> On Sat, Jul 31, 2021 at 12:41 AM Stefan Roese wrote:
> >>
Hi Walter,
On Sun, 25 Jul 2021 at 18:43, Walter Lozano wrote:
>
> Hi Simon,
>
> Thanks for the patch, it will be a nice way to spot errors and avoid
> headaches!
>
> On 7/7/21 8:22 AM, Simon Glass wrote:
> > With of-platdata-inst we want to set up a reference to each devices'
> > parent device, i
On Fri, 30 Jul 2021 at 09:05, Heinrich Schuchardt
wrote:
>
> With glibc 2.33 (Ubuntu package glibc6 2.33-0ubuntu9) building
> sifive_unmatched_defconfig results in:
>
> In file included from /usr/include/stdio.h:866,
> from ././include/compiler.h:26,
> from :
>
On Fri, 30 Jul 2021 at 04:13, Patrick Delaunay
wrote:
>
> Migrate the dm_warn function to log macro with
> LOGC_DM category and LOGL_WARNING level.
>
> This macro allows filtering with log command and allows
> output on all log backend.
>
> Signed-off-by: Patrick Delaunay
> ---
>
> drivers/core/
On Fri, 30 Jul 2021 at 04:12, Patrick Delaunay
wrote:
>
> Add a message on probe in driver model core when the default
> pinctrl selection failed.
>
> This message is displayed only when the pinctrl API is
> implemented, i.e. when result is not ENOSYS.
>
> Signed-off-by: Patrick Delaunay
> ---
>
Hi,
On Wed, 28 Jul 2021 at 23:13, Jonathan Gray wrote:
>
> On Wed, Jul 28, 2021 at 09:04:33PM +0300, Artem Panfilov wrote:
> > Fix LibreSSL compilation for versions before v2.7.0.
>
> Why 2.7.0? I had to disable CONFIG_FIT_SIGNATURE to get the qemu
> targets to build on OpenBSD-current (3.4.0) a
On Fri, 30 Jul 2021 at 04:12, Patrick Delaunay
wrote:
>
> Use the return value ENOSYS for unsupported API
> - pinctrl_generic_set_state
> - pinctrl_select_state
>
> Signed-off-by: Patrick Delaunay
> ---
>
> include/dm/pinctrl.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewe
Hi Takahiro,
On Tue, 27 Jul 2021 at 03:12, AKASHI Takahiro
wrote:
>
> This new configuration, which was derived from sandbox_defconfig, will be
> used solely to run efi capsule authentication test as the test requires
> a public key (esl file) to be embedded in U-Boot binary.
>
> Signed-off-by: A
On Mon, 26 Jul 2021 at 21:51, Sean Anderson wrote:
>
> Having to copy-paste the same 3 lines makes adding new test cases
> error-prone. Use a macro.
>
> Signed-off-by: Sean Anderson
> ---
>
> test/dm/k210_pll.c | 30 --
> 1 file changed, 12 insertions(+), 18 deletions
Hi Roman,
On Sat, 31 Jul 2021 at 02:26, Roman Kopytin wrote:
>
> Thank, but my question was about adding of the public key to dtb file without
> private key. We won't have private key in our side.
(please try not to top-post on the mailing list)
Presumably this means that you know what the pub
Hi Donald,
On Wed, 28 Jul 2021 at 18:35, Donald Chan wrote:
>
> If the 'keyfile' (-G) argument is used, there is little value to require
> 'keydir' (-k) argument since the public key can also be extracted from the
> private key itself.
>
> Signed-off-by: Donald Chan
> ---
> lib/rsa/rsa-sign.c |
The board routes the Integrated Graphics Device (IGD) to an LVDS
panel, which is less popular than a PCIe based graphics card.
Disable the IGD so that it does not show up in the PCI configuration
space as a VGA display controller, so we can use an external PCIe
graphics card with whatever cable we
Initialize igd and sdvo to NULL so that we don't need to test the
return value of dm_pci_bus_find_bdf() later.
Signed-off-by: Bin Meng
---
arch/x86/cpu/queensbay/tnc.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/arch/x86/cpu/queensbay/tnc.c b/arch/x86/cpu/queensba
Config option CONFIG_SYS_TCLK is set by kw88f6281.h and kw88f6192.h files
to correct SOC/platform value. So do not overwrite it in board config
include files.
Kirkwood 88F6180 and 88F6192 uses 166 MHz TCLK and Kirkwood 88F6281 uses
200 MHz TCLK.
Signed-off-by: Pali Rohár
---
arch/arm/mach-kirkw
This mvebu axp platform always uses fixed 250 MHz TCLK. So specify this
CONFIG_SYS_TCLK option in msys section of global file soc.h file instead of
manual configuration in every board file.
Now every #if-#else case of soc.h file defines CONFIG_SYS_TCLK, so remove
useless default CONFIG_SYS_TCLK va
This mvebu msys platform always uses fixed 200 MHz TCLK. So specify this
CONFIG_SYS_TCLK option in msys section of global file soc.h file instead of
manual configuration in every board file.
Signed-off-by: Pali Rohár
---
arch/arm/mach-mvebu/include/mach/soc.h | 2 ++
include/configs/crs3xx-98dx3
Bit 20 in SAR register specifies if TCLK is running at 200 MHz or 166 MHz.
Use this information instead of manual configuration in every board file.
Signed-off-by: Pali Rohár
---
arch/arm/mach-mvebu/include/mach/soc.h | 3 +++
include/configs/db-88f6720.h | 1 -
2 files changed, 3 inse
Bit 15 in SAR register specifies if TCLK is running at 200 MHz or 250 MHz.
Use this information instead of manual configuration in every board file.
Signed-off-by: Pali Rohár
---
arch/arm/mach-mvebu/include/mach/soc.h | 13 -
include/configs/clearfog.h | 1 -
include/con
This patch series automatically detects CONFIG_SYS_TCLK value, set it in
global mach/soc.h file and remove manual CONFIG_SYS_TCLK value from all
mvebu/kirkwood board config files.
So it just a cleanup to avoid defining custom macros in board config
files which can be automatically detected.
Could
On 31.07.21 12:27, Stefan Roese wrote:
Hi Tony,
(added Joe & Ramon as network custodians)
On 31.07.21 11:55, Tony Dinh wrote:
Hi Stefan,
On Sat, Jul 31, 2021 at 12:41 AM Stefan Roese wrote:
On 26.07.21 08:01, Tony Dinh wrote:
In DM Ethernet, the old "egiga0" and 'egiga1" names are no longe
Thank, but my question was about adding of the public key to dtb file without
private key. We won't have private key in our side.
-Original Message-
From: Thomas Perrot
Sent: Saturday, July 31, 2021 9:52 AM
To: Roman Kopytin ; Michael Nazzareno Trimarchi
Cc: U-Boot-Denx ; Simon Glass
Hi Tony,
(added Joe & Ramon as network custodians)
On 31.07.21 11:55, Tony Dinh wrote:
Hi Stefan,
On Sat, Jul 31, 2021 at 12:41 AM Stefan Roese wrote:
On 26.07.21 08:01, Tony Dinh wrote:
In DM Ethernet, the old "egiga0" and 'egiga1" names are no longer valid,
so replace these with Ethernet
Hi Rasmus,
On 15.07.21 10:15, Stefan Roese wrote:
Hi Rasmus,
On 05.07.21 17:30, Simon Glass wrote:
Hi Rasmus,
On Fri, 2 Jul 2021 at 06:45, Rasmus Villemoes
wrote:
A board can have and make use of more than one watchdog device, say
one built into the SOC and an external gpio-petted one. Hav
Hi Tom,
please pull the next batch of Marvell MVEBU related patches. Here the
summary log:
- kwboot / kwbimage improvements reducing image size (Pali & Marek)
- a37xx: pinctrl: Correct PWM pins definitions (Marek)
- Convert the Dre
On 26.07.21 14:58, Pali Rohár wrote:
Static inline function _debug_uart_init() should avoid calling external
(non-inline) functions. Therefore do not call get_ref_clk() in
_debug_uart_init() and reimplement its functionality without external
function calls.
Signed-off-by: Pali Rohár
Applied t
On 26.07.21 14:58, Pali Rohár wrote:
CONFIG_BAUDRATE should be used for setting the baudrate for the early debug
UART. This replaces current hardcoded 115200 value.
Signed-off-by: Pali Rohár
Applied to u-boot-marvell/master
Thanks,
Stefan
---
drivers/serial/serial_mvebu_a3700.c | 5 ++---
On 23.07.21 19:57, Marek Behún wrote:
The PWM pins on North Bridge on Armada 37xx can be configured into PWM
or GPIO functions. When in PWM function, each pin can also be configured
to drive low on 0 and tri-state on 1 (LED mode).
The current definitions handle this by declaring two pin groups f
On 23.07.21 11:13, Marek Behún wrote:
Hi Stefan and others,
this is v3 of the series of improvements to kwboot, kwbimage and mvebu,
with main motivation the reduction of KWB image size.
CI successful, see
https://dev.azure.com/u-boot/u-boot/_build/results?buildId=2569&view=results
Changes sinc
On 26.07.21 08:01, Tony Dinh wrote:
- Enable DM SATA, removed IDE driver, add SATA MV driver.
- Use Ethernet PHY names from device tree.
- Replace the old Ethernet PHY addr lookup with a device tree parsing
lookup function.
Tony Dinh (3):
arm: kirkwood: Dreamplug: Add DM Ethernet and DM SAT
Hi Stefan,
On Sat, Jul 31, 2021 at 12:41 AM Stefan Roese wrote:
>
> On 26.07.21 08:01, Tony Dinh wrote:
> > In DM Ethernet, the old "egiga0" and 'egiga1" names are no longer valid,
> > so replace these with Ethernet PHY names from device tree. Also, read
> > Ethernet PHY address for each port fro
Hi Stefan,
On Sat, Jul 31, 2021 at 12:31 AM Stefan Roese wrote:
>
> Hi Tony,
>
> On 25.07.21 23:57, Tony Dinh wrote:
> > During cold start, with some HDDs, mv_sata_identify() does not populate
> > the ID words on the 1st ATA ID command. In fact, the first ATA ID
> > command will only power up th
For FSP1, there is no such INIT_PHASE_END_FIRMWARE.
Fixes: 7c73cea44290 ("x86: Notify the FSP of the 'end firmware' event")
Signed-off-by: Bin Meng
---
arch/x86/lib/fsp/fsp_common.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/arch/x86/lib/fsp/fsp_common
The resource type for system memory is currently displayed as
"unknown", which is wrong.
Fixes: 51af144eb7a0 ("x86: Allow showing details about a HOB entry")
Signed-off-by: Bin Meng
---
cmd/x86/hob.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/x86/hob.c b/cmd/x86/hob
At present the hob command usage and help messages are messed up
in a single line. They should be separated.
This was a regression introduced when [seq] and [-v] were added
to the command.
Fixes: d11544dfa9f4 ("x86: hob: Add way to show a single hob entry")
Fixes: 51af144eb7a0 ("x86: Allow showin
The size parameter of mtrr_add_request() and mtrr_set_next_var()
shall be power of 2, otherwise the logic creates a mask that does
not meet the requirement of IA32_MTRR_PHYSMASK register.
Programming such a mask value to IA32_MTRR_PHYSMASK generates #GP.
Signed-off-by: Bin Meng
---
arch/x86/cp
At present Intel Crown Bay does not boot. This was caused by various
regression issues introduced when supporting FSP2, and some flaws in
MTRR codes.
With this series, U-Boot boot again on Intel Crown Bay board.
This series is available at u-boot-x86/crownbay for testing.
Bin Meng (7):
x86: f
Current mtrr_commit() logic assumes that MTRR MSRs are programmed
consecutively from index 0 to its maximum number, and whenever it
detects an unused one, it clears all other MTRRs starting from that
one. However this may not always be the case.
In fact, the clear is not much helpful because these
At present mtrr_commit() programs the MTRR MSRs starting from
index 0, which may overwrite MSRs that were already programmed
by previous boot stage or FSP.
Switch to call mtrr_set_next_var() instead.
Signed-off-by: Bin Meng
---
arch/x86/cpu/mtrr.c | 2 +-
1 file changed, 1 insertion(+), 1 dele
This actually reverts the following 2 commits:
commit 427911001809 ("x86: Set up the MTRR for SDRAM")
commit d46c0932a9d4 ("x86: fsp: Adjust calculations for MTRR range and DRAM
top")
There are several outstanding issues as to why:
* For FSP1, the system memory and reserved memory used by FSP a
Thanks, Michael.
Can we sign in the separate state on special server for example?
Looks like we can work with public key only in this step.
From: Michael Nazzareno Trimarchi
Sent: Friday, July 30, 2021 8:50 PM
To: Roman Kopytin
Cc: U-Boot-Denx ; Simon Glass
Subject: Re: U-boot
Caution: This is
On 26.07.21 14:58, Pali Rohár wrote:
Static inline function _debug_uart_init() should avoid calling external
(non-inline) functions. Therefore do not call get_ref_clk() in
_debug_uart_init() and reimplement its functionality without external
function calls.
Signed-off-by: Pali Rohár
Apart fro
On 26.07.21 14:58, Pali Rohár wrote:
CONFIG_BAUDRATE should be used for setting the baudrate for the early debug
UART. This replaces current hardcoded 115200 value.
Signed-off-by: Pali Rohár
Reviewed-by: Stefan Roese
Thanks,
Stefan
---
drivers/serial/serial_mvebu_a3700.c | 5 ++---
1 f
On 26.07.21 08:01, Tony Dinh wrote:
- Enable DM SATA, removed IDE driver, and add SATA MV driver.
- Use ethernet PHY names from device tree in default boot command
Signed-off-by: Tony Dinh
Reviewed-by: Stefan Roese
Thanks,
Stefan
---
include/configs/dreamplug.h | 12 ++--
1 fi
On 26.07.21 08:01, Tony Dinh wrote:
Add DM_ETH, SATA_MV and associated configs to dreamplug_defconfig
Signed-off-by: Tony Dinh
Reviewed-by: Stefan Roese
Thanks,
Stefan
---
configs/dreamplug_defconfig | 4
1 file changed, 4 insertions(+)
diff --git a/configs/dreamplug_defconfig b
On 26.07.21 08:01, Tony Dinh wrote:
In DM Ethernet, the old "egiga0" and 'egiga1" names are no longer valid,
so replace these with Ethernet PHY names from device tree. Also, read
Ethernet PHY address for each port from device tree.
Signed-off-by: Tony Dinh
---
board/Marvell/dreamplug/dreampl
On 23.07.21 19:57, Marek Behún wrote:
The PWM pins on North Bridge on Armada 37xx can be configured into PWM
or GPIO functions. When in PWM function, each pin can also be configured
to drive low on 0 and tri-state on 1 (LED mode).
The current definitions handle this by declaring two pin groups f
Hi Tony,
On 25.07.21 23:57, Tony Dinh wrote:
During cold start, with some HDDs, mv_sata_identify() does not populate
the ID words on the 1st ATA ID command. In fact, the first ATA ID
command will only power up the drive, and then the ATA ID command
processing is lost in the process.
Tests with:
65 matches
Mail list logo
