Re: [PATCH v2 0/2] omap4/twl6030: typical connection to omap4 asa separate dtsi file

From: Benoit Cousson
Date: Mon Aug 19 2013 - 06:04:40 EST


Hi Ruslan,

On 19/08/2013 08:14, Ruslan Bilovol wrote:
Hi Benoit,

On Fri, Aug 16, 2013 at 4:20 PM, Benoit Cousson <bcousson@xxxxxxxxxxxx> wrote:
Hi Ruslan,

On 16/08/2013 14:04, Ruslan Bilovol wrote:
Hi Benoit,

On Wed, Aug 14, 2013 at 4:51 PM, Benoit Cousson <bcousson@xxxxxxxxxxxx> wrote:
Hi Ruslan,

On 14/08/2013 10:35, Ruslan Bilovol wrote:

Hello,

There is no functional changes between v1 and v2 - just
added the patch for omap4-var-som - Uri Yosef confirmed
this board have the same connection of OMAP4<->TWL6030 as
SDP4430 board


The series looks good to me, but it will be good to have a test for Panda
and Variscite boards before merging it.

Okay, so I just verified this patch series on PandaBoard ES2. Should I
resubmit this series with
fixed commit message?

No, that's fine, I already applied it and fixed the subject and the changelog.

Here it is:

commit 2e25df1e5a4af906a9b25332719ace000063eb0d
Author: Ruslan Bilovol <ruslan.bilovol@xxxxxx>
Date: Wed Aug 14 11:35:47 2013 +0300

ARM: dts: twl6030: Move common configuration for OMAP4 boards in a separate dtsi file

The OMAP4 SoC family uses specially-designed PMIC (power management IC)
companion chip for power management needs: TWL6030/TWL6032.
Therefore there is a typical connection of PMIC to OMAP4 so we can
move it into separate .dtsi file and do not duplicate over
board-specific files.

Tested on OMAP4 SDP board and compile-tested for Panda board

Just for archives: I've successfully tested it on PandaBoard ES2 last
week as well.

Great, I'll update the changelog before pushing it.

Signed-off-by: Ruslan Bilovol <ruslan.bilovol@xxxxxx>
Signed-off-by: Benoit Cousson <bcousson@xxxxxxxxxxxx>


Just let me know if you are OK with the updated version.

Yes, this version looks good to me. Thanks for picking it up!


However I cannot verify the patch for Variscite board because I do not
own any such board so
you can drop that patch. But maybe Uri Yosef can verify it. Uri?

It seems that Uri cannot test it right now, so I will have to drop that one.

Okay, let's wait for Uri's verification.

I'll keep it for 3.13.

Thanks,
Benoit

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/