* [PATCH 1/8] ARM: optee-early: Fix 'memcpy' implicit declaration
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 2/8] ata: ahci: Fix format string Alexander Shiyan
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
arch/arm/lib32/optee-early.c: In function 'start_optee_early':
arch/arm/lib32/optee-early.c:26:2: error: implicit declaration of function 'memcpy' [-Werror=implicit-function-declaration]
memcpy((void *)hdr->init_load_addr_lo, tee + sizeof(*hdr), hdr->init_size);
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
arch/arm/lib32/optee-early.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/lib32/optee-early.c b/arch/arm/lib32/optee-early.c
index 197325b8a0..735d829c99 100644
--- a/arch/arm/lib32/optee-early.c
+++ b/arch/arm/lib32/optee-early.c
@@ -9,6 +9,7 @@
#include <asm/setjmp.h>
#include <tee/optee.h>
#include <debug_ll.h>
+#include <string.h>
static jmp_buf tee_buf;
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 2/8] ata: ahci: Fix format string
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
2022-06-10 5:40 ` [PATCH 1/8] ARM: optee-early: Fix 'memcpy' implicit declaration Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 3/8] gui: image_renderer: Fix empty prototype for image_renderer_image() Alexander Shiyan
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
drivers/ata/ahci.c: In function 'ahci_print_info':
drivers/ata/ahci.c:517:9: warning: too many arguments for format [-Wformat-extra-args]
Fixes: b5da6970bf23 ("ata: ahci: add missing capability in ahci_print_info()")
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
drivers/ata/ahci.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
index eb9e1bd133..f4b93183ce 100644
--- a/drivers/ata/ahci.c
+++ b/drivers/ata/ahci.c
@@ -304,8 +304,8 @@ static int ahci_init_port(struct ahci_port *ahci_port)
ahci_port->cmd_slot = mem;
ahci_port->cmd_slot_dma = mem_dma;
- ahci_port_debug(ahci_port, "cmd_slot = 0x%p (0x%pa)\n",
- ahci_port->cmd_slot, ahci_port->cmd_slot_dma);
+ ahci_port_debug(ahci_port, "cmd_slot = 0x%p (0x%pad)\n",
+ ahci_port->cmd_slot, &ahci_port->cmd_slot_dma);
/*
* Second item: Received-FIS area
@@ -320,8 +320,8 @@ static int ahci_init_port(struct ahci_port *ahci_port)
ahci_port->cmd_tbl = mem + AHCI_CMD_LIST_SZ + AHCI_RX_FIS_SZ;
ahci_port->cmd_tbl_dma = mem_dma + AHCI_CMD_LIST_SZ + AHCI_RX_FIS_SZ;
- ahci_port_debug(ahci_port, "cmd_tbl = 0x%p (0x%pa)\n",
- ahci_port->cmd_tbl, ahci_port->cmd_tbl_dma);
+ ahci_port_debug(ahci_port, "cmd_tbl = 0x%p (0x%pad)\n",
+ ahci_port->cmd_tbl, &ahci_port->cmd_tbl_dma);
ahci_port->cmd_tbl_sg = ahci_port->cmd_tbl + AHCI_CMD_TBL_HDR_SZ;
@@ -517,7 +517,7 @@ void ahci_print_info(struct ahci_device *ahci)
printf("flags: "
"%s%s%s%s%s%s%s"
"%s%s%s%s%s%s%s"
- "%s%s%s%s%s%s\n",
+ "%s%s%s%s%s%s%s\n",
cap & HOST_CAP_64 ? "64bit " : "",
cap & HOST_CAP_NCQ ? "ncq " : "",
cap & HOST_CAP_SNTF ? "sntf " : "",
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 3/8] gui: image_renderer: Fix empty prototype for image_renderer_image()
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
2022-06-10 5:40 ` [PATCH 1/8] ARM: optee-early: Fix 'memcpy' implicit declaration Alexander Shiyan
2022-06-10 5:40 ` [PATCH 2/8] ata: ahci: Fix format string Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 4/8] ARM: vexpress: Add missing "noreturn" attribute to restart function Alexander Shiyan
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
In file included from lib/gui/png.c:9:
include/gui/image_renderer.h: In function 'image_renderer_file':
include/gui/image_renderer.h:67:29: warning: passing argument 1 of 'image_renderer_image' from incompatible pointer type [-Wincompatible-pointer-types]
ret = image_renderer_image(sc, s, img);
include/gui/image_renderer.h:56:42: note: expected 'struct surface *' but argument is of type 'struct screen *'
int image_renderer_image(struct surface *s, struct image *img);
include/gui/image_renderer.h:67:33: warning: passing argument 2 of 'image_renderer_image' from incompatible pointer type [-Wincompatible-pointer-types]
ret = image_renderer_image(sc, s, img);
include/gui/image_renderer.h:56:59: note: expected 'struct image *' but argumentis of type 'struct surface *'
int image_renderer_image(struct surface *s, struct image *img);
include/gui/image_renderer.h:67:8: error: too many arguments to function 'image_renderer_image'
ret = image_renderer_image(sc, s, img);
include/gui/image_renderer.h:56:5: note: declared here
int image_renderer_image(struct surface *s, struct image *img);
Fixes: 3fa8d74abea3 ("gui: introduce screen and surface to factorize and simplify code")
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
include/gui/image_renderer.h | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/include/gui/image_renderer.h b/include/gui/image_renderer.h
index bfdea1b14e..323ad741b3 100644
--- a/include/gui/image_renderer.h
+++ b/include/gui/image_renderer.h
@@ -53,7 +53,10 @@ static inline struct image *image_renderer_open(const char* file)
static inline void image_renderer_close(struct image *img) {}
-int image_renderer_image(struct surface *s, struct image *img);
+static inline int image_renderer_image(struct screen *sc, struct surface *s, struct image *img)
+{
+ return -EINVAL;
+}
#endif
static inline int image_renderer_file(struct screen *sc, struct surface *s, const char* file)
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 4/8] ARM: vexpress: Add missing "noreturn" attribute to restart function
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (2 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 3/8] gui: image_renderer: Fix empty prototype for image_renderer_image() Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 5/8] ARM: versatile: " Alexander Shiyan
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
arch/arm/mach-vexpress/reset.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-vexpress/reset.c b/arch/arm/mach-vexpress/reset.c
index 78e452936d..01cd01e6df 100644
--- a/arch/arm/mach-vexpress/reset.c
+++ b/arch/arm/mach-vexpress/reset.c
@@ -14,7 +14,7 @@
void __iomem *v2m_wdt_base;
-static void vexpress_reset_soc(struct restart_handler *rst)
+static void __noreturn vexpress_reset_soc(struct restart_handler *rst)
{
writel(LOAD_MIN, v2m_wdt_base + WDTLOAD);
writeb(RESET_ENABLE, v2m_wdt_base + WDTCONTROL);
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 5/8] ARM: versatile: Add missing "noreturn" attribute to restart function
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (3 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 4/8] ARM: vexpress: Add missing "noreturn" attribute to restart function Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 6/8] lib: logo: Ignore more generated files Alexander Shiyan
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
arch/arm/mach-versatile/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-versatile/core.c b/arch/arm/mach-versatile/core.c
index 9a2a9cad80..9a81b5b6f3 100644
--- a/arch/arm/mach-versatile/core.c
+++ b/arch/arm/mach-versatile/core.c
@@ -173,7 +173,7 @@ void versatile_register_uart(unsigned id)
amba_apb_device_add(NULL, "uart-pl011", id, start, 4096, NULL, 0);
}
-static void versatile_reset_soc(struct restart_handler *rst)
+static void __noreturn versatile_reset_soc(struct restart_handler *rst)
{
u32 val;
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 6/8] lib: logo: Ignore more generated files
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (4 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 5/8] ARM: versatile: " Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 7/8] mtd: nand: orion: Disallow driver compile for ARMv4T Alexander Shiyan
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
lib/logo/.gitignore | 15 +++++----------
1 file changed, 5 insertions(+), 10 deletions(-)
diff --git a/lib/logo/.gitignore b/lib/logo/.gitignore
index 800a44c414..62ee7d270e 100644
--- a/lib/logo/.gitignore
+++ b/lib/logo/.gitignore
@@ -1,12 +1,7 @@
# SPDX-License-Identifier: GPL-2.0-only
-barebox-logo-w64.bblogo
-barebox-logo-w64.bblogo.S
-barebox-logo-w240.bblogo
-barebox-logo-w240.bblogo.S
-barebox-logo-w320.bblogo
-barebox-logo-w320.bblogo.S
-barebox-logo-w400.bblogo
-barebox-logo-w400.bblogo.S
-barebox-logo-w640.bblogo
-barebox-logo-w640.bblogo.S
+barebox-logo-*.bblogo
+barebox-logo-*.bblogo.S
+barebox-logo-*.bmp
+barebox-logo-*.png
+barebox-logo-*.qoi
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 7/8] mtd: nand: orion: Disallow driver compile for ARMv4T
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (5 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 6/8] lib: logo: Ignore more generated files Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 5:40 ` [PATCH 8/8] commands: Add CONSOLE dependency for EDIT command Alexander Shiyan
2022-06-10 7:54 ` [PATCH 0/8] Misc fixes part II Sascha Hauer
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
This fixes the following error (when ARMv4T and COMPILE_TEST are set):
{standard input}: Assembler messages:
{standard input}:197: Error: selected processor does not support "ldrd r2,[r4]" in ARM mode
CC drivers/virtio/virtio.o
make[3]: *** [/home/ARM/barebox/scripts/Makefile.build:140: drivers/mtd/nand/nand_orion.o] Error 1
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
drivers/mtd/nand/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index 3bdfabb439..2cd52f3820 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -82,7 +82,7 @@ config MTD_NAND_OMAP_ELM
config NAND_ORION
bool
prompt "Marvell Orion NAND driver"
- depends on ARM && (ARCH_KIRKWOOD || COMPILE_TEST)
+ depends on (ARM && !CPU_32v4T) && (ARCH_KIRKWOOD || COMPILE_TEST)
help
Support for the Orion NAND controller, present in Kirkwood SoCs.
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 8/8] commands: Add CONSOLE dependency for EDIT command
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (6 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 7/8] mtd: nand: orion: Disallow driver compile for ARMv4T Alexander Shiyan
@ 2022-06-10 5:40 ` Alexander Shiyan
2022-06-10 7:54 ` [PATCH 0/8] Misc fixes part II Sascha Hauer
8 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2022-06-10 5:40 UTC (permalink / raw)
To: barebox; +Cc: Alexander Shiyan
This fixes the following error (without CONSOLE support):
arm-linux-ld: commands/edit.o: in function "do_edit":
/home/ARM/barebox/commands/edit.c:684: undefined reference to "console_list"
make: *** [Makefile:872: .tmp_barebox1] Error 1
Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
commands/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/commands/Kconfig b/commands/Kconfig
index c5505321cf..86e4714849 100644
--- a/commands/Kconfig
+++ b/commands/Kconfig
@@ -1327,6 +1327,7 @@ config CMD_ECHO_E
config CMD_EDIT
tristate
+ depends on CONSOLE_FULL || CONSOLE_SIMPLE
prompt "edit"
help
A small fill-screen editor.
--
2.32.0
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH 0/8] Misc fixes part II
2022-06-10 5:40 [PATCH 0/8] Misc fixes part II Alexander Shiyan
` (7 preceding siblings ...)
2022-06-10 5:40 ` [PATCH 8/8] commands: Add CONSOLE dependency for EDIT command Alexander Shiyan
@ 2022-06-10 7:54 ` Sascha Hauer
8 siblings, 0 replies; 10+ messages in thread
From: Sascha Hauer @ 2022-06-10 7:54 UTC (permalink / raw)
To: Alexander Shiyan; +Cc: barebox
On Fri, Jun 10, 2022 at 08:40:42AM +0300, Alexander Shiyan wrote:
> Unsorted various fixes for the barebox.
>
> Alexander Shiyan (8):
> ARM: optee-early: Fix 'memcpy' implicit declaration
> ata: ahci: Fix format string
> gui: image_renderer: Fix empty prototype for image_renderer_image()
> ARM: vexpress: Add missing "noreturn" attribute to restart function
> ARM: versatile: Add missing "noreturn" attribute to restart function
> lib: logo: Ignore more generated files
> mtd: nand: orion: Disallow driver compile for ARMv4T
> commands: Add CONSOLE dependency for EDIT command
Applied, thanks
Sascha
>
> arch/arm/lib32/optee-early.c | 1 +
> arch/arm/mach-versatile/core.c | 2 +-
> arch/arm/mach-vexpress/reset.c | 2 +-
> commands/Kconfig | 1 +
> drivers/ata/ahci.c | 10 +++++-----
> drivers/mtd/nand/Kconfig | 2 +-
> include/gui/image_renderer.h | 5 ++++-
> lib/logo/.gitignore | 15 +++++----------
> 8 files changed, 19 insertions(+), 19 deletions(-)
>
> --
> 2.32.0
>
>
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
^ permalink raw reply [flat|nested] 10+ messages in thread