On 10/8/22 08:50, Niklas Cassel wrote: > On Sat, Oct 08, 2022 at 07:31:00AM +0900, Damien Le Moal wrote: >> On 10/7/22 22:23, Niklas Cassel wrote: >>> Clean up the code by making use of the newly introduced >>> ata_port_is_frozen() helper function. >> >> Looks good, but I think this should be squashed with patch 1. > > I could, but by that same logic, shouldn't patch 3/4 also be squashed into > patch 1? Well, I didn't get patch 3 :) Looking at it now, since patch 3 are changes for libsas, better keep it a separate patch, and we can see with Martin which tree to use to queue it (it probably should go through the ata tree). > > > Kind regards, > Niklas -- Damien Le Moal Western Digital Research