История изменений
Исправление Andrey_Utkin, (текущая версия) :
Встретив неоднозначность, отбрось искушение угадать.
В случае nand/au1550, похоже, явная ошибка, в случае ralink (проверка на отрицательность) - плохой стиль. Можно уточнить у разрабов и послать патчики.
Я бы рекомендовал следующую, более простую как для написания, так и чтения форму, которая делает то же самое:
err = pci_request_regions(pci_dev, dev->name);
if (err) {
dev_err(&dev->pci->dev, "Cannot request regions for MMIO\n");
goto disable_pci;
}
С последующим
dev->mmio = pci_ioremap_bar(pci_dev, 0);
if (!dev->mmio) {
err = -EIO;
dev_err(&dev->pci->dev, "can't ioremap() MMIO memory\n");
goto release_mmio;
}
Исходная версия Andrey_Utkin, :
Встретив неоднозначность, отбрось искушение угадать.
Похоже, в случае nand/au1550, похоже, явная ошибка, в случае ralink (проверка на отрицательность) - плохой стиль.
Я бы рекомендовал следующую, более простую как для написания, так и чтения форму, которая делает то же самое:
err = pci_request_regions(pci_dev, dev->name);
if (err) {
dev_err(&dev->pci->dev, "Cannot request regions for MMIO\n");
goto disable_pci;
}
С последующим
dev->mmio = pci_ioremap_bar(pci_dev, 0);
if (!dev->mmio) {
err = -EIO;
dev_err(&dev->pci->dev, "can't ioremap() MMIO memory\n");
goto release_mmio;
}