From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 13 Aug 2015 00:10:26 +0200 Subject: [U-Boot] [PATCH v6 5/5] usb: lpc32xx: add host USB driver In-Reply-To: <55CBBC1D.1080404@mleia.com> References: <1439208994-19072-4-git-send-email-slemieux.tyco@gmail.com> <4F172219764C784B84C2C1FF44E7DFB10301123F@003FCH1MPN2-041.003f.mgd2.msft.net> <55CBBC1D.1080404@mleia.com> Message-ID: <201508130010.27048.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 Wednesday, August 12, 2015 at 11:35:25 PM, Vladimir Zapolskiy wrote: > On 12.08.2015 22:47, LEMIEUX, SYLVAIN wrote: > > Hi Vladimir, > > > >> -----Original Message----- > >> From: Marek Vasut [mailto:marex at denx.de] > >> Sent: 12-Aug-15 3:42 PM > >> > >> On Wednesday, August 12, 2015 at 07:55:29 PM, Vladimir Zapolskiy wrote: > >>> Hi Sylvain, > >>> > >>> On 10.08.2015 15:16, slemieux.tyco at gmail.com wrote: > >>>> From: Sylvain Lemieux > >>>> > >>>> Incorporate USB driver from legacy LPCLinux NXP BSP. > >>>> The files taken from the legacy patch are: > >>>> - lpc32xx USB driver > >>>> - lpc3250 header file USB registers definition. > >>>> > >>>> The legacy driver was updated and clean-up as part of the integration > >>>> with the latest u-boot. > >>>> > >>>> Signed-off-by: Sylvain Lemieux > >> > >> [...] > >> > >>>> +static int wait_for_bit(void *reg, const u32 mask, bool set) > >>>> +{ > >>> > >>> (set == false) argument is not in use, and hence there is a piece of > >>> dead code in the function. > >> > >> I'd prefer this to be the way it is, since this function can be > >> extracted and made into generic code (probably in subsequent patch). > > > > Vladimir, are you OK with keeping the code as-is? > > It will make it easier when we attempt to create a generic > > "wait_for_bit()" function. > > Okay, if maintainer says, let it remain :) Heh, thanks :-) Best regards, Marek Vasut