Re: CVS commit: src/sys/dev/ic

[Michael <macallan%netbsd.org@localhost>]

Hello,

On Thu, 12 Jan 2012 18:19:26 +0900
Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost> wrote:

> > Module Name:        src
> > Committed By:       macallan
> > Date:               Wed Jan 11 20:41:28 UTC 2012
> > 
> > Modified Files:
> >     src/sys/dev/ic: igsfb.c vga.c vga_raster.c
> > 
> > Log Message:
> > wsfont_matches() and wsfont_find() take an extra parameter now
> 
> Isn't it possible to provide compatible wrapper functions (without
> an extra parameter) rather than changing all existing API callers?
> 
> So that you don't have to bump kernel version for modular(7) and
> all third parties (including ongoing porting efforts) don't
> have to fix their drivers on updating code base.

That would be trivial to do. The reason I didn't do it right away was that I 
had no idea there were so many drivers that call wsfont_find() - at least that 
shouldn't be necessary anymore in most cases, now that rasops_init() at least 
tries to pick a sensible font for the screen / terminal size requested. In vga 
it makes sense ( in order to get an 8 pixels wide font and nothing else ), in 
igsfb not so much.
Also, I'm not aware of any modular wsdisplay drivers.

have fun
Michael