linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: w.sang@pengutronix.de (Wolfram Sang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/4] MXS: I2C improvements
Date: Sat, 12 May 2012 13:36:30 +0200	[thread overview]
Message-ID: <20120512113630.GE20673@pengutronix.de> (raw)
In-Reply-To: <201205092033.46812.marex@denx.de>

On Wed, May 09, 2012 at 08:33:46PM +0200, Marek Vasut wrote:
> > This patchset implements support for adjustment of I2C bus speed via
> > platform data. Further, it implements support for DMA transfers into the
> > mxs-i2c driver. Lastly, there is a fix for GCC4.7 warning included.
> > 
> > Marek Vasut (4):
> >   MXS: Allow passing i2c bus speed via platform data
> >   MXS: Set I2C timing registers for mxs-i2c
> >   MXS: Implement DMA support into mxs-i2c
> >   MXS: Fix GCC4.7 complaint in mxs-i2c
> 
> Bump?

Regarding I2C, I think we will have DT support for it already in 3.5.
So, I am with Shawn here to not introduce new features with new
platform_data when there is a more recent alternative. So, drop 1/4 and
refactor 2/4 according to DT (you only need to get and parse the
"clock-frequency" property).

3/4 is the most interesting patch of this series, since it is a major
bugfix. Due to holidays, I won't be able to test it in the next 10 days,
but I am very eager to get it into 3.5. Since it is a bugfix, we could
probably get it in after the merge window, but I'll try to make the
merge window nontheless. If somebody donates Tested or Acked or Reviewed
tags, that would also help.

I already NACKed 4/4, so I wonder why you resent it.

Thanks,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120512/dce3f5fa/attachment.sig>

      parent reply	other threads:[~2012-05-12 11:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-01  2:00 [PATCH 0/4] MXS: I2C improvements Marek Vasut
2012-05-01  2:00 ` [PATCH 1/4] MXS: Allow passing i2c bus speed via platform data Marek Vasut
2012-05-01  2:00 ` [PATCH 2/4] MXS: Set I2C timing registers for mxs-i2c Marek Vasut
2012-05-01  2:00 ` [PATCH 3/4] MXS: Implement DMA support into mxs-i2c Marek Vasut
2012-05-01  2:00 ` [PATCH 4/4] MXS: Fix GCC4.7 complaint in mxs-i2c Marek Vasut
2012-05-01  2:05 ` [PATCH 0/4] MXS: I2C improvements Marek Vasut
2012-05-09 18:33 ` Marek Vasut
2012-05-10  7:35   ` Shawn Guo
2012-05-10 11:17     ` Marek Vasut
2012-05-10 14:37       ` Shawn Guo
2012-05-12 11:36   ` Wolfram Sang [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=20120512113630.GE20673@pengutronix.de \
    --to=w.sang@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.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).