]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/blobdiff - lib/logic_pio.c
lib/test_string.c: avoid masking memset16/32/64 failures
[mirror_ubuntu-bionic-kernel.git] / lib / logic_pio.c
index bacc0b314e7ebcb3b3b741a047e8c0d65f577db6..feea48fd1a0dd6ae7913b6fad19cc52e20664785 100644 (file)
@@ -1,9 +1,8 @@
 // SPDX-License-Identifier: GPL-2.0+
 /*
- * Copyright (C) 2017 Hisilicon Limited, All Rights Reserved.
+ * Copyright (C) 2017 HiSilicon Limited, All Rights Reserved.
  * Author: Gabriele Paoloni <gabriele.paoloni@huawei.com>
  * Author: Zhichang Yuan <yuanzhichang@hisilicon.com>
- *
  */
 
 #define pr_fmt(fmt)    "LOGIC PIO: " fmt
 #include <linux/sizes.h>
 #include <linux/slab.h>
 
-/* The unique hardware address list. */
+/* The unique hardware address list */
 static LIST_HEAD(io_range_list);
 static DEFINE_MUTEX(io_range_mutex);
 
+/* Consider a kernel general helper for this */
+#define in_range(b, first, len)        ((b) >= (first) && (b) < (first) + (len))
+
 /**
  * logic_pio_register_range - register logical PIO range for a host
- * @new_range: pointer to the io range to be registered.
+ * @new_range: pointer to the IO range to be registered.
  *
- * returns 0 on success, the error code in case of failure
+ * Returns 0 on success, the error code in case of failure.
  *
- * Register a new io range node in the io range list.
+ * Register a new IO range node in the IO range list.
  */
 int logic_pio_register_range(struct logic_pio_hwaddr *new_range)
 {
        struct logic_pio_hwaddr *range;
-       resource_size_t start = new_range->hw_start;
-       resource_size_t end = new_range->hw_start + new_range->size;
-       resource_size_t allocated_mmio_size = 0;
-       resource_size_t allocated_iio_size = MMIO_UPPER_LIMIT;
+       resource_size_t start;
+       resource_size_t end;
+       resource_size_t mmio_sz = 0;
+       resource_size_t iio_sz = MMIO_UPPER_LIMIT;
        int ret = 0;
 
        if (!new_range || !new_range->fwnode || !new_range->size)
                return -EINVAL;
 
+       start = new_range->hw_start;
+       end = new_range->hw_start + new_range->size;
+
        mutex_lock(&io_range_mutex);
        list_for_each_entry_rcu(range, &io_range_list, list) {
                if (range->fwnode == new_range->fwnode) {
                        /* range already there */
-                       ret = -EFAULT;
                        goto end_register;
                }
-               if (range->flags == PIO_CPU_MMIO &&
-                               new_range->flags == PIO_CPU_MMIO) {
+               if (range->flags == LOGIC_PIO_CPU_MMIO &&
+                   new_range->flags == LOGIC_PIO_CPU_MMIO) {
                        /* for MMIO ranges we need to check for overlap */
                        if (start >= range->hw_start + range->size ||
-                           end < range->hw_start)
-                               allocated_mmio_size += range->size;
-                       else {
+                           end < range->hw_start) {
+                               mmio_sz += range->size;
+                       else {
                                ret = -EFAULT;
                                goto end_register;
                        }
-               } else if (range->flags == PIO_INDIRECT &&
-                          new_range->flags == PIO_INDIRECT) {
-                       allocated_iio_size += range->size;
+               } else if (range->flags == LOGIC_PIO_INDIRECT &&
+                          new_range->flags == LOGIC_PIO_INDIRECT) {
+                       iio_sz += range->size;
                }
        }
 
        /* range not registered yet, check for available space */
-       if (new_range->flags == PIO_CPU_MMIO) {
-               if (allocated_mmio_size + new_range->size - 1 >
-                       MMIO_UPPER_LIMIT) {
+       if (new_range->flags == LOGIC_PIO_CPU_MMIO) {
+               if (mmio_sz + new_range->size - 1 > MMIO_UPPER_LIMIT) {
                        /* if it's too big check if 64K space can be reserved */
-                       if (allocated_mmio_size + SZ_64K - 1 >
-                       MMIO_UPPER_LIMIT) {
+                       if (mmio_sz + SZ_64K - 1 > MMIO_UPPER_LIMIT) {
                                ret = -E2BIG;
                                goto end_register;
                        }
                        new_range->size = SZ_64K;
                        pr_warn("Requested IO range too big, new size set to 64K\n");
                }
-               new_range->io_start = allocated_mmio_size;
-       } else if (new_range->flags == PIO_INDIRECT) {
-               if (allocated_iio_size + new_range->size - 1 >
-                   IO_SPACE_LIMIT) {
+               new_range->io_start = mmio_sz;
+       } else if (new_range->flags == LOGIC_PIO_INDIRECT) {
+               if (iio_sz + new_range->size - 1 > IO_SPACE_LIMIT) {
                        ret = -E2BIG;
                        goto end_register;
                }
-               new_range->io_start = allocated_iio_size;
+               new_range->io_start = iio_sz;
        } else {
                /* invalid flag */
                ret = -EINVAL;
@@ -101,10 +102,9 @@ end_register:
  * find_io_range_by_fwnode - find logical PIO range for given FW node
  * @fwnode: FW node handle associated with logical PIO range
  *
- * Returns pointer to node on success, NULL otherwise
+ * Returns pointer to node on success, NULL otherwise.
  *
- * Traverse the io_range_list to find the registered node whose device node
- * and/or physical IO address match to.
+ * Traverse the io_range_list to find the registered node for @fwnode.
  */
 struct logic_pio_hwaddr *find_io_range_by_fwnode(struct fwnode_handle *fwnode)
 {
@@ -123,11 +123,10 @@ static struct logic_pio_hwaddr *find_io_range(unsigned long pio)
        struct logic_pio_hwaddr *range;
 
        list_for_each_entry_rcu(range, &io_range_list, list) {
-               if (pio >= range->io_start &&
-                   pio < range->io_start + range->size)
+               if (in_range(pio, range->io_start, range->size))
                        return range;
        }
-       pr_err("PIO entry token invalid\n");
+       pr_err("PIO entry token %lx invalid\n", pio);
        return NULL;
 }
 
@@ -135,21 +134,20 @@ static struct logic_pio_hwaddr *find_io_range(unsigned long pio)
  * logic_pio_to_hwaddr - translate logical PIO to HW address
  * @pio: logical PIO value
  *
- * Returns HW address if valid, -1 otherwise
+ * Returns HW address if valid, ~0 otherwise.
  *
- * Translate the input logical pio to the corresponding hardware address.
- * The input pio should be unique in the whole logical PIO space.
+ * Translate the input logical PIO to the corresponding hardware address.
+ * The input PIO should be unique in the whole logical PIO space.
  */
 resource_size_t logic_pio_to_hwaddr(unsigned long pio)
 {
        struct logic_pio_hwaddr *range;
-       resource_size_t hwaddr = -1;
 
        range = find_io_range(pio);
        if (range)
-               hwaddr = range->hw_start + pio - range->io_start;
+               return range->hw_start + pio - range->io_start;
 
-       return hwaddr;
+       return (resource_size_t)~0;
 }
 
 /**
@@ -158,49 +156,46 @@ resource_size_t logic_pio_to_hwaddr(unsigned long pio)
  * @addr: Host-relative HW address
  * @size: size to translate
  *
- * Returns Logical PIO value if successful, -1 otherwise
+ * Returns Logical PIO value if successful, ~0UL otherwise
  */
-unsigned long
-logic_pio_trans_hwaddr(struct fwnode_handle *fwnode, resource_size_t addr,
-                      resource_size_t size)
+unsigned long logic_pio_trans_hwaddr(struct fwnode_handle *fwnode,
+                                    resource_size_t addr, resource_size_t size)
 {
        struct logic_pio_hwaddr *range;
 
        range = find_io_range_by_fwnode(fwnode);
-       if (!range || range->flags == PIO_CPU_MMIO) {
-               pr_err("range not found or invalid\n");
-               return -1;
+       if (!range || range->flags == LOGIC_PIO_CPU_MMIO) {
+               pr_err("IO range not found or invalid\n");
+               return ~0UL;
        }
        if (range->size < size) {
                pr_err("resource size %pa cannot fit in IO range size %pa\n",
                       &size, &range->size);
-               return -1;
+               return ~0UL;
        }
        return addr - range->hw_start + range->io_start;
 }
 
-unsigned long
-logic_pio_trans_cpuaddr(resource_size_t addr)
+unsigned long logic_pio_trans_cpuaddr(resource_size_t addr)
 {
        struct logic_pio_hwaddr *range;
 
        list_for_each_entry_rcu(range, &io_range_list, list) {
-               if (range->flags != PIO_CPU_MMIO)
+               if (range->flags != LOGIC_PIO_CPU_MMIO)
                        continue;
-               if (addr >= range->hw_start &&
-                   addr < range->hw_start + range->size)
-                       return addr - range->hw_start +
-                               range->io_start;
+               if (in_range(addr, range->hw_start, range->size))
+                       return addr - range->hw_start + range->io_start;
        }
-       pr_err("addr not registered in io_range_list\n");
-       return -1;
+       pr_err("addr %llx not registered in io_range_list\n",
+              (unsigned long long) addr);
+       return ~0UL;
 }
 
 #if defined(CONFIG_INDIRECT_PIO) && defined(PCI_IOBASE)
 #define BUILD_LOGIC_IO(bw, type)                                       \
 type logic_in##bw(unsigned long addr)                                  \
 {                                                                      \
-       type ret = -1;                                                  \
+       type ret = (type)~0;                                            \
                                                                        \
        if (addr < MMIO_UPPER_LIMIT) {                                  \
                ret = read##bw(PCI_IOBASE + addr);                      \