[coreboot] [PATCH] refactor SMI handler
Stefan Reinauer
stepan at coresystems.de
Sun Oct 25 15:14:06 CET 2009
Peter Stuge wrote:
> Myles Watson wrote:
>
>>> No, but it would make sense... What's the right place for this?
>>> include/uart8250.h ?
>>>
>> maybe include/pc80/serial.h or include/pc80/uart8250.h?
>>
>
> If either exists, use that. If neither exists I prefer the simpler
> serial.h.
>
include/uart8250.h is the only one that exists, hence my suggestion.
It occurs to me that we should have pc80/serial.c and lib/uart8250.c in the same directory. Maybe even the same file. They share a lot functionality.
Stefan
More information about the coreboot
mailing list