[U-Boot-Users] IDE and x86 port
Wolfgang Denk
wd at denx.de
Thu Aug 14 18:17:38 CEST 2003
In message <20030814154333.GD1468 at buici.com> you wrote:
>
> The other change appears below. For some reason, the existing IDE
> code reorders the half-words of the capacity value read from the IDE
> device. I don't have any MSB CPUs, so I cannot verify the need for
> this. AFAICT, it should not be necessary.
Well, I have no LE CPU's where I can test a CF card easily.
But I can say that the current code works fine of PPC systems.
> I'd make the conditional test hinge on endianness if I knew that was
> the reason for the swapping. So, either we add an __X86__ check, or
> we change the conditional.
>
> @@ -1056,7 +1099,12 @@
> #endif /* CONFIG_ATAPI */
>
> /* swap shorts */
> +#if !defined (__ARM__)
I will reject such a patch. This is definitely not an ARM / not ARM
issue.
> +#else
> + /* elf: it isn't clear why this code swaps the words. There's nothing i
This comment puzzles me: what's "elf"? ELF file format??
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-4596-87 Fax: (+49)-8142-4596-88 Email: wd at denx.de
"Intelligence without character is a dangerous thing." - G. Steinem
More information about the U-Boot
mailing list