2017-11-28 08:33:34

by 冯锐

[permalink] [raw]
Subject: 答复: [PATCH v7,1/2] misc: rtsx: Move Realt ek Card Reader Driver to misc

> On Tue, 28 Nov 2017, [email protected] wrote:
> > Hi Greg.
> > The Patches have been tested basing on V4.15-rc1 successfully.
> > The sdcard function is working well.
>
> Not sure why you're a) top posting, b) doing so with an incompatible email
> client and c) sending this to Greg?
>
> There doesn't appear to be a Memstick Maintainer at the moment, which
> means we have all of the Maintainer Acks we need (Arnd Acked in Greg's place
> for Misc) to apply this patch.
>
I can't get any Memstick Maintainer by following command:

$ ./scripts/get_maintainer.pl drivers/memstick/
Jens Axboe <[email protected]> (commit_signer:3/6=50%)
Arnd Bergmann <[email protected]> (commit_signer:2/6=33%,authored:1/6=17%)
Andrew Morton <[email protected]> (commit_signer:1/6=17%)
Thomas Gleixner <[email protected]> (commit_signer:1/6=17%)
Kate Stewart <[email protected]> (commit_signer:1/6=17%)
Rui Feng <[email protected]> (authored:1/6=17%)
Greg Kroah-Hartman <[email protected]> (authored:1/6=17%)
[email protected] (open list)

I can see Arnd and Greg are also in the list, but do I need to send the patch to all people in the list?

> If no one objects, I plan to take this through the MFD tree and provide everyone
> else with a pull-request which they can do with as they see fit.
>
> > Tested-by: Perry Yuan <[email protected]>
>
> Thanks for testing.
>
> > keep up the good work
> > Perry  Yuan
> > Dell | Client Software Group | Firmware Principal Engineer
> >
> > -----Original Message-----
> > From: [email protected]
> > [mailto:[email protected]] On Behalf Of
> > [email protected]
> > Sent: Monday, November 27, 2017 3:35 PM
> > To: [email protected]; [email protected]; [email protected];
> > [email protected]
> > Cc: [email protected]; [email protected];
> > [email protected]; Rui Feng <[email protected]>
> > Subject: [PATCH v7,1/2] misc: rtsx: Move Realtek Card Reader Driver to
> > misc
> >
> > From: Rui Feng <[email protected]>
> >
> > Because Realtek card reader drivers are pcie and usb drivers, and they
> > bridge mmc subsystem and memstick subsystem, they are not mfd drivers.
> > Greg and Lee Jones had a discussion about where to put the drivers,
> > the result is that misc is a good place for them, so I move all files
> > to misc. If I don't move them to a right place, I can't add any patch
> > for this driver.
> >
> > Signed-off-by: Rui Feng <[email protected]>
> > Acked-by: Arnd Bergmann <[email protected]>
> > Acked-for-MFD-by: Lee Jones <[email protected]>
> > Reviewed-by: Daniel Bristot de Oliveira <[email protected]>
> > Acked-by: Ulf Hansson <[email protected]>
> >
> > v7 changes:
> > * update missing modification in memstic
> > * add Acked-by
> > ---
> > drivers/memstick/host/Kconfig | 4 ++--
> > drivers/memstick/host/rtsx_pci_ms.c | 2 +-
> > drivers/memstick/host/rtsx_usb_ms.c | 2 +-
> > drivers/mfd/Kconfig | 21 ---------------------
> > drivers/mfd/Makefile | 4 ----
> > drivers/misc/Kconfig | 5 +++++
> > drivers/misc/Makefile | 1 +
> > drivers/misc/cardreader/Kconfig | 20
> ++++++++++++++++++++
> > drivers/misc/cardreader/Makefile | 4 ++++
> > drivers/{mfd => misc/cardreader}/rtl8411.c | 2 +- drivers/{mfd =>
> > misc/cardreader}/rts5209.c | 2 +- drivers/{mfd =>
> > misc/cardreader}/rts5227.c | 2 +- drivers/{mfd =>
> > misc/cardreader}/rts5229.c | 2 +- drivers/{mfd =>
> > misc/cardreader}/rts5249.c | 3 +-- drivers/{mfd =>
> > misc/cardreader}/rtsx_pcr.c | 2 +- drivers/{mfd =>
> > misc/cardreader}/rtsx_pcr.h | 2 +- drivers/{mfd =>
> > misc/cardreader}/rtsx_usb.c | 2 +-
> > drivers/mmc/host/Kconfig | 4 ++--
> > drivers/mmc/host/rtsx_pci_sdmmc.c | 2 +-
> > drivers/mmc/host/rtsx_usb_sdmmc.c | 2 +-
> > include/linux/{mfd => }/rtsx_common.h | 0
> > include/linux/{mfd => }/rtsx_pci.h | 2 +-
> > include/linux/{mfd => }/rtsx_usb.h | 0
> > 23 files changed, 47 insertions(+), 43 deletions(-) create mode
> > 100644 drivers/misc/cardreader/Kconfig create mode 100644
> > drivers/misc/cardreader/Makefile rename drivers/{mfd =>
> > misc/cardreader}/rtl8411.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rts5209.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rts5227.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rts5229.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rts5249.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rtsx_pcr.c (99%) rename drivers/{mfd =>
> > misc/cardreader}/rtsx_pcr.h (99%) rename drivers/{mfd =>
> > misc/cardreader}/rtsx_usb.c (99%) rename include/linux/{mfd =>
> > }/rtsx_common.h (100%) rename include/linux/{mfd => }/rtsx_pci.h
> > (99%) rename include/linux/{mfd => }/rtsx_usb.h (100%)
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source
> software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
>
> ------Please consider the environment before printing this e-mail.
?&ן9???ۯ;??;????S??h?o4????~~?M4?M{_???1? ????v?v??:??:???f????m?l??ۣ?jנ???????????+?ƥRz