Sorry, I forgot to send a reply.
(B
(BOn Monday 21 February 2005 13:38, Omniflux wrote:
(B> Here is a patch to add serial support to x86.
(B
(BGreat.
(B
(B> If this is headed in the right direction, and likely to be accepted
(B> for merging with a little more work, I'll keep working on it.
(B
(BI think this is a good way. But some problems must be solved:
(B
(B- Naming issues. Since you copied the code from GRUB Legacy, some names 
(Bare not appropriate in GRUB 2. For example, the global function 
(Bdeclared in terminfo.h do not use the prefix 'grub_' at all.
(B
(B- Module separation. I think it would be better to have a separate 
(Bmodule for terminfo, because we might be able to use it for other 
(Bterminals as well as serial console, say, parallel console.
(B
(B- ChangeLog. Please write ChangeLog entries for your changes.
(B
(BBTW, do you have any suggestion about the so-called "dumb terminal"? 
(BBecause supporting dumb terminal is horrible, I hesitate to support it 
(Bin GRUB 2, if there is no clean way to do it.
(B
(BOkuji
(B
(B
(B_______________________________________________
(BGrub-devel mailing list
([email protected]
(Bhttp://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to