From: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Maarten ter Huurne <maarten@treewalker.org>,
Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
Geert Uytterhoeven <geert@linux-m68k.org>,
Apelete Seketeli <apelete@seketeli.net>,
linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] fonts: Add 6x10 font
Date: Thu, 02 Oct 2014 12:40:37 +0000 [thread overview]
Message-ID: <20141002134037.4d437d7b@alan.etchedpixels.co.uk> (raw)
In-Reply-To: <542A7ADA.4010500@ti.com>
On Tue, 30 Sep 2014 12:41:46 +0300
Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> Hi,
>
> On 09/09/14 14:46, Maarten ter Huurne wrote:
> > This font is suitable for framebuffer consoles on devices with a
> > 320x240 screen, to get a reasonable number of characters (53x24) that
> > are still at a readable size.
> >
> > The font is derived from the existing 6x11 font, but gets 3 extra
> > lines without sacrificing readability. Also I redesigned a some glyhps
> > so they are more distinct and better fill the available space.
> >
> > Signed-off-by: Maarten ter Huurne <maarten@treewalker.org>
> > ---
> > include/linux/font.h | 4 +-
> > lib/fonts/Kconfig | 9 +
> > lib/fonts/Makefile | 1 +
> > lib/fonts/font_6x10.c | 3086 +++++++++++++++++++++++++++++++++++++++++++++++++
> > lib/fonts/fonts.c | 4 +
> > 5 files changed, 3103 insertions(+), 1 deletion(-)
> > create mode 100644 lib/fonts/font_6x10.c
>
> I think the patch is fine, but there hasn't been any comments or acks
> for this. I'm a bit reluctant to add this much data to the kernel if
> there are no users (except you) for this.
>
> So I'd like to hear at least one "yeah, I could maybe use that font"
> from someone =).
It only going to get linked if someone asks for it so that seems ok to
me. We support a variety of panels and the like with 320x240 displays and
ucLinux platforms so IMHO it makes sense to have such a font available.
I am planning to use the font - just ermm.. not in Linux ;)
Alan
next prev parent reply other threads:[~2014-10-02 12:40 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-09 11:46 [PATCH] fonts: Add 6x10 font Maarten ter Huurne
2014-09-30 9:41 ` Tomi Valkeinen
2014-10-02 12:40 ` One Thousand Gnomes [this message]
2014-10-08 7:25 ` Maarten ter Huurne
2014-10-09 9:02 ` Tomi Valkeinen
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=20141002134037.4d437d7b@alan.etchedpixels.co.uk \
--to=gnomes@lxorguk.ukuu.org.uk \
--cc=apelete@seketeli.net \
--cc=geert@linux-m68k.org \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten@treewalker.org \
--cc=plagnioj@jcrosoft.com \
--cc=tomi.valkeinen@ti.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;
as well as URLs for NNTP newsgroup(s).