Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-07-02 Thread Ben Warren
Tsi-Chung Liew wrote: From: TsiChung Liew [EMAIL PROTECTED] z The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu() in dm9000_rx_status_8/16/32bit(). Signed-off-by: TsiChung Liew [EMAIL PROTECTED] Applied to 'next'

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-07-01 Thread Liew Tsi Chung
Ben, May I know the status on this patch? Regards, TsiChung - Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-07-01 Thread Ben Warren
Liew Tsi Chung wrote: Ben, May I know the status on this patch? Regards, TsiChung Sorry, I'll address it tonight. regards, Ben - Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown

[U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Tsi-Chung Liew
From: TsiChung Liew [EMAIL PROTECTED] The received status and len was in little endian format and caused the ethernet unable to proceed further. Add define CFG_DM9000_BYTESWAP_STATUS_LEN in dm9000_rx_status_16bit() to byte swap RxStatus and RxLen Signed-off-by: TsiChung Liew [EMAIL PROTECTED]

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Ben Warren
Hi Tsi-Chung, Tsi-Chung Liew wrote: From: TsiChung Liew [EMAIL PROTECTED] The received status and len was in little endian format and caused the ethernet unable to proceed further. Add define CFG_DM9000_BYTESWAP_STATUS_LEN in dm9000_rx_status_16bit() to byte swap RxStatus and RxLen

[U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Tsi-Chung Liew
From: TsiChung Liew [EMAIL PROTECTED] The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu() in dm9000_rx_status_16bit(). Signed-off-by: TsiChung Liew [EMAIL PROTECTED] --- drivers/net/dm9000x.c |4 ++-- 1 files

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Remy Bohmer
Hello Tsi-Chung, The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu() in dm9000_rx_status_16bit(). I can imagine that similar problems are also valid for the 32bit code (and maybe the 8 bit code too) It is probably

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Liew Tsi Chung
To: Liew Tsi Chung Cc: U-Boot-Users; Rigby John; Ben Warren Subject: Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue Hello Tsi-Chung, The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu

[U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Tsi-Chung Liew
From: TsiChung Liew [EMAIL PROTECTED] The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu() in dm9000_rx_status_8/16/32bit(). Signed-off-by: TsiChung Liew [EMAIL PROTECTED] --- drivers/net/dm9000x.c | 16 ++--

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Ben Warren
Tsi-Chung Liew wrote: From: TsiChung Liew [EMAIL PROTECTED] The received status and len was in little endian format and caused the ethernet unable to proceed further. Add __le16_to_cpu() in dm9000_rx_status_8/16/32bit(). Signed-off-by: TsiChung Liew [EMAIL PROTECTED] ---

Re: [U-Boot-Users] [PATCH] Fix dm9000 receive status and len little endian issue

2008-06-25 Thread Liew Tsi Chung
Ben, The data (packet data) and the status/len are different in LE/BE, cannot use just the macro dm9000_x to cover both. In my case, packet data is already in BE, but the status/len is in LE; therefore, status/len require byteswap. This has to do with cpu endian mode. Regards, TsiChung