Re: [PATCH v4 02/27] ata: expose ncq_enable_prio sysfs attribute only on NCQ capable hosts

From: Christoph Hellwig
Date: Thu Mar 26 2020 - 05:45:42 EST


On Tue, Mar 17, 2020 at 03:43:08PM +0100, Bartlomiej Zolnierkiewicz wrote:
> There is no point in exposing ncq_enable_prio sysfs attribute for
> devices on PATA and non-NCQ capable SATA hosts so:
>
> * remove dev_attr_ncq_prio_enable from ata_common_sdev_attrs[]
>
> * add ata_ncq_sdev_attrs[]
>
> * update ATA_NCQ_SHT() macro to use ata_ncq_sdev_attrs[]
>
> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>