From: Felipe Balbi <felipe.balbi@linux.intel.com>
To: Randy Dunlap <rdunlap@infradead.org>, yann.morin.1998@free.fr
Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [PATCH] scripts: kconfig: implement a sort method
Date: Wed, 12 Apr 2017 10:49:32 +0300 [thread overview]
Message-ID: <87d1ciawvn.fsf@linux.intel.com> (raw)
In-Reply-To: <7d017fc8-9e0b-f8a8-a877-edcc07c91949@infradead.org>
[-- Attachment #1: Type: text/plain, Size: 12654 bytes --]
Hi,
Randy Dunlap <rdunlap@infradead.org> writes:
> On 04/11/17 04:12, Felipe Balbi wrote:
>> With a growing amount of Kernel configuration, it's
>> getting ever more difficult to find anything on
>> menuconfig. Because of that, implement mergesort for
>> kconfig to make it a little easier for anybody
>> building kernels.
>
> Search works for me, but I don't mind Sort either.
yes, it works. It does a different thing, though. Here's what happens
when I want to find DWC3:
/ DWC3 RET
| Symbol: USB_DWC3 [=m] │
│ Type : tristate │
│ Prompt: DesignWare USB3 DRD Core Support │
│ Location: │
│ -> Device Drivers │
│ (1) -> USB support (USB_SUPPORT [=y]) │
│ Defined at drivers/usb/dwc3/Kconfig:1 │
│ Depends on: USB_SUPPORT [=y] && (USB [=y] || USB_GADGET [=y]) && HAS_DMA │
│ Selects: USB_XHCI_PLATFORM [=m] │
│ │
│ │
│ Symbol: USB_DWC3_DUAL_ROLE [=n] │
│ Type : boolean │
│ Prompt: Dual Role mode │
│ Location: │
│ -> Device Drivers │
│ -> USB support (USB_SUPPORT [=y]) │
│ -> DesignWare USB3 DRD Core Support (USB_DWC3 [=m]) │
│ (2) -> DWC3 Mode Selection (<choice> [=y]) │
│ Defined at drivers/usb/dwc3/Kconfig:41 │
│ Depends on: <choice> && (USB [=y]=y || USB [=y]=USB_DWC3 [=m]) && (USB_G │
│ │
│ │
│ Symbol: USB_DWC3_EXYNOS [=n] │
│ Type : tristate │
│ Prompt: Samsung Exynos Platform │
│ Location: │
│ -> Device Drivers │
│ -> USB support (USB_SUPPORT [=y]) │
│ (3) -> DesignWare USB3 DRD Core Support (USB_DWC3 [=m]) │
│ Defined at drivers/usb/dwc3/Kconfig:63 │
│ Depends on: USB_SUPPORT [=y] && USB_DWC3 [=m] && (ARCH_EXYNOS || COMPILE │
│ │
│ │
│ Symbol: USB_DWC3_GADGET [=y] │
│ Type : boolean │
│ Prompt: Gadget only mode │
│ Location: │
│ -> Device Drivers │
│ -> USB support (USB_SUPPORT [=y]) │
│ -> DesignWare USB3 DRD Core Support (USB_DWC3 [=m]) │
│ (4) -> DWC3 Mode Selection (<choice> [=y]) │
│ Defined at drivers/usb/dwc3/Kconfig:34 │
│ Depends on: <choice> && (USB_GADGET [=y]=y || USB_GAD
Now I know where DWC3 is defined, so I navigate to Device Driver, USB
Support and search for DesignWare USB3 DRD Core Support:
│ │ <M> USB Test and Measurement Class support │ │
│ │ *** NOTE: USB_STORAGE depends on SCSI but BLK_DEV_SD may ***│ │
│ │ *** also be needed; see USB_STORAGE Help for more info *** │ │
│ │ <M> USB Mass Storage support │ │
│ │ [ ] USB Mass Storage verbose debug │ │
│ │ <M> Realtek Card Reader support │ │
│ │ [*] Realtek Card Reader autosuspend support │ │
│ │ <M> Datafab Compact Flash Reader support │ │
│ │ <M> Freecom USB/ATAPI Bridge support │ │
│ │ <M> ISD-200 USB/ATA Bridge support │ │
│ │ <M> USBAT/USBAT02-based storage support │ │
│ │ <M> SanDisk SDDR-09 (and other SmartMedia, including DPCM) sup│ │
│ │ <M> SanDisk SDDR-55 SmartMedia support │ │
│ │ <M> Lexar Jumpshot Compact Flash Reader │ │
│ │ <M> Olympus MAUSB-10/Fuji DPC-R1 support │ │
│ │ <M> Support OneTouch Button on Maxtor Hard Drives │ │
│ │ <M> Support for Rio Karma music player │ │
│ │ <M> SAT emulation on Cypress USB/ATA Bridge with ATACB │ │
│ │ <M> USB ENE card reader support │ │
│ │ <M> USB Attached SCSI │ │
│ │ *** USB Imaging devices *** │ │
│ │ < > USB Mustek MDC800 Digital Camera support │ │
│ │ < > Microtek X6USB scanner support │ │
│ │ < > USB/IP support │ │
│ │ < > Inventra Highspeed Dual Role Controller (TI, ADI, AW, ...) │ │
│ │ < > DesignWare USB3 DRD Core Support │ │
│ │ < > DesignWare USB2 DRD Core Support │ │
│ │ < > ChipIdea Highspeed Dual Role Controller │ │
│ │ < > NXP ISP 1760/1761 support │ │
│ │ *** USB port drivers *** │ │
│ │ <M> USB Serial Converter support ---> │ │
│ │ *** USB Miscellaneous drivers *** │ │
│ │ < > EMI 6|2m USB Audio interface support │ │
│ │ < > EMI 2|6 USB Audio interface support │ │
│ │ < > ADU devices from Ontrak Control Systems │ │
│ │ < > USB 7-Segment LED Display │ │
│ │ < > USB Diamond Rio500 support │ │
│ │ < > USB Lego Infrared Tower support │ │
│ │ < > USB LCD driver support │ │
Compare that to the sorted view of this same entry:
│ │ --- USB support │ │
│ │ < > ADU devices from Ontrak Control Systems │ │
│ │ < > Apple Cinema Display support │ │
│ │ < > ChaosKey random number generator driver support │ │
│ │ < > ChipIdea Highspeed Dual Role Controller │ │
│ │ < > Cypress CY7C63xxx USB driver support │ │
│ │ < > Cypress USB thermometer driver support │ │
│ │ < > DesignWare USB2 DRD Core Support │ │
│ │ < > DesignWare USB3 DRD Core Support │ │
│ │ < > EMI 2|6 USB Audio interface support │ │
│ │ < > EMI 6|2m USB Audio interface support │ │
│ │ < > Elan PCMCIA CardBus Adapter USB Client │ │
│ │ {M} Functions for loading firmware on EZUSB chips │ │
│ │ < > IO Warrior driver support │ │
│ │ < > Inventra Highspeed Dual Role Controller (TI, ADI, AW, ...) │ │
│ │ < > NXP ISP 1760/1761 support │ │
│ │ < > PlayStation 2 Trance Vibrator driver support │ │
│ │ < > Siemens ID USB Mouse Fingerprint sensor support │ │
│ │ <*> Support for Host-side USB │ │
│ │ < > BCMA usb host driver │ │
│ │ < > Cypress C67x00 HCD support │ │
│ │ [*] Dynamic USB minor allocation │ │
│ │ < > EHCI HCD (USB 2.0) support │ │
│ │ [*] Enable USB persist by default │ │
│ │ < > FOTG210 HCD support │ │
│ │ [ ] HCD test mode support │ │
│ │ < > ISP116X HCD support │ │
│ │ < > ISP1362 HCD support │ │
│ │ < > Microtek X6USB scanner support │ │
│ │ *** Miscellaneous USB options *** │ │
│ │ *** NOTE: USB_STORAGE depends on SCSI but BLK_DEV_SD may ***│ │
│ │ < > OHCI HCD (USB 1.1) support │ │
│ │ [ ] OTG support │ │
│ │ < > OXU210HP HCD support │ │
│ │ < > R8A66597 HCD support │ │
│ │ [ ] Rely on OTG and EH Targeted Peripherals List │ │
│ │ < > SL811HS HCD support │ │
│ │ < > SSB usb host driver │ │
│ │ < > Support WUSB Cable Based Association (CBA) │ │
much easier to find, don't you think?
> Why only menuconfig? Is it the one that you use mostly?
shouldn't be difficult to port to nconfig.
> Any Help text for Sort? or is it obvious what it does? (no)
wait, it's not obvious what "Sort" means?
> In an 80x25 terminal (window), the < Sort > option wraps around
> past column 80.
not what I see here [1]
> 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
> │ <Select> < Exit > < Help > < Save > < Load > < Sort
> >
> just have
> │ <Select> <Exit> <Help> <Save> <Load> <Sort>
>
> Or the <Select> does not need to be indented as much as it is.
I changed that for this very reason
> And BTW, Yann made kconfig an orphan today. :(
oh well
[1] https://imgur.com/a/qyvG6
--
balbi
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
next prev parent reply other threads:[~2017-04-12 7:50 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-11 11:12 [PATCH] scripts: kconfig: implement a sort method Felipe Balbi
2017-04-11 18:53 ` Randy Dunlap
2017-04-12 7:49 ` Felipe Balbi [this message]
2017-04-12 16:06 ` Yann E. MORIN
2017-04-12 18:19 ` Randy Dunlap
-- strict thread matches above, loose matches on Subject: below --
2016-04-21 20:07 Felipe Balbi
2016-04-21 23:12 ` Randy Dunlap
2016-04-22 7:45 ` Felipe Balbi
2016-04-22 17:03 ` Randy Dunlap
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=87d1ciawvn.fsf@linux.intel.com \
--to=felipe.balbi@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--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;
as well as URLs for NNTP newsgroup(s).