* [PATCH] button: add udevice forward declaration
@ 2021-02-19 7:41 ` Neil Armstrong
0 siblings, 0 replies; 5+ messages in thread
From: Neil Armstrong @ 2021-02-19 7:41 UTC (permalink / raw)
To: u-boot
After 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
build fails with :
drivers/button/button-uclass.c:13:5: error: conflicting types for 'button_get_by_label'
int button_get_by_label(const char *label, struct udevice **devp)
^~~~~~~~~~~~~~~~~~~
Adding struct udevice forward declaration in button.h solves the build error.
Fixes: 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
Hi,
This is needed to fix build on vim3/vim3l after [1] is applied.
[1] https://patchwork.ozlabs.org/project/uboot/cover/20210218103318.18915-1-m.szyprowski at samsung.com/
include/button.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/button.h b/include/button.h
index 688b63b082..ee14fadf0c 100644
--- a/include/button.h
+++ b/include/button.h
@@ -6,6 +6,8 @@
#ifndef __BUTTON_H
#define __BUTTON_H
+struct udevice;
+
/**
* struct button_uc_plat - Platform data the uclass stores about each device
*
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] button: add udevice forward declaration
@ 2021-02-19 7:41 ` Neil Armstrong
0 siblings, 0 replies; 5+ messages in thread
From: Neil Armstrong @ 2021-02-19 7:41 UTC (permalink / raw)
To: Simon Glass, xypron.glpk
Cc: Neil Armstrong, m.szyprowski, u-boot, u-boot-amlogic, philippe.reynes
After 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
build fails with :
drivers/button/button-uclass.c:13:5: error: conflicting types for 'button_get_by_label'
int button_get_by_label(const char *label, struct udevice **devp)
^~~~~~~~~~~~~~~~~~~
Adding struct udevice forward declaration in button.h solves the build error.
Fixes: 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
Hi,
This is needed to fix build on vim3/vim3l after [1] is applied.
[1] https://patchwork.ozlabs.org/project/uboot/cover/20210218103318.18915-1-m.szyprowski@samsung.com/
include/button.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/button.h b/include/button.h
index 688b63b082..ee14fadf0c 100644
--- a/include/button.h
+++ b/include/button.h
@@ -6,6 +6,8 @@
#ifndef __BUTTON_H
#define __BUTTON_H
+struct udevice;
+
/**
* struct button_uc_plat - Platform data the uclass stores about each device
*
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] button: add udevice forward declaration
2021-02-19 7:41 ` Neil Armstrong
(?)
@ 2021-02-20 11:55 ` Simon Glass
2021-02-22 17:39 ` Neil Armstrong
-1 siblings, 1 reply; 5+ messages in thread
From: Simon Glass @ 2021-02-20 11:55 UTC (permalink / raw)
To: u-boot
On Fri, 19 Feb 2021 at 00:41, Neil Armstrong <narmstrong@baylibre.com> wrote:
>
> After 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
> build fails with :
>
> drivers/button/button-uclass.c:13:5: error: conflicting types for 'button_get_by_label'
> int button_get_by_label(const char *label, struct udevice **devp)
> ^~~~~~~~~~~~~~~~~~~
>
> Adding struct udevice forward declaration in button.h solves the build error.
>
> Fixes: 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
> Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> Hi,
>
> This is needed to fix build on vim3/vim3l after [1] is applied.
>
> [1] https://patchwork.ozlabs.org/project/uboot/cover/20210218103318.18915-1-m.szyprowski at samsung.com/
>
> include/button.h | 2 ++
> 1 file changed, 2 insertions(+)
>
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] button: add udevice forward declaration
2021-02-20 11:55 ` Simon Glass
@ 2021-02-22 17:39 ` Neil Armstrong
0 siblings, 0 replies; 5+ messages in thread
From: Neil Armstrong @ 2021-02-22 17:39 UTC (permalink / raw)
To: u-boot
On 20/02/2021 12:55, Simon Glass wrote:
> On Fri, 19 Feb 2021 at 00:41, Neil Armstrong <narmstrong@baylibre.com> wrote:
>>
>> After 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
>> build fails with :
>>
>> drivers/button/button-uclass.c:13:5: error: conflicting types for 'button_get_by_label'
>> int button_get_by_label(const char *label, struct udevice **devp)
>> ^~~~~~~~~~~~~~~~~~~
>>
>> Adding struct udevice forward declaration in button.h solves the build error.
>>
>> Fixes: 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
>> Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>> ---
>> Hi,
>>
>> This is needed to fix build on vim3/vim3l after [1] is applied.
>>
>> [1] https://patchwork.ozlabs.org/project/uboot/cover/20210218103318.18915-1-m.szyprowski at samsung.com/
>>
>> include/button.h | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
Thanks, applied to u-boot-amlogic
Neil
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] button: add udevice forward declaration
@ 2021-02-22 17:39 ` Neil Armstrong
0 siblings, 0 replies; 5+ messages in thread
From: Neil Armstrong @ 2021-02-22 17:39 UTC (permalink / raw)
To: Simon Glass
Cc: Heinrich Schuchardt, Marek Szyprowski, U-Boot Mailing List,
u-boot-amlogic, Philippe Reynes
On 20/02/2021 12:55, Simon Glass wrote:
> On Fri, 19 Feb 2021 at 00:41, Neil Armstrong <narmstrong@baylibre.com> wrote:
>>
>> After 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
>> build fails with :
>>
>> drivers/button/button-uclass.c:13:5: error: conflicting types for 'button_get_by_label'
>> int button_get_by_label(const char *label, struct udevice **devp)
>> ^~~~~~~~~~~~~~~~~~~
>>
>> Adding struct udevice forward declaration in button.h solves the build error.
>>
>> Fixes: 401d1c4f5d2d ("common: Drop asm/global_data.h from common header")
>> Suggested-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>> ---
>> Hi,
>>
>> This is needed to fix build on vim3/vim3l after [1] is applied.
>>
>> [1] https://patchwork.ozlabs.org/project/uboot/cover/20210218103318.18915-1-m.szyprowski@samsung.com/
>>
>> include/button.h | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>
> Reviewed-by: Simon Glass <sjg@chromium.org>
>
Thanks, applied to u-boot-amlogic
Neil
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-02-22 17:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-19 7:41 [PATCH] button: add udevice forward declaration Neil Armstrong
2021-02-19 7:41 ` Neil Armstrong
2021-02-20 11:55 ` Simon Glass
2021-02-22 17:39 ` Neil Armstrong
2021-02-22 17:39 ` Neil Armstrong
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.