All of lore.kernel.org
 help / color / mirror / Atom feed
From: wwang <wei_wang@realsil.com.cn>
To: Borislav Petkov <bp@alien8.de>, Chris Ball <cjb@laptop.org>,
	gregkh@linuxfoundation.org,
	Andrew Morton <akpm@linux-foundation.org>,
	Phil Turmel <philip@turmel.org>,
	sameo@linux.intel.com, devel@linuxdriverproject.org,
	linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org,
	arnd@arndb.de, oakad@yahoo.com
Subject: Re: [PATCH v6 0/3] Add modules to support realtek PCIE card reader
Date: Sat, 6 Oct 2012 15:23:56 +0800	[thread overview]
Message-ID: <506FDC8C.3030707@realsil.com.cn> (raw)
In-Reply-To: <20120929152942.GA11412@liondog.tnic>

于 2012年09月29日 23:29, Borislav Petkov 写道:
> On Sat, Sep 29, 2012 at 09:41:17AM -0400, Chris Ball wrote:
>> On Sat, Sep 29 2012, Borislav Petkov wrote:
>>> So, I think that this driver has seen a bunch of testing and reviewing
>>> and we might start considering upstreaming it. There are a bunch of
>>> laptops integrating those readers and maybe a single three-month-ish -rc
>>> period of testing wouldn't hurt before it appears upstream.
>>>
>>> Now, AFAICT, there is another driver in staging which supports the same
>>> or a subset of the cardreaders this driver does so I don't know what
>>> Realsil want to do with that. Wei?
>>>
>>> And since this driver touches multiple drivers/{mfd,mmc,memstick}
>>> subdirs, maybe akpm would like to merge it? At least I haven't seen any
>>> other maintainers complaining about its design or wanting to take it so
>> I think you've missed the mails -- Alex Dubov has acked the memstick
>> portion, Samuel Ortiz has said he'll push the MFD driver for 3.7,
>> and I've pushed the MMC driver into mmc-next for 3.7.  So, I think
>> we should merge it.  I don't think we need akpm, since there are no
>> build-time dependencies involved.
> Great, all is taken care of then.
>
> Only the staging driver remains, I believe the CONFIG_RTS_PSTOR one. Wei?
>
> Thanks.
>
Hi all:

Sorry for my late reply. I'm on my vacation these days.

At first I want to explain the relationship between the staging driver 
"rts_pstor" with this one here.
Initially, we tried to build a SCSI driver for Realtek PCI-E card 
reader, which has the same architecture and code base with our Windows 
driver. And I pushed rts_pstor to staging tree last year. But Realtek 
PCI-E card reader is not a real SCSI device, which doesn't  understand 
SCSI command at all. So SCSI subsystem won't accept rts_pstor, and I 
can't find a proper subsystem to push it to.  That's why I create this 
series of drivers, which are more compatible with the whole Linux 
architecture.
We are still maintaining the SCSI driver for Realtek card reader, and 
will release the latest source code in the Github in the future.  But 
maybe we won't push it to the staging tree any more.

BR,
Wei WANG


WARNING: multiple messages have this Message-ID (diff)
From: wwang <wei_wang@realsil.com.cn>
To: Borislav Petkov <bp@alien8.de>, Chris Ball <cjb@laptop.org>,
	<gregkh@linuxfoundation.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Phil Turmel <philip@turmel.org>, <sameo@linux.intel.com>,
	<devel@linuxdriverproject.org>, <linux-kernel@vger.kernel.org>,
	<linux-mmc@vger.kernel.org>, <arnd@arndb.de>, <oakad@yahoo.com>
Subject: Re: [PATCH v6 0/3] Add modules to support realtek PCIE card reader
Date: Sat, 6 Oct 2012 15:23:56 +0800	[thread overview]
Message-ID: <506FDC8C.3030707@realsil.com.cn> (raw)
In-Reply-To: <20120929152942.GA11412@liondog.tnic>

于 2012年09月29日 23:29, Borislav Petkov 写道:
> On Sat, Sep 29, 2012 at 09:41:17AM -0400, Chris Ball wrote:
>> On Sat, Sep 29 2012, Borislav Petkov wrote:
>>> So, I think that this driver has seen a bunch of testing and reviewing
>>> and we might start considering upstreaming it. There are a bunch of
>>> laptops integrating those readers and maybe a single three-month-ish -rc
>>> period of testing wouldn't hurt before it appears upstream.
>>>
>>> Now, AFAICT, there is another driver in staging which supports the same
>>> or a subset of the cardreaders this driver does so I don't know what
>>> Realsil want to do with that. Wei?
>>>
>>> And since this driver touches multiple drivers/{mfd,mmc,memstick}
>>> subdirs, maybe akpm would like to merge it? At least I haven't seen any
>>> other maintainers complaining about its design or wanting to take it so
>> I think you've missed the mails -- Alex Dubov has acked the memstick
>> portion, Samuel Ortiz has said he'll push the MFD driver for 3.7,
>> and I've pushed the MMC driver into mmc-next for 3.7.  So, I think
>> we should merge it.  I don't think we need akpm, since there are no
>> build-time dependencies involved.
> Great, all is taken care of then.
>
> Only the staging driver remains, I believe the CONFIG_RTS_PSTOR one. Wei?
>
> Thanks.
>
Hi all:

Sorry for my late reply. I'm on my vacation these days.

At first I want to explain the relationship between the staging driver 
"rts_pstor" with this one here.
Initially, we tried to build a SCSI driver for Realtek PCI-E card 
reader, which has the same architecture and code base with our Windows 
driver. And I pushed rts_pstor to staging tree last year. But Realtek 
PCI-E card reader is not a real SCSI device, which doesn't  understand 
SCSI command at all. So SCSI subsystem won't accept rts_pstor, and I 
can't find a proper subsystem to push it to.  That's why I create this 
series of drivers, which are more compatible with the whole Linux 
architecture.
We are still maintaining the SCSI driver for Realtek card reader, and 
will release the latest source code in the Github in the future.  But 
maybe we won't push it to the staging tree any more.

BR,
Wei WANG


  reply	other threads:[~2012-10-06  7:24 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-11  4:54 [PATCH v6 0/3] Add modules to support realtek PCIE card reader wei_wang
2012-09-11  4:54 ` wei_wang
2012-09-11  4:54 ` [PATCH v6 1/3] drivers/mfd: Add realtek pcie card reader driver wei_wang
2012-09-11  4:54   ` wei_wang
2012-09-30 22:49   ` Samuel Ortiz
2012-10-06  9:05     ` wwang
2012-09-11  4:54 ` [PATCH v6 2/3] drivers/mmc: Add realtek pcie sdmmc host driver wei_wang
2012-09-11  4:54   ` wei_wang
2012-09-11  4:54 ` [PATCH v6 3/3] drivers/memstick: Add realtek pcie memstick " wei_wang
2012-09-11  4:54   ` wei_wang
2012-09-12  6:28   ` Alex Dubov
2012-09-12  6:28     ` Alex Dubov
2012-09-29  1:00 ` [PATCH v6 0/3] Add modules to support realtek PCIE card reader Phil Turmel
2012-09-29 11:07   ` Borislav Petkov
2012-09-29 11:07     ` Borislav Petkov
2012-09-29 13:41     ` Chris Ball
2012-09-29 15:29       ` Borislav Petkov
2012-10-06  7:23         ` wwang [this message]
2012-10-06  7:23           ` wwang
2012-10-20  3:46           ` Dan Carpenter
2012-10-20  3:46             ` Dan Carpenter
2012-10-25 18:50             ` Greg KH
2012-10-26  1:10               ` wwang
2012-10-26  2:45                 ` Greg KH
2012-10-26  2:51                   ` wwang
2012-10-26 16:04                     ` Greg KH

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=506FDC8C.3030707@realsil.com.cn \
    --to=wei_wang@realsil.com.cn \
    --cc=akpm@linux-foundation.org \
    --cc=arnd@arndb.de \
    --cc=bp@alien8.de \
    --cc=cjb@laptop.org \
    --cc=devel@linuxdriverproject.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=oakad@yahoo.com \
    --cc=philip@turmel.org \
    --cc=sameo@linux.intel.com \
    /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.