Commit daadabfb authored by Anupama K Patil's avatar Anupama K Patil Committed by Rafael J. Wysocki
Browse files

drivers: pnp: isapnp: proc.c: Remove unnecessary local variables 



In the PNP code, there are two redundant local variables that can be
dropped.

This also fixes a coding style issue reported by checkpatch about an
assignment made under an if () statement.

Reviewed-by: default avatarJaroslav Kysela <perex@perex.cz>
Signed-off-by: default avatarAnupama K Patil <anupamakpatil123@gmail.com>
[ rjw: Subject and changelog edits ]
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent d07f6ca9

drivers/pnp/isapnp/proc.c

+6 −7
Original line number Diff line number Diff line
@@ -57,21 +57,20 @@ static const struct proc_ops isapnp_proc_bus_proc_ops = { 
static int isapnp_proc_attach_device(struct pnp_dev *dev)

{

	struct pnp_card *bus = dev->card;

	struct proc_dir_entry *de, *e;

	char name[16];



	if (!(de = bus->procdir)) {

	if (!bus->procdir) {

		sprintf(name, "%02x", bus->number);

		de = bus->procdir = proc_mkdir(name, isapnp_proc_bus_dir);

		if (!de)

		bus->procdir = proc_mkdir(name, isapnp_proc_bus_dir);

		if (!bus->procdir)

			return -ENOMEM;

	}

	sprintf(name, "%02x", dev->number);

	e = dev->procent = proc_create_data(name, S_IFREG | S_IRUGO, de,

	dev->procent = proc_create_data(name, S_IFREG | S_IRUGO, bus->procdir,

					    &isapnp_proc_bus_proc_ops, dev);

	if (!e)

	if (!dev->procent)

		return -ENOMEM;

	proc_set_size(e, 256);

	proc_set_size(dev->procent, 256);

	return 0;

}