video console: allow font size configuration at runtime

Allow font size configuration at runtime for console_simple.c
driver. This needed for unit testing different fonts.

Configuring is done by `font` command, also used for font
selection in true type console.

Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 2caa4af71cbbd68b03582dfd7aabbea974d6118b..a3512836c1a3fd63b308a121f5eea29f51cde0ec 100644 (file)
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -2226,6 +2226,14 @@ config CMD_VIDCONSOLE
          The name 'lcdputs' is a bit of a misnomer, but so named because the
          video device is often an LCD.
 
+config CMD_SELECT_FONT
+       bool "select font size"
+       depends on VIDEO
+       default n
+       help
+         Enabling this will provide 'font' command.
+         Allows font selection at runtime.
+
 endmenu
 
 source "cmd/ti/Kconfig"

diff --git a/cmd/Makefile b/cmd/Makefile
index 36d2daf22a1b405cf480e51abfeee2451c77c2cb..2d8bb4fc0529de7f58e8c619a42934d7b98a605b 100644 (file)
--- a/cmd/Makefile
+++ b/cmd/Makefile
@@ -78,7 +78,7 @@ obj-$(CONFIG_CMD_EXT2) += ext2.o
 obj-$(CONFIG_CMD_FAT) += fat.o
 obj-$(CONFIG_CMD_FDT) += fdt.o
 obj-$(CONFIG_CMD_SQUASHFS) += sqfs.o
-obj-$(CONFIG_CONSOLE_TRUETYPE) += font.o
+obj-$(CONFIG_CMD_SELECT_FONT) += font.o
 obj-$(CONFIG_CMD_FLASH) += flash.o
 obj-$(CONFIG_CMD_FPGA) += fpga.o
 obj-$(CONFIG_CMD_FPGAD) += fpgad.o

diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index ce97eb47270779b0f4decd0fa84497fc35dfa8c4..e1bcc89b30ac9dd44fed101b3970411e265119fd 100644 (file)
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -165,6 +165,7 @@ config CONSOLE_ROTATION
 
 config CONSOLE_TRUETYPE
        bool "Support a console that uses TrueType fonts"
+       select CMD_SELECT_FONT
        help
          TrueTrype fonts can provide outline-drawing capability rather than
          needing to provide a bitmap for each font and size that is needed.

diff --git a/drivers/video/console_core.c b/drivers/video/console_core.c
index d019b985b4876c4ca16da5f6927eb9cd78c50186..d4f79c656a9b1490d3a43922dab345f765477585 100644 (file)
--- a/drivers/video/console_core.c
+++ b/drivers/video/console_core.c
@@ -180,3 +180,33 @@ int console_probe(struct udevice *dev)
 {
        return console_set_font(dev, fonts);
 }
+
+const char *console_simple_get_font_size(struct udevice *dev, uint *sizep)
+{
+       struct console_simple_priv *priv = dev_get_priv(dev);
+
+       *sizep = priv->fontdata->width;
+
+       return priv->fontdata->name;
+}
+
+int console_simple_get_font(struct udevice *dev, int seq, struct vidfont_info *info)
+{
+       info->name = fonts[seq].name;
+
+       return 0;
+}
+
+int console_simple_select_font(struct udevice *dev, const char *name, uint size)
+{
+       struct video_fontdata *font;
+
+       for (font = fonts; font->name; font++) {
+               if (!strcmp(name, font->name)) {
+                       console_set_font(dev, font);
+                       return 0;
+               }
+       };
+       printf("no such font: %s, make sure it's name has <width>x<height> format\n", name);
+       return -ENOENT;
+}

diff --git a/drivers/video/console_normal.c b/drivers/video/console_normal.c
index 03e859898c39de5d466f4f6a9f41056c7627f40e..413c7abee9e12a42e95f288c22fdd479a108ff94 100644 (file)
--- a/drivers/video/console_normal.c
+++ b/drivers/video/console_normal.c
@@ -101,6 +101,9 @@ struct vidconsole_ops console_ops = {
        .putc_xy        = console_putc_xy,
        .move_rows      = console_move_rows,
        .set_row        = console_set_row,
+       .get_font_size  = console_simple_get_font_size,
+       .get_font       = console_simple_get_font,
+       .select_font    = console_simple_select_font,
 };
 
 U_BOOT_DRIVER(vidconsole_normal) = {

diff --git a/drivers/video/console_rotate.c b/drivers/video/console_rotate.c
index b924bc345948339ab8bd7a67a200f9aaa1969c7e..65358a1c6e7436b07f2bbb998aad071a4069fd34 100644 (file)
--- a/drivers/video/console_rotate.c
+++ b/drivers/video/console_rotate.c
@@ -262,18 +262,27 @@ struct vidconsole_ops console_ops_1 = {
        .putc_xy        = console_putc_xy_1,
        .move_rows      = console_move_rows_1,
        .set_row        = console_set_row_1,
+       .get_font_size  = console_simple_get_font_size,
+       .get_font       = console_simple_get_font,
+       .select_font    = console_simple_select_font,
 };
 
 struct vidconsole_ops console_ops_2 = {
        .putc_xy        = console_putc_xy_2,
        .move_rows      = console_move_rows_2,
        .set_row        = console_set_row_2,
+       .get_font_size  = console_simple_get_font_size,
+       .get_font       = console_simple_get_font,
+       .select_font    = console_simple_select_font,
 };
 
 struct vidconsole_ops console_ops_3 = {
        .putc_xy        = console_putc_xy_3,
        .move_rows      = console_move_rows_3,
        .set_row        = console_set_row_3,
+       .get_font_size  = console_simple_get_font_size,
+       .get_font       = console_simple_get_font,
+       .select_font    = console_simple_select_font,
 };
 
 U_BOOT_DRIVER(vidconsole_1) = {

diff --git a/drivers/video/vidconsole_internal.h b/drivers/video/vidconsole_internal.h
index 99d3c876d72a2d8bd759d7e8a7c44cefafcf8e3d..c41edd45249a930edf1f8071fd355fe026d42545 100644 (file)
--- a/drivers/video/vidconsole_internal.h
+++ b/drivers/video/vidconsole_internal.h
@@ -100,3 +100,21 @@ int fill_char_horizontally(uchar *pfont, void **line, struct video_priv *vid_pri
  * @returns 0, if success, or else error code.
  */
 int console_probe(struct udevice *dev);
+
+/**
+ * Internal function to be used in as ops.
+ * See details in video_console.h get_font_size function
+ **/
+const char *console_simple_get_font_size(struct udevice *dev, uint *sizep);
+
+/**
+ * Internal function to be used in as ops.
+ * See details in video_console.h get_font function
+ **/
+int console_simple_get_font(struct udevice *dev, int seq, struct vidfont_info *info);
+
+/**
+ * Internal function to be used in as ops.
+ * See details in video_console.h select_font function
+ **/
+int console_simple_select_font(struct udevice *dev, const char *name, uint size);