From mboxrd@z Thu Jan  1 00:00:00 1970
From: Troy Kisky <troy.kisky@boundarydevices.com>
Date: Fri, 20 Sep 2013 11:34:11 -0700
Subject: [U-Boot] [PATCH V4 03/17] usb: gadget: ether set wMaxPacketSize
In-Reply-To: <201309201253.00098.marex@denx.de>
References: <1379647780-2623-1-git-send-email-troy.kisky@boundarydevices.com>
	<1379647780-2623-4-git-send-email-troy.kisky@boundarydevices.com>
	<201309201253.00098.marex@denx.de>
Message-ID: <523C9523.20504@boundarydevices.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 9/20/2013 3:52 AM, Marek Vasut wrote:
> Dear Troy Kisky,
>
>> set wMaxPacketSize for full speed descriptors
>> fs_source_desc, fs_sink_desc to 64.
>>
>> Full-speed bulk endpoint can have a maximum packet size of
>> 8, 16, 32, or 64 bytes, so choice 64.
>>
>> The hs_source_desc, hs_sink_desc, already have their wMaxPacketSize
>> set to 512. That is the only legal value for high speed bulk endpoints.
>>
>> Signed-off-by: Troy Kisky <troy.kisky@boundarydevices.com>
> Why do we need this patch? What issue does this fix ?
>
> Best regards,
> Marek Vasut
>
I could try full speed mode without this and see how linux behaves when 
given bad data,
but that would not says anything about other O.S.es. It seems safer just 
not to give out
bad data.

Troy