From: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
To: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>
Cc: vitb-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org,
htoa-hi6Y0CQ0nG0@public.gmane.org,
i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org,
tmbinc-hi6Y0CQ0nG0@public.gmane.org,
akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org
Subject: Re: [patch 3/3] i2c: add support for i2c bus on Freescale CPM1/CPM2 controllers
Date: Sat, 17 May 2008 15:46:45 +0200 [thread overview]
Message-ID: <20080517134645.GA4404@pengutronix.de> (raw)
In-Reply-To: <20080516210818.26bf8cb8-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
[-- Attachment #1.1: Type: text/plain, Size: 1018 bytes --]
On Fri, May 16, 2008 at 09:08:18PM +0200, Jean Delvare wrote:
> Would you have the possibility to review and ack this patch (with the
> suggested fix above, of course)? I'd like to include it in my i2c tree
> now as it has been around for some time but I'm a bit short of time to
> review it and I also can't test it. If you can do that then I'll trust
> your review and take the patch quickly.
Well, I could review it and give further testing on Monday. Still, I
wouldn't call my experiences in the linux-kernelspace (yet) big enough
to trust only this review. Still, thanks for your trust :)
> With the time saved, I could, say, review the at24c driver ;)
Hehe, nice offer. Though, I'd suggest to wait for the upcoming version
of the at24-driver, which will include proper device table support. I
already made up my mind how to implement it, just have to get hacking.
Wolfram
--
Dipl.-Ing. Wolfram Sang | http://www.pengutronix.de
Pengutronix - Linux Solutions for Science and Industry
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
i2c mailing list
i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org
http://lists.lm-sensors.org/mailman/listinfo/i2c
next prev parent reply other threads:[~2008-05-17 13:46 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-14 23:14 [patch 3/3] i2c: add support for i2c bus on Freescale CPM1/CPM2 controllers akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b
[not found] ` <200805142314.m4ENEjPV026316-AB4EexQrvXRQetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org>
2008-05-16 8:37 ` Wolfram Sang
[not found] ` <20080516083743.GA4180-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2008-05-16 19:08 ` Jean Delvare
[not found] ` <20080516210818.26bf8cb8-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-05-16 19:27 ` Jochen Friedrich
2008-05-17 13:46 ` Wolfram Sang [this message]
2008-05-19 15:54 ` Wolfram Sang
[not found] ` <20080519155443.GA4279-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2008-05-19 16:49 ` Jean Delvare
[not found] ` <20080519184907.651a4e48-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-06-01 22:24 ` Ben Dooks
[not found] ` <20080601222428.GC6226-elnMNo+KYs3pIgCt6eIbzw@public.gmane.org>
2008-06-02 7:08 ` Jean Delvare
[not found] ` <20080602090850.1b8db039-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-06-02 19:26 ` Ben Dooks
[not found] ` <20080602192630.GD6226-elnMNo+KYs3pIgCt6eIbzw@public.gmane.org>
2008-06-02 19:53 ` Jean Delvare
[not found] ` <20080602215343.07ad6e02-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-06-02 21:03 ` David Brownell
[not found] ` <200806021403.46232.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2008-06-02 22:19 ` Ben Dooks
[not found] ` <20080602221923.GF6226-elnMNo+KYs3pIgCt6eIbzw@public.gmane.org>
2008-06-03 10:06 ` Jean Delvare
[not found] ` <20080603120625.7bde7698-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-06-03 20:13 ` David Brownell
2008-06-03 20:49 ` Trent Piepho
2008-05-19 20:43 ` Jochen Friedrich
[not found] ` <4831E654.4020802-NIgtFMG+Po8@public.gmane.org>
2008-05-20 6:54 ` Wolfram Sang
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=20080517134645.GA4404@pengutronix.de \
--to=w.sang-bicnvbalz9megne8c9+irq@public.gmane.org \
--cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
--cc=htoa-hi6Y0CQ0nG0@public.gmane.org \
--cc=i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org \
--cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
--cc=tmbinc-hi6Y0CQ0nG0@public.gmane.org \
--cc=vitb-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
/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