From: Khiem Nguyen <khiem.nguyen.xt@renesas.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH v2] ARM: shmobile: Lager: Correct I2C bus for VDD MPU regulator
Date: Thu, 10 Jul 2014 08:43:55 +0000 [thread overview]
Message-ID: <53BE524B.9000909@renesas.com> (raw)
In-Reply-To: <53B5F327.5040301@renesas.com>
On 7/10/2014 5:23 PM, Magnus Damm wrote:
> Hi Khiem-san, Wolfram, everyone,
>
> [CC Wolfram]
>
> On Fri, Jul 4, 2014 at 2:19 AM, Khiem Nguyen
> <khiem.nguyen.xt@renesas.com> wrote:
>> I2C bus for VDD MPU regulator is IIC3, not I2C3.
>>
>> Signed-off-by: Khiem Nguyen <khiem.nguyen.xt@renesas.com>
>> ---
>> v2:
>> - re-based on top of tag renesas-devel-v3.16-rc3-20140701.
>> - removed redundant spaces.
>
> This patch is changing I2C master controller device for the I2C bus
> where the power chip is located. I've heard that all of our different
> I2C controllers come with a range of features and issues, so I think
> we should check with the I2C maintainer Wolfram Sang about this
> matter.
>
> Wolfram, can you give us your opinion about this patch? Does it
> improve our I2C software support on Lager?
IICDVFS is used for controlling power chip, according to H2 datasheet.
In H2/Lager board, name of IICDVFS is IIC3.
This patch is to correct I2C bus in H2/Lager board (revise to IIC3 from I2C3).
In M2/Koelsch board, name of IICDVFS is I2C6.
>
> Khiem-san, in the future please CC Wolfram about issues related to I2C.
I got it. Thanks.
>
> / magnus
>
--
Best regards,
KHIEM Nguyen
next prev parent reply other threads:[~2014-07-10 8:43 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-04 0:19 [PATCH v2] ARM: shmobile: Lager: Correct I2C bus for VDD MPU regulator Khiem Nguyen
2014-07-04 5:40 ` Gaku Inami
2014-07-04 6:41 ` Khiem Nguyen
2014-07-06 14:37 ` Simon Horman
2014-07-06 14:58 ` Simon Horman
2014-07-06 22:39 ` Khiem Nguyen
2014-07-10 8:23 ` Magnus Damm
2014-07-10 8:43 ` Khiem Nguyen [this message]
2014-07-10 9:40 ` Wolfram Sang
2014-07-10 9:46 ` Magnus Damm
2014-07-10 10:01 ` Khiem Nguyen
2014-07-10 10:03 ` Magnus Damm
2014-07-11 8:45 ` Simon Horman
2014-07-13 23:19 ` Khiem Nguyen
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=53BE524B.9000909@renesas.com \
--to=khiem.nguyen.xt@renesas.com \
--cc=linux-sh@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.