From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Sat, 8 Mar 2014 12:32:13 +0100 Subject: [U-Boot] net: asix: don't pad odd-length TX packets In-Reply-To: <20140307222737.GH16805@bill-the-cat> References: <1393532822-28217-1-git-send-email-swarren@wwwdotorg.org> <20140307222737.GH16805@bill-the-cat> Message-ID: <201403081232.13919.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 Friday, March 07, 2014 at 11:27:37 PM, Tom Rini wrote: > On Thu, Feb 27, 2014 at 01:27:02PM -0700, Stephen Warren wrote: > > From: Stephen Warren > > > > For Ethernet/USB RX packets, the ASIX HW pads odd-sized packets so that > > they have an even size. Currently, asix_recv() does remove this padding, > > and asic_send() adds equivalent padding in the TX path. However, the HW > > does not appear to need this packing for TX packets in practical testing > > with "ASIX Elec. Corp. AX88x72A 000001" Vendor: 0x0b95 Product 0x7720 > > Version 0.1. The Linux kernel does no such padding for the TX path. > > > > Remove the padding from the TX path: > > > > * For consistency with the Linux kernel. > > * NVIDIA has a Tegra simulator which validates that the length of USB > > > > packets sent to an ASIX device matches the packet length value inside > > the packet data. Having U-Boot and the kernel do the same thing when > > creating the TX packets simplifies the simulator's validation. > > > > Cc: Lucas Stach > > Cc: Marek Vasut > > Cc: Simon Glass > > Signed-off-by: Stephen Warren > > Acked-by: Simon Glass > > Acked-by: Marek Vasut > > Tested-by: Marek Vasut > > Tested-by: Gerhard Sittig > > Applied to u-boot/master, thanks! Uh, where did Joe disappear to anyway ? Best regards, Marek Vasut