Hello, On Sun, Jul 06, 2014 at 02:28:40PM +0800, Kevin Hao wrote: > +int ata_host_set_queue_depth(struct ata_host *host, unsigned int queue_depth) Hmmm... is there a reason we're doing this separately when the same information is available from scsi_host_template->can_queue from ata_host_register()? > @@ -5731,6 +5752,7 @@ struct ata_host *ata_host_alloc(struct device *dev, int max_ports) > mutex_init(&host->eh_mutex); > host->dev = dev; > host->n_ports = max_ports; > + host->queue_depth = ATA_MAX_QUEUE; And shouldn't this be one for !NCQ controllers? Thanks. -- tejun -- To unsubscribe from this list: 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
