ata: sata_mv: Fix the error handling of mv_chip_id()
authorZheyu Ma <zheyuma97@gmail.com>
Fri, 22 Oct 2021 09:12:26 +0000 (09:12 +0000)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Sun, 24 Oct 2021 23:53:04 +0000 (08:53 +0900)
commita0023bb9dd9bc439d44604eeec62426a990054cd
treeb3d0dcf247ebd8ef3add3c8af88f41228e5713c3
parent776c75010803849c1cc4f11031a2b3960ab05202
ata: sata_mv: Fix the error handling of mv_chip_id()

mv_init_host() propagates the value returned by mv_chip_id() which in turn
gets propagated by mv_pci_init_one() and hits local_pci_probe().

During the process of driver probing, the probe function should return < 0
for failure, otherwise, the kernel will treat value > 0 as success.

Since this is a bug rather than a recoverable runtime error we should
use dev_alert() instead of dev_err().

Signed-off-by: Zheyu Ma <zheyuma97@gmail.com>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/sata_mv.c