[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-embedded
Subject:    Re: [PATCH 5/5] KBUS source file
From:       Sam Ravnborg <sam () ravnborg ! org>
Date:       2011-03-01 5:42:32
Message-ID: 20110301054232.GA2331 () merkur ! ravnborg ! org
[Download RAW message or body]

On Mon, Feb 28, 2011 at 10:25:41PM +0000, Tony Ibbs wrote:
> 
> On 28 Feb 2011, at 19:23, Sam Ravnborg wrote:
> 
> > You are using /proc/* to something that is not process related.
> > At lkml you will be requested to use something else.
> 
> OK. That's mainly through ignorance of what I *should* be using
> instead (and perhaps poor reading of Linux Device Drivers). Do you
> have an opinion on where such information should be going?
I think sysfs is the better place.
But my experience in this ares is limited.

> >> If there is a better way for me to submit this, please let me know.
> > 
> > One way you would not like....
> > Let the first patches introduce core-functionality and add features
> > in later patches - touching the same file many times.
> 
> Hmm. That's very sensible. Assuming I can figure out a sensible way
> to do it (which I assume is what you expected me to object).
It is a lot of work when you already have something that works.
But sometimes it helps to do so. I have previously done a complete
rewrite in the process (becuase my initial version was bad).

	Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]