Hi Weiyi,
On 03/09/2020 05:22, Weiyi Lu wrote:
This series is based on v5.9-rc1 and MT8192 dts[1].
[1] https://patchwork.kernel.org/cover/11713555/
changes since v2:
- update and split dt-binding documents by functionalities
- add error checking in probe() function
- fix incorrect clock relation and add critical clocks
- update license identifier and minor fix of coding style
changes since v1:
- fix asymmetrical control of PLL
- have en_mask used as divider enable mask on all MediaTek SoC
Weiyi Lu (9):
dt-bindings: ARM: Mediatek: Document bindings for MT8192 BSP
dt-bindings: ARM: Mediatek: Document bindings for MT8192 Audio
dt-bindings: ARM: Mediatek: Document bindings for MT8192 Multimedia
dt-bindings: ARM: Mediatek: Document bindings for MT8192 Camera
dt-bindings: ARM: Mediatek: Document bindings for MT8192 APU and GPU
clk: mediatek: Add dt-bindings for MT8192 clocks
clk: mediatek: Fix asymmetrical PLL enable and disable control
clk: mediatek: Add configurable enable control to mtk_pll_data
clk: mediatek: Add MT8192 clock support
This looks like a big mix of different changes and I don't see the relation
between them.
To help facilitate the review process, I'd propose:
Any new binding with a new yaml file should be a single patch. Don't mix just
adding one string with adding a new binding file in the same patch.
Split patch 9/9 into different patches, one for each driver. Otherwise it is
nearly impossible to review such a big chunk of code.
Regards,
Matthias
.../arm/mediatek/mediatek,apmixedsys.txt | 1 +
.../bindings/arm/mediatek/mediatek,audsys.txt | 1 +
.../arm/mediatek/mediatek,camsys-raw.yaml | 54 +
.../bindings/arm/mediatek/mediatek,camsys.txt | 1 +
.../bindings/arm/mediatek/mediatek,imgsys.txt | 2 +
.../arm/mediatek/mediatek,imp_iic_wrap.yaml | 78 +
.../arm/mediatek/mediatek,infracfg.txt | 1 +
.../bindings/arm/mediatek/mediatek,ipesys.txt | 1 +
.../arm/mediatek/mediatek,mdpsys.yaml | 38 +
.../bindings/arm/mediatek/mediatek,mfgcfg.txt | 1 +
.../bindings/arm/mediatek/mediatek,mmsys.txt | 1 +
.../bindings/arm/mediatek/mediatek,msdc.yaml | 46 +
.../arm/mediatek/mediatek,pericfg.yaml | 1 +
.../arm/mediatek/mediatek,scp-adsp.yaml | 38 +
.../arm/mediatek/mediatek,topckgen.txt | 1 +
.../arm/mediatek/mediatek,vdecsys-soc.yaml | 38 +
.../arm/mediatek/mediatek,vdecsys.txt | 1 +
.../arm/mediatek/mediatek,vencsys.txt | 1 +
drivers/clk/mediatek/Kconfig | 146 ++
drivers/clk/mediatek/Makefile | 24 +
drivers/clk/mediatek/clk-mt2701.c | 26 +-
drivers/clk/mediatek/clk-mt2712.c | 30 +-
drivers/clk/mediatek/clk-mt6765.c | 20 +-
drivers/clk/mediatek/clk-mt6779.c | 24 +-
drivers/clk/mediatek/clk-mt6797.c | 20 +-
drivers/clk/mediatek/clk-mt7622.c | 18 +-
drivers/clk/mediatek/clk-mt7629.c | 12 +-
drivers/clk/mediatek/clk-mt8173.c | 28 +-
drivers/clk/mediatek/clk-mt8183.c | 22 +-
drivers/clk/mediatek/clk-mt8192-aud.c | 118 ++
drivers/clk/mediatek/clk-mt8192-cam.c | 72 +
drivers/clk/mediatek/clk-mt8192-cam_rawa.c | 59 +
drivers/clk/mediatek/clk-mt8192-cam_rawb.c | 59 +
drivers/clk/mediatek/clk-mt8192-cam_rawc.c | 59 +
drivers/clk/mediatek/clk-mt8192-img.c | 60 +
drivers/clk/mediatek/clk-mt8192-img2.c | 62 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_c.c | 61 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_e.c | 58 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_n.c | 59 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_s.c | 60 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_w.c | 58 +
.../clk/mediatek/clk-mt8192-imp_iic_wrap_ws.c | 60 +
drivers/clk/mediatek/clk-mt8192-ipe.c | 64 +
drivers/clk/mediatek/clk-mt8192-mdp.c | 89 ++
drivers/clk/mediatek/clk-mt8192-mfg.c | 57 +
drivers/clk/mediatek/clk-mt8192-mm.c | 108 ++
drivers/clk/mediatek/clk-mt8192-msdc.c | 57 +
drivers/clk/mediatek/clk-mt8192-msdc_top.c | 71 +
drivers/clk/mediatek/clk-mt8192-scp_adsp.c | 57 +
drivers/clk/mediatek/clk-mt8192-vdec.c | 82 +
drivers/clk/mediatek/clk-mt8192-vdec_soc.c | 82 +
drivers/clk/mediatek/clk-mt8192-venc.c | 60 +
drivers/clk/mediatek/clk-mt8192.c | 1340 +++++++++++++++++
drivers/clk/mediatek/clk-mtk.h | 2 +
drivers/clk/mediatek/clk-mux.h | 15 +
drivers/clk/mediatek/clk-pll.c | 28 +-
include/dt-bindings/clock/mt8192-clk.h | 592 ++++++++
57 files changed, 4116 insertions(+), 108 deletions(-)
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,camsys-raw.yaml
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,imp_iic_wrap.yaml
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,mdpsys.yaml
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,msdc.yaml
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,scp-adsp.yaml
create mode 100644
Documentation/devicetree/bindings/arm/mediatek/mediatek,vdecsys-soc.yaml
create mode 100644 drivers/clk/mediatek/clk-mt8192-aud.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-cam.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-cam_rawa.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-cam_rawb.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-cam_rawc.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-img.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-img2.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_c.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_e.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_n.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_s.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_w.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-imp_iic_wrap_ws.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-ipe.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-mdp.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-mfg.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-mm.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-msdc.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-msdc_top.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-scp_adsp.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-vdec.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-vdec_soc.c
create mode 100644 drivers/clk/mediatek/clk-mt8192-venc.c
create mode 100644 drivers/clk/mediatek/clk-mt8192.c
create mode 100644 include/dt-bindings/clock/mt8192-clk.h
