On Wed, 19 Nov 2014 11:14:24 +0100 Ulf Hansson <ulf.hans...@linaro.org> wrote:
> On 8 November 2014 00:52, NeilBrown <ne...@suse.de> wrote: > > omap_hsmmc currently duplicates some work that can be done for > > it by common code, and consequently does not benefit from extra > > functionality in that common code. > > > > In particular, mmc_of_parse and the slot-gpio library are not used. > > > > This set of patches allows omap_hsmmc to use that common > > functionality, and benefit from any extra devicetree parsing > > that it performs. > > > > The one awkward part of this change is that omap_hsmmc has an > > interrupt handler for 'card detect' which does more than the > > common code. > > I see three options: > > 1 - move that functionality into common code > > 2 - discard that functionality > > 3 - allow the common code to be configured to use a device-specific > > card detect interrupt. > > > > This series implements '3'. I suspect a mix of '1' and '2' would > > be a better choice but I know no of the history or justification > > for those differences. > > > > My preference would be for this series to be applied (if there are > > no other issues) and if there are opinions about effecting '1' or '2', > > they can be done with subsequent patches. > > > > Thanks, > > NeilBrown > > > > I like the looks of this patchset, but it needs a rebase. Thanks. What should I rebase against? Is 3.18-rc sufficient or is there some other tree I should work against? Thanks, NeilBrown > > Kind regards > Uffe > > > > > --- > > > > NeilBrown (3): > > mmc: omap_hsmmc: remove prepare/complete system suspend support. > > mmc: omap_hsmmc: use slot-gpio library for gpio support. > > mmc: omap_hsmmc: use mmc_of_parse to parse common mmc configuration. > > > > > > drivers/mmc/core/slot-gpio.c | 21 ++++ > > drivers/mmc/host/omap_hsmmc.c | 158 > > +++++--------------------------- > > include/linux/mmc/slot-gpio.h | 2 > > include/linux/platform_data/mmc-omap.h | 4 - > > 4 files changed, 47 insertions(+), 138 deletions(-) > > > > -- > > Signature > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
pgpxkCv4riEiB.pgp
Description: OpenPGP digital signature
