On 2/3/22 05:07, Sergey Shtylyov wrote: > ata_std_prereset() always returns 0, hence the check in ata_sff_prereset() > is pointless and thus it also can return only 0 (however, we cannot change > the prototypes of ata_{sff|std}_prereset() as they implement the driver's > prereset() method). > > Found by Linux Verification Center (linuxtesting.org) with the SVACE static > analysis tool. > > Signed-off-by: Sergey Shtylyov <s.shtylyov@xxxxxx> Applied to for-5.18. Thanks ! -- Damien Le Moal Western Digital Research