firmware/dmi_scan: drop obsolete comment
authorJean Delvare <jdelvare@suse.de>
Wed, 11 Sep 2013 21:24:07 +0000 (14:24 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 11 Sep 2013 22:58:50 +0000 (15:58 -0700)
This comment predates the introduction of early_ioremap.  Since then the
missing calls to dmi_iounmap have been added by Ingo and Yinghai in
commits 0d64484f7ea1 ("x86: fix DMI ioremap leak") and 3212bff370c2
("x86: left over fix for leak of early_ioremp in dmi_scan") .  That was
over 5 years ago so it is about time to drop this now misleading
comment.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: Joe Perches <joe@perches.com>
Cc: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/firmware/dmi_scan.c

index 232fa8fce26a5d0ffddf5fdc3ee01ed3c63eed7f..9e50cb997a42d06074a20de422bf4c15d8cf12b4 100644 (file)
@@ -504,11 +504,6 @@ void __init dmi_scan_machine(void)
                }
        }
        else {
-               /*
-                * no iounmap() for that ioremap(); it would be a no-op, but
-                * it's so early in setup that sucker gets confused into doing
-                * what it shouldn't if we actually call it.
-                */
                p = dmi_ioremap(0xF0000, 0x10000);
                if (p == NULL)
                        goto error;