public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Evgeniy Polyakov <zbr@ioremap.net>
Cc: Markus Franke <markus.franke@s2002.tu-chemnitz.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] w1: Add 1-wire slave device driver for DS28E04-100
Date: Wed, 9 May 2012 20:43:48 -0700	[thread overview]
Message-ID: <20120510034348.GB18484@kroah.com> (raw)
In-Reply-To: <20120510004350.GA8362@ioremap.net>

On Thu, May 10, 2012 at 04:43:50AM +0400, Evgeniy Polyakov wrote:
> On Wed, May 09, 2012 at 05:01:26PM -0700, Greg KH (greg@kroah.com) wrote:
> > Binary sysfs files should be "pass through" only, the kernel should not
> > touch the data involved in them at all, it is a pipe directly from the
> > kernel to userspace for binary blob data, like firmware images.  You
> > should never do any processing of any binary file data at all in the
> > kernel.
> 
> And if some of the data should be somehow changed, what interfact should
> be used? Also, Markus, does DS28E04 change written/read data when doing
> IO?
> 
> I must admit, I never heared that binary sysfs files have to follow this
> constraint.

For some reason I thought I had documented it years ago, but in digging
through the Documentation directory, I don't see it anywhere, sorry
about that.

I'll add it to my ever-growing TODO list...

thanks,

greg k-h

  reply	other threads:[~2012-05-10  3:43 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-30  2:13 [PATCH] w1: Add 1-wire slave device driver for DS28E04-100 Greg KH
2012-05-02 20:12 ` Markus Franke
2012-05-02 20:21   ` Greg KH
2012-05-03 18:00     ` Evgeniy Polyakov
2012-05-09 20:37       ` Markus Franke
2012-05-09 22:06         ` Greg KH
2012-05-09 22:16           ` Markus Franke
2012-05-09 22:24             ` Greg KH
2012-05-09 22:37               ` Markus Franke
2012-05-09 23:57                 ` Evgeniy Polyakov
2012-05-10  0:01                   ` Greg KH
2012-05-10  0:43                     ` Evgeniy Polyakov
2012-05-10  3:43                       ` Greg KH [this message]
2012-05-10  4:55                       ` Markus Franke
2012-05-10 15:16                         ` Greg KH
2012-05-10 22:57                           ` Markus Franke
2012-05-10 23:04                             ` Greg KH
2012-05-11  6:13                               ` Markus Franke
2012-05-15  1:21                                 ` Evgeniy Polyakov
2012-05-25 22:45                               ` Markus Franke
2012-05-10 23:22                             ` Andrew Morton
2012-05-11  6:15                               ` Markus Franke

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120510034348.GB18484@kroah.com \
    --to=greg@kroah.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=markus.franke@s2002.tu-chemnitz.de \
    --cc=zbr@ioremap.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox