Message-ID: <1491480665.6485.4.camel@mtkswgap22>
Subject: Re: [PATCH v3 00/12] Add Basic SoC support for MT6797
From: Mars Cheng <mars.cheng@mediatek.com>
To: Matthias Brugger <matthias.bgg@gmail.com>
CC: Rob Herring <robh+dt@kernel.org>, Marc Zyngier <marc.zyngier@arm.com>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@codeaurora.org>, CC Hwang <cc.hwang@mediatek.com>,
        Loda Chou <loda.chou@mediatek.com>,
        Miles Chen <miles.chen@mediatek.com>,
        Jades Shih <jades.shih@mediatek.com>,
        Yingjoe Chen <yingjoe.chen@mediatek.com>,
        "My Chuang" <my.chuang@mediatek.com>, <linux-kernel@vger.kernel.org>,
        <linux-mediatek@lists.infradead.org>, <devicetree@vger.kernel.org>,
        <wsd_upstream@mediatek.com>, <linux-clk@vger.kernel.org>
Date: Thu, 6 Apr 2017 20:11:05 +0800
In-Reply-To: <1490229971.25996.2.camel@mtkswgap22>
References: <1489937193-2953-1-git-send-email-mars.cheng@mediatek.com>
         <1490229971.25996.2.camel@mtkswgap22>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 4028
Lines: 90

Hi Matthias, Stephen

Would you like to give some comments for this patch set?
If there are any suggestions, I will try my best to fix the errors.

Thanks. 

On Thu, 2017-03-23 at 08:46 +0800, Mars Cheng wrote:
> Hi Matthias, Rob, Marc, Stephen
> 
> gentle ping for this patch set.
> 
> Thanks.
> 
> On Sun, 2017-03-19 at 23:26 +0800, Mars Cheng wrote:
> > This patch set adds basic SoC support for mediatek's first 10-core
> > chip, X20, also known as MT6797.
> > 
> > - based on 4.11-rc1
> > - support multiple base address for sysirq
> > - support common clk framework
> > 
> > Changes since v2:
> > - prevent uncessary #intpol-bases for mtk-sysirq
> > - add fast path for mtk-sysirq set_type when introducing multiple bases
> > - add acked-by and tested-by
> > - remove wrong usage for timer node
> > 
> > Changes since v1:
> > - add multiple base addresses support, v1 only allow 2 bases
> > - clean up clk driver
> > 
> > 
> > Kevin-CW Chen (2):
> >   dt-bindings: arm: mediatek: document clk bindings for MT6797
> >   clk: mediatek: add clk support for MT6797
> > 
> > Mars Cheng (10):
> >   dt-bindings: mediatek: multiple bases support for sysirq
> >   irqchip: mtk-sysirq: extend intpol base to arbitrary number
> >   irqchip: mtk-sysirq: prevent unnecessary visibility when set_type
> >   dt-bindings: mediatek: Add bindings for mediatek MT6797 Platform
> >   arm64: dts: mediatek: add mt6797 support
> >   soc: mediatek: avoid using fixed spm power status defines
> >   soc: mediatek: add vdec item for scpsys
> >   dt-bindings: mediatek: add MT6797 power dt-bindings
> >   soc: mediatek: add MT6797 scysys support
> >   arm64: dts: mediatek: add clk and scp nodes for MT6797
> > 
> >  Documentation/devicetree/bindings/arm/mediatek.txt |    4 +
> >  .../bindings/arm/mediatek/mediatek,apmixedsys.txt  |    1 +
> >  .../bindings/arm/mediatek/mediatek,imgsys.txt      |    1 +
> >  .../bindings/arm/mediatek/mediatek,infracfg.txt    |    1 +
> >  .../bindings/arm/mediatek/mediatek,mmsys.txt       |    1 +
> >  .../bindings/arm/mediatek/mediatek,topckgen.txt    |    1 +
> >  .../bindings/arm/mediatek/mediatek,vdecsys.txt     |    1 +
> >  .../bindings/arm/mediatek/mediatek,vencsys.txt     |    3 +-
> >  .../interrupt-controller/mediatek,sysirq.txt       |   12 +-
> >  .../devicetree/bindings/serial/mtk-uart.txt        |    1 +
> >  .../devicetree/bindings/soc/mediatek/scpsys.txt    |    6 +-
> >  arch/arm64/boot/dts/mediatek/Makefile              |    1 +
> >  arch/arm64/boot/dts/mediatek/mt6797-evb.dts        |   36 +
> >  arch/arm64/boot/dts/mediatek/mt6797.dtsi           |  245 +++++++
> >  drivers/clk/mediatek/Kconfig                       |   32 +
> >  drivers/clk/mediatek/Makefile                      |    5 +
> >  drivers/clk/mediatek/clk-mt6797-img.c              |   76 +++
> >  drivers/clk/mediatek/clk-mt6797-mm.c               |  136 ++++
> >  drivers/clk/mediatek/clk-mt6797-vdec.c             |   93 +++
> >  drivers/clk/mediatek/clk-mt6797-venc.c             |   78 +++
> >  drivers/clk/mediatek/clk-mt6797.c                  |  716 ++++++++++++++++++++
> >  drivers/irqchip/irq-mtk-sysirq.c                   |  116 +++-
> >  drivers/soc/mediatek/mtk-scpsys.c                  |  149 +++-
> >  include/dt-bindings/clock/mt6797-clk.h             |  281 ++++++++
> >  include/dt-bindings/power/mt6797-power.h           |   30 +
> >  25 files changed, 1993 insertions(+), 33 deletions(-)
> >  create mode 100644 arch/arm64/boot/dts/mediatek/mt6797-evb.dts
> >  create mode 100644 arch/arm64/boot/dts/mediatek/mt6797.dtsi
> >  create mode 100644 drivers/clk/mediatek/clk-mt6797-img.c
> >  create mode 100644 drivers/clk/mediatek/clk-mt6797-mm.c
> >  create mode 100644 drivers/clk/mediatek/clk-mt6797-vdec.c
> >  create mode 100644 drivers/clk/mediatek/clk-mt6797-venc.c
> >  create mode 100644 drivers/clk/mediatek/clk-mt6797.c
> >  create mode 100644 include/dt-bindings/clock/mt6797-clk.h
> >  create mode 100644 include/dt-bindings/power/mt6797-power.h
> > 
> > --
> > 1.7.9.5
> > 
>