On Sun, Jan 14, 2018 at 10:00 PM, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote: > Hi Ulf. > > On Sun, Jan 14, 2018 at 12:33:43PM +0100, Ulf Magnusson wrote: >> These together should make the automatic submenu logic a lot clearer. >> >> Ulf Magnusson (2): >> kconfig: Document 'if' flattening logic >> kconfig: Improve auto. menu documentation accuracy >> >> scripts/kconfig/menu.c | 33 +++++++++++++++++++++++++++++---- >> 1 file changed, 29 insertions(+), 4 deletions(-) > > Very good with extra comments in menu_finalize. > If you can mamage to explain the logic related to select > then even better. I added a short comment in https://patchwork.kernel.org/patch/9986947/, which is already in linux-next: /* * Handle selects and implies, which modify the * dependencies of the selected/implied symbol */ if (prop->type == P_SELECT) { ... There's also https://patchwork.kernel.org/patch/9983881/: /* Reverse dependencies through being selected by other symbols */ struct expr_value rev_dep; I guess it could be made more explicit. I don't want to heap on too many patches at once though. > > While on the subject on improving readability, then > menu_finalize would benefit from a split in a number of > well named and well documented helper functions. > > It would be great if you could try this out. I had a stab at it earlier, but never got done. I think the high-level structure in Kconfiglib should work: https://github.com/ulfalizer/Kconfiglib/blob/master/kconfiglib.py#L3990 Hopefully I'll get back to it later once the current batch of patches has been merged. I'm just happy to have to have less mystery code in there for now. Definitely one of the more opaque parts of Kconfig. Cheers, Ulf -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
