Re: [Buildroot] ARC no console output (was Re: [PATCH 1/2] init/console: Use ttynull as a fallback when there is no console)

2021-01-07 Thread Yann E. MORIN
fs/cpio/?id=98a6f1fc02e41 > Please advise what needs to be done to unbork boot. This has been present since the 2020.08 release, and has been backported to the maintenance branches: 2020.02.x (LTS) -> f1a83afe2df2a 2020.05.x -> 797f9e40224c9 > Otherwise this seems

Re: [PATCH] scripts: kconfig: implement a sort method

2017-04-12 Thread Yann E. MORIN
ere: $ stty size 25 80 https://imgur.com/a/ku0sG > > I haven't looked at the source code (lately), but I think that it > > would OK to not have the (extra) spaces inside the <...> brackets. > > I.e., instead of > > │< Exit >

[PATCH] MAINTAINERS: relinquish kconfig

2017-04-10 Thread Yann E. MORIN
Since about the time I was actually added in that file, I've been almost entirely absent, which totally sucks, and there is no excuse for my behaviour and for not having relinquished this earlier. My deepest apologies. Signed-off-by: "Yann E. MORIN" Cc: Michal Marek Cc: Masa

Re: [patch core/stackprotector] stackprotector: Fix build when compiler lacks support

2014-01-01 Thread Yann E. MORIN
Linus, All, On 2014-01-01 11:33 -0800, Linus Torvalds spake thusly: > On Wed, Jan 1, 2014 at 3:42 AM, Yann E. MORIN wrote: > > > > On 2013-12-31 16:16 -0800, Linus Torvalds spake thusly: > >> > >> What we really really want to do is to have some way to add c

Re: [patch core/stackprotector] stackprotector: Fix build when compiler lacks support

2014-01-01 Thread Yann E. MORIN
tack-protector strong" depends on CC_SUPPORTS_STACKPROTECTOR_STRONG Would that be something that match what you suggested above? Sam, there were some comments on that patch of yours. Do you want to update it and resubmit it? And, Happy New Year to All! Regards, Yann E. MORIN. -- .-

Re: [PATCH 1/2] [trivial] scripts: kconfig: Fix trailing whitespace in zconf.lex.c_shipped

2014-01-01 Thread Yann E. MORIN
nly really care about trailing spaces in actual source files. Ditto for your patch 2/2. Regards, Yann E. MORIN. -- .-..--.--------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 6

Re: [PATCH] scripts/config: allow setting a string which value contains a colon

2013-11-18 Thread Yann E. MORIN
#x27; commands that script `config' > relies on. > > Escape colons in `sed' commands to allow values containing colons. > > Signed-off-by: Clement Chauplannaz Acked-by: "Yann E. MORIN" It'll be in my tree soon, as a rc-fix. Thank you. Regards, Yann E. M

Re: make oldnoconfig fix?

2013-11-11 Thread Yann E. MORIN
thread, please add a pointer (URL) to this thread in the archives (Use: http://marc.info/?l=linux-kbuild ). Regards, Yann E. MORIN. -- .-.----.--.. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' consp

[PATCH] Makefile: fix out-of-tree builds for multiple targets with 'all'

2013-11-03 Thread Yann E. MORIN
From: "Yann E. MORIN" For out-of-tree builds, this use-case fails to build: $ make clean all This is because 'all' is filtered-out in the Makefile wrapper, since the wrapper itself has a 'all' target. The 'all' target is just the usual naming for t

Re: [PATCH] MAINTAINERS: New kconfig maintainer

2013-10-30 Thread Yann E. MORIN
Michal, All, On 2013-10-30 12:07 +0100, Michal Marek spake thusly: > Yann has been the de facto maintainer of kconfig for some time. Update > the KCONFIG entry with his emails address and git tree. > > Cc: "Yann E. MORIN" > Signed-off-by: Michal Marek Acked-by: "

Re: [PATCH] scripts/checkkconfig.py: find unused Kconfig parameters

2013-10-29 Thread Yann E. MORIN
onder if it would not be possible to invert the loop (in pseudo Python code): for f in all_interesting_files: read f in memory for s in all_symbols: if symbol is in f: remove f from all_symbols break the inner-most loop This way: - you scan the tree onl

Re: [PATCH] scripts/kconfig/menu.c: warning: jump may be used uninitialized in this function

2013-10-29 Thread Yann E. MORIN
de reall issues in case the code is changed in the future, so it is not appropriate a fix. Hence why the previously- submitted patches were not applied. I'll take this on me to properly fix this. Thanks for the heads up. Regards, Yann E. MORIN. -- .-.--

Re: [PATCH v2 0/5] kconfig: cleanups, documentation updates, and one bug fix

2013-10-08 Thread Yann E. MORIN
| 2 +- > scripts/kconfig/zconf.l | 1 - > 5 files changed, 37 insertions(+), 39 deletions(-) > > -- > 1.8.1.5 > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majord...@vger.kernel

Re: [PATCH 4/4] kconfig: remove unused definition from scanner

2013-10-02 Thread Yann E. MORIN
Martin, All, On Thursday 03 October 2013 02:30:21 Martin Walch wrote: > On Wednesday 02 October 2013 08:57:54 Yann E. MORIN wrote: > > On Wednesday 02 October 2013 08:42:59 Dirk Gouders wrote: > > > I think you should also regenerate the scanner and add > > > the new

Re: [PATCH 4/4] kconfig: remove unused definition from scanner

2013-10-02 Thread Yann E. MORIN
th patch. BTW, regenerating the parser can be done with: make REGENERATE_PARSERS=1 config Regards, Yann E. MORIN. > > Signed-off-by: Martin Walch > > --- > > scripts/kconfig/zconf.l | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/scripts/k

[pull request] Pull request for branch yem/kconfig-rc-fixes

2013-09-30 Thread Yann E. MORIN
From: "Yann E. MORIN" Michal, Please pull these two changes for 3.12-rc: - a fix for "make install" on ARM, by Robert Richter, acked by Linus and yourself; - a fix for "make allmodconfig" when KCONFIG_ALLCONFIG is set, which already flew on the list, bu

Re: [PATCH] arm, kbuild: make "make install" not depend on vmlinux

2013-09-30 Thread Yann E. MORIN
if needed. Otherwise you can add > > Acked-by: Michal Marek . > > This didn't make it upstream yet, can somebody at it to a tree? Since it's been acked-by Linus, I'll queue it in my tree, for Michal to pu

Re: [PATCH V2] kconfig/menu.c: fix uninitialized variable warning

2013-09-26 Thread Yann E. MORIN
* with gcc != 4.8.0 */ struct jump_key *jump; This is so, if sometime in the future the code changes, and the variable is indeeded used uninitialised, we have a proper warning by gcc in this case. Care to resend, please? Regards, Yann E. MORIN. > Signed-off-by: Madhavan Srinivasan >

Re: [PATCH] Documentation/kbuild/kconfig.txt: Doesn't make listnewconfig replacing yes "" | make oldconfig ?

2013-09-21 Thread Yann E. MORIN
ee for -next. Thanks! Regards, Yann E. MORIN. > diff --git a/Documentation/kbuild/kconfig.txt > b/Documentation/kbuild/kconfig.txt > index 8ef6dbb..14cc40c 100644 > --- a/Documentation/kbuild/kconfig.txt > +++ b/Documentation/kbuild/kconfig.txt > @@ -20,16 +20,9 @@ symbols have bee

Re: [PATCH]kconfig/menu.c: fix uninitialized variable warning

2013-09-20 Thread Yann E. MORIN
Benjamin, Madhavan, All, On 2013-09-20 13:58 -0400, Benjamin Poirier spake thusly: > On 2013/09/20 08:05, Madhavan Srinivasan wrote: > > On Friday 20 September 2013 01:34 AM, Yann E. MORIN wrote: > > > Benjamin, All, > > > > > > On 2013-09-19 15:13 -0400, Benj

Re: [PATCH]kconfig/menu.c: fix uninitialized variable warning

2013-09-19 Thread Yann E. MORIN
Benjamin, All, On 2013-09-19 15:13 -0400, Benjamin Poirier spake thusly: > On 2013/09/19 19:27, Yann E. MORIN wrote: > > Benjamin, Madhavan, All, > > > > On 2013-09-19 11:22 -0400, Benjamin Poirier spake thusly: > > > On 2013/09/19 12:58, Madhavan Srinivasan wrote:

Re: [PATCH]kconfig/menu.c: fix uninitialized variable warning

2013-09-19 Thread Yann E. MORIN
perty *prop, > > { > > int i, j; > > struct menu *submenu[8], *menu, *location = NULL; > > - struct jump_key *jump; > > + struct jump_key *jump = NULL; > > > > str_printf(r, _("Prompt: %s\n"), _(prop->text)); > > menu = pr

Re: scripts/config: fix variable substitution command

2013-09-14 Thread Yann E. MORIN
offline linux-kbuild ML. That would be Michal's suggestion there: http://marc.info/?l=linux-kbuild&m=137907001305533&w=2 > BTW, the GIT repo of Yann is not browsable (which is sh*t for checking > commits quickl, I don't want to be forced to checkout). Yes, I know abou

Re: scripts/config: fix variable substitution command

2013-09-14 Thread Yann E. MORIN
ert this change. [--SNIP--] > Is this patch even "CC: stable"? It was not in any previous released version, since it went in Linus' tree for 3.12. As such, it does not make sense to Cc: stable. Regards, Yann E. MORIN. -- .-.--------.--

Re: [PATCH] Documentation/kbuild/kconfig.txt: Doesn't make listnewconfig replacing yes "" | make oldconfig ?

2013-09-12 Thread Yann E. MORIN
Regid, All, On 2013-09-12 03:35 +0300, Regid Ichira spake thusly: > Signed-off-by: Regid Ichira Acked-by: "Yann E. MORIN" > --- > Documentation/kbuild/kconfig.txt | 11 ++- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/Documenta

Re: [PATCH tip/core/rcu 8/9] nohz_full: Add full-system-idle state machine

2013-09-08 Thread Yann E. MORIN
Paul, All, On 2013-09-07 11:57 -0700, Paul E. McKenney spake thusly: > On Sat, Sep 07, 2013 at 11:13:48AM +0200, Yann E. MORIN wrote: [--SNIP--] > > I'll see if I can come up with a meaningfull construct that fixes your > > use-case. Don't hold your breath, though! ;-) &g

Re: [PATCH tip/core/rcu 8/9] nohz_full: Add full-system-idle state machine

2013-09-07 Thread Yann E. MORIN
k. The dependency expression is appended to all enclosed menu entries. ---8<--- There's no equivlaent to '#ifdef' in Kconfig. I'll see if I can come up with a meaningfull construct that fixes your use-case. Don't hold your breath, though! ;-) Regards, Y

Re: [PATCH 1/3] kconfig: do not special-case 'MODULES' symbol

2013-09-06 Thread Yann E. MORIN
Michal, All, On 2013-09-05 12:02 +0200, Michal Marek spake thusly: > On 5.9.2013 11:38, Michal Marek wrote: > > On 3.9.2013 23:04, Yann E. MORIN wrote: > >> From: "Yann E. MORIN" > >> This patch changes the way we enable/disable tristates: if a symbo

Re: [PATCH 1/3] kconfig: do not special-case 'MODULES' symbol

2013-09-06 Thread Yann E. MORIN
Michal, All, On 2013-09-05 11:38 +0200, Michal Marek spake thusly: > On 3.9.2013 23:04, Yann E. MORIN wrote: > > From: "Yann E. MORIN" > > > > Currently, the 'MODULES' symbol is hard-coded to be the default symbol > > that enables/disables

[PATCH 3/3] kconfig: do not allow more than one symbol to have 'option modules'

2013-09-04 Thread Yann E. MORIN
From: "Yann E. MORIN" Previously, it was possible to have more than one symbol with the 'option modules' attached to them, although only the last one would in fact control tristates. Since this does not make much sense, only allow at most one symbol to control tristates

[PATCH 2/3] kconfig: regenerate bison parser

2013-09-04 Thread Yann E. MORIN
From: "Yann E. MORIN" Regenerate bison parser after changes made in: b1a4fe3: kconfig: do not special-case 'MODULES' symbol Signed-off-by: "Yann E. MORIN" Cc: Sam Ravnborg Cc: Michal Marek --- scripts/kconfig/zconf.tab.c_shipped | 562 +++---

[PATCH 1/3] kconfig: do not special-case 'MODULES' symbol

2013-09-04 Thread Yann E. MORIN
From: "Yann E. MORIN" Currently, the 'MODULES' symbol is hard-coded to be the default symbol that enables/disables tristates, if no other symbol was declared with 'option modules'. While this used to be needed for the Linux kernel, we now have an explicit &#

[pull request] Pull request for branch yem/kconfig-for-next

2013-09-04 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello All! Michal, please pull for 3.12 this series that removes special-casing of the 'MODULES' symbol, as previously discussed with Sam and you. Regards, Yann E. MORIN. The following changes since commit b020bae0467b490797d61e6a658fb7df4568c

Re: [PATCH 1/3] kconfig: do not special-case 'MODULES' symbol

2013-09-03 Thread Yann E. MORIN
Michal, All, On 2013-09-03 23:04 +0200, Yann E. MORIN spake thusly: > Currently, the 'MODULES' symbol is hard-coded to be the default symbol > that enables/disables tristates, if no other symbol was declared with > 'option modules'. > > While this used to be

[PATCH 3/3] kconfig: do not allow more than one symbol to have 'option modules'

2013-09-03 Thread Yann E. MORIN
From: "Yann E. MORIN" Previously, it was possible to have more than one symbol with the 'option modules' attached to them, although only the last one would in fact controls tristates. Since this does not make much sense, only allow at most one symbol to control tristates

[PATCH 2/3] kconfig: regenerate bison parser

2013-09-03 Thread Yann E. MORIN
From: "Yann E. MORIN" Regenerate bison parser after changes made in: b1a4fe3: kconfig: do not special-case 'MODULES' symbol Signed-off-by: "Yann E. MORIN" Cc: Sam Ravnborg Cc: Michal Marek --- scripts/kconfig/zconf.tab.c_shipped | 562 +++---

[PATCH 1/3] kconfig: do not special-case 'MODULES' symbol

2013-09-03 Thread Yann E. MORIN
From: "Yann E. MORIN" Currently, the 'MODULES' symbol is hard-coded to be the default symbol that enables/disables tristates, if no other symbol was declared with 'option modules'. While this used to be needed for the Linux kernel, we now have an explicit &#

Re: linux-next: build failure after merge of the ext4 tree

2013-08-16 Thread Yann E. MORIN
Sam, Michal, All, On 2013-08-16 19:14 +0200, Sam Ravnborg spake thusly: > On Fri, Aug 16, 2013 at 03:10:38PM +0200, Michal Marek wrote: > > On 11.8.2013 23:39, Yann E. MORIN wrote: > > > On 2013-08-09 13:42 +0200, Sam Ravnborg spake thusly: > > >> If we drop the spe

Re: [pull request] Pull request for branch yem/kconfig-for-next

2013-08-16 Thread Yann E. MORIN
Michal, All, On Friday 16 August 2013 14:48:47 Michal Marek wrote: > On 15.8.2013 23:17, Yann E. MORIN wrote: > > From: "Yann E. MORIN" > > > > Hello Michal, > > > > Please pull a few more changes queued for 3.12 since your last pull: > > &

[PATCH 2/4] scripts/config: use sed's POSIX interface

2013-08-15 Thread Yann E. MORIN
alls to `a' command, while some implementations require a leading newline before the parameter. Rewrite calls to `sed' to comply with POSIX interface, and move them to helper functions. Signed-off-by: Clement Chauplannaz Tested-by: "Yann E. MORIN" Reviewed-by: "Ya

[PATCH 1/4] kconfig: switch to "long long" for sanity

2013-08-15 Thread Yann E. MORIN
Kees Cook Acked-by: Geert Uytterhoeven Tested-by: "Yann E. MORIN" Signed-off-by: "Yann E. MORIN" --- scripts/kconfig/symbol.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c index

[PATCH 3/4] kconfig: silence warning when parsing auto.conf when a symbol has changed type

2013-08-15 Thread Yann E. MORIN
From: "Yann E. MORIN" When a symbol changes type from tristate to bool, and was previously set to 'm', a subsequent silentoldconfig would warn about inconsistency, such as: include/config/auto.conf:3014:warning: symbol value 'm' invalid for HOTPLUG_PCI_PCIE

[pull request] Pull request for branch yem/kconfig-for-next

2013-08-15 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello Michal, Please pull a few more changes queued for 3.12 since your last pull: - POSIX compliance when using sed in scripts/config(CLément) - use 'long long' to represent hex and ranges, so their width is not dependent on the arch

[PATCH 4/4] modules: do not depend on kconfig to set 'modules' option to symbol MODULES

2013-08-15 Thread Yann E. MORIN
From: "Yann E. MORIN" Currently, the MODULES symbol is special-cased in different places in the kconfig language. For example, if no symbol is defined to enable tristates, then kconfig looks up for a symbol named 'MODULES', and forces the 'modules' option onto that

Re: linux-next: build failure after merge of the ext4 tree

2013-08-11 Thread Yann E. MORIN
Sam, All, On 2013-08-09 13:42 +0200, Sam Ravnborg spake thusly: > On Thu, Aug 08, 2013 at 11:54:49PM +0200, Yann E. MORIN wrote: > > Stephen, All, > > > > On 2013-08-08 21:16 +0200, Yann E. MORIN spake thusly: > > > On 2013-08-08 10:36 +1000, Stephen Rothwell spake

Re: linux-next: build failure after merge of the ext4 tree

2013-08-08 Thread Yann E. MORIN
Stephen, All, On 2013-08-08 21:16 +0200, Yann E. MORIN spake thusly: > On 2013-08-08 10:36 +1000, Stephen Rothwell spake thusly: > > On Thu, 8 Aug 2013 10:22:28 +1000 Stephen Rothwell > > wrote: > > > > > > More quick testing with an empty file: v3.9 is OK, v3.10

Re: linux-next: build failure after merge of the ext4 tree

2013-08-08 Thread Yann E. MORIN
ve. Can you elaborate on what you did, what you got, what expected, so I can try to reproduce and fix this, please? > Bisecting gives: > > cfa98f2e0ae956feca935573e977d7661a9561b9 is the first bad commit > commit cfa98f2e0ae956feca935573e977d7661a9561b9 > Author: Yann E. MOR

[PATCH] kconfig: silence warning when parsing auto.conf when a symbol has changed type

2013-08-06 Thread Yann E. MORIN
From: "Yann E. MORIN" When a symbol changes type from tristate to bool, and was previously set to 'm', a subsequent silentoldconfig would warn about inconsistency, such as: include/config/auto.conf:3014:warning: symbol value 'm' invalid for HOTPLUG_PCI_PCIE

Re: Warnings from silentoldconfig (Re: [GIT PULL] PCI updates for v3.11)

2013-08-05 Thread Yann E. MORIN
n tackle this again. In any case, I've found the kconfig code to be relatively difficult to read and follow... :-( It's mostly an agglomerate of orgnic changes accumulated over time, it is missing coments in the right places, is rather complex with a lot of special cases spread out all over,

Re: Warnings from silentoldconfig (Re: [GIT PULL] PCI updates for v3.11)

2013-08-05 Thread Yann E. MORIN
/auto.conf, > which may not exist yet or may be out of date. Yann, can anything break > if we simply do not read that file from conf_split_config(), like this? Sorry, that's a part I'm not sure about, and seems non-trivial. I'll look at it tonight (UTC+2) when I'm b

Re: Build regressions/improvements in v3.11-rc2

2013-07-27 Thread Yann E. MORIN
n how to reproduce this error? randconfig now should print the RNF seed, so that's d be nice to have to reproduce the error. Also, having the problematic changeset would be nice. Regards, Yann E. MORIN. -- .-.----.--..

Re: [pull request] Pull request for branch yem-kconfig-rc-fixes

2013-07-23 Thread Yann E. MORIN
Michal, All, On Tuesday 23 July 2013 15:37:37 Michal Marek wrote: > On 17.7.2013 11:59, Jean Delvare wrote: > > Le Tuesday 16 July 2013 à 22:39 +0200, Yann E. MORIN a écrit : > >> From: "Yann E. MORIN" > >> > >> Hello Michal, > >> >

Re: [PATCH v2] kconfig: switch to "long long" for sanity

2013-07-18 Thread Yann E. MORIN
or can it wait in -for-next? My gut feelings is that it does not fix an existing breakage, but only a potential problem in the future, so it would be a candidate for -for-next. Agreed? Regards, Yann E. MORIN. -- .-----..--.. | Y

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-18 Thread Yann E. MORIN
Andrew, All, On Thursday 18 July 2013 09:34:08 Andrew Morton wrote: > On Thu, 18 Jul 2013 09:22:58 +0200 Geert Uytterhoeven > wrote: > > On Thu, Jul 18, 2013 at 12:30 AM, Yann E. MORIN > > wrote: > > > On 2013-07-17 23:16 +0200, Sam Ravnborg spake thusly: > >

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-17 Thread Yann E. MORIN
menu_warn(current_entry, > + "command '%s' - invalid (empty?) > return value: \"%s\"", > + command, buffer); > + ret

[pull request] Pull request for branch yem-kconfig-rc-fixes

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello Michal, Please pull these post-rc1 kconfig cleanups, all after review and comments by Jean: - simplify and cleanup the symbol-search code - better documentation about the symbols ordering in search results - eye-candy in the title of the search-box i

[PATCH 1/6] Documentation/kconfig: more concise and straightforward search explanation

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Re-phrase the explanations on the sorting of search results, in a more concise and complete way. Drop reference to the user's locale when sorting alphabetically, since this is implicit. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" C

[PATCH 4/6] kconfig: minor style fixes in symbol-search code

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Two minor style fixes: - no space before/after parenthesis in function definition - no {} for single-line if() And one grammar fix in a comment. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare Reviewed-by: Jean D

[PATCH 2/6] kconfig: avoid multiple calls to strlen

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" Calls to strlen are costly, so avoid calling strln as much as we can. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 10 +- 1 file changed, 5 insert

[PATCH 5/6] kconfig: don't allocate n+1 elements in temporary array

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" The temporary array that stores the search results is not NULL-terminated, so there is no reason to allocate n+1 elements. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 2 +

[PATCH 6/6] kconfig: simplify symbol-search code

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" There is no need for a double indirection in the temporary array that stores the internediate search results. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Reviewed-by: Jean Delvare --- scripts/kconfig/symbol.c | 30 +++---

[PATCH 3/6] kconfig/[mn]conf: shorten title in search-box

2013-07-16 Thread Yann E. MORIN
From: "Yann E. MORIN" No need to repeat the 'CONFIG_' string in the title, once is explicit enough. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare --- scripts/kconfig/mconf.c | 4 ++-- scripts/kconfig/nconf.c | 4 ++-- 2 files changed

Re: [PATCH 3/3] kconfig: cleanup symbol-search code

2013-07-16 Thread Yann E. MORIN
Jean, All, On Tuesday 16 July 2013 16:33:47 Jean Delvare wrote: > Le Saturday 13 July 2013 à 20:08 +0200, Yann E. MORIN a écrit : > > From: "Yann E. MORIN" > > > > - no need for a double-indirection for the temporary sym_match_arr array > > - the temp

Re: [PATCH 13/15] kconfig/[mn]conf: make it explicit in the search box that a regexp is possible

2013-07-16 Thread Yann E. MORIN
Jean, All, On Tuesday 16 July 2013 16:31:20 Jean Delvare wrote: > Le Monday 08 July 2013 à 19:37 +0200, Yann E. MORIN a écrit : > > On 2013-07-08 13:25 +0200, Jean Delvare spake thusly: > > > Le Monday 24 June 2013 à 20:11 +0200, Yann E. MORIN a écrit : [--SNIP--] > >

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-16 Thread Yann E. MORIN
Borislav, All, On Tuesday 16 July 2013 11:38:20 Borislav Petkov wrote: > On Tue, Jul 16, 2013 at 11:32:39AM +0200, Yann E. MORIN wrote: > > That does not work in two cases: > > - older distros that do not have the tool packaged > > This point is moot - distro kernels ar

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-16 Thread Yann E. MORIN
Borislav, All, On Tuesday 16 July 2013 11:22:42 Borislav Petkov wrote: > On Tue, Jul 16, 2013 at 11:12:17AM +0200, Yann E. MORIN wrote: > > The goal is to avoid generating a non-buildable kernel in the first place. > > I know, that's why I'm proposing to fail the build

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-16 Thread Yann E. MORIN
gt; > fi > > > > then munge the output of that script into the Kconfig run and do > > > > depends on HAVE_LZ4C > > That does sound nice to avoid the build error, but will eventually > make it harder to diagnose why the kernel has not been compre

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-16 Thread Yann E. MORIN
calling into Kconfig, we can simply hide non-available config options, and even inform the user *why* the option is not available. Regards, Yann E. MORIN. -- .-----..--.. | Yann E. MORIN | Real-Time Embedded | /&q

Re: [PATCH -next 2/2] kbuild: fix for updated LZ4 tool with the new streaming format

2013-07-16 Thread Yann E. MORIN
is what I'm doing in crosstool-NG: a script checks for optional pre-requisites, spits out a Kconfig blob which is included by the top-level Kconfig file. Here is a snippet of generated Kconfig blob: config HAVE_XZ def_bool

Re: [PATCH 1/3] Documentation/kconfig: more concise and straightforward search explanation

2013-07-13 Thread Yann E. MORIN
All, On 2013-07-13 20:08 +0200, Yann E. MORIN spake thusly: > From: "Yann E. MORIN" > > Re-phrase the explanations on the sorting of search results, in a more > concise and complete way. > > Drop reference to the user's locale when sorting alphabe

[PATCH 1/3] Documentation/kconfig: more concise and straightforward search explanation

2013-07-13 Thread Yann E. MORIN
From: "Yann E. MORIN" Re-phrase the explanations on the sorting of search results, in a more concise and complete way. Drop reference to the user's locale when sorting alphabetically, since this is implicit. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORI

[PATCH 2/3] kconfig: avoid multiple calls to strlen

2013-07-13 Thread Yann E. MORIN
From: "Yann E. MORIN" Calls to strlen are costly, so avoid calling strln as much as we can. Reported-by: Jean Delvare Signed-off-by: "Yann E. MORIN" Cc: Jean Delvare --- scripts/kconfig/symbol.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --g

[PATCH 0/3] Misc cleanups in kconfig for symbol-search

2013-07-13 Thread Yann E. MORIN
request to Michal a bit later during the rc phase (in case more fixups accumulate). Regards, Yann E. MORIN. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majord

[PATCH 3/3] kconfig: cleanup symbol-search code

2013-07-13 Thread Yann E. MORIN
From: "Yann E. MORIN" - no need for a double-indirection for the temporary sym_match_arr array - the temporary sym_match_arr array is not NULL terminated, so no need to allocate n+1 elements - two minor style fixes - grammar fix in comment Reported-by: Jean Delvare Signed-off-b

Re: [PATCH 12/15] kconfig: sort found symbols by relevance

2013-07-13 Thread Yann E. MORIN
r warn you if a function changes data it was not supposed to. > > So what you want to do is: > > static int sym_rel_comp(const void *sym1, const void *sym2) > { > const struct sym_match *s1 = sym1; > const struct sym_match *s2 = sym2;

Re: [PATCH 12/15] kconfig: sort found symbols by relevance

2013-07-10 Thread Yann E. MORIN
Michal, All, On 2013-07-08 19:35 +0200, Yann E. MORIN spake thusly: > On 2013-07-08 13:19 +0200, Jean Delvare spake thusly: > > Le Monday 24 June 2013 à 20:11 +0200, Yann E. MORIN a écrit : > > > From: "Yann E. MORIN" > [--SNIP--] > > > Since the search c

Re: [PATCH 13/15] kconfig/[mn]conf: make it explicit in the search box that a regexp is possible

2013-07-08 Thread Yann E. MORIN
Jean, All, On 2013-07-08 13:25 +0200, Jean Delvare spake thusly: > Le Monday 24 June 2013 à 20:11 +0200, Yann E. MORIN a écrit : > > From: "Yann E. MORIN" > > > > Reported-by: Jean Delvare > > Signed-off-by: "Yann E. MORIN" > > Cc: Jea

Re: [PATCH 12/15] kconfig: sort found symbols by relevance

2013-07-08 Thread Yann E. MORIN
Jean, All, On 2013-07-08 13:19 +0200, Jean Delvare spake thusly: > Le Monday 24 June 2013 à 20:11 +0200, Yann E. MORIN a écrit : > > From: "Yann E. MORIN" [--SNIP--] > > Since the search can be a regexp, it is possible that more than one symbol > > matches exactly

kconfig-frontends-3.10.0.0 released

2013-07-04 Thread Yann E. MORIN
feedback, any patch is welcome! ;-) Regards, Yann E. MORIN. -- .-..--.--------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN |

Re: [RFC PATCH] Allow optional module parameters

2013-07-03 Thread Yann E. MORIN
module? I was going to suggest that, or a new kernel option: noloadmodules=module1[,module2...] The option may well be cumulative, too, so we could do: noloadmodules=module1,module2 noloadmodules=module3 and none of module{1,

[PATCH 1/1] kconfig: allow "hex" and "range" to support longs

2013-06-29 Thread Yann E. MORIN
From: Kees Cook The parsing routines for Kconfig files use strtol(), but store and render values as int. Switch types and formating to long to support a wider range of values. For example, 0x8000 wasn't representable. Signed-off-by: Kees Cook Tested-by: "Yann E. MORIN" Rev

[pull request] Pull request for branch yem-kconfig-for-next

2013-06-29 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello Michal, All, Please pull this kconfig fix from Kees that enables 64-bit-wide (ie. signed long) [int,hex] ranges. Regards, Yann E. MORIN. The following changes since commit 490f16171119a16e05d670306c105f3b45c38837: Revert "kconfig: fix randomising

Re: linux-next: Tree for Jun 25

2013-06-25 Thread Yann E. MORIN
Sedat, All, On 2013-06-25 11:26 +0200, Sedat Dilek spake thusly: > On Tue, Jun 25, 2013 at 11:21 AM, Yann E. MORIN > wrote: > > On Tuesday 25 June 2013 10:55:20 Sedat Dilek wrote: > >> Not sure what's going on, but I get this with today's Linux-Next. > >

Re: [PATCH 15/15] kconfig: fix randomising choice entries in presence of KCONFIG_ALLCONFIG

2013-06-25 Thread Yann E. MORIN
Michal, All, On 2013-06-24 20:11 +0200, Yann E. MORIN spake thusly: > Currently, randconfig does randomise choice entries, unless KCONFIG_ALLCONFIG > is specified. [--SNIP--] > This patch defers setting that a choice has a value until a symbol for > that choice is indeed set, so that

Re: linux-next: Tree for Jun 25 [ kbuild or rapidio? ]

2013-06-25 Thread Yann E. MORIN
onfig] Error 1 > >> make: *** [silentoldconfig] Error 2 [--SNIP--] > So, it's a kbuild issue (info to the rapidio maintainers). To be true, it's a kconfig (not kbuild) issue I introduced when trying to fix choice randomisation. Regards, Yann E. MOR

Re: linux-next: Tree for Jun 25

2013-06-25 Thread Yann E. MORIN
Michal, All, [Sorry for duplicates, I inadvertently sent HTML] On Tuesday 25 June 2013 11:21:53 Yann E. MORIN wrote: > On Tuesday 25 June 2013 10:55:20 Sedat Dilek wrote: > > Not sure what's going on, but I get this with today's Linux-Next. > > ( Yesterd

Re: linux-next: Tree for Jun 25

2013-06-25 Thread Yann E. MORIN
r as a side effect of another bug (which 8357b48 tries to fix), and now this new one has been exposed. I have to think in a calm place tonight, when I'm back from work... This one is starting to drive me crazy, really... :-( Regards, Yann E. MORIN. -- .-..

Re: [PATCH 15/15] kconfig: fix randomising choice entries in presence of KCONFIG_ALLCONFIG

2013-06-25 Thread Yann E. MORIN
Michal, All, On Monday 24 June 2013 20:11:59 Yann E. MORIN wrote: > Currently, randconfig does randomise choice entries, unless KCONFIG_ALLCONFIG > is specified. [--SNIP--] > This patch defers setting that a choice has a value until a symbol for > that choice is indeed set, so that

[PATCH 03/15] kconfig/lxdialog: Use new mininimum resize definitions in conf_choice()

2013-06-24 Thread Yann E. MORIN
From: Sedat Dilek This is a cleanup which uses the proper (new) definitions and does not change current behaviour. Signed-off-by: Sedat Dilek Reviewed-by: "Yann E. MORIN" Tested-by: "Yann E. MORIN" Signed-off-by: "Yann E. MORIN" --- Yann had some more ideas on

[PATCH 01/15] kconfig: Fix defconfig when one choice menu selects options that another choice menu depends on

2013-06-24 Thread Yann E. MORIN
-8<--- when the expected result would be: .config: ---8<--- CONFIG_MODULES=y CONFIG_CONFIGFS_FS=y CONFIG_USB_LIBCOMPOSITE=y CONFIG_USB_ZERO=y CONFIG_CRC32=y CONFIG_CRC32_SLICEBY8=y ---8<--- Signed-off-by: Arve Hjønnevåg [yann.morin.1...@free.fr: add the resulting .config to commit log,

[pull request v2] Pull request for branch yem-kconfig-for-next

2013-06-24 Thread Yann E. MORIN
From: "Yann E. MORIN" Hello Michal, All! │ Michal, please pull these patches against kconfig t

[PATCH 02/15] kconfig/lxdialog: Add definitions for mininimum (re)size values

2013-06-24 Thread Yann E. MORIN
YanQing Tested-by: "Yann E. MORIN" Reviewed-by: "Yann E. MORIN" Signed-off-by: "Yann E. MORIN" --- scripts/kconfig/lxdialog/checklist.c | 4 ++-- scripts/kconfig/lxdialog/dialog.h| 14 ++ scripts/kconfig/lxdialog/inputbox.c | 4 ++-- scripts

[PATCH 05/15] mconf: use function calls instead of ncurses' variables LINES and COLS

2013-06-24 Thread Yann E. MORIN
calls to get window dimensions. init_dialog() could make use of the variables, but for the sake of consistency we do not change it's current use of the macro getmaxyx(). [1] ncurses(3X) Signed-off-by: Dirk Gouders Tested-by: "Yann E. MORIN" Reviewed-by: "Yann E. MORIN&quo

[PATCH 04/15] kconfig/lxdialog: handle newline characters in print_autowrap()

2013-06-24 Thread Yann E. MORIN
ed, so let print_autowrap() handle newlines propperly. Also, reword that dialog's second phrase with a real sentence. Signed-off-by: Dirk Gouders Tested-by: "Yann E. MORIN" Reviewed-by: "Yann E. MORIN" [yann.morin.1...@free.fr: very slightly tweak the commit me

[PATCH 06/15] nconf: use function calls instead of ncurses' variables LINES and COLS

2013-06-24 Thread Yann E. MORIN
calls or macros to get window dimensions. The use of the variables in main() was OK, but for the sake of consistency it was modified to use the macro getmaxyx(). [1] ncurses(3X) Signed-off-by: Dirk Gouders Reviewed-by: "Yann E. MORIN" [yann.morin.1...@free.fr: declare 'lines'

[PATCH 08/15] scripts/config: replace hard-coded script name by a dynamic value

2013-06-24 Thread Yann E. MORIN
f the script at runtime. Signed-off-by: Clement Chauplannaz Acked-by: Andi Kleen Signed-off-by: Yann E. MORIN --- scripts/config | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/scripts/config b/scripts/config index bb4d3de..6b3272e 100755 --- a/scripts/confi

[PATCH 09/15] kconfig/conf: fix randconfig setting multiple symbols in a choice

2013-06-24 Thread Yann E. MORIN
From: "Yann E. MORIN" Currently, randconfig may set more than one symbol in a given choice. Given this config file: config A bool "A" if A choice bool "B/C/D" config B bool "B" config C bool "C

[PATCH 07/15] mconf/nconf: mark empty menus/menuconfigs different from non-empty ones

2013-06-24 Thread Yann E. MORIN
sed to non-empty ones that are marked by "--->". This scheme was suggested by "Yann E. MORIN" . Signed-off-by: Dirk Gouders Tested-by: "Yann E. MORIN" Reviewed-by: "Yann E. MORIN" Signed-off-by: "Yann E. MORIN" --- scrip

[PATCH 11/15] kconfig/conf: print the seed used to initialise the RNG for randconfig

2013-06-24 Thread Yann E. MORIN
From: "Yann E. MORIN" ... so the user has a chance to reproduce a test-case. Signed-off-by: "Yann E. MORIN" --- scripts/kconfig/conf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c index 94521c7..38616c1 100644 --- a/sc

[PATCH 12/15] kconfig: sort found symbols by relevance

2013-06-24 Thread Yann E. MORIN
From: "Yann E. MORIN" When searching for symbols, return the symbols sorted by relevance. Sorting is done as thus: - first, symbols that match exactly - then, alphabetical sort Since the search can be a regexp, it is possible that more than one symbol matches exactly. In thi

  1   2   3   >