* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-16 23:59 Kim Phillips 2013-01-17 0:36 ` Scott Wood 2013-01-18 12:02 ` Jerry Van Baren 0 siblings, 2 replies; 15+ messages in thread From: Kim Phillips @ 2013-01-16 23:59 UTC (permalink / raw) To: u-boot and, if including libfdt.h which includes libfdt_env.h in the correct order, don't include fdt.h before libfdt.h. this is needed to get the fdt type definitions set from the project environment before fdt.h uses them. Signed-off-by: Kim Phillips <kim.phillips@freescale.com> Cc: Jerry Van Baren <gvb.uboot@gmail.com> --- arch/arm/lib/bootm.c | 1 - arch/powerpc/lib/bootm.c | 1 - arch/x86/include/asm/arch-coreboot/sysinfo.h | 1 + common/cmd_bootm.c | 1 - common/cmd_fdt.c | 1 - common/fdt_support.c | 1 - common/image.c | 1 - include/fdt_support.h | 1 + include/image.h | 1 - tools/fdt_host.h | 1 - tools/imls/imls.c | 1 - 11 files changed, 2 insertions(+), 9 deletions(-) diff --git a/arch/arm/lib/bootm.c b/arch/arm/lib/bootm.c index 1bd2730..f3b30c5 100644 --- a/arch/arm/lib/bootm.c +++ b/arch/arm/lib/bootm.c @@ -30,7 +30,6 @@ #include <image.h> #include <u-boot/zlib.h> #include <asm/byteorder.h> -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #include <asm/bootm.h> diff --git a/arch/powerpc/lib/bootm.c b/arch/powerpc/lib/bootm.c index 7088293..33d013f 100644 --- a/arch/powerpc/lib/bootm.c +++ b/arch/powerpc/lib/bootm.c @@ -36,7 +36,6 @@ #include <asm/mp.h> #if defined(CONFIG_OF_LIBFDT) -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> diff --git a/arch/x86/include/asm/arch-coreboot/sysinfo.h b/arch/x86/include/asm/arch-coreboot/sysinfo.h index 77ae304..bd88eb5 100644 --- a/arch/x86/include/asm/arch-coreboot/sysinfo.h +++ b/arch/x86/include/asm/arch-coreboot/sysinfo.h @@ -32,6 +32,7 @@ #include <common.h> #include <compiler.h> +#include <libfdt_env.h> #include <fdt.h> #include <asm/arch/tables.h> diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c index 1b8a8c1..e886c4a 100644 --- a/common/cmd_bootm.c +++ b/common/cmd_bootm.c @@ -47,7 +47,6 @@ #endif #if defined(CONFIG_OF_LIBFDT) -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #endif diff --git a/common/cmd_fdt.c b/common/cmd_fdt.c index 9e2de34..6eec947 100644 --- a/common/cmd_fdt.c +++ b/common/cmd_fdt.c @@ -29,7 +29,6 @@ #include <linux/ctype.h> #include <linux/types.h> #include <asm/global_data.h> -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> diff --git a/common/fdt_support.c b/common/fdt_support.c index 6b9fa05..7a3fee4 100644 --- a/common/fdt_support.c +++ b/common/fdt_support.c @@ -28,7 +28,6 @@ #include <linux/ctype.h> #include <linux/types.h> #include <asm/global_data.h> -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #include <exports.h> diff --git a/common/image.c b/common/image.c index 95498e6..ae1a9d3 100644 --- a/common/image.c +++ b/common/image.c @@ -47,7 +47,6 @@ #include <image.h> #if defined(CONFIG_FIT) || defined(CONFIG_OF_LIBFDT) -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #endif diff --git a/include/fdt_support.h b/include/fdt_support.h index 4b9f84a..a08c90b 100644 --- a/include/fdt_support.h +++ b/include/fdt_support.h @@ -26,6 +26,7 @@ #ifdef CONFIG_OF_LIBFDT +#include <libfdt_env.h> #include <fdt.h> u32 fdt_getprop_u32_default(const void *fdt, const char *path, diff --git a/include/image.h b/include/image.h index f5adc50..8e285f9 100644 --- a/include/image.h +++ b/include/image.h @@ -52,7 +52,6 @@ #endif /* USE_HOSTCC */ #if defined(CONFIG_FIT) -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #define CONFIG_MD5 /* FIT images need MD5 support */ diff --git a/tools/fdt_host.h b/tools/fdt_host.h index 085013e..b08bf86 100644 --- a/tools/fdt_host.h +++ b/tools/fdt_host.h @@ -21,7 +21,6 @@ #define __FDT_HOST_H__ /* Make sure to include u-boot version of libfdt include files */ -#include "../include/fdt.h" #include "../include/libfdt.h" #include "../include/fdt_support.h" diff --git a/tools/imls/imls.c b/tools/imls/imls.c index b21c505..a982a13 100644 --- a/tools/imls/imls.c +++ b/tools/imls/imls.c @@ -38,7 +38,6 @@ #endif #include <sha1.h> -#include <fdt.h> #include <libfdt.h> #include <fdt_support.h> #include <image.h> -- 1.8.1.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-16 23:59 [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h Kim Phillips @ 2013-01-17 0:36 ` Scott Wood 2013-01-17 17:54 ` [U-Boot] " Kim Phillips 2013-01-18 12:02 ` Jerry Van Baren 1 sibling, 1 reply; 15+ messages in thread From: Scott Wood @ 2013-01-17 0:36 UTC (permalink / raw) To: u-boot On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > and, if including libfdt.h which includes libfdt_env.h in > the correct order, don't include fdt.h before libfdt.h. > > this is needed to get the fdt type definitions set from > the project environment before fdt.h uses them. > > Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > Cc: Jerry Van Baren <gvb.uboot@gmail.com> Maybe fdt.h should include libfdt_env.h? Or just always use libfdt.h as the public header. -Scott ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-17 0:36 ` Scott Wood @ 2013-01-17 17:54 ` Kim Phillips 0 siblings, 0 replies; 15+ messages in thread From: Kim Phillips @ 2013-01-17 17:54 UTC (permalink / raw) To: Scott Wood; +Cc: u-boot, devicetree-discuss, Gibson, David On Wed, 16 Jan 2013 18:36:03 -0600 Scott Wood <scottwood@freescale.com> wrote: > On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > > and, if including libfdt.h which includes libfdt_env.h in > > the correct order, don't include fdt.h before libfdt.h. > > > > this is needed to get the fdt type definitions set from > > the project environment before fdt.h uses them. > > > > Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > > Cc: Jerry Van Baren <gvb.uboot@gmail.com> > > Maybe fdt.h should include libfdt_env.h? > > Or just always use libfdt.h as the public header. Was just following along the same lines as the dtc commits 38ad79d3 "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David G. I don't know why some only include fdt.h. devicetree-discuss/David: is there a prescribed way to go here? Change all fdt.h includers to just always include libfdt.h instead of libfdt_env.h prior to fdt.h? Kim ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-17 17:54 ` Kim Phillips 0 siblings, 0 replies; 15+ messages in thread From: Kim Phillips @ 2013-01-17 17:54 UTC (permalink / raw) To: u-boot On Wed, 16 Jan 2013 18:36:03 -0600 Scott Wood <scottwood@freescale.com> wrote: > On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > > and, if including libfdt.h which includes libfdt_env.h in > > the correct order, don't include fdt.h before libfdt.h. > > > > this is needed to get the fdt type definitions set from > > the project environment before fdt.h uses them. > > > > Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > > Cc: Jerry Van Baren <gvb.uboot@gmail.com> > > Maybe fdt.h should include libfdt_env.h? > > Or just always use libfdt.h as the public header. Was just following along the same lines as the dtc commits 38ad79d3 "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David G. I don't know why some only include fdt.h. devicetree-discuss/David: is there a prescribed way to go here? Change all fdt.h includers to just always include libfdt.h instead of libfdt_env.h prior to fdt.h? Kim ^ permalink raw reply [flat|nested] 15+ messages in thread
[parent not found: <20130117115456.71a38e3275230cdbb175bfe4-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-17 17:54 ` [U-Boot] " Kim Phillips @ 2013-01-17 18:32 ` Jerry Van Baren -1 siblings, 0 replies; 15+ messages in thread From: Jerry Van Baren @ 2013-01-17 18:32 UTC (permalink / raw) To: Kim Phillips Cc: Scott Wood, u-boot-0aAXYlwwYIKGBzrmiIFOJg, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ Hi Scott, Kim, David, On 01/17/2013 12:54 PM, Kim Phillips wrote: > On Wed, 16 Jan 2013 18:36:03 -0600 > Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: > >> On 01/16/2013 05:59:04 PM, Kim Phillips wrote: >>> and, if including libfdt.h which includes libfdt_env.h in >>> the correct order, don't include fdt.h before libfdt.h. >>> >>> this is needed to get the fdt type definitions set from >>> the project environment before fdt.h uses them. >>> >>> Signed-off-by: Kim Phillips <kim.phillips-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >>> Cc: Jerry Van Baren <gvb.uboot-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> >> Maybe fdt.h should include libfdt_env.h? >> >> Or just always use libfdt.h as the public header. > > Was just following along the same lines as the dtc commits 38ad79d3 > "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > G. I don't know why some only include fdt.h. > > devicetree-discuss/David: is there a prescribed way to go here? > Change all fdt.h includers to just always include libfdt.h instead > of libfdt_env.h prior to fdt.h? I started applying Kim's "sparse" patches to the u-boot source and ran into this issue pretty hard. In u-boot, there is an added complexity that the "tools" (host-based u-boot support tools) support flattened device trees, but explicitly include the u-boot version of libfdt declarations so they don't fall out of sync if the host has a non-compatible libfdt version. Having them out of sync would be a *horrible* situation to sort out - everything would build OK but nothing would work right, probably with no useful diagnostic information. This originated in 2008, so life may be better nowadays. Or maybe not. I would be in favor of explicitly including all the *fdt* headers in the sources. Alternately, Scott's suggestion of just including libfdt.h as the public header seems good, but I'm pretty sure it will mess me up with the explicit #including in the host-based "tools" build, leaving us with nasty work-arounds or a risk of hard to identify incompatible host vs. u-boot fdt versions. Who Includes Who fdt.h - no includes fdt_support.h - (u-boot only file) 29 #include <fdt.h> libfdt.h 54 #include <libfdt_env.h> 55 #include <fdt.h> libfdt_env.h - u-boot version is minimal, uses pre-existing macros for byte swapping - dtc version implements byte swapping, includes: 4 #include <stddef.h> 5 #include <stdint.h> 6 #include <string.h> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t I suspect the original intent was to have <libfdt.h> be the file that people #included. For whatever reason, most includes are (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. It sorts out OK in dtc because libfdt_env.h includes stdint.h and defines fdt*_t, but it messes me up in u-boot where (currently) libfdt_env.h does *not* include stdint.h... and then things get really wonky in the u-boot "tools" directory due to the need to use the u-boot version of the *fdt*.h headers, not random stuff installed on the computer. HTH, gvb ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-17 18:32 ` Jerry Van Baren 0 siblings, 0 replies; 15+ messages in thread From: Jerry Van Baren @ 2013-01-17 18:32 UTC (permalink / raw) To: u-boot Hi Scott, Kim, David, On 01/17/2013 12:54 PM, Kim Phillips wrote: > On Wed, 16 Jan 2013 18:36:03 -0600 > Scott Wood <scottwood@freescale.com> wrote: > >> On 01/16/2013 05:59:04 PM, Kim Phillips wrote: >>> and, if including libfdt.h which includes libfdt_env.h in >>> the correct order, don't include fdt.h before libfdt.h. >>> >>> this is needed to get the fdt type definitions set from >>> the project environment before fdt.h uses them. >>> >>> Signed-off-by: Kim Phillips <kim.phillips@freescale.com> >>> Cc: Jerry Van Baren <gvb.uboot@gmail.com> >> >> Maybe fdt.h should include libfdt_env.h? >> >> Or just always use libfdt.h as the public header. > > Was just following along the same lines as the dtc commits 38ad79d3 > "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > G. I don't know why some only include fdt.h. > > devicetree-discuss/David: is there a prescribed way to go here? > Change all fdt.h includers to just always include libfdt.h instead > of libfdt_env.h prior to fdt.h? I started applying Kim's "sparse" patches to the u-boot source and ran into this issue pretty hard. In u-boot, there is an added complexity that the "tools" (host-based u-boot support tools) support flattened device trees, but explicitly include the u-boot version of libfdt declarations so they don't fall out of sync if the host has a non-compatible libfdt version. Having them out of sync would be a *horrible* situation to sort out - everything would build OK but nothing would work right, probably with no useful diagnostic information. This originated in 2008, so life may be better nowadays. Or maybe not. I would be in favor of explicitly including all the *fdt* headers in the sources. Alternately, Scott's suggestion of just including libfdt.h as the public header seems good, but I'm pretty sure it will mess me up with the explicit #including in the host-based "tools" build, leaving us with nasty work-arounds or a risk of hard to identify incompatible host vs. u-boot fdt versions. Who Includes Who fdt.h - no includes fdt_support.h - (u-boot only file) 29 #include <fdt.h> libfdt.h 54 #include <libfdt_env.h> 55 #include <fdt.h> libfdt_env.h - u-boot version is minimal, uses pre-existing macros for byte swapping - dtc version implements byte swapping, includes: 4 #include <stddef.h> 5 #include <stdint.h> 6 #include <string.h> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t I suspect the original intent was to have <libfdt.h> be the file that people #included. For whatever reason, most includes are (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. It sorts out OK in dtc because libfdt_env.h includes stdint.h and defines fdt*_t, but it messes me up in u-boot where (currently) libfdt_env.h does *not* include stdint.h... and then things get really wonky in the u-boot "tools" directory due to the need to use the u-boot version of the *fdt*.h headers, not random stuff installed on the computer. HTH, gvb ^ permalink raw reply [flat|nested] 15+ messages in thread
[parent not found: <50F843CD.1030309-He//nVnquyzQT0dZR+AlfA@public.gmane.org>]
* Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-17 18:32 ` Jerry Van Baren @ 2013-01-17 23:50 ` David Gibson -1 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-17 23:50 UTC (permalink / raw) To: Jerry Van Baren Cc: Scott Wood, u-boot-0aAXYlwwYIKGBzrmiIFOJg, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ [-- Attachment #1.1: Type: text/plain, Size: 3834 bytes --] On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: > Hi Scott, Kim, David, > > On 01/17/2013 12:54 PM, Kim Phillips wrote: > >On Wed, 16 Jan 2013 18:36:03 -0600 > >Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: > > > >>On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > >>>and, if including libfdt.h which includes libfdt_env.h in > >>>the correct order, don't include fdt.h before libfdt.h. > >>> > >>>this is needed to get the fdt type definitions set from > >>>the project environment before fdt.h uses them. > >>> > >>>Signed-off-by: Kim Phillips <kim.phillips-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > >>>Cc: Jerry Van Baren <gvb.uboot-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > >> > >>Maybe fdt.h should include libfdt_env.h? > >> > >>Or just always use libfdt.h as the public header. > > > >Was just following along the same lines as the dtc commits 38ad79d3 > >"dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > >"dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > >G. I don't know why some only include fdt.h. > > > >devicetree-discuss/David: is there a prescribed way to go here? > >Change all fdt.h includers to just always include libfdt.h instead > >of libfdt_env.h prior to fdt.h? > > I started applying Kim's "sparse" patches to the u-boot source and > ran into this issue pretty hard. > > In u-boot, there is an added complexity that the "tools" (host-based > u-boot support tools) support flattened device trees, but explicitly > include the u-boot version of libfdt declarations so they don't fall > out of sync if the host has a non-compatible libfdt version. Having > them out of sync would be a *horrible* situation to sort out - > everything would build OK but nothing would work right, probably > with no useful diagnostic information. This originated in 2008, so > life may be better nowadays. Or maybe not. > > I would be in favor of explicitly including all the *fdt* headers in > the sources. Alternately, Scott's suggestion of just including > libfdt.h as the public header seems good, but I'm pretty sure it > will mess me up with the explicit #including in the host-based > "tools" build, leaving us with nasty work-arounds or a risk of hard > to identify incompatible host vs. u-boot fdt versions. > > Who Includes Who > > fdt.h - no includes > > fdt_support.h - (u-boot only file) > 29 #include <fdt.h> > > libfdt.h > 54 #include <libfdt_env.h> > 55 #include <fdt.h> > > libfdt_env.h > - u-boot version is minimal, uses pre-existing macros for byte swapping > - dtc version implements byte swapping, includes: > 4 #include <stddef.h> > 5 #include <stdint.h> > 6 #include <string.h> > > libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t > > I suspect the original intent was to have <libfdt.h> be the file > that people #included. For whatever reason, most includes are > (picking on fdt_ro.c arbitrarily): > 51 #include "libfdt_env.h" > 53 #include <fdt.h> > 54 #include <libfdt.h> > Since libfdt.h #includes fdt.h and libfdt_env.h, lines 51 and 53 > (above) are redundant. It sorts out OK in dtc because libfdt_env.h > includes stdint.h and defines fdt*_t, but it messes me up in u-boot > where (currently) libfdt_env.h does *not* include stdint.h... Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t and fdtXX_t (either by including stdint or my other means). The purpose of libfdt_env.h is to define the things that libfdt requires, and those types are (now) such a requirement. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 198 bytes --] [-- Attachment #2: Type: text/plain, Size: 192 bytes --] _______________________________________________ devicetree-discuss mailing list devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org https://lists.ozlabs.org/listinfo/devicetree-discuss ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-17 23:50 ` David Gibson 0 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-17 23:50 UTC (permalink / raw) To: u-boot On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: > Hi Scott, Kim, David, > > On 01/17/2013 12:54 PM, Kim Phillips wrote: > >On Wed, 16 Jan 2013 18:36:03 -0600 > >Scott Wood <scottwood@freescale.com> wrote: > > > >>On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > >>>and, if including libfdt.h which includes libfdt_env.h in > >>>the correct order, don't include fdt.h before libfdt.h. > >>> > >>>this is needed to get the fdt type definitions set from > >>>the project environment before fdt.h uses them. > >>> > >>>Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > >>>Cc: Jerry Van Baren <gvb.uboot@gmail.com> > >> > >>Maybe fdt.h should include libfdt_env.h? > >> > >>Or just always use libfdt.h as the public header. > > > >Was just following along the same lines as the dtc commits 38ad79d3 > >"dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > >"dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > >G. I don't know why some only include fdt.h. > > > >devicetree-discuss/David: is there a prescribed way to go here? > >Change all fdt.h includers to just always include libfdt.h instead > >of libfdt_env.h prior to fdt.h? > > I started applying Kim's "sparse" patches to the u-boot source and > ran into this issue pretty hard. > > In u-boot, there is an added complexity that the "tools" (host-based > u-boot support tools) support flattened device trees, but explicitly > include the u-boot version of libfdt declarations so they don't fall > out of sync if the host has a non-compatible libfdt version. Having > them out of sync would be a *horrible* situation to sort out - > everything would build OK but nothing would work right, probably > with no useful diagnostic information. This originated in 2008, so > life may be better nowadays. Or maybe not. > > I would be in favor of explicitly including all the *fdt* headers in > the sources. Alternately, Scott's suggestion of just including > libfdt.h as the public header seems good, but I'm pretty sure it > will mess me up with the explicit #including in the host-based > "tools" build, leaving us with nasty work-arounds or a risk of hard > to identify incompatible host vs. u-boot fdt versions. > > Who Includes Who > > fdt.h - no includes > > fdt_support.h - (u-boot only file) > 29 #include <fdt.h> > > libfdt.h > 54 #include <libfdt_env.h> > 55 #include <fdt.h> > > libfdt_env.h > - u-boot version is minimal, uses pre-existing macros for byte swapping > - dtc version implements byte swapping, includes: > 4 #include <stddef.h> > 5 #include <stdint.h> > 6 #include <string.h> > > libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t > > I suspect the original intent was to have <libfdt.h> be the file > that people #included. For whatever reason, most includes are > (picking on fdt_ro.c arbitrarily): > 51 #include "libfdt_env.h" > 53 #include <fdt.h> > 54 #include <libfdt.h> > Since libfdt.h #includes fdt.h and libfdt_env.h, lines 51 and 53 > (above) are redundant. It sorts out OK in dtc because libfdt_env.h > includes stdint.h and defines fdt*_t, but it messes me up in u-boot > where (currently) libfdt_env.h does *not* include stdint.h... Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t and fdtXX_t (either by including stdint or my other means). The purpose of libfdt_env.h is to define the things that libfdt requires, and those types are (now) such a requirement. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130118/7037f6f0/attachment.pgp> ^ permalink raw reply [flat|nested] 15+ messages in thread
[parent not found: <20130117235048.GF26321-W9XWwYn+TF0XU02nzanrWNbf9cGiqdzd@public.gmane.org>]
* Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-17 23:50 ` David Gibson @ 2013-01-18 11:59 ` Gerald Van Baren -1 siblings, 0 replies; 15+ messages in thread From: Gerald Van Baren @ 2013-01-18 11:59 UTC (permalink / raw) To: David Gibson Cc: Scott Wood, u-boot-0aAXYlwwYIKGBzrmiIFOJg, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ On 01/17/2013 06:50 PM, David Gibson wrote: > On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: >> Hi Scott, Kim, David, [snip] >> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t >> >> I suspect the original intent was to have <libfdt.h> be the file >> that people #included. For whatever reason, most includes are >> (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 >> #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes >> fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. >> It sorts out OK in dtc because libfdt_env.h includes stdint.h and >> defines fdt*_t, but it messes me up in u-boot where (currently) >> libfdt_env.h does *not* include stdint.h... > > Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t > and fdtXX_t (either by including stdint or my other means). The > purpose of libfdt_env.h is to define the things that libfdt > requires, and those types are (now) such a requirement. I like the move, but have not done it (yet). I made a trial patch (see below) that uses libfdt.h as the interface and cleans out the (now redundant) other *fdt*.h includes. If this is in the right direction, I'll move the fdtXX_t typedefs and formally submit it. The test suite has one failure, but it fails with or without my changes. $ make check | grep FAIL fdtget-runtest.sh 77 121 66 111 97 114 100 78 97 109 101 0 77 121 66 111 97 114 100 70 97 109 105 108 121 78 97 109 101 0 label01.dts.fdtget.test.dtb / compatible: FAIL Results differ from expected * FAIL: 1 ********** TEST SUMMARY * Total testcases: 1443 * PASS: 1442 * FAIL: 1 * Bad configuration: 0 * Strange test result: 0 ********** gvb >From 1fb0193670e012072d4a9e5ac480e3201aaf30fd Mon Sep 17 00:00:00 2001 From: Gerald Van Baren <gvb-LFPmMGy0VdYAvxtiuMwx3w@public.gmane.org> Date: Thu, 17 Jan 2013 21:14:01 -0500 Subject: [PATCH] Use the libfdt.h include as the interface definition libfdt.h is the libfdt interface definition. It includes fdt.h and libfdt_env.h, so there is no reason to include those in the general code. Signed-off-by: Gerald Van Baren <vanbaren-He//nVnquyzQT0dZR+AlfA@public.gmane.org> --- dtc.h | 3 +-- fdtdump.c | 3 +-- libfdt/fdt.c | 3 --- libfdt/fdt_empty_tree.c | 3 --- libfdt/fdt_ro.c | 3 --- libfdt/fdt_rw.c | 3 --- libfdt/fdt_strerror.c | 3 --- libfdt/fdt_sw.c | 3 --- libfdt/fdt_wip.c | 3 --- libfdt/libfdt_internal.h | 1 - 10 files changed, 2 insertions(+), 26 deletions(-) diff --git a/dtc.h b/dtc.h index 3e42a07..a032645 100644 --- a/dtc.h +++ b/dtc.h @@ -32,8 +32,7 @@ #include <errno.h> #include <unistd.h> -#include <libfdt_env.h> -#include <fdt.h> +#include <libfdt.h> #include "util.h" diff --git a/fdtdump.c b/fdtdump.c index b2c5b37..3919490 100644 --- a/fdtdump.c +++ b/fdtdump.c @@ -8,8 +8,7 @@ #include <string.h> #include <ctype.h> -#include <libfdt_env.h> -#include <fdt.h> +#include <libfdt.h> #include "util.h" diff --git a/libfdt/fdt.c b/libfdt/fdt.c index 57faba3..19b9043 100644 --- a/libfdt/fdt.c +++ b/libfdt/fdt.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" int fdt_check_header(const void *fdt) diff --git a/libfdt/fdt_empty_tree.c b/libfdt/fdt_empty_tree.c index f72d13b..30c5525 100644 --- a/libfdt/fdt_empty_tree.c +++ b/libfdt/fdt_empty_tree.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" int fdt_create_empty_tree(void *buf, int bufsize) diff --git a/libfdt/fdt_ro.c b/libfdt/fdt_ro.c index 42da2bd..53c61ed 100644 --- a/libfdt/fdt_ro.c +++ b/libfdt/fdt_ro.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" static int _fdt_nodename_eq(const void *fdt, int offset, diff --git a/libfdt/fdt_rw.c b/libfdt/fdt_rw.c index fdba618..b6a8815 100644 --- a/libfdt/fdt_rw.c +++ b/libfdt/fdt_rw.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" static int _fdt_blocks_misordered(const void *fdt, diff --git a/libfdt/fdt_strerror.c b/libfdt/fdt_strerror.c index e6c3cee..c55793c 100644 --- a/libfdt/fdt_strerror.c +++ b/libfdt/fdt_strerror.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" struct fdt_errtabent { diff --git a/libfdt/fdt_sw.c b/libfdt/fdt_sw.c index f422754..85aaedf 100644 --- a/libfdt/fdt_sw.c +++ b/libfdt/fdt_sw.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" static int _fdt_sw_check_header(void *fdt) diff --git a/libfdt/fdt_wip.c b/libfdt/fdt_wip.c index c5bbb68..67fa7ca 100644 --- a/libfdt/fdt_wip.c +++ b/libfdt/fdt_wip.c @@ -48,11 +48,8 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include "libfdt_env.h" -#include <fdt.h> #include <libfdt.h> - #include "libfdt_internal.h" int fdt_setprop_inplace(void *fdt, int nodeoffset, const char *name, diff --git a/libfdt/libfdt_internal.h b/libfdt/libfdt_internal.h index 381133b..a8b36f4 100644 --- a/libfdt/libfdt_internal.h +++ b/libfdt/libfdt_internal.h @@ -50,7 +50,6 @@ * OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, * EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include <fdt.h> #define FDT_ALIGN(x, a) (((x) + (a) - 1) & ~((a) - 1)) #define FDT_TAGALIGN(x) (FDT_ALIGN((x), FDT_TAGSIZE)) -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-18 11:59 ` Gerald Van Baren 0 siblings, 0 replies; 15+ messages in thread From: Gerald Van Baren @ 2013-01-18 11:59 UTC (permalink / raw) To: u-boot On 01/17/2013 06:50 PM, David Gibson wrote: > On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: >> Hi Scott, Kim, David, [snip] >> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t >> >> I suspect the original intent was to have <libfdt.h> be the file >> that people #included. For whatever reason, most includes are >> (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 >> #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes >> fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. >> It sorts out OK in dtc because libfdt_env.h includes stdint.h and >> defines fdt*_t, but it messes me up in u-boot where (currently) >> libfdt_env.h does *not* include stdint.h... > > Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t > and fdtXX_t (either by including stdint or my other means). The > purpose of libfdt_env.h is to define the things that libfdt > requires, and those types are (now) such a requirement. I like the move, but have not done it (yet). I made a trial patch (see below) that uses libfdt.h as the interface and cleans out the (now redundant) other *fdt*.h includes. If this is in the right direction, I'll move the fdtXX_t typedefs and formally submit it. The test suite has one failure, but it fails with or without my changes. $ make check | grep FAIL fdtget-runtest.sh 77 121 66 111 97 114 100 78 97 109 101 0 77 121 66 111 97 114 100 70 97 109 105 108 121 78 97 109 101 0 label01.dts.fdtget.test.dtb / compatible: FAIL Results differ from expected * FAIL: 1 ********** TEST SUMMARY * Total testcases: 1443 * PASS: 1442 * FAIL: 1 * Bad configuration: 0 * Strange test result: 0 ********** gvb ^ permalink raw reply [flat|nested] 15+ messages in thread
[parent not found: <50F93932.60700-He//nVnquyzQT0dZR+AlfA@public.gmane.org>]
* Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-18 11:59 ` Gerald Van Baren @ 2013-01-21 0:17 ` David Gibson -1 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-21 0:17 UTC (permalink / raw) To: Gerald Van Baren Cc: Scott Wood, u-boot-0aAXYlwwYIKGBzrmiIFOJg, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ [-- Attachment #1.1: Type: text/plain, Size: 2349 bytes --] On Fri, Jan 18, 2013 at 06:59:46AM -0500, Gerald Van Baren wrote: > On 01/17/2013 06:50 PM, David Gibson wrote: > > On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: > >> Hi Scott, Kim, David, > > [snip] > > >> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t > >> > >> I suspect the original intent was to have <libfdt.h> be the file > >> that people #included. For whatever reason, most includes are > >> (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 > >> #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes > >> fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. > >> It sorts out OK in dtc because libfdt_env.h includes stdint.h and > >> defines fdt*_t, but it messes me up in u-boot where (currently) > >> libfdt_env.h does *not* include stdint.h... > > > > Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t > > and fdtXX_t (either by including stdint or my other means). The > > purpose of libfdt_env.h is to define the things that libfdt > > requires, and those types are (now) such a requirement. > > I like the move, but have not done it (yet). I made a trial patch > (see below) that uses libfdt.h as the interface and cleans out the > (now redundant) other *fdt*.h includes. If this is in the right > direction, I'll move the fdtXX_t typedefs and formally submit it. > > The test suite has one failure, but it fails with or without my changes. > > $ make check | grep FAIL > fdtget-runtest.sh 77 121 66 111 97 114 100 78 97 109 101 0 77 121 66 > 111 97 114 100 70 97 109 105 108 121 78 97 109 101 0 > label01.dts.fdtget.test.dtb / compatible: FAIL Results differ from > expected > * FAIL: 1 > > ********** TEST SUMMARY > * Total testcases: 1443 > * PASS: 1442 > * FAIL: 1 > * Bad configuration: 0 > * Strange test result: 0 > ********** Yeah, Jon and I are aware of that recently introduced bug. I've had other projects to attend to and haven't had a chance to fix it yet. It's basically a testsuite bug though, so it should be harmless. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 198 bytes --] [-- Attachment #2: Type: text/plain, Size: 192 bytes --] _______________________________________________ devicetree-discuss mailing list devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org https://lists.ozlabs.org/listinfo/devicetree-discuss ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-21 0:17 ` David Gibson 0 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-21 0:17 UTC (permalink / raw) To: u-boot On Fri, Jan 18, 2013 at 06:59:46AM -0500, Gerald Van Baren wrote: > On 01/17/2013 06:50 PM, David Gibson wrote: > > On Thu, Jan 17, 2013 at 01:32:45PM -0500, Jerry Van Baren wrote: > >> Hi Scott, Kim, David, > > [snip] > > >> libfdt_env.h is where Kim typedef'ed fdt16_t, fdt32_t, fdt64_t > >> > >> I suspect the original intent was to have <libfdt.h> be the file > >> that people #included. For whatever reason, most includes are > >> (picking on fdt_ro.c arbitrarily): 51 #include "libfdt_env.h" 53 > >> #include <fdt.h> 54 #include <libfdt.h> Since libfdt.h #includes > >> fdt.h and libfdt_env.h, lines 51 and 53 (above) are redundant. > >> It sorts out OK in dtc because libfdt_env.h includes stdint.h and > >> defines fdt*_t, but it messes me up in u-boot where (currently) > >> libfdt_env.h does *not* include stdint.h... > > > > Ok, so, the uboot libfdt_env.h should be fixed to define uintXX_t > > and fdtXX_t (either by including stdint or my other means). The > > purpose of libfdt_env.h is to define the things that libfdt > > requires, and those types are (now) such a requirement. > > I like the move, but have not done it (yet). I made a trial patch > (see below) that uses libfdt.h as the interface and cleans out the > (now redundant) other *fdt*.h includes. If this is in the right > direction, I'll move the fdtXX_t typedefs and formally submit it. > > The test suite has one failure, but it fails with or without my changes. > > $ make check | grep FAIL > fdtget-runtest.sh 77 121 66 111 97 114 100 78 97 109 101 0 77 121 66 > 111 97 114 100 70 97 109 105 108 121 78 97 109 101 0 > label01.dts.fdtget.test.dtb / compatible: FAIL Results differ from > expected > * FAIL: 1 > > ********** TEST SUMMARY > * Total testcases: 1443 > * PASS: 1442 > * FAIL: 1 > * Bad configuration: 0 > * Strange test result: 0 > ********** Yeah, Jon and I are aware of that recently introduced bug. I've had other projects to attend to and haven't had a chance to fix it yet. It's basically a testsuite bug though, so it should be harmless. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130121/30bec7f4/attachment.pgp> ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-17 17:54 ` [U-Boot] " Kim Phillips @ 2013-01-17 23:48 ` David Gibson -1 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-17 23:48 UTC (permalink / raw) To: Kim Phillips Cc: Scott Wood, u-boot-0aAXYlwwYIKGBzrmiIFOJg, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ [-- Attachment #1.1: Type: text/plain, Size: 1779 bytes --] On Thu, Jan 17, 2013 at 11:54:56AM -0600, Kim Phillips wrote: > On Wed, 16 Jan 2013 18:36:03 -0600 > Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote: > > > On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > > > and, if including libfdt.h which includes libfdt_env.h in > > > the correct order, don't include fdt.h before libfdt.h. > > > > > > this is needed to get the fdt type definitions set from > > > the project environment before fdt.h uses them. > > > > > > Signed-off-by: Kim Phillips <kim.phillips-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > > > Cc: Jerry Van Baren <gvb.uboot-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > > > > Maybe fdt.h should include libfdt_env.h? > > > > Or just always use libfdt.h as the public header. > > Was just following along the same lines as the dtc commits 38ad79d3 > "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > G. I don't know why some only include fdt.h. > > devicetree-discuss/David: is there a prescribed way to go here? > Change all fdt.h includers to just always include libfdt.h instead > of libfdt_env.h prior to fdt.h? Yeah, I think just including libfdt.h instead of fdt.h is the way to go. The distinction is that fdt.h contains only "passive" declarations. That is, defines and structure/type declarations but no function prorotypes or other code. In particular that means that it is strictly about the FDT tree structure only, and not any of the libfdt specific entry points. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 198 bytes --] [-- Attachment #2: Type: text/plain, Size: 192 bytes --] _______________________________________________ devicetree-discuss mailing list devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org https://lists.ozlabs.org/listinfo/devicetree-discuss ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h @ 2013-01-17 23:48 ` David Gibson 0 siblings, 0 replies; 15+ messages in thread From: David Gibson @ 2013-01-17 23:48 UTC (permalink / raw) To: u-boot On Thu, Jan 17, 2013 at 11:54:56AM -0600, Kim Phillips wrote: > On Wed, 16 Jan 2013 18:36:03 -0600 > Scott Wood <scottwood@freescale.com> wrote: > > > On 01/16/2013 05:59:04 PM, Kim Phillips wrote: > > > and, if including libfdt.h which includes libfdt_env.h in > > > the correct order, don't include fdt.h before libfdt.h. > > > > > > this is needed to get the fdt type definitions set from > > > the project environment before fdt.h uses them. > > > > > > Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > > > Cc: Jerry Van Baren <gvb.uboot@gmail.com> > > > > Maybe fdt.h should include libfdt_env.h? > > > > Or just always use libfdt.h as the public header. > > Was just following along the same lines as the dtc commits 38ad79d3 > "dtc/tests: don't include fdt.h prior to libfdt.h" and 20b866a7 > "dtc/fdtdump: include libfdt_env.h prior to fdt.h", acked by David > G. I don't know why some only include fdt.h. > > devicetree-discuss/David: is there a prescribed way to go here? > Change all fdt.h includers to just always include libfdt.h instead > of libfdt_env.h prior to fdt.h? Yeah, I think just including libfdt.h instead of fdt.h is the way to go. The distinction is that fdt.h contains only "passive" declarations. That is, defines and structure/type declarations but no function prorotypes or other code. In particular that means that it is strictly about the FDT tree structure only, and not any of the libfdt specific entry points. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130118/f251d1d7/attachment.pgp> ^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h 2013-01-16 23:59 [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h Kim Phillips 2013-01-17 0:36 ` Scott Wood @ 2013-01-18 12:02 ` Jerry Van Baren 1 sibling, 0 replies; 15+ messages in thread From: Jerry Van Baren @ 2013-01-18 12:02 UTC (permalink / raw) To: u-boot Hi Kim, On 01/16/2013 06:59 PM, Kim Phillips wrote: > and, if including libfdt.h which includes libfdt_env.h in > the correct order, don't include fdt.h before libfdt.h. > > this is needed to get the fdt type definitions set from > the project environment before fdt.h uses them. > > Signed-off-by: Kim Phillips <kim.phillips@freescale.com> > Cc: Jerry Van Baren <gvb.uboot@gmail.com> Thanks, that was very helpful. I applied the series to my personal tree, will push when the dust settles. I created a patch that uses just libfdt.h as the interface #include, will publish when the dust settles on the dtc (device tree) list discussion. Thanks, gvb ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-01-21 0:17 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-16 23:59 [U-Boot] [PATCH 1/5] treewide: include libfdt_env.h before fdt.h Kim Phillips
2013-01-17 0:36 ` Scott Wood
2013-01-17 17:54 ` Kim Phillips
2013-01-17 17:54 ` [U-Boot] " Kim Phillips
[not found] ` <20130117115456.71a38e3275230cdbb175bfe4-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2013-01-17 18:32 ` Jerry Van Baren
2013-01-17 18:32 ` Jerry Van Baren
[not found] ` <50F843CD.1030309-He//nVnquyzQT0dZR+AlfA@public.gmane.org>
2013-01-17 23:50 ` David Gibson
2013-01-17 23:50 ` David Gibson
[not found] ` <20130117235048.GF26321-W9XWwYn+TF0XU02nzanrWNbf9cGiqdzd@public.gmane.org>
2013-01-18 11:59 ` Gerald Van Baren
2013-01-18 11:59 ` Gerald Van Baren
[not found] ` <50F93932.60700-He//nVnquyzQT0dZR+AlfA@public.gmane.org>
2013-01-21 0:17 ` David Gibson
2013-01-21 0:17 ` David Gibson
2013-01-17 23:48 ` David Gibson
2013-01-17 23:48 ` David Gibson
2013-01-18 12:02 ` Jerry Van Baren
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.