Subject: Re: com rumblings...
To: Garrett D'Amore <garrett_damore@tadpole.com>
From: Jason Thorpe <thorpej@shagadelic.org>
List: tech-kern
Date: 06/16/2006 09:21:01
On Jun 16, 2006, at 9:17 AM, Garrett D'Amore wrote:

> I'm trying to understand what you mean by this comment;  
> particularly in
> the context of this discussion.  Specifically, are you saying its a  
> bad
> idea to initialize a softc that isn't really a softc (as Tsutsui-san
> suggests), or that it is a bad idea to use some pseudo-softc structure
> (ala struct com_regs) like I've done?

The former.  It is bad practice, IMO, to create a softc that isn't  
really a softc.

> Personally, I find the idea of initializing a softc structure with a
> bunch of stuff in it that won't be used (and maybe isn't initialized)
> ugly -- and possibly error prone.  (Some assumes they are dealing  
> with a
> normal softc and prints sc_dev.dv_xname, for instance.  Is that
> initialized to a sane value?  Who knows?)

I agree.  It will get even uglier come the say when softc structures  
do not directly contain a "struct device".

> I think my approach is still pretty sane.

I haven't looked at the diffs yet, but in general I agree.

-- thorpej