Niklas, > This series moves the assignment of ap->print_id, which is used as a > unique id for each port, earlier, such that we can use the ata_port_* > print functions even before the ata_host has been registered. > > While the patch series was orginally meant to simply assign a unique > id used for printing earlier (ap->print_id), it has since grown to > also include cleanups related to ata_port_alloc() (since ap->print_id > is now assigned in ata_port_alloc()). Looks good to me. Reviewed-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx> -- Martin K. Petersen Oracle Linux Engineering