On Sat, 2008-05-03 at 19:35 +0200, Marco Vega Trucillo wrote:
> Hi to all,
>
> I tried to modify ata.c simply add more ioaddress (I attach the diff file
> from 1.96 version), and it work and recognize
> my four sata disk on different pc.
I think it's generally considered unsafe to probe addresses above 0x3ff
without making sure they belong to a compatible device. The device
should be found on the PCI bus, and enabled if necessary.
> -/* try address of commons sata controller */
> -static const int grub_ata_ioaddress[] = {0x1f0, 0x170
> ,0x9800,0x9000,0x8800,0x18f8,0x18f0};
Your patch is reversed. Also, it would be better to make a patch
against the current CVS version.
> - for (port = 0; port <= 6; port++)
> + for (port = 0; port <= 1; port++)
Using sizeof would be great here. Unless we switch to using a dynamic
list of interface.
--
Regards,
Pavel Roskin
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
