From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Pierre Ossman <pierre@ossman.eu>
Cc: "Michał Mirosław" <mirq@rere.qmqm.pl>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH/RFC 2.6.29.2] Driver for CB710/720 memory card reader (MMC part) - v4
Date: Fri, 22 May 2009 21:04:06 +0200 [thread overview]
Message-ID: <200905222104.07148.bzolnier@gmail.com> (raw)
In-Reply-To: <20090522132755.144efcca@mjolnir.ossman.eu>
On Friday 22 May 2009 13:27:55 Pierre Ossman wrote:
> On Fri, 8 May 2009 17:16:22 +0200
> Michał Mirosław <mirq@rere.qmqm.pl> wrote:
>
> > Here is next version of the CB710 SD/MMC reader driver. As in previous
> > versions the code is divided in two parts - virtual 'bus' driver that
> > handles PCI device and registers three new devices one per card reader
> > type. The other device handles SD/MMC part of the reader.
> >
> > Key changes from v3:
> > - kerneldoc for sg_mapping_iterator extension
> > - free/restore IRQ during suspend/resume
> > - protect slot->irq_handler by spinlock instead of RCU
> > - remove verify_serialization()
> > - Kconfig
> > - complete patch for linux kernel 2.6.29.2
> >
>
> Looks good to me. I just want you to add a MAINTAINERS entry so that
> people know who to contact if they have problems with this driver.
>
> > My patches for sg_mapping_iterator extension went to /dev/null, so I have
> > incorporated them to the cb710 core driver module for now. Probably someone
> > with more karma points should look at them (sgbuf2.{c,h}) and push further
> > if they are of any value to other driver writers.
I like the general idea and I think that this could serve as a good starting
point into making generic PIO handling library (which has been on my personal
TODO of things to look into for a long time).
Michał, I'll try ping you back about it once I'm done with 2.6.31 IDE stuff..
> Give them to Andrew Morton. He's good at finding the right people and
> making sure they notice. :)
Hehe, please keep me on cc:.
Thanks.
Bart
prev parent reply other threads:[~2009-05-22 19:06 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-07 21:52 MMC host driver requirements Michał Mirosław
2008-09-09 7:18 ` Pierre Ossman
2008-09-09 9:06 ` Michał Mirosław
2008-09-11 20:13 ` Driver for CB710/720 memory card reader (MMC part) Michał Mirosław
2008-09-12 23:43 ` RFC: " Michał Mirosław
2008-09-20 11:00 ` Pierre Ossman
2008-09-25 6:29 ` RFC: Driver for CB710/720 memory card reader (MMC part) - v2 Michał Mirosław
2008-10-04 19:51 ` Pierre Ossman
2008-10-29 14:11 ` RFC: Driver for CB710/720 memory card reader (MMC part) - v3 Michał Mirosław
2008-11-14 21:06 ` Pierre Ossman
2009-02-01 18:54 ` Michał Mirosław
2009-02-21 12:46 ` Pierre Ossman
2009-05-08 15:16 ` [PATCH/RFC 2.6.29.2] Driver for CB710/720 memory card reader (MMC part) - v4 Michał Mirosław
2009-05-22 11:27 ` Pierre Ossman
2009-05-22 17:55 ` [PATCH 2.6.29.3] Driver for CB710/720 memory card reader (MMC part) - v5 Michał Mirosław
2009-05-22 18:33 ` [PATCH 2.6.29.4] Driver for CB710/720 memory card reader (MMC part) - v5 fixed Michał Mirosław
2009-05-27 20:13 ` Pierre Ossman
2009-06-04 10:24 ` Michał Mirosław
2009-06-05 9:26 ` Michał Mirosław
2009-06-13 10:39 ` Pierre Ossman
2009-05-22 19:04 ` Bartlomiej Zolnierkiewicz [this message]
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=200905222104.07148.bzolnier@gmail.com \
--to=bzolnier@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mirq@rere.qmqm.pl \
--cc=pierre@ossman.eu \
/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;
as well as URLs for NNTP newsgroup(s).