diff options
author | Jean Delvare <jdelvare@suse.de> | 2013-09-11 17:24:07 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-11 18:58:50 -0400 |
commit | 3d267f24d4c7bcc829ce9daa92e41c3f390c95dd (patch) | |
tree | 988b7f6ee69671aae9ef43d653cceb826a647679 /drivers/firmware/dmi_scan.c | |
parent | 91cf5ab60ff82ecf4550a596867787c1e360dd3f (diff) |
firmware/dmi_scan: drop obsolete comment
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>
Diffstat (limited to 'drivers/firmware/dmi_scan.c')
-rw-r--r-- | drivers/firmware/dmi_scan.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware/dmi_scan.c index 232fa8fce26a..9e50cb997a42 100644 --- a/drivers/firmware/dmi_scan.c +++ b/drivers/firmware/dmi_scan.c | |||
@@ -504,11 +504,6 @@ void __init dmi_scan_machine(void) | |||
504 | } | 504 | } |
505 | } | 505 | } |
506 | else { | 506 | else { |
507 | /* | ||
508 | * no iounmap() for that ioremap(); it would be a no-op, but | ||
509 | * it's so early in setup that sucker gets confused into doing | ||
510 | * what it shouldn't if we actually call it. | ||
511 | */ | ||
512 | p = dmi_ioremap(0xF0000, 0x10000); | 507 | p = dmi_ioremap(0xF0000, 0x10000); |
513 | if (p == NULL) | 508 | if (p == NULL) |
514 | goto error; | 509 | goto error; |