All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rodolfo Giometti <giometti@enneenne.com>
To: Greg KH <greg@kroah.com>
Cc: Andrew Morton <akpm@linux-foundation.org>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] Add c2 port support.
Date: Thu, 23 Oct 2008 09:39:57 +0200	[thread overview]
Message-ID: <20081023073957.GU4390@tekkaman> (raw)
In-Reply-To: <20081022210352.GA29754@kroah.com>

On Wed, Oct 22, 2008 at 02:03:52PM -0700, Greg KH wrote:

> > Why is that good?  We have hundreds and hundreds of drivers which use a
> > device node interface and they work OK.
> > 
> > sysfs was never intended for use as the sole interface between device
> > drivers and userspace.  Using it that way is somewhat unconventional. 
> > At the very least I do think it needs more justification and
> > explanation than we're being provded here.
> 
> For firmware download to devices, using sysfs is the "normal" interface
> these days, that is what the firmware class is for.
> 
> Which reminds me, Rodolfo, why didn't you just use that interface for
> your firmware downloading instead of rolling your own here?

Because I need also to know revision ID, device ID and other minor
info from the microcontroller. Also I can reset the microcontroller
when I need it without downloading any new firmware.

Ciao,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail: giometti@enneenne.com
Linux Device Driver                          giometti@linux.it
Embedded Systems                     phone:  +39 349 2432127
UNIX programming                     skype:  rodolfo.giometti

  reply	other threads:[~2008-10-23  7:40 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-22 13:47 c2port (Version 2): support for Silicon C2 Interface Rodolfo Giometti
2008-10-22 13:47 ` [PATCH 1/2] Add c2 port support Rodolfo Giometti
2008-10-22 13:47   ` [PATCH 2/2] c2port: add c2port support for Eurotech Duramar 2150 Rodolfo Giometti
2008-10-22 17:40     ` Greg KH
2008-10-22 17:40   ` [PATCH 1/2] Add c2 port support Greg KH
2008-10-22 18:11     ` Andrew Morton
2008-10-22 19:52       ` Rodolfo Giometti
2008-10-22 20:01         ` Andrew Morton
2008-10-22 20:48           ` Rodolfo Giometti
2008-10-22 20:58             ` Andrew Morton
2008-10-22 21:03               ` Greg KH
2008-10-23  7:39                 ` Rodolfo Giometti [this message]
2008-10-23 15:37                   ` Greg KH
2008-10-27  8:54                     ` Rodolfo Giometti
2008-10-27 16:14                       ` Greg KH
  -- strict thread matches above, loose matches on Subject: below --
2008-10-22 20:53 c2port (Version 3): support for Silicon C2 Interface Rodolfo Giometti
2008-10-22 20:53 ` [PATCH 1/2] Add c2 port support Rodolfo Giometti

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=20081023073957.GU4390@tekkaman \
    --to=giometti@enneenne.com \
    --cc=akpm@linux-foundation.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.