On Fri, Apr 12, 2024 at 09:15:28AM +0900, Damien Le Moal wrote: > ata_exec_internal() is the only caller of ata_exec_internal_sg() and > always calls this function with a single element scattergather list. > Remove ata_exec_internal_sg() and code it directly in > ata_exec_internal(), simplifying a little the sgl handling for the > command. > > While at it, change the function signature to use the proper enum > dma_data_direction type for the dma_dir argument, cleanup comments > (capitalization and remove useless comments) and change the variable > auto_timeout type to a boolean. > > No functional change. > > Signed-off-by: Damien Le Moal <dlemoal@xxxxxxxxxx> > Reviewed-by: John Garry <john.g.garry@xxxxxxxxxx> > --- Reviewed-by: Niklas Cassel <cassel@xxxxxxxxxx>