PNP: Remove pnp_alloc()
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 12 May 2021 20:36:12 +0000 (22:36 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 17 May 2021 14:54:01 +0000 (16:54 +0200)
The kernel will complain anyway if it runs out of memory, so it is
not necessary to print an extra error message when that happens and
kzalloc() can be called directly instead of pnp_alloc() which then
becomes redundant and can be dropped.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
[ rjw: Subject and changelog edits ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/pnp/base.h
drivers/pnp/card.c
drivers/pnp/core.c
drivers/pnp/interface.c

index cdcfa39cf1673152d305f897041d9280cff988b1..e74a0f6a31572bf53c87f4dfbb34cf7d76208f8f 100644 (file)
@@ -6,7 +6,6 @@
 
 extern struct mutex pnp_lock;
 extern const struct attribute_group *pnp_dev_groups[];
-void *pnp_alloc(long size);
 
 int pnp_register_protocol(struct pnp_protocol *protocol);
 void pnp_unregister_protocol(struct pnp_protocol *protocol);
index c2464ee08e4aca066a260b9aeaad35537754ff55..2430c14f472d26adda08e6d980dd9c2b3eed771b 100644 (file)
@@ -80,7 +80,7 @@ static int card_probe(struct pnp_card *card, struct pnp_card_driver *drv)
        if (!id)
                return 0;
 
-       clink = pnp_alloc(sizeof(*clink));
+       clink = kzalloc(sizeof(*clink), GFP_KERNEL);
        if (!clink)
                return 0;
        clink->card = card;
index a50ab002e9e41c7f3b2fcc5026777a24c9fdec21..ccdfbf3978c08c793c293965022cbf005c7c0af9 100644 (file)
@@ -31,18 +31,6 @@ DEFINE_MUTEX(pnp_lock);
 int pnp_platform_devices;
 EXPORT_SYMBOL(pnp_platform_devices);
 
-void *pnp_alloc(long size)
-{
-       void *result;
-
-       result = kzalloc(size, GFP_KERNEL);
-       if (!result) {
-               printk(KERN_ERR "pnp: Out of Memory\n");
-               return NULL;
-       }
-       return result;
-}
-
 static void pnp_remove_protocol(struct pnp_protocol *protocol)
 {
        mutex_lock(&pnp_lock);
index 602c46893e8313640b09065e39eaefef975186dd..44efcdb87e6fe36de7b0f813a7ccff38afadb887 100644 (file)
@@ -214,7 +214,7 @@ static ssize_t options_show(struct device *dmdev, struct device_attribute *attr,
        int ret, dep = 0, set = 0;
        char *indent;
 
-       buffer = pnp_alloc(sizeof(pnp_info_buffer_t));
+       buffer = kzalloc(sizeof(*buffer), GFP_KERNEL);
        if (!buffer)
                return -ENOMEM;
 
@@ -257,7 +257,7 @@ static ssize_t resources_show(struct device *dmdev,
        if (!dev)
                return -EINVAL;
 
-       buffer = pnp_alloc(sizeof(pnp_info_buffer_t));
+       buffer = kzalloc(sizeof(*buffer), GFP_KERNEL);
        if (!buffer)
                return -ENOMEM;