PCI: Use consistent put_user() pointer types
authorBjorn Helgaas <bhelgaas@google.com>
Mon, 25 Jan 2021 17:57:38 +0000 (11:57 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Fri, 25 Aug 2023 13:15:13 +0000 (08:15 -0500)
We used u8, u16, and u32 for get_user() pointer types, but "unsigned char",
"unsigned short", and "unsigned int" for put_user().

Use u8, u16, and u32 for put_user() for consistency.  No functional change
intended.

Link: https://lore.kernel.org/r/20230824193712.542167-6-helgaas@kernel.org
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/pci/syscall.c

index 61a6fe3..803acbf 100644 (file)
@@ -52,13 +52,13 @@ SYSCALL_DEFINE5(pciconfig_read, unsigned long, bus, unsigned long, dfn,
 
        switch (len) {
        case 1:
-               err = put_user(byte, (unsigned char __user *)buf);
+               err = put_user(byte, (u8 __user *)buf);
                break;
        case 2:
-               err = put_user(word, (unsigned short __user *)buf);
+               err = put_user(word, (u16 __user *)buf);
                break;
        case 4:
-               err = put_user(dword, (unsigned int __user *)buf);
+               err = put_user(dword, (u32 __user *)buf);
                break;
        }
        pci_dev_put(dev);
@@ -70,13 +70,13 @@ error:
           they get instead of a machine check on x86.  */
        switch (len) {
        case 1:
-               put_user(-1, (unsigned char __user *)buf);
+               put_user(-1, (u8 __user *)buf);
                break;
        case 2:
-               put_user(-1, (unsigned short __user *)buf);
+               put_user(-1, (u16 __user *)buf);
                break;
        case 4:
-               put_user(-1, (unsigned int __user *)buf);
+               put_user(-1, (u32 __user *)buf);
                break;
        }
        pci_dev_put(dev);