On Fri, Jul 26, 2024 at 12:19:54PM +0900, Damien Le Moal wrote:
> Rename the function hpt_dma_blacklisted() to the more neutral
> hpt_dma_broken().
>
> Signed-off-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
> ---
> drivers/ata/pata_hpt37x.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/ata/pata_hpt37x.c b/drivers/ata/pata_hpt37x.c
> index c0329cf01135..4af22b819416 100644
> --- a/drivers/ata/pata_hpt37x.c
> +++ b/drivers/ata/pata_hpt37x.c
> @@ -218,8 +218,8 @@ static u32 hpt37x_find_mode(struct ata_port *ap, int speed)
> return 0xffffffffU; /* silence compiler warning */
> }
>
> -static int hpt_dma_blacklisted(const struct ata_device *dev, char *modestr,
> - const char * const list[])
> +static int hpt_dma_broken(const struct ata_device *dev, char *modestr,
> + const char * const list[])
> {
> unsigned char model_num[ATA_ID_PROD_LEN + 1];
> int i;
> @@ -281,9 +281,9 @@ static const char * const bad_ata100_5[] = {
> static unsigned int hpt370_filter(struct ata_device *adev, unsigned int mask)
> {
> if (adev->class == ATA_DEV_ATA) {
> - if (hpt_dma_blacklisted(adev, "UDMA", bad_ata33))
> + if (hpt_dma_broken(adev, "UDMA", bad_ata33))
> mask &= ~ATA_MASK_UDMA;
> - if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
> + if (hpt_dma_broken(adev, "UDMA100", bad_ata100_5))
> mask &= ~(0xE0 << ATA_SHIFT_UDMA);
> }
> return mask;
> @@ -300,7 +300,7 @@ static unsigned int hpt370_filter(struct ata_device *adev, unsigned int mask)
> static unsigned int hpt370a_filter(struct ata_device *adev, unsigned int mask)
> {
> if (adev->class == ATA_DEV_ATA) {
> - if (hpt_dma_blacklisted(adev, "UDMA100", bad_ata100_5))
> + if (hpt_dma_broken(adev, "UDMA100", bad_ata100_5))
> mask &= ~(0xE0 << ATA_SHIFT_UDMA);
> }
> return mask;
> --
> 2.45.2
>
Reviewed-by: Niklas Cassel <cassel@xxxxxxxxxx>
