From: Mike Turquette <mturquette@linaro.org> To: Lee Jones <lee.jones@linaro.org>, "Sascha Hauer" <s.hauer@pengutronix.de> Cc: "Matthias Brugger" <matthias.bgg@gmail.com>, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Rob Herring" <robh+dt@kernel.org>, "Eddie Huang" <eddie.huang@mediatek.com>, "\"Yingjoe Chen (陳英洲)" <Yingjoe.Chen@mediatek.com>, "Henry Chen" <henryc.chen@mediatek.com>, "\"YH Chen (陳昱豪)" <yh.chen@mediatek.com>, kernel@pengutronix.de, "Flora Fu" <flora.fu@mediatek.com>, "Samuel Ortiz" <sameo@linux.intel.com> Subject: Re: [PATCH 13/13] mfd: Add support for the MediaTek MT6397 PMIC Date: Thu, 19 Feb 2015 13:41:58 -0800 [thread overview] Message-ID: <20150219214158.421.17521@quantum> (raw) In-Reply-To: <20150219121304.GH12212@x1> Quoting Lee Jones (2015-02-19 04:13:04) > On Thu, 19 Feb 2015, Sascha Hauer wrote: > > > On Thu, Feb 19, 2015 at 08:43:49AM +0000, Lee Jones wrote: > > > On Thu, 19 Feb 2015, Sascha Hauer wrote: > > > > > > > > > Looks okay to me now. > > > > > > > > > > > > Acked-by: Lee Jones <lee.jones@linaro.org> > > > > > > > > > > > > What's the merge plan for this set? > > > > > > > > > > Patches 1-9 are clock related an several of them have review comments > > > > > that need to be addressed. I wonder if a V2 series can break out the > > > > > various subsystems bits from each other? > > > > > > > > I'll send a new series later this day. These used to be two series, but > > > > the PMIC wrapper patches depend on the clock and reset controllers, also > > > > the device nodes depend on the clock/reset defines from the clock > > > > support patches. What do you suggest? In the early days of a SoC > > > > everything seems to depend on everything. > > > > > > Only build dependencies count. So long as the Kconfigs are setup > > > correct, there shouldn't be any issue in taking patches in one > > > subsystem at a time. > > > > The dts snippets need the files in include/dt-bindings, so indeed this > > is a build dependency. However, this comes only in with the dts changes. > > > > So here's the plan: > > > > - Mike takes the clk patches > > - Matthias takes the pmic wrapper driver (in drivers/soc/mediatek/) > > - You take the MT6397 core driver. > > Sounds reasonable. Just ensure that each set is orthogonal and builds > (or doesn't attempt to) and we'll be in a good place. Agreed. Regards, Mike > > > I'll queue up the dts changes locally and ask Arnd to take these after > > next -rc1 so that all dependencies are in. Unfortunately this means that > > the patches can't be tested until everything is together after next > > -rc1. > > I'm sure you will be diligent enough to test the interoperability of > the sets combined. Failing that we can deal with any unavoidable > fall-out during the -rcs. > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: mturquette@linaro.org (Mike Turquette) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 13/13] mfd: Add support for the MediaTek MT6397 PMIC Date: Thu, 19 Feb 2015 13:41:58 -0800 [thread overview] Message-ID: <20150219214158.421.17521@quantum> (raw) In-Reply-To: <20150219121304.GH12212@x1> Quoting Lee Jones (2015-02-19 04:13:04) > On Thu, 19 Feb 2015, Sascha Hauer wrote: > > > On Thu, Feb 19, 2015 at 08:43:49AM +0000, Lee Jones wrote: > > > On Thu, 19 Feb 2015, Sascha Hauer wrote: > > > > > > > > > Looks okay to me now. > > > > > > > > > > > > Acked-by: Lee Jones <lee.jones@linaro.org> > > > > > > > > > > > > What's the merge plan for this set? > > > > > > > > > > Patches 1-9 are clock related an several of them have review comments > > > > > that need to be addressed. I wonder if a V2 series can break out the > > > > > various subsystems bits from each other? > > > > > > > > I'll send a new series later this day. These used to be two series, but > > > > the PMIC wrapper patches depend on the clock and reset controllers, also > > > > the device nodes depend on the clock/reset defines from the clock > > > > support patches. What do you suggest? In the early days of a SoC > > > > everything seems to depend on everything. > > > > > > Only build dependencies count. So long as the Kconfigs are setup > > > correct, there shouldn't be any issue in taking patches in one > > > subsystem at a time. > > > > The dts snippets need the files in include/dt-bindings, so indeed this > > is a build dependency. However, this comes only in with the dts changes. > > > > So here's the plan: > > > > - Mike takes the clk patches > > - Matthias takes the pmic wrapper driver (in drivers/soc/mediatek/) > > - You take the MT6397 core driver. > > Sounds reasonable. Just ensure that each set is orthogonal and builds > (or doesn't attempt to) and we'll be in a good place. Agreed. Regards, Mike > > > I'll queue up the dts changes locally and ask Arnd to take these after > > next -rc1 so that all dependencies are in. Unfortunately this means that > > the patches can't be tested until everything is together after next > > -rc1. > > I'm sure you will be diligent enough to test the interoperability of > the sets combined. Failing that we can deal with any unavoidable > fall-out during the -rcs. > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead > Linaro.org ? Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2015-02-19 21:42 UTC|newest] Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-02-09 10:47 [PATCH v5]: clk: Add common clock support for Mediatek MT8135 and MT8173 Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 01/13] clk: dts: mediatek: add Mediatek MT8135 clock bindings Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 13:35 ` Philipp Zabel 2015-02-09 13:35 ` Philipp Zabel 2015-02-09 10:47 ` [PATCH 02/13] clk: mediatek: Add initial common clock support for Mediatek SoCs Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-13 7:41 ` Tomasz Figa 2015-02-13 7:41 ` Tomasz Figa 2015-02-13 12:06 ` Sascha Hauer 2015-02-13 12:06 ` Sascha Hauer 2015-02-13 13:22 ` Tomasz Figa 2015-02-13 13:22 ` Tomasz Figa 2015-02-09 10:47 ` [PATCH 03/13] clk: mediatek: Add reset controller support Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 13:35 ` Philipp Zabel 2015-02-09 13:35 ` Philipp Zabel 2015-02-09 10:47 ` [PATCH 04/13] clk: mediatek: Add basic clocks for Mediatek MT8135 Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 05/13] clk: dts: mediatek: add Mediatek MT8173 clock bindings Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 06/13] clk: mediatek: Add basic clocks for Mediatek MT8173 Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-13 9:56 ` Tomasz Figa 2015-02-13 9:56 ` Tomasz Figa 2015-02-19 8:24 ` Sascha Hauer 2015-02-19 8:24 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 07/13] dt: bindings: Add MediaTek MT8135/MT8173 reset controller defines Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 08/13] soc: mediatek: Add PMIC wrapper for MT8135 and MT6397 SoC Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 09/13] ARM: dts: mediatek: Enable clock support for Mediatek MT8135 Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:51 ` Russell King - ARM Linux 2015-02-09 10:51 ` Russell King - ARM Linux 2015-02-09 11:25 ` Sascha Hauer 2015-02-09 11:25 ` Sascha Hauer 2015-02-09 11:27 ` Russell King - ARM Linux 2015-02-09 11:27 ` Russell King - ARM Linux 2015-02-09 11:44 ` Sascha Hauer 2015-02-09 11:44 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 10/13] ARM: dts: mt8135: Add pmic wrapper nodes Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 11/13] ARM: dts: mt8135-evbp1: Add PMIC support Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 12/13] mfd: dt-bindings: Add bindings for the MediaTek MT6397 PMIC Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-09 10:47 ` [PATCH 13/13] mfd: Add support " Sascha Hauer 2015-02-09 10:47 ` Sascha Hauer 2015-02-16 9:56 ` Lee Jones 2015-02-16 9:56 ` Lee Jones [not found] ` <20150218181904.421.59675@quantum> [not found] ` <20150219082655.GV12209@pengutronix.de> [not found] ` <20150219084349.GA12212@x1> [not found] ` <20150219120409.GW12209@pengutronix.de> [not found] ` <20150219121304.GH12212@x1> 2015-02-19 21:41 ` Mike Turquette [this message] 2015-02-19 21:41 ` Mike Turquette
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=20150219214158.421.17521@quantum \ --to=mturquette@linaro.org \ --cc=Yingjoe.Chen@mediatek.com \ --cc=eddie.huang@mediatek.com \ --cc=flora.fu@mediatek.com \ --cc=henryc.chen@mediatek.com \ --cc=kernel@pengutronix.de \ --cc=lee.jones@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=matthias.bgg@gmail.com \ --cc=robh+dt@kernel.org \ --cc=s.hauer@pengutronix.de \ --cc=sameo@linux.intel.com \ --cc=yh.chen@mediatek.com \ /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: linkBe 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.