[EMAIL PROTECTED] wrote:
> Hello, 
> 
> The last patch I'have sent to you changes the following

Hmm - I never got it, sorry.

>  - New device entry for unusual_dev.h with rev id 0x0103 instead of
>    0x0000-0x9999 and with the new flag

Awesome.

>  - New flag MAX_SECTORS_128 in include/linux/usb_usual.h
>  - Sets max_sectors per request of queue to 128 sectors if
>    flag is set in drivers/usb/storage/scsiglue.c

Cool - unfortunately, this part of the code isn't up to me alone to
accept or deny.

Matt/Alan - can you weigh in here? There had been some talk about making
a more generic flag of MAX_SECTORS_32 and pulling out the GENESYS stuff,
and putting this and the GENESYS stuff under that.

Is there still desire for that? If so, I can whip up a patch, or perhaps
Benjamin would prefer to do it.

Personally, I think a single, more generic override is better that a
generic one and a non-generic one - but I'm fine if the conesus is to
proceed with Benjamin's patch.

> -     US_FLAG(SINGLE_LUN,     0x00000001)                     \
> +     US_FLAG(SINGLE_LUN,             0x00000001)             \
>               /* allow access to only LUN 0 */                \
> -     US_FLAG(NEED_OVERRIDE,  0x00000002)                     \
> +     US_FLAG(NEED_OVERRIDE,          0x00000002)             \
...

Benjamin, why are you changing all the spacing here?


-- 
Phil Dibowitz                             [EMAIL PROTECTED]
Freeware and Technical Pages              Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"Be who you are and say what you feel, because those who mind don't
matter and those who matter don't mind."
 - Dr. Seuss


Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to