Re: RFC: cleaning up j720ssp.c

"Valeriy E. Ushakov" <[email protected]>
Newsgroups gmane.os.netbsd.ports.hpcarm
Message-ID <[email protected]>
On Tue, Jan 17, 2006 at 15:30:21 +0100, Peter Postma wrote:

> j720ssp.c contains drivers for the touch-panel, keyboard, lcd control
> and battery status. I'd like to split this into multiple files, because
> it's a bit messy now and having each driver in a separate file is much
> cleaner.

Absolutely.


> - j720ssp.c 	- SSP interface (parent to the above drivers)
> 
> The last file might possibly be moved to arm/sa11x0/sa11x0_ssp.c
> (plus some general functions to access it?) because the SSP seems to
> be part of the SA-1110 according to the Intel docs.

You can always start with j720ssp.c and move it to arm/sa1100 later.


> Also, we might rename j720 to j72x or even j7xx because the 720 is
> not the only supported model.

I'm not sure it's necessary.  Everyone knows(tm) :) that Jornada 7xx
series is 720 plus some trivial variations thereof.  So it doesn't buy
much, but disrupts custom configs with device renaming.


SY, Uwe
-- 
[email protected]                         |       Zu Grunde kommen
http://snark.ptc.spbu.ru/~uwe/          |       Ist zu Grunde gehen
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.