On 05/22/2017 04:14 PM, Matthias Brugger wrote:

On 22/05/17 11:09, Hans Verkuil wrote:

quoted

On 05/12/2017 05:22 AM, Minghsiu Tsai wrote:

Who should take care of the dtsi changes? I'm not sure who maintains the mdp dts.

I will take care of the dtsi patches.

quoted

The driver change and the dtsi change need to be in sync, so it is probably easiest
to merge this via one tree.

Here is my Acked-by for these three patches:

Acked-by: Hans Verkuil <redacted>

I can take all three, provided I have the Ack of the mdp dts maintainer. Or it can
go through him with my Ack.

I think we should provide backwards compability instead, as proposed here:
http://lists.infradead.org/pipermail/linux-mediatek/2017-May/008811.html

If this change is ok for you, please let Minghsiu know so that he can 
provide a v4.

That's a lot better. In that case I can take the media patches and you the dts.

I'll wait for the v4.

Regards,

	Hans

Regards,
Matthias

quoted

Regards,

	Hans

quoted

Changes in v3:
- Upload patches again because forget to add v2 in title

Changes in v2:
- Update commit message

If the mdp_* nodes are under an mdp sub-node, their corresponding
platform device does not automatically get its iommu assigned properly.

Fix this by moving the mdp component nodes up a level such that they are
siblings of mdp and all other SoC subsystems.  This also simplifies the
device tree.

Although it fixes iommu assignment issue, it also break compatibility
with old device tree. So, the patch in driver is needed to iterate over
sibling mdp device nodes, not child ones, to keep driver work properly.

Daniel Kurtz (2):
   arm64: dts: mt8173: Fix mdp device tree
   media: mtk-mdp: Fix mdp device tree

Minghsiu Tsai (1):
   dt-bindings: mt8173: Fix mdp device tree

  .../devicetree/bindings/media/mediatek-mdp.txt     |  12 +-
  arch/arm64/boot/dts/mediatek/mt8173.dtsi           | 126 ++++++++++-----------
  drivers/media/platform/mtk-mdp/mtk_mdp_core.c      |   2 +-
  3 files changed, 64 insertions(+), 76 deletions(-)