From: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>
To: Linus Torvalds
<torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Jean Delvare <jdelvare-l3A5Bk7waGM@public.gmane.org>,
Mika Westerberg
<mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Subject: [PULL REQUEST] i2c for 3.17
Date: Thu, 25 Sep 2014 20:41:11 +0200 [thread overview]
Message-ID: <20140925184111.GA1924@katana> (raw)
[-- Attachment #1: Type: text/plain, Size: 3018 bytes --]
Linus,
this is probably not the kind of pull request you want to see that late
in the cycle. Yet, the ACPI refactorization was problematic again and
caused another two issues which need fixing. My holidays with limited
internet (plus travelling) and the developer's illness didn't help
either :(
The details:
- ACPI code was refactored out into a seperate file and as a
side-effect, the i2c-core module got renamed. Jean Delvare rightfully
complained about the rename being problematic for distributions. So,
Mika and I thought the least problematic way to deal with it is to move
all the code back into the main i2c core source file. This is mainly a
huge code move with some #ifdeffery applied. No functional code changes.
Our personal tests and the testbots did not find problems. (I was
thinking about reverting, too, yet that would also have ~800 lines
changed)
- The new ACPI code also had a NULL pointer exception, thanks to Peter
for finding and fixing it.
- Mikko fixed a locking problem by decoupling clock_prepare and
clock_enable.
- Addy learnt that the datasheet was wrong and reimplemented the
frequency setup according to the new algorithm.
- Fan fixed an off-by-one error when copying data
- Janusz fixed a copy'n'paste bug which gave a wrong error message
- Sergei made sure that "don't touch" bits are not accessed
Please pull.
Thanks,
Wolfram
The following changes since commit 9e82bf014195d6f0054982c463575cdce24292be:
Linux 3.17-rc5 (2014-09-14 17:50:12 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-current
for you to fetch changes up to 0aef44e84ad16cd87a88df78773fd81ecca34f11:
i2c: acpi: Fix NULL Pointer dereference (2014-09-25 16:08:15 +0200)
----------------------------------------------------------------
Fan Du (1):
i2c: ismt: use correct length when copy buffer
Janusz Użycki (1):
i2c: mxs: fix error message in pio transfer
Mikko Perttunen (1):
i2c: tegra: Move clk_prepare/clk_set_rate to probe
Peter Hüwe (1):
i2c: acpi: Fix NULL Pointer dereference
Sergei Shtylyov (1):
i2c: rcar: fix RCAR_IRQ_ACK_{RECV|SEND}
Wolfram Sang (1):
i2c: move acpi code back into the core
addy ke (1):
i2c: rk3x: fix divisor calculation for SCL frequency
MAINTAINERS | 1 -
drivers/i2c/Makefile | 5 +-
drivers/i2c/busses/i2c-ismt.c | 4 +-
drivers/i2c/busses/i2c-mxs.c | 2 +-
drivers/i2c/busses/i2c-rcar.c | 4 +-
drivers/i2c/busses/i2c-rk3x.c | 11 +-
drivers/i2c/busses/i2c-tegra.c | 57 +++++--
drivers/i2c/i2c-acpi.c | 364 -----------------------------------------
drivers/i2c/i2c-core.c | 364 +++++++++++++++++++++++++++++++++++++++++
include/linux/i2c.h | 16 --
10 files changed, 420 insertions(+), 408 deletions(-)
delete mode 100644 drivers/i2c/i2c-acpi.c
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next reply other threads:[~2014-09-25 18:41 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-25 18:41 Wolfram Sang [this message]
-- strict thread matches above, loose matches on Subject: below --
2014-10-03 21:10 [PULL REQUEST] i2c for 3.17 Wolfram Sang
2014-09-05 19:41 Wolfram Sang
2014-08-21 3:25 Wolfram Sang
2014-08-09 13:56 Wolfram Sang
2014-08-09 17:34 ` Linus Torvalds
[not found] ` <CA+55aFy0UVY=60pCr85AvhedfO-MEoD-iMGdWmBYt8fKgF7kWA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-08-11 1:39 ` Lan Tianyu
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=20140925184111.GA1924@katana \
--to=wsa-z923lk4zbo2bacvfa/9k2g@public.gmane.org \
--cc=jdelvare-l3A5Bk7waGM@public.gmane.org \
--cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
--cc=torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@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;
as well as URLs for NNTP newsgroup(s).