Use list_for_each_entry() instead of list_for_each() where
applicable.
Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zou Wei <zou_wei@huawei.com>
[ rjw: Subject and changelog rewrite ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
 
 static int __init insert_device(struct pnp_bios_node *node)
 {
-       struct list_head *pos;
        struct pnp_dev *dev;
        char id[8];
        int error;
 
        /* check if the device is already added */
-       list_for_each(pos, &pnpbios_protocol.devices) {
-               dev = list_entry(pos, struct pnp_dev, protocol_list);
+       list_for_each_entry(dev, &pnpbios_protocol.devices, protocol_list) {
                if (dev->number == node->handle)
                        return -EEXIST;
        }