linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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 --]

             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).