From: Finn Thain <fthain@telegraphics.com.au>
To: Thorsten Glaser <tg@debian.org>
Cc: linux-m68k@vger.kernel.org
Subject: Re: [PATCH 1/1] m68k: add missing I/O macros {in,out}{w,l}_p() for !CONFIG_ISA
Date: Mon, 4 Oct 2010 11:51:03 +1100 (EST) [thread overview]
Message-ID: <alpine.OSX.2.00.1010041149030.285@localhost> (raw)
In-Reply-To: <Pine.BSM.4.64L.1010031846060.15165@herc.mirbsd.org>
[-- Attachment #1: Type: TEXT/PLAIN, Size: 1434 bytes --]
On Sun, 3 Oct 2010, Thorsten Glaser wrote:
> On m68k, I/O macros like inb() outw() etc. are only defined to something
> useful if CONFIG_ISA is set; dummies are in place if not, but four
> macros were missing from the !CONFIG_ISA case. Adding these makes some
> drivers, such as speakup, compile again. The patch was made against
> Debian’s linux-2.6_2.6.32-24, but
I was able to compile that source package. Can you please send the .config
you used?
Finn
> applies cleanly on linux-m68k master.
>
> Signed-off-by: Thorsten Glaser <tg@debian.org>
> ---
> arch/m68k/include/asm/io_mm.h | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/m68k/include/asm/io_mm.h b/arch/m68k/include/asm/io_mm.h
> index 56a6555..be9d7da 100644
> --- a/arch/m68k/include/asm/io_mm.h
> +++ b/arch/m68k/include/asm/io_mm.h
> @@ -387,9 +387,13 @@ static inline void isa_delay(void)
> #define outb(val,port) ((void)0)
> #define outb_p(val,port) ((void)0)
> #define inw(port) 0xffff
> +#define inw_p(port) 0xffff
> #define outw(val,port) ((void)0)
> +#define outw_p(val,port) ((void)0)
> #define inl(port) 0xffffffffUL
> +#define inl_p(port) 0xffffffffUL
> #define outl(val,port) ((void)0)
> +#define outl_p(val,port) ((void)0)
>
> #define insb(port,buf,nr) ((void)0)
> #define outsb(port,buf,nr) ((void)0)
>
next prev parent reply other threads:[~2010-10-04 0:51 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-03 18:38 [PATCH 0/1] m68k: add missing I/O macros {in,out}{w,l}_p() for !CONFIG_ISA Thorsten Glaser
2010-10-03 18:48 ` [PATCH 1/1] " Thorsten Glaser
2010-10-04 0:51 ` Finn Thain [this message]
2010-10-04 7:40 ` Thorsten Glaser
2010-10-04 8:47 ` Finn Thain
2010-10-04 9:10 ` Thorsten Glaser
2010-10-04 10:22 ` Finn Thain
2010-10-04 11:30 ` Thorsten Glaser
2010-10-04 13:18 ` Geert Uytterhoeven
2010-10-04 13:33 ` Finn Thain
2010-10-04 13:40 ` Thorsten Glaser
2010-10-04 14:01 ` Geert Uytterhoeven
2010-10-04 14:13 ` Thorsten Glaser
2010-10-04 19:20 ` Thorsten Glaser
2010-10-13 20:17 ` Thorsten Glaser
2010-10-05 12:56 ` Stephen R. Marenka
2010-10-05 14:10 ` debian kernel config, was " Finn Thain
2010-10-06 0:57 ` Stephen R. Marenka
2010-10-06 9:26 ` Thorsten Glaser
2010-10-06 13:09 ` Stephen R. Marenka
2010-10-06 14:15 ` Finn Thain
2010-10-04 7:57 ` Michael Schmitz
2010-10-04 8:13 ` Thorsten Glaser
2010-10-04 9:08 ` Finn Thain
2010-10-04 10:18 ` Thorsten Glaser
2010-10-04 10:51 ` Finn Thain
2010-10-04 11:37 ` Thorsten Glaser
2010-10-04 12:45 ` Finn Thain
2010-10-16 17:12 ` Geert Uytterhoeven
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=alpine.OSX.2.00.1010041149030.285@localhost \
--to=fthain@telegraphics.com.au \
--cc=linux-m68k@vger.kernel.org \
--cc=tg@debian.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.