ata: pata_atiixp: make static read-only arrays const
authorColin Ian King <colin.i.king@gmail.com>
Sun, 23 Jan 2022 22:16:22 +0000 (22:16 +0000)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Mon, 31 Jan 2022 00:58:32 +0000 (09:58 +0900)
The static arrays pio_timings and mwdma_timings are read-only so
it make sense to make them const.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/pata_atiixp.c

index c3a65cc..efdb94c 100644 (file)
@@ -102,7 +102,7 @@ static int atiixp_prereset(struct ata_link *link, unsigned long deadline)
 
 static void atiixp_set_pio_timing(struct ata_port *ap, struct ata_device *adev, int pio)
 {
-       static u8 pio_timings[5] = { 0x5D, 0x47, 0x34, 0x22, 0x20 };
+       static const u8 pio_timings[5] = { 0x5D, 0x47, 0x34, 0x22, 0x20 };
 
        struct pci_dev *pdev = to_pci_dev(ap->host->dev);
        int dn = 2 * ap->port_no + adev->devno;
@@ -149,7 +149,7 @@ static void atiixp_set_piomode(struct ata_port *ap, struct ata_device *adev)
 
 static void atiixp_set_dmamode(struct ata_port *ap, struct ata_device *adev)
 {
-       static u8 mwdma_timings[5] = { 0x77, 0x21, 0x20 };
+       static const u8 mwdma_timings[5] = { 0x77, 0x21, 0x20 };
 
        struct pci_dev *pdev = to_pci_dev(ap->host->dev);
        int dma = adev->dma_mode;