Re: [PATCH] fonts: Add 6x10 font

[Maarten ter Huurne]

On Tuesday 30 September 2014 12:41:46 Tomi Valkeinen 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@xxxxxxxxxxxxxx>
> > ---
> > 
> >  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.

Are you counting end users, developers or projects? We've got at least 
several thousand end users in OpenDingux for Dingoo A320 and GCW Zero 
combined. I could ask a few fellow developers for signed-off-by lines, if 
that helps. But it's all from the same project.

I think the font fills a space that is currently unoccupied: the 4x6 font is 
space efficient but barely readable, the 8x8 font gets only 40 columns on 
320x240 and the 6x11 font doesn't use the vertical space efficiently.

Bye,
		Maarten

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/