Re: [U-Boot] [PATCH V2 1/2] USB-CDC: correct wrong alignment in ether.c

2010-08-15 Thread Stefano Babic
Sergei Shtylyov wrote: > Hello. > > Stefano Babic wrote: > >> The buffer for the status request must be word aligned >> because it is accessed with 32 bit pointer in the >> eth_status_complete function. > >> Signed-off-by: Stefano Babic > [...] >> diff --git a/drivers/usb/gadget/ether.c b/drive

Re: [U-Boot] [PATCH V2 2/2] USB-CDC: called handle_interrupts inside usb_eth_send

2010-08-15 Thread Stefano Babic
Remy Bohmer wrote: > Hi, > > 2010/8/14 Stefano Babic : >> The patch removes an endless loop in the usb_eth_send >> if the tx_complete is not called before going >> in the loop. The driver interrupt routine is called >> allowing the driver to check if the TX is completed. >> >> Signed-off-by: Stef

[U-Boot] [PATCH V3 2/2] USB-CDC: called handle_interrupts inside usb_eth_send

2010-08-15 Thread Stefano Babic
The patch removes an endless loop in the usb_eth_send if the tx_complete is not called before going in the loop. The driver interrupt routine is called allowing the driver to check if the TX is completed. Signed-off-by: Stefano Babic --- drivers/usb/gadget/ether.c | 10 +- 1 files cha

[U-Boot] [PATCH V3 1/2] USB-CDC: correct wrong alignment in ether.c

2010-08-15 Thread Stefano Babic
The buffer for the status request must be word aligned because it is accessed with 32 bit pointer in the eth_status_complete function. Signed-off-by: Stefano Babic --- drivers/usb/gadget/ether.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/usb/gadget/ether.c

Re: [U-Boot] [PATCH V3 2/2] USB-CDC: called handle_interrupts inside usb_eth_send

2010-08-15 Thread Remy Bohmer
Hi, 2010/8/15 Stefano Babic : > The patch removes an endless loop  in the usb_eth_send > if the tx_complete is not called before going > in the loop. The driver interrupt routine is called > allowing the driver to check if the TX is completed. > > Signed-off-by: Stefano Babic > --- >  drivers/usb

Re: [U-Boot] [PATCH V3 1/2] USB-CDC: correct wrong alignment in ether.c

2010-08-15 Thread Remy Bohmer
Hi, 2010/8/15 Stefano Babic : > The buffer for the status request must be word aligned > because it is accessed with 32 bit pointer in the > eth_status_complete function. > > Signed-off-by: Stefano Babic > --- >  drivers/usb/gadget/ether.c |    2 +- >  1 files changed, 1 insertions(+), 1 deletion

[U-Boot] [PATCH] IDE: Don't assume there are always two devices per bus

2010-08-15 Thread Rogan Dawes
Some SATA controllers can operate in an IDE compatible mode (e.g. mvsata) but will only ever have a single device per bus. This allows the upcoming DNS323 port to properly identify and use a drive on both SATA interfaces. --- include/ide.h |3 ++- 1 files changed, 2 insertions(+), 1 deletions

Re: [U-Boot] [PATCH] IDE: Don't assume there are always two devices per bus

2010-08-15 Thread Wolfgang Denk
Dear Rogan Dawes, In message <1281904542-11694-1-git-send-email-ro...@dawes.za.net> you wrote: > > -#define IDE_BUS(dev)(dev >> 1) > +#define IDE_BUS(dev)(dev >> (CONFIG_SYS_IDE_MAXDEVICE / \ > + CONFIG_SYS_IDE_MAXBUS - 1)) Please add parens to make c

Re: [U-Boot] [PATCH v2a] AT91: add SD/MMC support

2010-08-15 Thread Xu, Hong
Hi Alex, Thanks to Reinhard, his previous work does well on SAM9260 SAM9G20 SAM9XE SAM9RL (I tested all of them). Since he said he'll provide a more fancy one, I'm just on hold :-) For your issue, could you try (If you're using the original EK board from Atmel) Remove R42 - or - In at91_mci_

Re: [U-Boot] UNS: [Nios2-dev] [PATCH] nios2: fix out of reach case for do_reset

2010-08-15 Thread Thomas Chou
On 08/14/2010 07:51 AM, Brad Parker wrote: > As a comment, that patch looks pretty ugly. I might suggest an inline > function (called "callr" or "nios2_callr") which > encapsulates the __asm__. That would make the code much easier to read. > > You could put the inline function in one of the asm

[U-Boot] [PATCH v2] nios2: fix out of reach case for do_reset

2010-08-15 Thread Thomas Chou
There is a limitation (or bug?) of nios2 toolchain. The nios2 gcc didn't generate correct code when the reset vector is passed as a constant. It just generated a direct "call", which was wrong when the reset vector was not located in the same 256MB span as u-boot. The "Nios II Processor Reference

[U-Boot] [PATCH 0/7] patch set for P1021MDS support

2010-08-15 Thread Haiying Wang
The patchset supports SRAM boot, P1021MDS NAND boot, P1021MDS QE/UEC, it is against the git://git.denx.de/u-boot-mpc85xx. Thanks. Haiying ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 0/7] patch set for P1021MDS support

2010-08-15 Thread Wolfgang Denk
Dear Haiying Wang, In message <1281928450.3071.0.ca...@localhost.localdomain> you wrote: > The patchset supports SRAM boot, P1021MDS NAND boot, P1021MDS QE/UEC, it > is against the git://git.denx.de/u-boot-mpc85xx. Please keep in mind that patches shall be submitted against the mainline reposito

[U-Boot] [PATCH v2] IDE: Don't assume there are always two devices per bus

2010-08-15 Thread Rogan Dawes
This addresses Wolfgang's suggestion to use additional parens ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] IDE: Don't assume there are always two devices per bus

2010-08-15 Thread Rogan Dawes
From: Rogan Dawes Some SATA controllers can operate in an IDE compatible mode (e.g. mvsata) but will only ever have a single device per bus. This allows the upcoming DNS323 port to properly identify and use a drive on both SATA interfaces. --- include/ide.h |3 ++- 1 files changed, 2 insert