linux-kernel - struct should not be named same way as function in sony-laptop

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080103110154.GA2376@elf.ucw.cz>
Date:	Thu, 3 Jan 2008 12:01:54 +0100
From:	Pavel Machek <pavel@....cz>
To:	malattia@...ux.it, kernel list <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...l.org>
Cc:	trivial@...nel.org
Subject: struct should not be named same way as function in sony-laptop

Naming struct and function by same name is evil. Rename the struct.

Signed-off-by: Pavel Machek <pavel@...e.cz>

diff --git a/drivers/misc/sony-laptop.c b/drivers/misc/sony-laptop.c
index b0f6803..31ce39d 100644
--- a/drivers/misc/sony-laptop.c
+++ b/drivers/misc/sony-laptop.c
@@ -1180,7 +1180,7 @@ struct sony_pic_ioport {
 	struct list_head	list;
 };
 
-struct sony_pic_irq {
+struct sony_pic_irqdesc {
 	struct acpi_resource_irq	irq;
 	struct list_head		list;
 };
@@ -1192,7 +1192,7 @@ struct sony_pic_dev {
 	u8			bluetooth_power;
 	u8			wwan_power;
 	struct acpi_device	*acpi_dev;
-	struct sony_pic_irq	*cur_irq;
+	struct sony_pic_irqdesc	*cur_irq;
 	struct sony_pic_ioport	*cur_ioport;
 	struct list_head	interrupts;
 	struct list_head	ioports;
@@ -2094,7 +2094,7 @@ sony_pic_read_possible_resource(struct a
 	case ACPI_RESOURCE_TYPE_IRQ:
 		{
 			struct acpi_resource_irq *p = &resource->data.irq;
-			struct sony_pic_irq *interrupt = NULL;
+			struct sony_pic_irqdesc *interrupt = NULL;
 			if (!p || !p->interrupt_count) {
 				/*
 				 * IRQ descriptors may have no IRQ# bits set,
@@ -2218,7 +2218,7 @@ static int sony_pic_disable(struct acpi_
  *  Call _SRS to set current resources
  */
 static int sony_pic_enable(struct acpi_device *device,
-		struct sony_pic_ioport *ioport, struct sony_pic_irq *irq)
+		struct sony_pic_ioport *ioport, struct sony_pic_irqdesc *irq)
 {
 	acpi_status status;
 	int result = 0;
@@ -2381,7 +2381,7 @@ found:
 static int sony_pic_remove(struct acpi_device *device, int type)
 {
 	struct sony_pic_ioport *io, *tmp_io;
-	struct sony_pic_irq *irq, *tmp_irq;
+	struct sony_pic_irqdesc *irq, *tmp_irq;
 
 	if (sony_pic_disable(device)) {
 		printk(KERN_ERR DRV_PFX "Couldn't disable device.\n");
@@ -2422,7 +2422,7 @@ static int sony_pic_add(struct acpi_devi
 {
 	int result;
 	struct sony_pic_ioport *io, *tmp_io;
-	struct sony_pic_irq *irq, *tmp_irq;
+	struct sony_pic_irqdesc *irq, *tmp_irq;
 
 	printk(KERN_INFO DRV_PFX "%s v%s.\n",
 		SONY_PIC_DRIVER_NAME, SONY_LAPTOP_DRIVER_VERSION);

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/