linux-kernel - RE: [PATCH V6 08/13] interconnect: mediatek: Add interconnect provider driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <06a8c29ef398431bb2023d939152342c@mtkmbs02n1.mediatek.inc>
Date:   Wed, 6 Jan 2021 13:11:44 +0000
From:   HenryC Chen (陳建豪) 
        <HenryC.Chen@...iatek.com>
To:     Georgi Djakov <georgi.djakov@...aro.org>
CC:     Rob Herring <robh+dt@...nel.org>,
        Matthias Brugger <matthias.bgg@...il.com>,
        Stephen Boyd <swboyd@...omium.org>,
        Ryan Case <ryandcase@...omium.org>,
        Mark Brown <broonie@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Nicolas Boichat <drinkcat@...gle.com>,
        Fan Chen (陳凡) <fan.chen@...iatek.com>,
        JamesJJ Liao (廖建智) 
        <jamesjj.liao@...iatek.com>,
        Arvin Wang (王志銘) 
        <Arvin.Wang@...iatek.com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-mediatek@...ts.infradead.org" 
        <linux-mediatek@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>
Subject: RE: [PATCH V6 08/13] interconnect: mediatek: Add interconnect
 provider driver



> -----Original Message-----
> From: Georgi Djakov [mailto:georgi.djakov@...aro.org]
> Sent: Wednesday, January 06, 2021 8:50 PM
> To: HenryC Chen (陳建豪)
> Cc: Rob Herring; Matthias Brugger; Stephen Boyd; Ryan Case; Mark Brown; Mark
> Rutland; Nicolas Boichat; Fan Chen (陳凡); JamesJJ Liao (廖建智); Arvin Wang
> (王志銘); devicetree@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> linux-mediatek@...ts.infradead.org; linux-kernel@...r.kernel.org; linux-
> pm@...r.kernel.org
> Subject: Re: [PATCH V6 08/13] interconnect: mediatek: Add interconnect
> provider driver
> 
> On 6.01.21 12:44, HenryC Chen (陳建豪) wrote:
> >> -----Original Message-----
> >> From: Georgi Djakov [mailto:georgi.djakov@...aro.org]
> >> Sent: Wednesday, January 06, 2021 4:14 PM
> >> To: HenryC Chen (陳建豪)
> >> Cc: Rob Herring; Matthias Brugger; Stephen Boyd; Ryan Case; Mark
> >> Brown; Mark Rutland; Nicolas Boichat; Fan Chen (陳凡); JamesJJ Liao
> >> (廖建智); Arvin Wang
> >> (王志銘); devicetree@...r.kernel.org;
> >> linux-arm-kernel@...ts.infradead.org;
> >> linux-mediatek@...ts.infradead.org; linux-kernel@...r.kernel.org;
> >> linux- pm@...r.kernel.org
> >> Subject: Re: [PATCH V6 08/13] interconnect: mediatek: Add
> >> interconnect provider driver
> >>
> >> On 1/6/21 09:30, Henry Chen wrote:
> >> > On Mon, 2021-01-04 at 20:36 +0200, Georgi Djakov wrote:
> >> >> On 12/24/20 08:08, Henry Chen wrote:
> >> >>> Introduce Mediatek MT6873/MT8183/MT8192 specific provider driver
> >> >>> using the interconnect framework.
> >> >>>
> >> >>>                ICC provider         ICC Nodes
> >> >>>                                 ----          ----
> >> >>>                ---------       |CPU |   |--- |VPU |
> >> >>>       -----   |         |-----  ----    |     ----
> >> >>>      |DRAM |--|DRAM     |       ----    |     ----
> >> >>>      |     |--|scheduler|----- |GPU |   |--- |DISP|
> >> >>>      |     |--|(EMI)    |       ----    |     ----
> >> >>>      |     |--|         |       -----   |     ----
> >> >>>       -----   |         |----- |MMSYS|--|--- |VDEC|
> >> >>>                ---------        -----   |     ----
> >> >>>                  /|\                    |     ----
> >> >>>                   |change DRAM freq     |--- |VENC|
> >> >>>                ----------               |     ----
> >> >>>               |  DVFSR   |              |
> >> >>>               |          |              |     ----
> >> >>>                ----------               |--- |IMG |
> >> >>>                                         |     ----
> >> >>>                                         |     ----
> >> >>>                                         |--- |CAM |
> >> >>>                                               ----
> >> >>>
> >> >>> Signed-off-by: Henry Chen <henryc.chen@...iatek.com>
> >> >>> ---
> >> >>>    drivers/interconnect/Kconfig            |   1 +
> >> >>>    drivers/interconnect/Makefile           |   1 +
> >> >>>    drivers/interconnect/mediatek/Kconfig   |  13 ++
> >> >>>    drivers/interconnect/mediatek/Makefile  |   3 +
> >> >>>    drivers/interconnect/mediatek/mtk-emi.c | 330
> >> ++++++++++++++++++++++++++++++++
> >> >>>    5 files changed, 348 insertions(+)
> >> >>>    create mode 100644 drivers/interconnect/mediatek/Kconfig
> >> >>>    create mode 100644 drivers/interconnect/mediatek/Makefile
> >> >>>    create mode 100644 drivers/interconnect/mediatek/mtk-emi.c
> >> >>>
> >> >>> diff --git a/drivers/interconnect/Kconfig
> >> >>> b/drivers/interconnect/Kconfig index 5b7204e..e939f5a 100644
> >> >>> --- a/drivers/interconnect/Kconfig
> >> >>> +++ b/drivers/interconnect/Kconfig
> >> >>> @@ -13,5 +13,6 @@ if INTERCONNECT
> >> >>>
> >> >>>    source "drivers/interconnect/imx/Kconfig"
> >> >>>    source "drivers/interconnect/qcom/Kconfig"
> >> >>> +source "drivers/interconnect/mediatek/Kconfig"
> >> >>
> >> >> Sort alphabetically please.
> >> > Ok
> >> >>
> >> >>>
> >> >>>    endif
> >> >>> diff --git a/drivers/interconnect/Makefile
> >> >>> b/drivers/interconnect/Makefile index d203520..0643a24 100644
> >> >>> --- a/drivers/interconnect/Makefile
> >> >>> +++ b/drivers/interconnect/Makefile
> >> >>> @@ -6,3 +6,4 @@ icc-core-objs:= core.o bulk.o
> >> >>>    obj-$(CONFIG_INTERCONNECT)+= icc-core.o
> >> >>>    obj-$(CONFIG_INTERCONNECT_IMX)+= imx/
> >> >>>    obj-$(CONFIG_INTERCONNECT_QCOM)+= qcom/
> >> >>> +obj-$(CONFIG_INTERCONNECT_MTK)+= mediatek/
> >> >>
> >> >> Ditto.
> >> > Ok
> >> >>
> >> >>> diff --git a/drivers/interconnect/mediatek/Kconfig
> >> >>> b/drivers/interconnect/mediatek/Kconfig
> >> >>> new file mode 100644
> >> >>> index 0000000..972d3bb
> >> >>> --- /dev/null
> >> >>> +++ b/drivers/interconnect/mediatek/Kconfig
> >> >>> @@ -0,0 +1,13 @@
> >> >>> +config INTERCONNECT_MTK
> >> >>> +bool "Mediatek Network-on-Chip interconnect drivers"
> >> >>> +depends on ARCH_MEDIATEK
> >> >>> +help
> >> >>> +  Support for Mediatek's Network-on-Chip interconnect hardware.
> >> >>> +
> >> >>> +config INTERCONNECT_MTK_EMI
> >> >>> +tristate "Mediatek EMI interconnect driver"
> >> >>> +depends on INTERCONNECT_MTK
> >> >>> +depends on (MTK_DVFSRC && OF)
> >> >>
> >> >> Would it be possible to enable COMPILE_TEST?
> >> > Do you means change to "depends on (MTK_DVFSRC && OF) || COMPILE_TEST" ?
> >>
> >> Yeah, there is a stub for mtk_dvfsrc_send_request(). Maybe we could
> >> even change it for INTERCONNECT_MTK to something like:
> >> depends on ARCH_MEDIATEK || COMPILE_TEST Will this work?
> > Hi Georgi,
> >
> > So..only change to as following?
> >
> > config INTERCONNECT_MTK
> > bool "Mediatek Network-on-Chip interconnect drivers"
> > depends on ARCH_MEDIATEK || COMPILE_TEST help
> >    Support for Mediatek's Network-on-Chip interconnect hardware.
> 
> I would say to add COMPILE_TEST for both INTERCONNECT_MTK and
> INTERCONNECT_MTK_EMI, unless there is some dependency which does not allow us
> to do so.
Understood, thanks.
> 
> Thanks,
> Georgi