public inbox for linux-kbuild@vger.kernel.org
 help / color / mirror / Atom feed
From: Michal Marek <mmarek@suse.cz>
To: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: linux-kbuild@vger.kernel.org,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [PATCH] kconfig: fix allmodconfig
Date: Wed, 02 Oct 2013 22:29:54 +0200	[thread overview]
Message-ID: <524C8242.2080405@suse.cz> (raw)
In-Reply-To: <52455BA2.2020805@suse.cz>

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

  parent reply	other threads:[~2013-10-02 20:30 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2013-10-03 22:02         ` Yann E. MORIN
2013-10-08 21:54           ` Yann E. MORIN

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=524C8242.2080405@suse.cz \
    --to=mmarek@suse.cz \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=sam@ravnborg.org \
    --cc=sfr@canb.auug.org.au \
    --cc=yann.morin.1998@free.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox