On Fri, 2020-10-02 at 12:58 +0200, Krzysztof Kozlowski wrote:

> On Wed, Sep 30, 2020 at 03:06:24PM +0800, Yong Wu wrote:
> > Convert MediaTek IOMMU to DT schema.
> > 
> > Signed-off-by: Yong Wu <yong.wu@mediatek.com>
> > ---
> >  .../bindings/iommu/mediatek,iommu.txt         | 103 ------------
> >  .../bindings/iommu/mediatek,iommu.yaml        | 154 ++++++++++++++++++
> >  2 files changed, 154 insertions(+), 103 deletions(-)
> >  delete mode 100644 Documentation/devicetree/bindings/iommu/mediatek,iommu.txt
> >  create mode 100644 Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
> > 


...


> > diff --git a/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
> > new file mode 100644
> > index 000000000000..eae773ad53a3
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
> > @@ -0,0 +1,154 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> 
> You relicense GPLv2 content so you need acks/signed-offs from every
> author:
> 
>  - Fabien Parent <fparent@baylibre.com>
>  - Chao Hao <chao.hao@mediatek.com>
>  - Matthias Brugger <matthias.bgg@gmail.com>
>  - Honghui Zhang <honghui.zhang@mediatek.com>
> (assuming yours is implicit).
> 
> Please resend CC-ing all the people.


Sorry, I really missed this mail.

I have sent v4 only for smi. the iommu part still need some time.

The license "GPL-2.0-only OR BSD-2-Clause" is requested when we run
check_patch. That means each converting YAML need all contributors'
ack/SoB? 

I have added their CC in v4 smi binding except honghui whose address is
not avalable now. I will act for him.


> 
> 
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/iommu/mediatek,iommu.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: MediaTek IOMMU Architecture Implementation
> > +


[snip..]


> > +  mediatek,larbs:
> > +    $ref: /schemas/types.yaml#/definitions/phandle-array
> > +    description: |
> > +      List of phandle to the local arbiters in the current Socs.
> > +      Refer to bindings/memory-controllers/mediatek,smi-larb.yaml. It must sort
> > +      according to the local arbiter index, like larb0, larb1, larb2...
> 
> How many items?


I will add maxitems property.

Thanks.


> 
> Best regards,
> Krzysztof
> 
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek