Re: [PATCH v2 0/8] Pad retentions support for Exynos5433

From: Marek Szyprowski
Date: Thu Jan 26 2017 - 05:37:41 EST

Hi Linus,

On 2017-01-26 10:50, Linus Walleij wrote:
On Thu, Jan 26, 2017 at 9:33 AM, Marek Szyprowski
<m.szyprowski@xxxxxxxxxxx> wrote:
Patches in this patchset depends on each other. They are order in such a
way to make the changes bisectable.

Patch #3 has runtime dependency on #1.
Patch #5 has runtime dependency on #3.
Patch #6 has runtime dependency on #4.

This patchset also directly depends on the "Move pad retention control to
Exynos pin controller driver" patchset:
Do we *have* to merge it runtime-bisectably?

I'm asking because we need a huge immutable branch
(I guess in the MFD subsystem) to deal with that.

It'd be great if I could just apply the pinctrl patches in
isolation, then Lee applies the MFD patches in isolation,
everything compiles in isolation but maye just work once both
pinctrl and MFD are merged upstream, as in linux-next
or Torvalds' tree.

I always thought that it is good to prepare patches in such a way that they
don't break runtime bisectability, especially if this is just a matter of
applying a few patches via the other tree. In this case it will work fine
if MFD patches gets applied with Lee Jones ack via pinctrl tree (I remember
that MFD changes were often applied via the other trees in the past).

That is one of the reasons why using a system-agnostic
syscon regmap lookup is so good, BTW.

I've just explained my rationale about the regmap lookup in the separate

Best regards
Marek Szyprowski, PhD
Samsung R&D Institute Poland