On 2/15/20 8:39 PM, Simon Goldschmidt wrote:
> Am 15.02.2020 um 15:02 schrieb Marek Vasut:
>> The default timer rate may be different than 25 MHz, permit overriding
>> the default rate in board configuration file. Ultimatelly, this should
>> be properly handled by a clock driver, however that is
Hello,
I'm trying to get latest U-Boot to work on the NextThing C.H.I.P.
After borrowing NFC config from the Chip Pro .dts, I can write the SPL to NAND
after loading U-Boot via sunxi-fel and the bootrom will pick it up fine.
So the "full" sunxi_nand.c works.
SPL's sunxi_nand_spl.c appears to
Hi,
I'm trying to build an image for a NanoPi-M4 but it is not able to get out of
TPL. The U-Boot build seems to be constructing the idbloader.img properly, but
the board is failing to boot. Note that this is an original NanoPi-M4 (i.e.
V1) and not the NanoPi-M4V2.
>From the build log:
The value of local variable ecc is immediately overwritten. So we can
remove the first assignment.
Signed-off-by: Heinrich Schuchardt
---
drivers/ddr/marvell/axp/ddr3_init.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/ddr/marvell/axp/ddr3_init.c
Variable count is initialized at the start of every round of the while
loop and it is not used after the while loop. So there is no need to
initialize it beforehand.
Identified by cppcheck.
Signed-off-by: Heinrich Schuchardt
---
drivers/core/of_access.c | 2 +-
1 file changed, 1 insertion(+),
The value of parent is not changed in the first if statement. So we can
merge the two if statements depending on parent.
Indicated by cppcheck.
Signed-off-by: Heinrich Schuchardt
---
drivers/core/device.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git
Do not calculate a unused value of n which is overwritten in both branches
of the subsequent if statement.
Identified by cppcheck.
Signed-off-by: Heinrich Schuchardt
---
drivers/clk/ics8n3qv01.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/clk/ics8n3qv01.c
If all branches of a switch statement have a return instruction, all
subsequent lines are unreachable.
Identified with cppcheck.
Signed-off-by: Heinrich Schuchardt
---
drivers/clk/intel/clk_intel.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/clk/intel/clk_intel.c
Add missing break for CLKID_PCIE_PLL in switch statement.
Reported by CppCheck.
Cc: Neil Armstrong
Fixes: 08e09c263fdf ("clk: meson-g12a: Add PCIE PLL support")
Signed-off-by: Heinrich Schuchardt
---
Looking at the code it seems obvious that something is wrong in commit
08e09c263fdf but I have
Am 15.02.2020 um 15:02 schrieb Marek Vasut:
> The default timer rate may be different than 25 MHz, permit overriding
> the default rate in board configuration file. Ultimatelly, this should
> be properly handled by a clock driver, however that is not available
> on Gen5 yet.
Sigh, yes, I still
The default timer rate may be different than 25 MHz, permit overriding
the default rate in board configuration file. Ultimatelly, this should
be properly handled by a clock driver, however that is not available
on Gen5 yet.
Signed-off-by: Marek Vasut
Cc: Ley Foon Tan
Cc: Simon Goldschmidt
---
The Denali SPL shim won't build without these options set,
set them accordingly to fix the build error and let the SPL
shim to work correctly.
Signed-off-by: Marek Vasut
Cc: Ley Foon Tan
Cc: Simon Goldschmidt
---
include/configs/socfpga_common.h | 2 ++
1 file changed, 2 insertions(+)
diff
This adds a compatible string for m41st87. This ensures that this driver
can be used for m41st87 .
Signed-off-by: Marek Vasut
Cc: Simon Goldschmidt
Cc: Stefan Roese
---
drivers/rtc/m41t62.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/rtc/m41t62.c b/drivers/rtc/m41t62.c
index
Currently when booting the kernel on i.MX8 U-Boot hangs in an
endless loop when switching off dma, connectivity or lsio power
domains during device removal. It hapens first when removing
gpio0 (gpio@5d08) device, here its power domain device
'lsio_gpio0' is obtained for switching off power.
If CONFIG_LOG=n, we still expect output for log_err(), log_warning(),
log_notice(), log_info() and in case of DEBUG=1 also for log_debug().
Provide unit tests verifying this.
The tests depend on:
CONFIG_CONSOLE_RECORD=y
CONFIG_LOG=n
CONFIG_UT_LOG=y
It may be necessary
For testing purposes enable the syslog logging driver.
Signed-off-by: Heinrich Schuchardt
---
v3:
new patch
---
configs/sandbox64_defconfig| 1 +
configs/sandbox_defconfig | 1 +
configs/sandbox_flattree_defconfig | 1 +
3 files changed, 3 insertions(+)
diff --git
Provide a log driver that broadcasts RFC 3164 messages to syslog servers.
rsyslog is one implementation of such a server.
The messages are sent to the local broadcast address 255.255.255.255 on
port 514.
The environment variable log_hostname can be used to provide the HOSTNAME
field for the
If CONFIG_LOG=n, we should still output errors, warnings, notices, infos,
and for DEBUG=1 also debug messages.
Signed-off-by: Heinrich Schuchardt
Reviewed-by: Simon Glass
---
v3:
no change
---
include/log.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
Provide unit tests for the syslog logging driver.
Signed-off-by: Heinrich Schuchardt
---
v3:
new patch
---
test/log/Makefile | 4 +
test/log/syslog_test.c | 186 +
2 files changed, 190 insertions(+)
create mode 100644
An error
undefined reference to `do_log_test'
occurs for CONFIG_CMD_LOG=y, CONFIG_LOG_TEST=y, CONGIG_UNIT_TEST=n
Make CONFIG_UNIT_TEST a prerequisite.
Signed-off-by: Heinrich Schuchardt
Reviewed-by: Simon Glass
---
v3:
no change
---
common/Kconfig | 2 +-
1 file changed, 1
This patch series provides a logging driver to send syslog messages via
UDP port 514 to a syslog server and a unit test for the syslog driver.
For testing LOG_SYSLOG is enabled on the sandbox.
For CONFIG_LOG=n a patch enables printf() or debug() output for log_*
functions. A unit test for this
21 matches
Mail list logo