linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [PATCH] Fonts: New Terminus large console font
       [not found] <20181126092115.754-1-amanoeladawod@gmail.com>
@ 2018-12-05 11:02 ` Greg KH
       [not found]   ` <20181205235637.1079-1-amanoeladawod@gmail.com>
  0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2018-12-05 11:02 UTC (permalink / raw)
  To: Amanoel Dawod; +Cc: akpm, linux-fbdev, linux-kernel

On Mon, Nov 26, 2018 at 04:21:15AM -0500, Amanoel Dawod wrote:
> This patch adds an option to compile-in a high resolution and large
> Terminus (ter16x32) bitmap console font for use with HiDPI and Retina screens.
> 
> The font was convereted from standard Terminus ter-i32b.psf (size 16x32)
> with the help of psftools and minor hand editing deleting useless characters.
> 
> This patch is non-intrusive, no options are enabled by default so most
> users won't notice a thing.
> 
> I am placing my changes under the GPL 2.0 just as source Terminus font.

Can you please run scripts/checkpatch.pl and fix up the issues that it
finds in this patch.  It's a lot:
	total: 1792 errors, 2058 warnings, 2127 lines checked

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] Fonts: New Terminus large console font
       [not found]   ` <20181205235637.1079-1-amanoeladawod@gmail.com>
@ 2018-12-06  0:42     ` Randy Dunlap
  2018-12-06 18:43       ` Amanoel Dawod
  0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2018-12-06  0:42 UTC (permalink / raw)
  To: Amanoel Dawod; +Cc: gregkh, akpm, linux-kernel, linux-fbdev

On 12/5/18 3:56 PM, Amanoel Dawod wrote:
> This patch adds an option to compile-in a high resolution
> and large Terminus (ter16x32) bitmap console font for use with
> HiDPI and Retina screens.
> 
> The font was convereted from standard Terminus ter-i32b.psf

               converted

Where does "ter-i32b.psf" come from?
I.e., where can I find it?

> (size 16x32) with the help of psftools and minor hand editing
> deleting useless characters.
> 
> This patch is non-intrusive, no options are enabled by default so most
> users won't notice a thing.
> 
> I am placing my changes under the GPL 2.0 just as source Terminus font.
> 
> Signed-off-by: Amanoel Dawod <amanoeladawod@gmail.com>
> ---


> +const struct font_desc font_ter_16x32 = {
> +	.idx	= TER16x32_IDX,
> +	.name	= "TER16x32",
> +	.width	= 16,
> +	.height = 32,
> +	.data	= fontdata_ter16x32,

Please update Documentation/fb/fbcon.txt with the addition of that font name
if that is applicable.

thanks,
-- 
~Randy

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] Fonts: New Terminus large console font
  2018-12-06  0:42     ` [PATCH v2] " Randy Dunlap
@ 2018-12-06 18:43       ` Amanoel Dawod
  0 siblings, 0 replies; 3+ messages in thread
From: Amanoel Dawod @ 2018-12-06 18:43 UTC (permalink / raw)
  To: rdunlap; +Cc: gregkh, akpm, linux-kernel, linux-fbdev

On Wed, Dec 5, 2018 at 7:42 PM Randy Dunlap <rdunlap@infradead.org> wrote:
>
>                converted
>
> Where does "ter-i32b.psf" come from?
> I.e., where can I find it?

The Unix/Linux Terminus sources are available for download
at SourceForge. Simply running "make" in source directory
will build the .psf font files.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-12-06 18:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20181126092115.754-1-amanoeladawod@gmail.com>
2018-12-05 11:02 ` [PATCH] Fonts: New Terminus large console font Greg KH
     [not found]   ` <20181205235637.1079-1-amanoeladawod@gmail.com>
2018-12-06  0:42     ` [PATCH v2] " Randy Dunlap
2018-12-06 18:43       ` Amanoel Dawod

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).