Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/30/2013 01:56 AM, Wolfgang Denk wrote: > Dear Stephen, > > In message <51a6df7c.30...@wwwdotorg.org> you wrote: >> >> It seems to be aimed specifically at enabling use of new dtc features >> when present. That seems to be specifically against Wolfgang's goal of >> not requiring new dtc featu

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen, In message <51a6df7c.30...@wwwdotorg.org> you wrote: > > It seems to be aimed specifically at enabling use of new dtc features > when present. That seems to be specifically against Wolfgang's goal of > not requiring new dtc features. There's no point allowing use of new dtc Please s

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen Warren, In message <51a68a4c.4060...@wwwdotorg.org> you wrote: > > > Sorry, instead we should strive to be compatible to a reasonably old, > > stable version of DTC, like we do for all other tools as well. As > > mentioned before - just because RHEL 5 ships an ancient version of - >

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen, In message <51a6869f.1020...@wwwdotorg.org> you wrote: > > Since DT is supposed to be a HW description, it shouldn't be using cpp's > built-in macros to compile in different ways; there really isn't a > concept of the "target arch of compilation"; a .dts file should simply > compile

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Hi Stephen, On Wed, May 29, 2013 at 10:11 PM, Stephen Warren wrote: > On 05/29/2013 10:46 PM, Simon Glass wrote: > > Hi, > > > > On Wed, May 29, 2013 at 4:07 PM, Stephen Warren > > wrote: > > > > On 05/29/2013 04:36 PM, Wolfgang Denk wrote: > > > Dear Stephe

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/29/2013 10:46 PM, Simon Glass wrote: > Hi, > > On Wed, May 29, 2013 at 4:07 PM, Stephen Warren > wrote: > > On 05/29/2013 04:36 PM, Wolfgang Denk wrote: > > Dear Stephen Warren, > > > > In message <51a67ec1.2000...@wwwdotorg.org >

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Hi, On Wed, May 29, 2013 at 4:07 PM, Stephen Warren wrote: > On 05/29/2013 04:36 PM, Wolfgang Denk wrote: > > Dear Stephen Warren, > > > > In message <51a67ec1.2000...@wwwdotorg.org> you wrote: > >> > >> To keep this process in check a bit, we could always pick a specific git > >> commit or relea

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/29/2013 04:36 PM, Wolfgang Denk wrote: > Dear Stephen Warren, > > In message <51a67ec1.2000...@wwwdotorg.org> you wrote: >> >> To keep this process in check a bit, we could always pick a specific git >> commit or release version of dtc that each U-Boot version (release) will >> be allowed to

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/29/2013 03:33 PM, Wolfgang Denk wrote: > Dear Stephen, > > In message <51a634b5.5060...@wwwdotorg.org> you wrote: >> >>> I think this is not a good way to address this issue. The GCC >>> documentation (section "System-specific Predefined Macros" [1]) >>> desribes how this should be handled.

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen Warren, In message <51a67ec1.2000...@wwwdotorg.org> you wrote: > > To keep this process in check a bit, we could always pick a specific git > commit or release version of dtc that each U-Boot version (release) will > be allowed to assume. That will limit the number of times people nee

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/29/2013 03:31 PM, Wolfgang Denk wrote: > Dear Stephen, > > In message <51a62f8d.9010...@wwwdotorg.org> you wrote: >> >> The Linux kernel chose to solve this by bundling the required dtc source >> inside the kernel source tree as a tool. This seems by far the simplest >> way to solve the prob

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen, In message <51a634b5.5060...@wwwdotorg.org> you wrote: > > > I think this is not a good way to address this issue. The GCC > > documentation (section "System-specific Predefined Macros" [1]) > > desribes how this should be handled. The "correct" (TM) way to fix > > this is by addin

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Stephen, In message <51a62f8d.9010...@wwwdotorg.org> you wrote: > > The Linux kernel chose to solve this by bundling the required dtc source > inside the kernel source tree as a tool. This seems by far the simplest > way to solve the problem for U-Boot too. If not, it's not exactly hard to:

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Simon, In message you wrote: > > > I think this is not a good way to address this issue. The GCC > > documentation (section "System-specific Predefined Macros" [1]) > > desribes how this should be handled. The "correct" (TM) way to fix > > this is by adding "-ansi" or any "-std" option t

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/28/2013 03:08 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message <1369769778-12455-1-git-send-email-...@chromium.org> you wrote: >> >> Some device tree files use the word 'linux' which gets replaced with '1' by >> many version of gcc, including version 4.7. So undefine this. > > I

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/29/2013 09:59 AM, Simon Glass wrote: > Hi Stephen, > > On Tue, May 28, 2013 at 1:57 PM, Stephen Warren wrote: >> On 05/28/2013 01:36 PM, Simon Glass wrote: >>> There are a few partially conflicting requirements in compiling the device >>> tree, since U-Boot relies on whatever is installed o

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Hi Wolfgang, On Tue, May 28, 2013 at 2:08 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message <1369769778-12455-1-git-send-email-...@chromium.org> you wrote: >> >> Some device tree files use the word 'linux' which gets replaced with '1' by >> many version of gcc, including version 4.7. So

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Hi Stephen, On Tue, May 28, 2013 at 1:57 PM, Stephen Warren wrote: > On 05/28/2013 01:36 PM, Simon Glass wrote: >> There are a few partially conflicting requirements in compiling the device >> tree, since U-Boot relies on whatever is installed on the build machine. >> >> Some versions of dtc supp

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Dear Simon Glass, In message <1369769778-12455-1-git-send-email-...@chromium.org> you wrote: > > Some device tree files use the word 'linux' which gets replaced with '1' by > many version of gcc, including version 4.7. So undefine this. I think this is not a good way to address this issue. The

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

On 05/28/2013 01:36 PM, Simon Glass wrote: > There are a few partially conflicting requirements in compiling the device > tree, since U-Boot relies on whatever is installed on the build machine. > > Some versions of dtc support -i, and this is often better than using #include > since you get corre

Re: [U-Boot] [PATCH] fdt: Enhance dts/Makefile to be all things to all men

Simon, > -Original Message- > From: Simon Glass [mailto:s...@chromium.org] > Sent: Tuesday, May 28, 2013 12:36 PM > To: U-Boot Mailing List > Cc: Tom Rini; Stephen Warren; Devicetree Discuss; u-boot- > rev...@google.com; Simon Glass; Tom Warren; Jerry Van Baren > Subject: [PATCH] fdt: Enha