* [PATCH] kconfig: fix allmodconfig
@ 2013-09-21 23:17 Yann E. MORIN
2013-09-21 23:21 ` Yann E. MORIN
0 siblings, 1 reply; 8+ messages in thread
From: Yann E. MORIN @ 2013-09-21 23:17 UTC (permalink / raw)
To: linux-kbuild; +Cc: Yann E. MORIN, Stephen Rothwell, Sam Ravnborg, Michal Marek
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Michal Marek <mmarek@suse.cz>
---
scripts/kconfig/confdata.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
index 87f7238..0cbfc67 100644
--- a/scripts/kconfig/confdata.c
+++ b/scripts/kconfig/confdata.c
@@ -399,8 +399,6 @@ setsym:
free(line);
fclose(in);
- if (modules_sym)
- sym_calc_value(modules_sym);
return 0;
}
@@ -1175,6 +1173,10 @@ bool conf_set_all_new_symbols(enum conf_def_mode mode)
sym->def[S_DEF_USER].tri = yes;
break;
case def_mod:
+ /* Note: although modules_sym must be 'yes' to
+ * enable tristates, 'mod' is promoted to 'yes'
+ * when applied to a boolean, which modules_sym
+ * is. So we need not special-case it here. */
sym->def[S_DEF_USER].tri = mod;
break;
case def_no:
--
1.8.1.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-09-21 23:17 [PATCH] kconfig: fix allmodconfig Yann E. MORIN
@ 2013-09-21 23:21 ` Yann E. MORIN
2013-09-26 21:36 ` Yann E. MORIN
0 siblings, 1 reply; 8+ messages in thread
From: Yann E. MORIN @ 2013-09-21 23:21 UTC (permalink / raw)
To: linux-kbuild; +Cc: Stephen Rothwell, Sam Ravnborg, Michal Marek
Stephen, All,
Please, test this patch to fix the allmodconfig issue reported by
Stephen.
From my little testing (some randconfig followed by silentoldconfig, as
well as allmodconfig), this patch seems to fix the issue without any
regression I could ientify.
But since this is sensitive code, I'd like some feedback before this
gets applied.
Thank you!
Regards,
Yann E. MORIN.
On 2013-09-22 01:17 +0200, Yann E. MORIN spake thusly:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Michal Marek <mmarek@suse.cz>
> ---
> scripts/kconfig/confdata.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
> index 87f7238..0cbfc67 100644
> --- a/scripts/kconfig/confdata.c
> +++ b/scripts/kconfig/confdata.c
> @@ -399,8 +399,6 @@ setsym:
> free(line);
> fclose(in);
>
> - if (modules_sym)
> - sym_calc_value(modules_sym);
> return 0;
> }
>
> @@ -1175,6 +1173,10 @@ bool conf_set_all_new_symbols(enum conf_def_mode mode)
> sym->def[S_DEF_USER].tri = yes;
> break;
> case def_mod:
> + /* Note: although modules_sym must be 'yes' to
> + * enable tristates, 'mod' is promoted to 'yes'
> + * when applied to a boolean, which modules_sym
> + * is. So we need not special-case it here. */
> sym->def[S_DEF_USER].tri = mod;
> break;
> case def_no:
> --
> 1.8.1.2
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-09-21 23:21 ` Yann E. MORIN
@ 2013-09-26 21:36 ` Yann E. MORIN
2013-09-27 10:19 ` Michal Marek
0 siblings, 1 reply; 8+ messages in thread
From: Yann E. MORIN @ 2013-09-26 21:36 UTC (permalink / raw)
To: linux-kbuild; +Cc: Stephen Rothwell, Sam Ravnborg, Michal Marek
Hello All!
On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
> Stephen, All,
>
> Please, test this patch to fix the allmodconfig issue reported by
> Stephen.
>
> From my little testing (some randconfig followed by silentoldconfig, as
> well as allmodconfig), this patch seems to fix the issue without any
> regression I could ientify.
>
> But since this is sensitive code, I'd like some feedback before this
> gets applied.
Ping? :-)
I wanted to push this right after -rc3 is out. If any one has an issue
with that patch, you've got a few days to chime in! ;-)
Regards,
Yann E. MORIN.
> On 2013-09-22 01:17 +0200, Yann E. MORIN spake thusly:
> > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> >
> > Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Michal Marek <mmarek@suse.cz>
> > ---
> > scripts/kconfig/confdata.c | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c
> > index 87f7238..0cbfc67 100644
> > --- a/scripts/kconfig/confdata.c
> > +++ b/scripts/kconfig/confdata.c
> > @@ -399,8 +399,6 @@ setsym:
> > free(line);
> > fclose(in);
> >
> > - if (modules_sym)
> > - sym_calc_value(modules_sym);
> > return 0;
> > }
> >
> > @@ -1175,6 +1173,10 @@ bool conf_set_all_new_symbols(enum conf_def_mode mode)
> > sym->def[S_DEF_USER].tri = yes;
> > break;
> > case def_mod:
> > + /* Note: although modules_sym must be 'yes' to
> > + * enable tristates, 'mod' is promoted to 'yes'
> > + * when applied to a boolean, which modules_sym
> > + * is. So we need not special-case it here. */
> > sym->def[S_DEF_USER].tri = mod;
> > break;
> > case def_no:
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-09-26 21:36 ` Yann E. MORIN
@ 2013-09-27 10:19 ` Michal Marek
2013-09-27 20:02 ` Yann E. MORIN
2013-10-02 20:29 ` Michal Marek
0 siblings, 2 replies; 8+ messages in thread
From: Michal Marek @ 2013-09-27 10:19 UTC (permalink / raw)
To: Yann E. MORIN; +Cc: linux-kbuild, Stephen Rothwell, Sam Ravnborg
On 26.9.2013 23:36, Yann E. MORIN wrote:
> Hello All!
>
> On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
>> Stephen, All,
>>
>> Please, test this patch to fix the allmodconfig issue reported by
>> Stephen.
>>
>> From my little testing (some randconfig followed by silentoldconfig, as
>> well as allmodconfig), this patch seems to fix the issue without any
>> regression I could ientify.
>>
>> But since this is sensitive code, I'd like some feedback before this
>> gets applied.
>
> Ping? :-)
>
> I wanted to push this right after -rc3 is out. If any one has an issue
> with that patch, you've got a few days to chime in! ;-)
It works for me, you can add my Tested-by: if you like. The only issue I
have is that a more verbose changelog is missing.
Michal
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-09-27 10:19 ` Michal Marek
@ 2013-09-27 20:02 ` Yann E. MORIN
2013-10-02 20:29 ` Michal Marek
1 sibling, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2013-09-27 20:02 UTC (permalink / raw)
To: Michal Marek; +Cc: linux-kbuild, Stephen Rothwell, Sam Ravnborg
Michal, All,
On 2013-09-27 12:19 +0200, Michal Marek spake thusly:
> On 26.9.2013 23:36, Yann E. MORIN wrote:
> > Hello All!
> >
> > On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
> >> Stephen, All,
> >>
> >> Please, test this patch to fix the allmodconfig issue reported by
> >> Stephen.
> >>
> >> From my little testing (some randconfig followed by silentoldconfig, as
> >> well as allmodconfig), this patch seems to fix the issue without any
> >> regression I could ientify.
> >>
> >> But since this is sensitive code, I'd like some feedback before this
> >> gets applied.
> >
> > Ping? :-)
> >
> > I wanted to push this right after -rc3 is out. If any one has an issue
> > with that patch, you've got a few days to chime in! ;-)
>
> It works for me, you can add my Tested-by: if you like. The only issue I
> have is that a more verbose changelog is missing.
OK. I'll rewrite the commit log with more details before I send the
final pull-request. Thank you! :-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-09-27 10:19 ` Michal Marek
2013-09-27 20:02 ` Yann E. MORIN
@ 2013-10-02 20:29 ` Michal Marek
2013-10-03 22:02 ` Yann E. MORIN
1 sibling, 1 reply; 8+ messages in thread
From: Michal Marek @ 2013-10-02 20:29 UTC (permalink / raw)
To: Yann E. MORIN; +Cc: linux-kbuild, Stephen Rothwell, Sam Ravnborg
Dne 27.9.2013 12:19, Michal Marek napsal(a):
> On 26.9.2013 23:36, Yann E. MORIN wrote:
>> Hello All!
>>
>> On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
>>> Stephen, All,
>>>
>>> Please, test this patch to fix the allmodconfig issue reported by
>>> Stephen.
>>>
>>> From my little testing (some randconfig followed by silentoldconfig, as
>>> well as allmodconfig), this patch seems to fix the issue without any
>>> regression I could ientify.
>>>
>>> But since this is sensitive code, I'd like some feedback before this
>>> gets applied.
>>
>> Ping? :-)
>>
>> I wanted to push this right after -rc3 is out. If any one has an issue
>> with that patch, you've got a few days to chime in! ;-)
>
> It works for me, you can add my Tested-by: if you like. The only issue I
> have is that a more verbose changelog is missing.
Sorry, I have to retract my Tested-by. When I merge your rc-fixes branch
into 3.12-rc1, I get this behavior:
$ make mrproper
$ make allmodconfig
...
scripts/kconfig/conf --allmodconfig Kconfig
#
# configuration written to .config
#$ make silentoldconfig
scripts/kconfig/conf --silentoldconfig Kconfig
*
* Restart config...
*
(this should not happen)
*
* PHY Device support and infrastructure
*
PHY Device support and infrastructure (PHYLIB) [Y/?] y
*
* MII PHY device drivers
*
Drivers for Atheros AT803X PHYs (AT803X_PHY) [M/n/y/?] m
Drivers for the AMD PHYs (AMD_PHY) [M/n/y/?] m
Drivers for Marvell PHYs (MARVELL_PHY) [M/n/y/?] m
Drivers for Davicom PHYs (DAVICOM_PHY) [M/n/y/?] m
Drivers for Quality Semiconductor PHYs (QSEMI_PHY) [M/n/y/?] m
Drivers for the Intel LXT PHYs (LXT_PHY) [M/n/y/?] m
Drivers for the Cicada PHYs (CICADA_PHY) [M/n/y/?] m
Drivers for the Vitesse PHYs (VITESSE_PHY) [M/n/y/?] m
Drivers for SMSC PHYs (SMSC_PHY) [M/y/?] m
Drivers for Broadcom PHYs (BROADCOM_PHY) [M/n/y/?] m
Driver for Broadcom BCM8706 and BCM8727 PHYs (BCM87XX_PHY) [M/n/y/?] m
Drivers for ICPlus PHYs (ICPLUS_PHY) [M/n/y/?] m
Drivers for Realtek PHYs (REALTEK_PHY) [M/n/y/?] m
Drivers for National Semiconductor PHYs (NATIONAL_PHY) [M/n/y/?] m
Driver for STMicroelectronics STe10Xp PHYs (STE10XP) [M/n/y/?] m
Driver for LSI ET1011C PHY (LSI_ET1011C_PHY) [M/n/y/?] m
Driver for Micrel PHYs (MICREL_PHY) [M/n/y/?] m
Driver for MDIO Bus/PHY emulation with fixed speed/link PHYs
(FIXED_PHY) [N/y/?] (NEW)
The .config generated by make allmodconfig is identical to a .config
generated with plain v3.12-rc1 and thus correct. It's (silent)oldconfig
that has a problem.
Michal
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-10-02 20:29 ` Michal Marek
@ 2013-10-03 22:02 ` Yann E. MORIN
2013-10-08 21:54 ` Yann E. MORIN
0 siblings, 1 reply; 8+ messages in thread
From: Yann E. MORIN @ 2013-10-03 22:02 UTC (permalink / raw)
To: Michal Marek; +Cc: linux-kbuild, Stephen Rothwell, Sam Ravnborg
Michal, All,
On 2013-10-02 22:29 +0200, Michal Marek spake thusly:
> Dne 27.9.2013 12:19, Michal Marek napsal(a):
> > On 26.9.2013 23:36, Yann E. MORIN wrote:
> >> Hello All!
> >>
> >> On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
> >>> Stephen, All,
> >>>
> >>> Please, test this patch to fix the allmodconfig issue reported by
> >>> Stephen.
> >>>
> >>> From my little testing (some randconfig followed by silentoldconfig, as
> >>> well as allmodconfig), this patch seems to fix the issue without any
> >>> regression I could ientify.
> >>>
> >>> But since this is sensitive code, I'd like some feedback before this
> >>> gets applied.
> >>
> >> Ping? :-)
> >>
> >> I wanted to push this right after -rc3 is out. If any one has an issue
> >> with that patch, you've got a few days to chime in! ;-)
> >
> > It works for me, you can add my Tested-by: if you like. The only issue I
> > have is that a more verbose changelog is missing.
>
> Sorry, I have to retract my Tested-by. When I merge your rc-fixes branch
> into 3.12-rc1, I get this behavior:
>
> $ make mrproper
> $ make allmodconfig
> ...
> scripts/kconfig/conf --allmodconfig Kconfig
> #
> # configuration written to .config
> #$ make silentoldconfig
> scripts/kconfig/conf --silentoldconfig Kconfig
> *
> * Restart config...
> *
> (this should not happen)
> *
> * PHY Device support and infrastructure
> *
> PHY Device support and infrastructure (PHYLIB) [Y/?] y
> *
> * MII PHY device drivers
> *
> Drivers for Atheros AT803X PHYs (AT803X_PHY) [M/n/y/?] m
> Drivers for the AMD PHYs (AMD_PHY) [M/n/y/?] m
> Drivers for Marvell PHYs (MARVELL_PHY) [M/n/y/?] m
> Drivers for Davicom PHYs (DAVICOM_PHY) [M/n/y/?] m
> Drivers for Quality Semiconductor PHYs (QSEMI_PHY) [M/n/y/?] m
> Drivers for the Intel LXT PHYs (LXT_PHY) [M/n/y/?] m
> Drivers for the Cicada PHYs (CICADA_PHY) [M/n/y/?] m
> Drivers for the Vitesse PHYs (VITESSE_PHY) [M/n/y/?] m
> Drivers for SMSC PHYs (SMSC_PHY) [M/y/?] m
> Drivers for Broadcom PHYs (BROADCOM_PHY) [M/n/y/?] m
> Driver for Broadcom BCM8706 and BCM8727 PHYs (BCM87XX_PHY) [M/n/y/?] m
> Drivers for ICPlus PHYs (ICPLUS_PHY) [M/n/y/?] m
> Drivers for Realtek PHYs (REALTEK_PHY) [M/n/y/?] m
> Drivers for National Semiconductor PHYs (NATIONAL_PHY) [M/n/y/?] m
> Driver for STMicroelectronics STe10Xp PHYs (STE10XP) [M/n/y/?] m
> Driver for LSI ET1011C PHY (LSI_ET1011C_PHY) [M/n/y/?] m
> Driver for Micrel PHYs (MICREL_PHY) [M/n/y/?] m
> Driver for MDIO Bus/PHY emulation with fixed speed/link PHYs
> (FIXED_PHY) [N/y/?] (NEW)
>
> The .config generated by make allmodconfig is identical to a .config
> generated with plain v3.12-rc1 and thus correct. It's (silent)oldconfig
> that has a problem.
Doh... :-(
I've played with this a bit, and I noticed that if you do:
$ git clean -dX; git clean -d # Make sure tree is clean
$ make allmodconfig
$ grep PHYLIB= .config
CONFIG_PHYLIB=m
$ make menuconfig
Device Drivers -->
Network device support -->
-*- PHY Device support and infrastructure -->
Notice how "PHY Device support and infrastructure" is forced to 'y', not
'm'. This is expected, because it is selected by (there are other terms
in the equation that also force it to 'y'):
ETHOC [=y] && NETDEVICES [=y] && ETHERNET [=y]
&& HAS_IOMEM [=y] && HAS_DMA [=y]
Hence, what we see is that "make allmodconfig" does not properly resolve
the symbols values.
I'll tackle this, but it may take some time...
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] kconfig: fix allmodconfig
2013-10-03 22:02 ` Yann E. MORIN
@ 2013-10-08 21:54 ` Yann E. MORIN
0 siblings, 0 replies; 8+ messages in thread
From: Yann E. MORIN @ 2013-10-08 21:54 UTC (permalink / raw)
To: Michal Marek; +Cc: linux-kbuild, Stephen Rothwell, Sam Ravnborg
Michal, Al,
On 2013-10-04 00:02 +0200, Yann E. MORIN spake thusly:
> On 2013-10-02 22:29 +0200, Michal Marek spake thusly:
> > Dne 27.9.2013 12:19, Michal Marek napsal(a):
> > > On 26.9.2013 23:36, Yann E. MORIN wrote:
> > >> Hello All!
> > >>
> > >> On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly:
> > >>> Stephen, All,
> > >>>
> > >>> Please, test this patch to fix the allmodconfig issue reported by
> > >>> Stephen.
> > >>>
> > >>> From my little testing (some randconfig followed by silentoldconfig, as
> > >>> well as allmodconfig), this patch seems to fix the issue without any
> > >>> regression I could ientify.
> > >>>
> > >>> But since this is sensitive code, I'd like some feedback before this
> > >>> gets applied.
> > >>
> > >> Ping? :-)
> > >>
> > >> I wanted to push this right after -rc3 is out. If any one has an issue
> > >> with that patch, you've got a few days to chime in! ;-)
> > >
> > > It works for me, you can add my Tested-by: if you like. The only issue I
> > > have is that a more verbose changelog is missing.
> >
> > Sorry, I have to retract my Tested-by. When I merge your rc-fixes branch
> > into 3.12-rc1, I get this behavior:
> >
> > $ make mrproper
> > $ make allmodconfig
> > ...
> > scripts/kconfig/conf --allmodconfig Kconfig
> > #
> > # configuration written to .config
> > #$ make silentoldconfig
> > scripts/kconfig/conf --silentoldconfig Kconfig
> > *
> > * Restart config...
> > *
> > (this should not happen)
> > *
> > * PHY Device support and infrastructure
> > *
> > PHY Device support and infrastructure (PHYLIB) [Y/?] y
> > *
> > * MII PHY device drivers
> > *
> > Drivers for Atheros AT803X PHYs (AT803X_PHY) [M/n/y/?] m
> > Drivers for the AMD PHYs (AMD_PHY) [M/n/y/?] m
> > Drivers for Marvell PHYs (MARVELL_PHY) [M/n/y/?] m
> > Drivers for Davicom PHYs (DAVICOM_PHY) [M/n/y/?] m
> > Drivers for Quality Semiconductor PHYs (QSEMI_PHY) [M/n/y/?] m
> > Drivers for the Intel LXT PHYs (LXT_PHY) [M/n/y/?] m
> > Drivers for the Cicada PHYs (CICADA_PHY) [M/n/y/?] m
> > Drivers for the Vitesse PHYs (VITESSE_PHY) [M/n/y/?] m
> > Drivers for SMSC PHYs (SMSC_PHY) [M/y/?] m
> > Drivers for Broadcom PHYs (BROADCOM_PHY) [M/n/y/?] m
> > Driver for Broadcom BCM8706 and BCM8727 PHYs (BCM87XX_PHY) [M/n/y/?] m
> > Drivers for ICPlus PHYs (ICPLUS_PHY) [M/n/y/?] m
> > Drivers for Realtek PHYs (REALTEK_PHY) [M/n/y/?] m
> > Drivers for National Semiconductor PHYs (NATIONAL_PHY) [M/n/y/?] m
> > Driver for STMicroelectronics STe10Xp PHYs (STE10XP) [M/n/y/?] m
> > Driver for LSI ET1011C PHY (LSI_ET1011C_PHY) [M/n/y/?] m
> > Driver for Micrel PHYs (MICREL_PHY) [M/n/y/?] m
> > Driver for MDIO Bus/PHY emulation with fixed speed/link PHYs
> > (FIXED_PHY) [N/y/?] (NEW)
> >
> > The .config generated by make allmodconfig is identical to a .config
> > generated with plain v3.12-rc1 and thus correct. It's (silent)oldconfig
> > that has a problem.
>
> Doh... :-(
>
> I've played with this a bit, and I noticed that if you do:
> $ git clean -dX; git clean -d # Make sure tree is clean
> $ make allmodconfig
> $ grep PHYLIB= .config
> CONFIG_PHYLIB=m
> $ make menuconfig
> Device Drivers -->
> Network device support -->
> -*- PHY Device support and infrastructure -->
>
> Notice how "PHY Device support and infrastructure" is forced to 'y', not
> 'm'. This is expected, because it is selected by (there are other terms
> in the equation that also force it to 'y'):
> ETHOC [=y] && NETDEVICES [=y] && ETHERNET [=y]
> && HAS_IOMEM [=y] && HAS_DMA [=y]
>
> Hence, what we see is that "make allmodconfig" does not properly resolve
> the symbols values.
>
> I'll tackle this, but it may take some time...
OK, I have no clue... :-(
Any suggestion?
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-10-08 21:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-21 23:17 [PATCH] kconfig: fix allmodconfig Yann E. MORIN
2013-09-21 23:21 ` Yann E. MORIN
2013-09-26 21:36 ` Yann E. MORIN
2013-09-27 10:19 ` Michal Marek
2013-09-27 20:02 ` Yann E. MORIN
2013-10-02 20:29 ` Michal Marek
2013-10-03 22:02 ` Yann E. MORIN
2013-10-08 21:54 ` Yann E. MORIN
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox