> Why not: > > if (pdev->device == 0xC409) > if (ap->port_no == 0) { > pci_read_config_dword(pdev, 0x52, &ata66); > return (ata66 & 0x10) ? ATA_CBL_PATA80 > : ATA_CBL_PATA40; > } else if (ap->port_no == 1) { > DPRINTK("C409 only has one pata channel\n"); > return ATA_CBL_PATA_UNK; > } > > Thx Sergei. We might refer to Alan's opinion for next patch. ��.n��������+%������w��{.n�����{��'^�)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥