Re: kconfig: support option env="" [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-13 Thread Sam Ravnborg
On Mon, Jan 14, 2008 at 04:49:48AM +0100, Roman Zippel wrote: > Hi, > > On Sun, 6 Jan 2008, Sam Ravnborg wrote: > > > Please get back to me so we can finsih this patch and have it applied. > > I will split the patch in two btw. > > I reworked the patch a little and split it into three. Thanks

Re: kconfig: support option env="" [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-13 Thread Roman Zippel
Hi, On Sun, 6 Jan 2008, Sam Ravnborg wrote: > Please get back to me so we can finsih this patch and have it applied. > I will split the patch in two btw. I reworked the patch a little and split it into three. > > + if (sym->flags & SYMBOL_AUTO) > > + sym->flags &= ~SYMBOL_WRITE; >

Re: kconfig: support option env= [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-13 Thread Roman Zippel
Hi, On Sun, 6 Jan 2008, Sam Ravnborg wrote: Please get back to me so we can finsih this patch and have it applied. I will split the patch in two btw. I reworked the patch a little and split it into three. + if (sym-flags SYMBOL_AUTO) + sym-flags = ~SYMBOL_WRITE; + Why

Re: kconfig: support option env= [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-13 Thread Sam Ravnborg
On Mon, Jan 14, 2008 at 04:49:48AM +0100, Roman Zippel wrote: Hi, On Sun, 6 Jan 2008, Sam Ravnborg wrote: Please get back to me so we can finsih this patch and have it applied. I will split the patch in two btw. I reworked the patch a little and split it into three. Thanks Roman. I

kconfig: support option env="" [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-06 Thread Sam Ravnborg
Hi Roman. Some time ago you sent the following patch which I have started to review properly and test. But it triggers a few questions / comments. For reference (since it is so long ago I have kept most of the patch but only a few places are commented. Please get back to me so we can finsih

kconfig: support option env= [Was: kconfig: use $K64BIT to set 64BIT with all*config targets]

2008-01-06 Thread Sam Ravnborg
Hi Roman. Some time ago you sent the following patch which I have started to review properly and test. But it triggers a few questions / comments. For reference (since it is so long ago I have kept most of the patch but only a few places are commented. Please get back to me so we can finsih