From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mathieu Poirier Subject: Re: [PATCH v2 00/11] dts: Update DT bindings for CoreSight replicator and funnel Date: Mon, 13 May 2019 09:28:19 -0600 Message-ID: <20190513152819.GA32523@xps15> References: <20190508021902.10358-1-leo.yan@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190508021902.10358-1-leo.yan@linaro.org> Sender: linux-kernel-owner@vger.kernel.org To: Leo Yan Cc: Rob Herring , Mark Rutland , Suzuki K Poulose , Mike Leach , Wei Xu , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Andy Gross , David Brown , Linus Walleij , Liviu Dudau , Sudeep Holla , Lorenzo Pieralisi , Orson Zhai , Baolin Wang , Chunyan Zhang , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Wed, May 08, 2019 at 10:18:51AM +0800, Leo Yan wrote: > Since the DT bindings consolidatoins for CoreSight replicator and funnel > is ready for kernel v5.2 merge window [1], this patch set is to update > the related CoreSight DT bindings for platforms; IIUC, this patch set > will be safe for merging into kernel v5.2 because the dependency > patches in [1] will be landed into mainline kernel v5.2 cycle. > > In this patch set, it tries to update below two compatible strings to > the latest strings: > > s/"arm,coresight-replicator"/"arm,coresight-static-replicator" > s/"arm,coresight-funnel"/"arm,coresight-dynamic-funnel" > > Please note, some platforms have two continuous patches, one is for > updating static replicator compatible string and another is for dynamic > funnel change; and other platforms have only one patch since it only > needs to change for dynamic funnel. > > Avoid to introduce merging confliction, I rebased this patch set on > linux-next branch with last commit fcdb095ad001 ("Add linux-next > specific files for 20190506"). > > This patch set has been tested on Arm Juno and Hikey620 boards, other > platforms are only compilation passing. > > P.s. when use scirpt/checkpatch.pl, it reports the warnings as below. I > think we can ignore this warnings for this patch set, please review if > this makes sense for you. > > WARNING: line over 80 characters > #29: FILE: arch/arm/boot/dts/imx7s.dtsi:178: > + compatible = "arm,coresight-dynamic-funnel", "arm,primecell"; > > [1] https://archive.armlinux.org.uk/lurker/message/20190412.102734.2afbb29a.en.html > > == Changes for v2 == > * Add explanation for the change in the commit logs. (Fabio) > * Merge the separate patches for funnel and replicator per DTS into > a single patch. (Suzuki) > > > Cc: Rob Herring > Cc: Mark Rutland > Cc: Mathieu Poirier > Cc: Suzuki K Poulose > Cc: Liviu Dudau > Cc: Sudeep Holla > Cc: Lorenzo Pieralisi > Cc: Wei Xu > Cc: Guodong Xu > Cc: Zhangfei Gao > Cc: Haojian Zhuang > Cc: Shawn Guo > Cc: Chris Healy > Cc: Andrew Lunn > Cc: Fabio Estevam > Cc: Sascha Hauer > Cc: Andy Gross > Cc: David Brown > Cc: Linus Walleij > Cc: Lee Jones > Cc: Chunyan Zhang > Cc: Orson Zhai > Cc: Baolin Wang > > > Leo Yan (11): > ARM: dts: hip04: Update coresight DT bindings > ARM: dts: imx7s: Update coresight DT bindings > ARM: dts: qcom-apq8064: Update coresight DT bindings > ARM: dts: ste: Update coresight DT bindings > ARM: dts: vexpress-v2p-ca15_a7: Update coresight DT bindings > ARM: dts: qcom-msm8974: Update coresight DT bindings > arm64: dts: hi6220: Update coresight DT bindings > arm64: dts: juno: Update coresight DT bindings > arm64: dts: qcom-msm8916: Update coresight DT bindings > arm64: dts: sc9836: Update coresight DT bindings > arm64: dts: sc9860: Update coresight DT bindings > > arch/arm/boot/dts/hip04.dtsi | 18 +++++++++--------- > arch/arm/boot/dts/imx7s.dtsi | 6 +++--- > arch/arm/boot/dts/qcom-apq8064.dtsi | 4 ++-- > arch/arm/boot/dts/qcom-msm8974.dtsi | 6 +++--- > arch/arm/boot/dts/ste-dbx5x0.dtsi | 4 ++-- > arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts | 4 ++-- > arch/arm64/boot/dts/arm/juno-base.dtsi | 6 +++--- > arch/arm64/boot/dts/arm/juno-cs-r1r2.dtsi | 4 ++-- > .../boot/dts/hisilicon/hi6220-coresight.dtsi | 6 +++--- > arch/arm64/boot/dts/qcom/msm8916.dtsi | 4 ++-- > arch/arm64/boot/dts/sprd/sc9836.dtsi | 2 +- > arch/arm64/boot/dts/sprd/sc9860.dtsi | 8 ++++---- > 12 files changed, 36 insertions(+), 36 deletions(-) For the set: Reviewed-by: Mathieu Poirier > > -- > 2.17.1 >