[coreboot] libpayload: make cursor position available to payloads

Jordan Crouse jordan.crouse at amd.com
Mon Aug 11 17:02:39 CEST 2008


On 11/08/08 15:48 +0200, Stefan Reinauer wrote:
> see patch...
> 
> -- 
> coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
>       Tel.: +49 761 7668825 • Fax: +49 761 7664613
> Email: info at coresystems.dehttp://www.coresystems.de/
> Registergericht: Amtsgericht Freiburg • HRB 7656
> Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866
> 

> make cursor position available to payloads
> 
> Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
> 
> 
> Index: include/libpayload.h
> ===================================================================
> --- include/libpayload.h	(revision 425)
> +++ include/libpayload.h	(working copy)
> @@ -123,6 +123,7 @@
>  void video_console_putc(u8 row, u8 col, unsigned int ch);
>  void video_console_clear(void);
>  void video_console_cursor_enable(int state);
> +void video_console_get_cursor(unsigned int *x, unsigned int *y, unsigned int *en);
>  
>  /* drivers/option.c */
>  int get_option(void *dest, char *name);
> Index: drivers/video/video.c
> ===================================================================
> --- drivers/video/video.c	(revision 425)
> +++ drivers/video/video.c	(working copy)
> @@ -140,6 +140,12 @@
>  	video_console_fixup_cursor();
>  }
>  
> +void video_console_get_cursor(unsigned int *x, unsigned int *y, unsigned int *en)
> +{
> +	if (console->get_cursor)
> +		console->get_cursor(&cursorx, &cursory, &cursor_enabled);

Oops - I think you forgot to rename some variables here.
Also, you might want to set the return values to zero if console->get_cursor
doesn't exist.

Jordan


-- 
Jordan Crouse
Systems Software Development Engineer 
Advanced Micro Devices, Inc.





More information about the coreboot mailing list