Re: [PATCH v2 00/11] dts: Update DT bindings for CoreSight replicator and funnel

[Mathieu Poirier]

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 <robh+dt@xxxxxxxxxx>
> Cc: Mark Rutland <mark.rutland@xxxxxxx>
> Cc: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
> Cc: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
> Cc: Liviu Dudau <liviu.dudau@xxxxxxx>
> Cc: Sudeep Holla <sudeep.holla@xxxxxxx>
> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
> Cc: Wei Xu <xuwei5@xxxxxxxxxxxxx>
> Cc: Guodong Xu <guodong.xu@xxxxxxxxxx>
> Cc: Zhangfei Gao <zhangfei.gao@xxxxxxxxxx>
> Cc: Haojian Zhuang <haojian.zhuang@xxxxxxxxxx>
> Cc: Shawn Guo <shawnguo@xxxxxxxxxx>
> Cc: Chris Healy <cphealy@xxxxxxxxx>
> Cc: Andrew Lunn <andrew@xxxxxxx>
> Cc: Fabio Estevam <festevam@xxxxxxxxx>
> Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> Cc: Andy Gross <agross@xxxxxxxxxx>
> Cc: David Brown <david.brown@xxxxxxxxxx>
> Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> Cc: Lee Jones <lee.jones@xxxxxxxxxx>
> Cc: Chunyan Zhang <zhang.chunyan@xxxxxxxxxx>
> Cc: Orson Zhai <orsonzhai@xxxxxxxxx>
> Cc: Baolin Wang <baolin.wang@xxxxxxxxxx>
> 
> 
> 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 <mathieu.poirier@xxxxxxxxxx>

> 
> -- 
> 2.17.1
>