From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus =?iso-8859-1?Q?Klotzb=FCcher?= Date: Fri, 29 Aug 2008 08:07:01 +0200 Subject: [U-Boot] [U-Boot-Users] [PATCH] fix USB devices with multiple configurations In-Reply-To: <20080828235516.GB30743@game.jcrosoft.org> References: <20080709221101.3E0B324304@gemini.denx.de> <20080828235516.GB30743@game.jcrosoft.org> Message-ID: <20080829060700.GA8422@lisa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, Aug 29, 2008 at 01:55:16AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 00:11 Thu 10 Jul , Wolfgang Denk wrote: > > In message <20080706165805.GG20299@prithivi.gnumonks.org> you wrote: > > > > > > This patch fixes bugs in usbdcore*.c related to the use of devices > > > with multiple configurations. > > > > > > The original code made mistakes about the meaning of configuration value and > > > configuration index, and the resulting off-by-one errors resulted in: > > > > > > * SET_CONFIGURATION always selected the first configuration, no matter what > > > wValue is being passed. > > > * GET_DESCRIPTOR/CONFIGURATION always returned the descriptor for the first > > > configuration (index 0). > > > > > > Signed-off-by: Harald Welte > > > > Markus, are you going to add this to the USB repo any time soon? > > > > Or you could just ACK it (if you think so) and tell me to pick up > > directly. > Any news about this patch? Has been merged a long time ago as cc83b27217f7380041fea386ddb6d6d9b261617d Or what type of news are you looking for? Best regards Markus -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de")