linux-kbuild.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: Michal Marek <mmarek@suse.cz>,
	linux-kbuild@vger.kernel.org, Felipe Balbi <balbi@ti.com>,
	USB list <linux-usb@vger.kernel.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Roger Quadros <rogerq@ti.com>
Subject: Re: choice =y selection becomes lost after having multiple entries =m with depends on
Date: Wed, 23 Oct 2013 12:23:31 +0100	[thread overview]
Message-ID: <20131023112331.GA3869@free.fr> (raw)
In-Reply-To: <5267AA3E.1040003@linutronix.de>

Sebastian, All,

On 2013-10-23 12:51 +0200, Sebastian Andrzej Siewior spake thusly:
> Hi,
> 
> in USB gadget menu (that is Device Drivers ---> USB support ---> USB
> Gadget Support --->  USB Gadget Drivers) I can create a configuration
> which is "lost". Here is how to reproduce it:
> 
> - first config two gadgets as M:
>  <M>   USB Gadget Drivers
>  <M>     Audio Gadget
>  <M>     Ethernet Gadget
>  <M>     MIDI Gadget
> 
>  save config & leave
> 
> - now start menu config again and go to the same menu, now select
>   built-in:
>   <*>   USB Gadget Drivers (Ethernet Gadget
>   the ethernet gadget is chosen automatically because we can have only
>   one gadget selected.
>   save config & leave
> 
> - step three, go back to the menu and you will see that everything is
>   as it was (the <*> is ignored).
> 
> With only Audio & Ethernet =m and Midi =n then the switch to =y
> behaves differently: The menu looks correct (=y) but the .config keeps
> the Audio gadget as =M and it is built. Having multiple entries =m
> which have no "depends on" behaves correctly.

So, I've tried your tests here, and indeed it does not behave as
expected. Yet, I can observe a slight deviation from your observations:
the third time I enter the "USB Gadget Support" sub-menu, the "USB
Gadget Drivers" choice is back to 'M', not 'Y'. But that's still
considered an issue.

> So the problem exists only if the item itself has a "depends on" line.
> Anyone aware of such kconfig limitation?

I'll try to come up with a smaller test-case to investigate, but since
I'm at the LinuxCon Europe and ELCE this week (hey! Party tonight!), I
won't have much time to look at it before the start of next week. Be
sure to ping me later next week if you do not see any reply (and in case
no one else solves the issue in the meantime, that is).

Thanks for this bug report.

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.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2013-10-23 11:23 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-23 10:51 choice =y selection becomes lost after having multiple entries =m with depends on Sebastian Andrzej Siewior
2013-10-23 11:23 ` Yann E. MORIN [this message]
2013-10-23 11:28   ` Sebastian Andrzej Siewior
2013-10-24 15:30     ` Dirk Gouders
2013-10-24 16:19       ` Sebastian Andrzej Siewior
2013-10-24 16:50         ` Dirk Gouders
2013-10-30 10:00         ` Dirk Gouders
2013-10-30 10:30           ` Daniele Forsi
2013-10-30 10:41             ` Dirk Gouders
2013-10-30 14:26           ` Dirk Gouders
2013-10-31 10:20             ` Sebastian Andrzej Siewior
2013-10-31 21:49             ` Yann E. MORIN
2013-11-01  8:45               ` Dirk Gouders
2013-10-31 23:39                 ` [PATCH v3] kconfig/symbol.c: handle choice_values that depend on 'm' symbols Dirk Gouders
2013-11-04 17:27                   ` Sebastian Andrzej Siewior
2013-11-04 20:46                     ` Yann E. MORIN
2013-11-05  8:45                       ` Sebastian Andrzej Siewior
2013-11-05 23:04                   ` Yann E. MORIN
2013-11-06 14:43                     ` Dirk Gouders
2013-11-06 18:59                       ` Yann E. MORIN
2013-11-07 14:02                         ` Dirk Gouders
2013-11-07 14:05                           ` [PATCH v4] " Dirk Gouders
2013-11-18 18:08                             ` Yann E. MORIN
2013-12-20 12:46                               ` Sebastian Andrzej Siewior
2014-08-13 15:35                               ` Bin Liu
2014-08-14  6:52                                 ` Dirk Gouders
2014-08-14 13:54                                   ` Bin Liu
2014-08-15  7:37                                     ` Dirk Gouders
2014-08-15  7:43                                       ` Sebastian Andrzej Siewior
2016-03-30 22:08                                       ` Bin Liu
2016-03-30 22:16                                         ` Ruslan Bilovol
2016-03-31  7:13                                           ` Roger Quadros
2016-03-31  9:38                                             ` Dirk Gouders
2016-03-31  9:53                                               ` Dirk Gouders
2016-04-20 10:19                                             ` [RESEND PATCH " Dirk Gouders
2016-04-20 11:04                                               ` kbuild test robot
2016-04-20 13:14                                                 ` Dirk Gouders
2016-04-29  8:24                                                 ` [PATCH v5] " Dirk Gouders
2016-05-02  8:43                                                   ` Roger Quadros
2016-05-10 19:15                                                     ` Michal Marek
2016-04-20 12:12                                               ` [RESEND PATCH v4] " kbuild test robot
2013-11-08  9:46                       ` [PATCH v3] " Dirk Gouders

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=20131023112331.GA3869@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=balbi@ti.com \
    --cc=bigeasy@linutronix.de \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=rogerq@ti.com \
    --cc=tomi.valkeinen@ti.com \
    /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;
as well as URLs for NNTP newsgroup(s).