On Llu, 2006-05-15 at 08:00 -0700, Andrew Morton wrote: > > /* Capable of UltraDMA modes? */ > - if (id->field_valid & 4) > - ultra_mask = id->dma_ultra & hwif->ultra_mask; > - else > + ultra_mask = id->dma_ultra & hwif->ultra_mask; > + > + if (!(id->field_valid & 4)) > mode = 0; /* fallback to MW/SW DMA if no UltraDMA */ > Looks fine to me, id-> is always the full 512 bytes of data so its safe to do that. - : 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