* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 8:07 ` [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets Wolfgang Denk
@ 2009-09-11 8:12 ` Stefan Roese
2009-09-11 8:25 ` Wolfgang Denk
2009-09-11 8:46 ` Hu Mingkai-B21284
` (2 subsequent siblings)
3 siblings, 1 reply; 12+ messages in thread
From: Stefan Roese @ 2009-09-11 8:12 UTC (permalink / raw)
To: u-boot
On Friday 11 September 2009 10:07:46 Wolfgang Denk wrote:
> Arghhh... It seems this patch has not been well tested. Obviously
> never ever ran a MAKEALL over all PPC boards. It causes tons of
> messages like that:
>
> include/config.h:2:25: warning: missing whitespace after the macro name
>
> This patch breaks building for a number of boards / configurations,
> especially:
>
> xilinx-ppc405-generic_flash
> xilinx-ppc405-generic
> xilinx-ppc440-generic_flash
> xilinx-ppc440-generic
> VoVPN-GW_66MHz
> VoVPN-GW_100MHz
> EB+MCF-EV123
> EB+MCF-EV123_internal
> microblaze-generic
> favr-32-ezkit
Yes, I just stumbled over the build failure here as well. It seems to be a
problem with the "-" in the board target name. Not sure how this could be
solved though.
Cheers,
Stefan
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de
^ permalink raw reply [flat|nested] 12+ messages in thread* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 8:12 ` Stefan Roese
@ 2009-09-11 8:25 ` Wolfgang Denk
0 siblings, 0 replies; 12+ messages in thread
From: Wolfgang Denk @ 2009-09-11 8:25 UTC (permalink / raw)
To: u-boot
Dear Stefan Roese,
In message <200909111012.07310.sr@denx.de> you wrote:
>
> > This patch breaks building for a number of boards / configurations,
> > especially:
> >
> > xilinx-ppc405-generic_flash
> > xilinx-ppc405-generic
> > xilinx-ppc440-generic_flash
> > xilinx-ppc440-generic
> > VoVPN-GW_66MHz
> > VoVPN-GW_100MHz
> > EB+MCF-EV123
> > EB+MCF-EV123_internal
> > microblaze-generic
> > favr-32-ezkit
>
> Yes, I just stumbled over the build failure here as well. It seems to be a
> problem with the "-" in the board target name. Not sure how this could be
> solved though.
Yes. The modification was based on the assumption that Make target
names _and_ board config file names could be used as valid C
identifiers, which is obviously not the case.
Of course we could rename the respective Make targets and
directories, but this is (1) some effort and (2) requires negotiation
with the respective board maintainers, as documentation, build
scripts and other things may be effected by such a change.
Alternatively, we could filter out such problematic characters, but
then again we have to check which side effects this may cause.
All together: this patch needs a lot more thinking and more careful
testing.
I tend to revert it.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
People are always a lot more complicated than you think. It's very
important to remember that. - Terry Pratchett, _Truckers_
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 8:07 ` [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets Wolfgang Denk
2009-09-11 8:12 ` Stefan Roese
@ 2009-09-11 8:46 ` Hu Mingkai-B21284
2009-09-11 15:20 ` Scott Wood
2009-09-14 12:47 ` Michal Simek
3 siblings, 0 replies; 12+ messages in thread
From: Hu Mingkai-B21284 @ 2009-09-11 8:46 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Friday, September 11, 2009 4:08 PM
> To: Hu Mingkai-B21284
> Cc: u-boot at lists.denx.de; galak at kernel.crashing.org; Wood Scott-B07421
> Subject: Re: [PATCH v2 1/2] mkconfig: split the board make
> target to multiple config targets
>
> Dear Mingkai Hu,
>
> In message
> <1252466603-25103-1-git-send-email-Mingkai.hu@freescale.com>
> you wrote:
> > To simplify the top level makefile it useful to be able to
> parse the
> > top level makefile target to multiple individual target,
> then put them
> > to the config.h, leave the board config file to handle the
> different
> > targets.
> >
> > Note that this method uses the '_'(underline) as the delimiter when
> > splits the board make target.
> >
> > Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> > ---
> >
> > According to the comments from Wolfgang and Scott, I modified the
> > patch and made some modification over v1:
> >
> > - remove the sectence thats puts the splited variables to the
> > config.mk, we can use the CONFIG_MK_* in the board config file
> > to override the variable in the board config file.
> >
> > - change CONFIG_OPT_* to CONFIG_MK_*
> >
> > mkconfig | 7 +++++++
> > 1 files changed, 7 insertions(+), 0 deletions(-)
>
> Arghhh... It seems this patch has not been well tested.
> Obviously never ever ran a MAKEALL over all PPC boards. It
> causes tons of messages like that:
>
> include/config.h:2:25: warning: missing whitespace after the
> macro name
>
> This patch breaks building for a number of boards / configurations,
> especially:
>
> xilinx-ppc405-generic_flash
> xilinx-ppc405-generic
> xilinx-ppc440-generic_flash
> xilinx-ppc440-generic
> VoVPN-GW_66MHz
> VoVPN-GW_100MHz
> EB+MCF-EV123
> EB+MCF-EV123_internal
> microblaze-generic
> favr-32-ezkit
>
>
> Shall I revert that patch?
>
Oh..., sorry, I don't run makeall to test and fell free to revert it,
but not this patch,
this patch doesn't be included on your git tree, I think you means the
following patch:
http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commit;h=511c02f611
cb5afa1b8ca5980caaaabaa0de377f
Obviously, this patch also causes these issues, so can I parse the board
config name
in the top make file by using the "findstring" method which will make
the top make file
a little lengthy? If yes, I'll resend the patchset for boot from
NAND/eSDHC/eSPI again.
Thanks,
Mingkai
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 8:07 ` [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets Wolfgang Denk
2009-09-11 8:12 ` Stefan Roese
2009-09-11 8:46 ` Hu Mingkai-B21284
@ 2009-09-11 15:20 ` Scott Wood
2009-09-11 15:35 ` Kumar Gala
2009-09-14 12:47 ` Michal Simek
3 siblings, 1 reply; 12+ messages in thread
From: Scott Wood @ 2009-09-11 15:20 UTC (permalink / raw)
To: u-boot
On Fri, Sep 11, 2009 at 10:07:46AM +0200, Wolfgang Denk wrote:
> In message <1252466603-25103-1-git-send-email-Mingkai.hu@freescale.com> you wrote:
> > To simplify the top level makefile it useful to be able to parse
> > the top level makefile target to multiple individual target, then
> > put them to the config.h, leave the board config file to handle
> > the different targets.
[snip]
> Arghhh... It seems this patch has not been well tested. Obviously
> never ever ran a MAKEALL over all PPC boards. It causes tons of
> messages like that:
>
> include/config.h:2:25: warning: missing whitespace after the macro name
>
> This patch breaks building for a number of boards / configurations,
Actually, I think the patch you quoted would not have had this problem --
unlike the one you merged, it doesn't create #defines for all boards,
only those which request it by passing -t to mkconfig.
-Scott
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 15:20 ` Scott Wood
@ 2009-09-11 15:35 ` Kumar Gala
2009-09-11 18:09 ` Wolfgang Denk
0 siblings, 1 reply; 12+ messages in thread
From: Kumar Gala @ 2009-09-11 15:35 UTC (permalink / raw)
To: u-boot
On Sep 11, 2009, at 10:20 AM, Scott Wood wrote:
> On Fri, Sep 11, 2009 at 10:07:46AM +0200, Wolfgang Denk wrote:
>> In message <1252466603-25103-1-git-send-email-Mingkai.hu@freescale.com
>> > you wrote:
>>> To simplify the top level makefile it useful to be able to parse
>>> the top level makefile target to multiple individual target, then
>>> put them to the config.h, leave the board config file to handle
>>> the different targets.
>
> [snip]
>
>> Arghhh... It seems this patch has not been well tested. Obviously
>> never ever ran a MAKEALL over all PPC boards. It causes tons of
>> messages like that:
>>
>> include/config.h:2:25: warning: missing whitespace after the macro
>> name
>>
>> This patch breaks building for a number of boards / configurations,
>
> Actually, I think the patch you quoted would not have had this
> problem --
> unlike the one you merged, it doesn't create #defines for all boards,
> only those which request it by passing -t to mkconfig.
I'm with Scott on this. Why not just have it be a new explicit option
to mkconfig?
- k
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 15:35 ` Kumar Gala
@ 2009-09-11 18:09 ` Wolfgang Denk
2009-09-14 2:44 ` Hu Mingkai-B21284
0 siblings, 1 reply; 12+ messages in thread
From: Wolfgang Denk @ 2009-09-11 18:09 UTC (permalink / raw)
To: u-boot
Dear Kumar Gala,
In message <C93D00E1-0D25-40AB-AAD9-707DDFD30735@kernel.crashing.org> you wrote:
>
> > Actually, I think the patch you quoted would not have had this
> > problem --
Seems I used a bad reference; sorry for that.
> > unlike the one you merged, it doesn't create #defines for all boards,
> > only those which request it by passing -t to mkconfig.
>
> I'm with Scott on this. Why not just have it be a new explicit option
> to mkconfig?
It's OK with me - and probably the least difficult way to solve the
issue. Who will prepare a patch?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Our OS who art in CPU, UNIX be thy name.
Thy programs run, thy syscalls done,
In kernel as it is in user!
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 18:09 ` Wolfgang Denk
@ 2009-09-14 2:44 ` Hu Mingkai-B21284
0 siblings, 0 replies; 12+ messages in thread
From: Hu Mingkai-B21284 @ 2009-09-14 2:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Saturday, September 12, 2009 2:10 AM
> To: Kumar Gala
> Cc: Wood Scott-B07421; Hu Mingkai-B21284; u-boot at lists.denx.de
> Subject: Re: [PATCH v2 1/2] mkconfig: split the board make
> target to multiple config targets
>
> Dear Kumar Gala,
>
> In message
> <C93D00E1-0D25-40AB-AAD9-707DDFD30735@kernel.crashing.org> you wrote:
> >
> > > Actually, I think the patch you quoted would not have had this
> > > problem --
>
> Seems I used a bad reference; sorry for that.
>
> > > unlike the one you merged, it doesn't create #defines for all
> > > boards, only those which request it by passing -t to mkconfig.
> >
> > I'm with Scott on this. Why not just have it be a new
> explicit option
> > to mkconfig?
>
> It's OK with me - and probably the least difficult way to
> solve the issue. Who will prepare a patch?
>
I've submitted this patch a few days ago which is you quoted:
http://lists.denx.de/pipermail/u-boot/2009-September/060268.html
Thanks,
Mingkai
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets
2009-09-11 8:07 ` [U-Boot] [PATCH v2 1/2] mkconfig: split the board make target to multiple config targets Wolfgang Denk
` (2 preceding siblings ...)
2009-09-11 15:20 ` Scott Wood
@ 2009-09-14 12:47 ` Michal Simek
3 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2009-09-14 12:47 UTC (permalink / raw)
To: u-boot
Wolfgang Denk wrote:
> Dear Mingkai Hu,
>
> In message <1252466603-25103-1-git-send-email-Mingkai.hu@freescale.com> you wrote:
>> To simplify the top level makefile it useful to be able to parse
>> the top level makefile target to multiple individual target, then
>> put them to the config.h, leave the board config file to handle
>> the different targets.
>>
>> Note that this method uses the '_'(underline) as the delimiter when
>> splits the board make target.
>>
>> Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
>> ---
>>
>> According to the comments from Wolfgang and Scott, I modified
>> the patch and made some modification over v1:
>>
>> - remove the sectence thats puts the splited variables to the
>> config.mk, we can use the CONFIG_MK_* in the board config file
>> to override the variable in the board config file.
>>
>> - change CONFIG_OPT_* to CONFIG_MK_*
>>
>> mkconfig | 7 +++++++
>> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> Arghhh... It seems this patch has not been well tested. Obviously
> never ever ran a MAKEALL over all PPC boards. It causes tons of
> messages like that:
>
> include/config.h:2:25: warning: missing whitespace after the macro name
>
> This patch breaks building for a number of boards / configurations,
> especially:
>
> xilinx-ppc405-generic_flash
> xilinx-ppc405-generic
> xilinx-ppc440-generic_flash
> xilinx-ppc440-generic
> VoVPN-GW_66MHz
> VoVPN-GW_100MHz
> EB+MCF-EV123
> EB+MCF-EV123_internal
> microblaze-generic
I can confirm problem on Microblaze.
Please revert that patches.
Thanks,
Michal
> favr-32-ezkit
>
>
> Shall I revert that patch?
>
> Best regards,
>
> Wolfgang Denk
>
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
^ permalink raw reply [flat|nested] 12+ messages in thread