Re: [PATCH v4 8/9] ata,scsi: Remove wrapper ata_sas_port_alloc()

John Garry
2024-07-04T07:05:10Z

On 03/07/2024 19:44, Niklas Cassel wrote:
> The ata_sas_port_alloc() wrapper mainly exists in order to export the
> internal libata function which it wraps. The secondary reason is that
> it initializes some ata_port struct members.
>
> However, ata_sas_port_alloc() is only used in a single location,
> sas_ata_init(), which already performs some ata_port struct member
> initialization, so it does not make sense to spread this initialization
> out over two separate locations.
>
> Thus, remove the wrapper and instead export the libata function directly,
> and move the libsas specific ata_port initialization to sas_ata_init(),
> which already does some ata_port initialization.
>
> Reviewed-by: Hannes Reinecke<hare@suse.de>
> Signed-off-by: Niklas Cassel<cassel@kernel.org>

Reviewed-by: John Garry <john.g.garry@oracle.com>