Jeff Garzik wrote: > Albert Lee wrote: > >> --- upstream/include/linux/libata.h 2006-05-16 11:08:54.000000000 >> +0800 >> +++ upstream1/include/linux/libata.h 2006-05-17 13:25:24.000000000 >> +0800 >> @@ -330,12 +330,12 @@ struct ata_host_set { >> unsigned long irq; >> void __iomem *mmio_base; >> unsigned int n_ports; >> - void *private_data; >> const struct ata_port_operations *ops; >> unsigned long flags; >> int simplex_claimed; /* Keep seperate in case we >> ever need to do this locked */ >> struct ata_port * ports[0]; >> + void *private_data; >> }; > > > The zero length array needs to remain the last element in the struct... > Ah, I see. Sorry for the noise. -- albert - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html