From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: edwin_rong <edwin_rong@realsil.com.cn>
Cc: "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
王炜 <wei_wang@realsil.com.cn>, "Greg KH" <greg@kroah.com>
Subject: Re: Is it possible for Realtek card reader driver to reside in SCSIsubsystem?
Date: Mon, 23 Apr 2012 09:11:19 +0100 [thread overview]
Message-ID: <1335168679.3051.9.camel@dabdike.lan> (raw)
In-Reply-To: <4F950C0B.10804@realsil.com.cn>
[updated Greg to non-SUSE address]
On Mon, 2012-04-23 at 16:00 +0800, edwin_rong wrote:
> On 04/23/2012 03:39 PM, James Bottomley wrote:
> > On Mon, 2012-04-23 at 11:49 +0800, edwin_rong wrote:
> >> Dear James and all :
> >>
> >> Sorry to disturb you again!
> >>
> >> I'm an software engineer of Realtek corporation, responsible for writing
> >> driver for Realtek Card Reader chips.
> >>
> >> Our device supports SD/MMC/MS/MSpro/xD series of cards, etc., which is
> >> implemented as an SCSI device in our driver,
> >> and now our driver rts_pstor is under staging folder of linux kernel, so
> >> I want to know whether it is possible to move it out of staging folder,
> >> and reside in SCSI subsystem?
> >>
> >> I also know that both "mmc" and "memstick" subsystem exist in kernel
> >> now, but our device is a composition of these types of cards,
> >> so it seems not suitable for keeping our driver there.
> >>
> >> All replies are appreciated.
> > The general rule is that if the device itself speaks SCSI ... as in
> > either the firmware or the underlying disk does, then you should be
> > using SCSI (This doesn't mean you have to have an actual SCSI device
> > anywhere ... lots of USB devices are some wierd flash or IDE device
> > fronted by a chip that does SCSI<->whatever translation [usually
> > badly]). Conversely if you would be writing your SCSI command emulation
> > in the driver, don't ... you should be using another subsystem.
> >
> > James
> >
> Dear James,
>
> Got it, thanks for your response, sincerely.
>
> As to my case, which subsystem do you think is fit for our driver to
> stay, could you give me some suggestions?
Well, you said it's a combination of mmc and memstick. If it can be
bound as two separate drivers, I'd say one in each. If there's magic
that has to be done in the binding (as in you need a single bind driver
that activates each component), I'd say be guided by the maintainers of
those components. My instinct would be to put it in one and use the
other via Kconfig, but whatever they find most appropriate.
James
next prev parent reply other threads:[~2012-04-23 8:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-23 3:49 Is it possible for Realtek card reader driver to reside in SCSI subsystem? edwin_rong
2012-04-23 7:39 ` James Bottomley
2012-04-23 8:00 ` Is it possible for Realtek card reader driver to reside in SCSIsubsystem? edwin_rong
2012-04-23 8:11 ` James Bottomley [this message]
2012-04-23 8:24 ` Is it possible for Realtek card reader driver to reside inSCSIsubsystem? edwin_rong
2012-04-23 15:48 ` Greg KH
2012-04-24 1:10 ` Is it possible for Realtek card reader driver to resideinSCSIsubsystem? edwin_rong
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=1335168679.3051.9.camel@dabdike.lan \
--to=james.bottomley@hansenpartnership.com \
--cc=edwin_rong@realsil.com.cn \
--cc=greg@kroah.com \
--cc=linux-scsi@vger.kernel.org \
--cc=wei_wang@realsil.com.cn \
/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