From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 7 Jan 2016 23:50:08 +0100 Subject: [U-Boot] [PATCH v1 05/16] usb: ulpi: Add Kconfig options for ULPI In-Reply-To: <568EE2CB.4080404@gmail.com> References: <1452104488-5502-1-git-send-email-mateusz.kulikowski@gmail.com> <201601062149.42745.marex@denx.de> <568EE2CB.4080404@gmail.com> Message-ID: <201601072350.08539.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thursday, January 07, 2016 at 11:12:27 PM, Mateusz Kulikowski wrote: > On 06.01.2016 21:49, Marek Vasut wrote: > > On Wednesday, January 06, 2016 at 07:21:17 PM, Mateusz Kulikowski wrote: > >> The following options can be now enabled via defconfig: > >> - CONFIG_USB_ULPI > >> - CONFIG_USB_ULPI_VIEWPORT > >> - CONFIG_USB_ULPI_VIEWPORT_OMAP > > [...] > > >> +++ b/drivers/usb/ulpi/Kconfig > >> @@ -0,0 +1,10 @@ > >> +comment "ULPI drivers" > >> + > >> +config USB_ULPI > >> + bool > >> + > >> +config USB_ULPI_VIEWPORT > >> + bool > >> + > >> +config USB_ULPI_VIEWPORT_OMAP > >> + bool > > > > Description of these options is missing. > > My intention here was to keep it away from "users". > > This options are not "optional" in any way. > They are rather like CPU_something or VBAR_something, > that is - they are required and should be selected by drivers that use > them, but without this drivers, they are useless. OK, so they're just options which are selected by drivers. Got it. All is good from my side then. > But maybe I'm missing something :) > If anyone insists on that - I'll make them visible and add description. > > > > Regards, > Mateusz Best regards, Marek Vasut