Re: [PATCH v5] Fonts: Adding all Terminus fronts from 12 to 32 in bold, and normal versions

[Thomas Zimmermann <tzimmermann@suse.de>]

Hi

Am 27.02.26 um 21:59 schrieb Helge Deller:
> Hi Pavel,
>
> please include the graphics mailing lists (fbdev, dri-devel).
>
> * Pavel Nikulin <pavel@noa-labs.com>:
>> This patch adds an option to compile-in all terminus fonts, and removed the old bold only terminus version
>>
>> These fonts were converted from Terminus psf files with the help of psftools and a script.
>>
>> This patch is non-intrusive, no options are enabled by default so most users won't notice a thing.
> Sure, but you add a whole lot of fonts.
> Why are both bold and non-bold fonts needed?
> Why do we need all in-kernel?
> They may be loaded after bootup via userspace too.
>
> What does other people think?

I'm strongly against adding more fonts to the kernel. Fonts can (and 
should) be loaded from user space.

AFAICT this patch is just about someone trying to get their favorite 
font merged. Doing that is not going to work out long term.

Best regards
Thomas

>
> Helge
>
>   
>> I am placing my changes under the GPL 2.0 just as source Terminus font.
>>
>> Signed-off-by: Pavel Nikulin <pavel@noa-labs.com>
>> ---
>>   V4 -> V5: Added changed font.h to the patch
>>
>>   include/linux/font.h       |   42 +-
>>   lib/fonts/Kconfig          |  142 +-
>>   lib/fonts/Makefile         |   15 +
>>   lib/fonts/font_ter10x18.c  | 6412 ++++++++----------------------------
>>   lib/fonts/font_ter10x18b.c | 1305 ++++++++
>>   lib/fonts/font_ter10x20.c  | 1305 ++++++++
>>   lib/fonts/font_ter10x20b.c | 1305 ++++++++
>>   lib/fonts/font_ter11x22.c  | 1561 +++++++++
>>   lib/fonts/font_ter11x22b.c | 1561 +++++++++
>>   lib/fonts/font_ter12x24.c  | 1561 +++++++++
>>   lib/fonts/font_ter12x24b.c | 1561 +++++++++
>>   lib/fonts/font_ter14x28.c  | 1817 ++++++++++
>>   lib/fonts/font_ter14x28b.c | 1817 ++++++++++
>>   lib/fonts/font_ter16x32.c  | 4107 +++++++++++------------
>>   lib/fonts/font_ter16x32b.c | 2073 ++++++++++++
>>   lib/fonts/font_ter6x12.c   |  537 +++
>>   lib/fonts/font_ter8x14.c   |  537 +++
>>   lib/fonts/font_ter8x14b.c  |  537 +++
>>   lib/fonts/font_ter8x16.c   |  537 +++
>>   lib/fonts/font_ter8x16b.c  |  537 +++
>>   lib/fonts/fonts.c          |   45 +
>>   21 files changed, 22124 insertions(+), 7190 deletions(-)
>>   create mode 100644 lib/fonts/font_ter10x18b.c
>>   create mode 100644 lib/fonts/font_ter10x20.c
>>   create mode 100644 lib/fonts/font_ter10x20b.c
>>   create mode 100644 lib/fonts/font_ter11x22.c
>>   create mode 100644 lib/fonts/font_ter11x22b.c
>>   create mode 100644 lib/fonts/font_ter12x24.c
>>   create mode 100644 lib/fonts/font_ter12x24b.c
>>   create mode 100644 lib/fonts/font_ter14x28.c
>>   create mode 100644 lib/fonts/font_ter14x28b.c
>>   create mode 100644 lib/fonts/font_ter16x32b.c
>>   create mode 100644 lib/fonts/font_ter6x12.c
>>   create mode 100644 lib/fonts/font_ter8x14.c
>>   create mode 100644 lib/fonts/font_ter8x14b.c
>>   create mode 100644 lib/fonts/font_ter8x16.c
>>   create mode 100644 lib/fonts/font_ter8x16b.c

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)