[U-Boot] [PATCH 2/3] usb: Allow up to 7 storage devices

Simon Glass sjg at chromium.org
Fri Jan 8 04:34:27 CET 2016


On 3 January 2016 at 14:23, Marek Vasut <marex at denx.de> wrote:
> On Sunday, January 03, 2016 at 09:50:30 PM, Simon Glass wrote:
>> The current limit of 5 is not enough for the driver model USB tests. Really
>> we should not have a limit but the driver model code still uses the
>> usb_dev_desc[] array, which has a limit.
>>
>> Increasing the limit by 2 should not bother anyone. Adjust it.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>
> Reviewed-by: Marek Vasut <marex at denx.de>
>
> btw moving this constant to one place would be nice ;)

OK, how about this:

http://patchwork.ozlabs.org/patch/564474/

>
>> ---
>>
>>  common/usb_storage.c | 2 +-
>>  include/usb.h        | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/common/usb_storage.c b/common/usb_storage.c
>> index 4fa6538..e61a8c8 100644
>> --- a/common/usb_storage.c
>> +++ b/common/usb_storage.c
>> @@ -65,7 +65,7 @@ static const unsigned char us_direction[256/8] = {
>>  static ccb usb_ccb __attribute__((aligned(ARCH_DMA_MINALIGN)));
>>  static __u32 CBWTag;
>>
>> -#define USB_MAX_STOR_DEV 5
>> +#define USB_MAX_STOR_DEV 7
>>  static int usb_max_devs; /* number of highest available usb device */
>>
>>  static block_dev_desc_t usb_dev_desc[USB_MAX_STOR_DEV];
>> diff --git a/include/usb.h b/include/usb.h
>> index 55b9268..7d6ad70 100644
>> --- a/include/usb.h
>> +++ b/include/usb.h
>> @@ -227,7 +227,7 @@ int board_usb_cleanup(int index, enum usb_init_type
>> init);
>>
>>  #ifdef CONFIG_USB_STORAGE
>>
>> -#define USB_MAX_STOR_DEV 5
>> +#define USB_MAX_STOR_DEV 7
>>  block_dev_desc_t *usb_stor_get_dev(int index);
>>  int usb_stor_scan(int mode);
>>  int usb_stor_info(void);
>
> Best regards,
> Marek Vasut


More information about the U-Boot mailing list