Received: by 10.223.176.46 with SMTP id f43csp495437wra; Thu, 18 Jan 2018 20:57:01 -0800 (PST) X-Google-Smtp-Source: ACJfBos+zee9FMjYEXnwLxlEtK4xwb9HViApBQc09BkWFtziKhVaNPr/ZFFX5GNxJsYmMRQ+K+qp X-Received: by 10.99.124.89 with SMTP id l25mr35904750pgn.38.1516337820858; Thu, 18 Jan 2018 20:57:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1516337820; cv=none; d=google.com; s=arc-20160816; b=cObqfEY+UtI4CBH0L3jHNeKMYKyv5o7FSC487YBuajDlKqbGtTUEeoKmjUA/aMtcvy XbcGz8W3EQXksb2jLH7B/aJaP33NdKKoqN1BEXjkPVbKPnezDeLD9+WtUFXaA21BkG8Q 8Q4RaX1ExtJjgNidmMTfLPCML8Zn8UgFSfodTUHjRhSp/zy1kuhYohe28pCg+tTBs9SF FhFBQ9oyZC0F3SdGQBdn/Mjl3Mkb5mTULQMJwlYZF+M7rHLt4fFSPzXN5DQcIywq2auP 7QoWvs8nbef+oxsbVv0OPoKuWtmpx7EefvGd4fTNd3HczdTcBt8QYOlsxKVrPYz8bASA 9d9w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:in-reply-to :references:subject:cc:to:mime-version:user-agent:from:date :message-id:arc-authentication-results; bh=S+z5zvP7Wrj8hkgJNvv2QmF6fJEzEOTHdyFhMaZpMWI=; b=Z36mGJwIaotZBFxXLUoTUnOk0inrbXo+N4DmBHrnS1ru/xkwvYxX0r5ZQRMJ/ytROR LOUoAyG5ib8nIdSebD10cqlr4+UvJhARps7fUr/GhFoSkA1T5jW9JFKnlTJIyxP71JVH ZBobgsGcTzp0BnRN9ftOhnUyDXm5RarRDVDtMfKOc/OMzs6Fa2qDOQTxto9hMdu5Oavr Yr0Da8sVQMd4bV35cC8lmswubsP63eLEnWtkduo1OdKx7IUP2ZZzLethy2KgFGm9uRkT O0K+I28CykQHMZ7dbIGoEjtIFVWpwhoEk0II3Dbu0Trx90PVsXRSZmRmtGuMhzG866Ua UFlw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h7si6593137pgc.827.2018.01.18.20.56.46; Thu, 18 Jan 2018 20:57:00 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753390AbeASE4R (ORCPT + 99 others); Thu, 18 Jan 2018 23:56:17 -0500 Received: from regular1.263xmail.com ([211.150.99.141]:49493 "EHLO regular1.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750757AbeASE4L (ORCPT ); Thu, 18 Jan 2018 23:56:11 -0500 Received: from jeffy.chen?rock-chips.com (unknown [192.168.167.87]) by regular1.263xmail.com (Postfix) with ESMTP id F3E8770; Fri, 19 Jan 2018 12:56:03 +0800 (CST) X-263anti-spam: KSV:0; X-MAIL-GRAY: 0 X-MAIL-DELIVERY: 1 X-KSVirus-check: 0 X-ABS-CHECKED: 4 Received: from [172.16.22.190] (localhost [127.0.0.1]) by smtp.263.net (Postfix) with ESMTPA id 67C873D6; Fri, 19 Jan 2018 12:55:59 +0800 (CST) X-RL-SENDER: jeffy.chen@rock-chips.com X-FST-TO: tfiga@chromium.org X-SENDER-IP: 103.29.142.67 X-LOGIN-NAME: jeffy.chen@rock-chips.com X-UNIQUE-TAG: <3c8c6b45e0156fd5d35e02acad273c11> X-ATTACHMENT-NUM: 0 X-SENDER: cjf@rock-chips.com X-DNS-TYPE: 0 Received: from [172.16.22.190] (unknown [103.29.142.67]) by smtp.263.net (Postfix) whith ESMTP id 9945GMWXLZ; Fri, 19 Jan 2018 12:56:03 +0800 (CST) Message-ID: <5A617A5E.6070606@rock-chips.com> Date: Fri, 19 Jan 2018 12:55:58 +0800 From: JeffyChen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:19.0) Gecko/20130126 Thunderbird/19.0 MIME-Version: 1.0 To: Tomasz Figa CC: linux-kernel@vger.kernel.org, Ricky Liang , Robin Murphy , simon xue , devicetree@vger.kernel.org, Heiko Stuebner , "open list:ARM/Rockchip SoC..." , Rob Herring , Mark Rutland , Russell King , "list@263.net:IOMMU DRIVERS" , Joerg Roedel , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v4 07/13] ARM: dts: rockchip: add clocks in vop iommu nodes References: <20180118115251.5542-1-jeffy.chen@rock-chips.com> <20180118115251.5542-8-jeffy.chen@rock-chips.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Tomasz, Thanks for your reply. On 01/19/2018 11:23 AM, Tomasz Figa wrote: > On Thu, Jan 18, 2018 at 8:52 PM, Jeffy Chen wrote: >> Add clocks in vop iommu nodes, since we are going to control clocks in >> rockchip iommu driver. >> >> Signed-off-by: Jeffy Chen >> --- >> >> 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 = ; >> 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. > hmmm, i'm trying to switch to clk_bulk APIs, the get and put are name based. or maybe i can use clk_get/put along with other clk_bulk APIs > 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 > > >