lkml.org 
[lkml]   [2018]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v4 07/13] ARM: dts: rockchip: add clocks in vop iommu nodes
    On Thu, Jan 18, 2018 at 8:52 PM, Jeffy Chen <jeffy.chen@rock-chips.com> wrote:
    > Add clocks in vop iommu nodes, since we are going to control clocks in
    > rockchip iommu driver.
    >
    > Signed-off-by: Jeffy Chen <jeffy.chen@rock-chips.com>
    > ---
    >
    > Changes in v4: None
    > Changes in v3: None
    > Changes in v2: None
    >
    > arch/arm/boot/dts/rk3036.dtsi | 2 ++
    > arch/arm/boot/dts/rk3288.dtsi | 4 ++++
    > 2 files changed, 6 insertions(+)
    >
    > diff --git a/arch/arm/boot/dts/rk3036.dtsi b/arch/arm/boot/dts/rk3036.dtsi
    > index 3b704cfed69a..95b0ebc7a40f 100644
    > --- a/arch/arm/boot/dts/rk3036.dtsi
    > +++ b/arch/arm/boot/dts/rk3036.dtsi
    > @@ -197,6 +197,8 @@
    > reg = <0x10118300 0x100>;
    > interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>;
    > interrupt-names = "vop_mmu";
    > + clocks = <&cru ACLK_LCDC>, <&cru SCLK_LCDC>, <&cru HCLK_LCDC>;
    > + clock-names = "aclk_vop", "dclk_vop", "hclk_vop";

    We should remove clock-names from IOMMU nodes. The Rockchip IOMMU
    bindings don't define clock names and only the clocks property should
    be given.

    Not even saying that the names currently listed are not good examples,
    they name SoC clock controller output, rather than device inputs.

    Best regards,
    Tomasz

    \
     
     \ /
      Last update: 2018-01-19 04:23    [W:3.581 / U:0.044 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site