From: Teodori Serge <teodori.serge@gmail.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Arnd Bergmann <arnd@arndb.de>,
linuxppc-dev@lists.ozlabs.org
Cc: Teodori Serge <teodori.serge@hotmail.com>
Subject: [PATCH 1/1] drivers/char/mem.c
Date: Sat, 22 Sep 2012 23:20:12 +0200 [thread overview]
Message-ID: <1348348812-17201-1-git-send-email-teodori.serge@hotmail.com> (raw)
In-Reply-To: <Teodori Serge <teodori.serge@hotmail.com>
replace 'arch_has_dev_port()' to 'CONFIG_DEVPORT'
modified: arch/powerpc/include/asm/io.h
modified: drivers/char/mem.c
modified: include/linux/io.h
why do we need 'arch_has_dev_port()' if we have a 'CONFIG_DEVPORT'?
Signed-off-by: Teodori Serge <teodori.serge@hotmail.com>
---
arch/powerpc/include/asm/io.h | 2 +-
drivers/char/mem.c | 6 ------
include/linux/io.h | 9 ---------
3 files changed, 1 insertion(+), 16 deletions(-)
diff --git a/arch/powerpc/include/asm/io.h b/arch/powerpc/include/asm/io.h
index f94ef42..cb8b663 100644
--- a/arch/powerpc/include/asm/io.h
+++ b/arch/powerpc/include/asm/io.h
@@ -25,7 +25,7 @@ extern struct pci_dev *isa_bridge_pcidev;
/*
* has legacy ISA devices ?
*/
-#define arch_has_dev_port() (isa_bridge_pcidev != NULL)
+#define CONFIG_DEVPORT (isa_bridge_pcidev != NULL)
#endif
#include <linux/device.h>
diff --git a/drivers/char/mem.c b/drivers/char/mem.c
index e5eedfa..7251a67 100644
--- a/drivers/char/mem.c
+++ b/drivers/char/mem.c
@@ -897,12 +897,6 @@ static int __init chr_dev_init(void)
if (!devlist[minor].name)
continue;
- /*
- * Create /dev/port?
- */
- if ((minor == DEVPORT_MINOR) && !arch_has_dev_port())
- continue;
-
device_create(mem_class, NULL, MKDEV(MEM_MAJOR, minor),
NULL, devlist[minor].name);
}
diff --git a/include/linux/io.h b/include/linux/io.h
index 069e407..7fd2d21 100644
--- a/include/linux/io.h
+++ b/include/linux/io.h
@@ -67,13 +67,4 @@ int check_signature(const volatile void __iomem *io_addr,
const unsigned char *signature, int length);
void devm_ioremap_release(struct device *dev, void *res);
-/*
- * Some systems do not have legacy ISA devices.
- * /dev/port is not a valid interface on these systems.
- * So for those archs, <asm/io.h> should define the following symbol.
- */
-#ifndef arch_has_dev_port
-#define arch_has_dev_port() (1)
-#endif
-
#endif /* _LINUX_IO_H */
--
1.7.9.5
next parent reply other threads:[~2012-09-22 21:21 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <Teodori Serge <teodori.serge@hotmail.com>
2012-09-22 21:20 ` Teodori Serge [this message]
2012-09-23 14:16 ` [PATCH 1/1] drivers/char/mem.c Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1348348812-17201-1-git-send-email-teodori.serge@hotmail.com \
--to=teodori.serge@gmail.com \
--cc=arnd@arndb.de \
--cc=gregkh@linuxfoundation.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=teodori.serge@hotmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox