From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Woodhouse Subject: Re: [PATCH] console - Add configurable support for console charset translation Date: Wed, 04 Jun 2008 14:55:53 +0100 Message-ID: <1212587753.32207.86.camel@pmac.infradead.org> References: <48447615.5050806@am.sony.com> <1212500751.16924.322.camel@pmac.infradead.org> <4845DB46.3020308@am.sony.com> <20080604125500.0fce2ea4@core> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20080604125500.0fce2ea4@core> Sender: linux-embedded-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" To: Alan Cox Cc: Tim Bird , linux-tiny , linux-embedded , linux kernel On Wed, 2008-06-04 at 12:55 +0100, Alan Cox wrote: > Can we please get the ifdefs tided up before this goes in. > > For the moment this has a NAK from the tty maintainer but if the > ifdefs > turned went into headers where they belong and the code looked like > say > > tc = vc_translate(vc, c); > > with two versions of vc_translate (one being vc_translate(x) (x)) it > might be more reasonable. Good point. I did think about that originally, but then just completely forgot to process that hunk of the original patch altogether, which was another way of avoiding the ifdefs :) If I merge this incremental patch, does that address your objections? (Note that I've changed the expression to evaluate (c) only once, just because that's best practice in macro definitions. Wasn't really worth doing it for (vc) though.) diff --git a/drivers/char/vt.c b/drivers/char/vt.c index de52f99..18b7fb0 100644 --- a/drivers/char/vt.c +++ b/drivers/char/vt.c @@ -2208,11 +2208,7 @@ rescan_last_byte: c = 0xfffd; tc = c; } else { /* no utf or alternate charset mode */ -#ifdef CONFIG_CONSOLE_TRANSLATIONS - tc = vc->vc_translate[vc->vc_toggle_meta ? (c | 0x80) : c]; -#else - tc = c; -#endif + tc = vc_translate(vc, c); } param.c = tc; diff --git a/include/linux/vt_kern.h b/include/linux/vt_kern.h index 81ed155..929ee80 100644 --- a/include/linux/vt_kern.h +++ b/include/linux/vt_kern.h @@ -72,6 +72,9 @@ int con_set_default_unimap(struct vc_data *vc); void con_free_unimap(struct vc_data *vc); void con_protect_unimap(struct vc_data *vc, int rdonly); int con_copy_unimap(struct vc_data *dst_vc, struct vc_data *src_vc); + +#define vc_translate(vc, c) ((vc)->vc_translate[(c) | \ + (vc)->vc_toggle_meta ? 0x80 : 0]) #else #define con_set_trans_old(arg) (0) #define con_get_trans_old(arg) (-EINVAL) @@ -83,6 +86,8 @@ int con_copy_unimap(struct vc_data *dst_vc, struct vc_data *src_vc); #define con_copy_unimap(d, s) (0) #define con_get_unimap(vc, ct, uct, list) (-EINVAL) #define con_free_unimap(vc) do { ; } while (0) + +#define vc_translate(vc, c) (c) #endif /* vt.c */ -- dwmw2