linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: ux500: resurrect I2C platform data
Date: Mon, 6 Aug 2012 10:51:45 +0100	[thread overview]
Message-ID: <20120806095144.GA16996@gmail.com> (raw)
In-Reply-To: <1344243146-6598-1-git-send-email-linus.walleij@stericsson.com>

On Mon, Aug 06, 2012 at 10:52:26AM +0200, Linus Walleij wrote:
> Commit:
> 98582d9562b4bea6b0eb6e2bfafcd3fab3b60ccb
> "ARM: ux500: Remove unused i2c platform_data initialisation code"
> 
> Deleted platform data intialization code that was used,
> not unused as indicated in the commit. The boot log (without
> devicetree) now looks like this:

The aforementioned commit was part of a patch-set. One of its affiliates
moved this code into the Nomadik I2C driver; however, it was contested
because Device Tree support was also added and the maintainer thought 
the properties introduced could be consolidated with other I2C driver's
DT properties. I don't think anyone has taken up the consolidation work
and for that reason it hasn't gone in.

I think moving this stuff into the driver is the right thing to do.
Instead, I'll split out the configuration and DT enablement from the
patch and resubmit.

Sound good to you? 

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2012-08-06  9:51 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-06  8:52 [PATCH] ARM: ux500: resurrect I2C platform data Linus Walleij
2012-08-06  9:51 ` Lee Jones [this message]
2012-08-06 11:37   ` Linus Walleij
2012-08-06 10:21 ` [PATCH 1/1] i2c: Add default configuration into the Nomadik I2C driver Lee Jones
2012-08-06 11:23   ` Linus Walleij
2012-08-06 11:43   ` Srinidhi Kasagar

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=20120806095144.GA16996@gmail.com \
    --to=lee.jones@linaro.org \
    --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).