[SeaBIOS] [PATCH 7/6] sercon: use a single entry point

Kevin O'Connor kevin at koconnor.net
Mon Sep 11 17:20:14 CEST 2017


On Mon, Sep 11, 2017 at 01:18:15PM +0200, Gerd Hoffmann wrote:
> ---
>  src/config.h    |  1 -
>  src/misc.c      |  9 ---------
>  src/sercon.c    | 49 ++++++++++++++++++-------------------------------
>  src/romlayout.S |  5 +++--
>  4 files changed, 21 insertions(+), 43 deletions(-)
> 
> diff --git a/src/config.h b/src/config.h
> index baca029f0e..93c8dbc2d5 100644
> --- a/src/config.h
> +++ b/src/config.h
> @@ -76,7 +76,6 @@
>  #define DEBUG_ISR_08 20
>  #define DEBUG_ISR_09 9
>  #define DEBUG_ISR_0e 9
> -#define DEBUG_HDL_10 20
>  #define DEBUG_HDL_11 2
>  #define DEBUG_HDL_12 2
>  #define DEBUG_HDL_13 10
> diff --git a/src/misc.c b/src/misc.c
> index f4b656dd35..387a3482c9 100644
> --- a/src/misc.c
> +++ b/src/misc.c
> @@ -52,15 +52,6 @@ handle_05(struct bregs *regs)
>      debug_enter(regs, DEBUG_HDL_05);
>  }
>  
> -// INT 10h Video Support Service Entry Point
> -void VISIBLE16
> -handle_10(struct bregs *regs)
> -{
> -    debug_enter(regs, DEBUG_HDL_10);
> -    // don't do anything, since the VGA BIOS handles int10h requests
> -    sercon_10(regs);
> -}

Thanks.  With all the entry points hardened, I'm good with this
series.  (All my other comments are minor.)

-Kevin



More information about the SeaBIOS mailing list