[LinuxBIOS] [PATCH] Add initial serial output for Via vt82c686
Corey Osgood
corey_osgood at verizon.net
Sun Mar 18 04:04:18 CET 2007
Uwe Hermann wrote:
> Committed in r2571, thanks!
>
> On Sat, Mar 17, 2007 at 12:01:28AM -0400, Corey Osgood wrote:
>>> If it works (and if you don't spot further problems in the code) I'd say
>>> this
>>> can be committed. Please send an Acked-by if the code looks good to you.
>>>
>>>
>> Awesome! I was starting to get concerned that I'd done something that
>> only worked on my board. I've got no problem with your changes (although
>> I can't _find_ an io base), and the code works fine on my board.
>
> Yep, iobase is not used yet. I guess we should make these two lines
> dependant on the value of iobase?
>
> + vt82c686_sio_write(VT82C686_SP1, 0xfe); /* SP1: 0x3f8 */
> + vt82c686_sio_write(VT82C686_SP2, 0xbe); /* SP2: 0x2f8 */
>
>
Oops, my bad, that wasn't what I meant. But yeah, that will get
done...eventually.
>> die() in console/console.c does send a post code, but if I try to use
>> it, gcc gives an error about function type pointers that I haven't been
>> able to track down the cause of. The die() src/arch/i386/lib/console.c
>> doesn't send a post code, and that's the one I'm currently using, since
>> that's what whichever motherboard I based my code on was using. So,
>> either that console.c should get fixed up, or I should figure out why I
>> can't use the other one (or even both).
>
> Hm, strange. I'll have a look later. Shall we hardcode a post_code()
> call in addition of die() for now?
>
Can't do that either, for the exact same reason, post_code() isn't in
that particular console.c, it needs to be outb(). Why do we have two
different console.c's, anyways? Is it a romcc or cashe-as-ram thing?
-Corey
More information about the coreboot
mailing list