Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1287160

Re: [PATCH v6 1/5] dt-bindings: iommu: Add binding for mediatek IOMMU

From Yong Wu <yong.wu@mediatek.com>
Newsgroups linux.kernel
Subject Re: [PATCH v6 1/5] dt-bindings: iommu: Add binding for mediatek IOMMU
Date 2015-12-09 08:00 +0100
Message-ID <qDGTf-4SH-1@gated-at.bofh.it> (permalink)
References <qDn4e-xE-1@gated-at.bofh.it> <qDndV-AW-23@gated-at.bofh.it> <qDDLI-2ST-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


also add Mark.

On Tue, 2015-12-08 at 21:33 -0600, Rob Herring wrote:
> On Tue, Dec 08, 2015 at 05:49:09PM +0800, Yong Wu wrote:
> > This patch add mediatek iommu dts binding document.
> > 
> > Signed-off-by: Yong Wu <yong.wu@mediatek.com>
> > ---
> >  .../devicetree/bindings/iommu/mediatek,iommu.txt   |  68 +++++++++++++
> >  include/dt-bindings/memory/mt8173-larb-port.h      | 111 +++++++++++++++++++++
> 
> This should be iommu rather than memory.

Hi Rob,
   Thanks very much for review.

   From the HW, all the larbs and ports are defined in the SMI.
About the position of the SMI code, We also discussed before in [1].
So I put the header file in /dt-bindings/memory/.

This is a smi larb-port header file, maybe we should move this header
file into this patch:
[2/5] dt-bindings: mediatek: Add smi dts binding

is it ok if we move it into [2/5]?

[1]
http://lists.infradead.org/pipermail/linux-mediatek/2015-March/000121.html

> 
> Otherwise, it looks okay to me.

Thanks.

> 
> >  2 files changed, 179 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/iommu/mediatek,iommu.txt
> >  create mode 100644 include/dt-bindings/memory/mt8173-larb-port.h

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

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v6 0/5] MT8173 IOMMU SUPPORT Yong Wu <yong.wu@mediatek.com> - 2015-12-08 10:50 +0100
  [PATCH v6 3/5] memory: mediatek: Add SMI driver Yong Wu <yong.wu@mediatek.com> - 2015-12-08 11:00 +0100
    Re: [PATCH v6 3/5] memory: mediatek: Add SMI driver Yong Wu <yong.wu@mediatek.com> - 2015-12-11 08:30 +0100
  [PATCH v6 2/5] dt-bindings: mediatek: Add smi dts binding Yong Wu <yong.wu@mediatek.com> - 2015-12-08 11:00 +0100
    Re: [PATCH v6 2/5] dt-bindings: mediatek: Add smi dts binding Rob Herring <robh@kernel.org> - 2015-12-09 04:40 +0100
  [PATCH v6 5/5] dts: mt8173: Add iommu/smi nodes for mt8173 Yong Wu <yong.wu@mediatek.com> - 2015-12-08 11:00 +0100
  [PATCH v6 1/5] dt-bindings: iommu: Add binding for mediatek IOMMU Yong Wu <yong.wu@mediatek.com> - 2015-12-08 11:00 +0100
    Re: [PATCH v6 1/5] dt-bindings: iommu: Add binding for mediatek IOMMU Rob Herring <robh@kernel.org> - 2015-12-09 04:40 +0100
      Re: [PATCH v6 1/5] dt-bindings: iommu: Add binding for mediatek  IOMMU Yong Wu <yong.wu@mediatek.com> - 2015-12-09 08:00 +0100
  [PATCH v6 4/5] iommu/mediatek: Add mt8173 IOMMU driver Yong Wu <yong.wu@mediatek.com> - 2015-12-08 11:00 +0100

csiph-web