Re: [PATCH v3 00/12] Add Basic SoC support for MT6797

[Mars Cheng]

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
> > 
> 

Re: [PATCH v3 00/12] Add Basic SoC support for MT6797

[Mars Cheng]

On Thu, 2017-03-23 at 15:24 +, Marc Zyngier wrote:
> Hi Mars,
> 
> On 23/03/17 00:46, Mars Cheng wrote:
> > Hi Matthias, Rob, Marc, Stephen
> > 
> > gentle ping for this patch set.
> 
> I appreciate that you're eager to see this reviewed, but less than 4
> days between a posting and a reminder is a bit too eager. We're not
> machines! ;-)
> 
> Thanks,
> 
>   M.
Hi Marc

My bad. I should wait for one week at least.

Thanks for reminding. :-) 

Re: [PATCH v3 00/12] Add Basic SoC support for MT6797

[Marc Zyngier]

Hi Mars,

On 23/03/17 00:46, Mars Cheng wrote:
> Hi Matthias, Rob, Marc, Stephen
> 
> gentle ping for this patch set.

I appreciate that you're eager to see this reviewed, but less than 4
days between a posting and a reminder is a bit too eager. We're not
machines! ;-)

Thanks,

M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH v3 00/12] Add Basic SoC support for MT6797

[Mars Cheng]

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
> 

[PATCH v3 00/12] Add Basic SoC support for MT6797

[Mars Cheng]

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