Hi > On Mon, Feb 11, 2008 at 10:30:08PM +0100, Jean-Christophe PLAGNIOL-VILLARD > wrote: > >> On 21:40 Mon 11 Feb , michael wrote: >> >>> --- a/common/cmd_ide.c >>> +++ b/common/cmd_ide.c >>> @@ -934,7 +934,14 @@ output_data(int dev, ulong *sect_buf, int words) >>> static void >>> output_data(int dev, ulong *sect_buf, int words) >>> { >>> +#ifndef __ARM__ >>> outsw(ATA_CURR_BASE(dev)+ATA_DATA_REG, sect_buf, words<<1); >>> +#else >>> + int i = words; >>> >> Why do you initialise "i" with words and re-initialize it at 0 in for? >> > And why do you need this ungly #ifdef at all? > > ladis > > The code is fill with ugly ifdef. Can you suggest me something better?
Michael ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users