Re: [U-Boot] [PATCH v1 03/11] include: kernel.h: include printk.h
On Thu, Oct 17, 2019 at 02:58:45PM +0900, AKASHI Takahiro wrote: > On Sat, Oct 12, 2019 at 01:47:06PM +0200, Heinrich Schuchardt wrote: > > On 10/11/19 9:41 AM, AKASHI Takahiro wrote: > > >Adding "printk.h" will help improve portability from linux kernel > > >code (in my case, lib/asn1_decoder.c). > > > > > >Signed-off-by: AKASHI Takahiro > > >--- > > > include/linux/kernel.h | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > >diff --git a/include/linux/kernel.h b/include/linux/kernel.h > > >index a85c15d8dc28..919d12bdf89c 100644 > > >--- a/include/linux/kernel.h > > >+++ b/include/linux/kernel.h > > >@@ -2,7 +2,9 @@ > > > #define _LINUX_KERNEL_H > > > > > > > > >+#include /* for printf utilities */ > > > > Kernel code has no vsprintf.h. Linux's lib/asn1_decoder.c has not a > > single print statement. So why are you inserting vsprintf.h here? > > I don't remember why I mentioned to asn1_decoder.c here, but For the record, asn1_parser.c uses pr_debug/pr_devel and others in many places. -Takahiro Akashi > When this statement is removed from kernel.h, some files cannot > be compiled. > The fact is that sprintf() is used in time.h without including any > related headers. > (There are still bunch of *missing headers* issues in U-Boot.) > > I will add one more patch against time.h here. > > Thanks, > -Takahiro Akashi > > > > > #include > > >+#include > > > > Linux include/linux/kernel.h also includes printk.h. OK. > > > > Best regards > > > > Heinrich > > > > > > #define USHRT_MAX((u16)(~0U)) > > > #define SHRT_MAX ((s16)(USHRT_MAX>>1)) > > > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v1 03/11] include: kernel.h: include printk.h
On Sat, Oct 12, 2019 at 01:47:06PM +0200, Heinrich Schuchardt wrote: > On 10/11/19 9:41 AM, AKASHI Takahiro wrote: > >Adding "printk.h" will help improve portability from linux kernel > >code (in my case, lib/asn1_decoder.c). > > > >Signed-off-by: AKASHI Takahiro > >--- > > include/linux/kernel.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > >diff --git a/include/linux/kernel.h b/include/linux/kernel.h > >index a85c15d8dc28..919d12bdf89c 100644 > >--- a/include/linux/kernel.h > >+++ b/include/linux/kernel.h > >@@ -2,7 +2,9 @@ > > #define _LINUX_KERNEL_H > > > > > >+#include /* for printf utilities */ > > Kernel code has no vsprintf.h. Linux's lib/asn1_decoder.c has not a > single print statement. So why are you inserting vsprintf.h here? I don't remember why I mentioned to asn1_decoder.c here, but When this statement is removed from kernel.h, some files cannot be compiled. The fact is that sprintf() is used in time.h without including any related headers. (There are still bunch of *missing headers* issues in U-Boot.) I will add one more patch against time.h here. Thanks, -Takahiro Akashi > > #include > >+#include > > Linux include/linux/kernel.h also includes printk.h. OK. > > Best regards > > Heinrich > > > > #define USHRT_MAX ((u16)(~0U)) > > #define SHRT_MAX ((s16)(USHRT_MAX>>1)) > > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH v1 03/11] include: kernel.h: include printk.h
On 10/11/19 9:41 AM, AKASHI Takahiro wrote: Adding "printk.h" will help improve portability from linux kernel code (in my case, lib/asn1_decoder.c). Signed-off-by: AKASHI Takahiro --- include/linux/kernel.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/linux/kernel.h b/include/linux/kernel.h index a85c15d8dc28..919d12bdf89c 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h @@ -2,7 +2,9 @@ #define _LINUX_KERNEL_H +#include /* for printf utilities */ Kernel code has no vsprintf.h. Linux's lib/asn1_decoder.c has not a single print statement. So why are you inserting vsprintf.h here? #include +#include Linux include/linux/kernel.h also includes printk.h. OK. Best regards Heinrich #define USHRT_MAX ((u16)(~0U)) #define SHRT_MAX ((s16)(USHRT_MAX>>1)) ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH v1 03/11] include: kernel.h: include printk.h
Adding "printk.h" will help improve portability from linux kernel code (in my case, lib/asn1_decoder.c). Signed-off-by: AKASHI Takahiro --- include/linux/kernel.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/linux/kernel.h b/include/linux/kernel.h index a85c15d8dc28..919d12bdf89c 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h @@ -2,7 +2,9 @@ #define _LINUX_KERNEL_H +#include /* for printf utilities */ #include +#include #define USHRT_MAX ((u16)(~0U)) #define SHRT_MAX ((s16)(USHRT_MAX>>1)) -- 2.21.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
