* [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming [not found] <1497278211-5001-1-git-send-email-suzuki.poulose@arm.com> @ 2017-06-12 14:36 ` Suzuki K Poulose [not found] ` <1497278211-5001-2-git-send-email-suzuki.poulose-5wv7dgnIgG8@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Suzuki K Poulose @ 2017-06-12 14:36 UTC (permalink / raw) To: linux-arm-kernel Cc: linux-kernel, mathieu.poirier, Suzuki K Poulose, Pratik Patel, Ivan T . Ivanov, devicetree, Rob Herring, Mark Rutland The Linux coresight drivers define the programmable ATB replicator as Qualcom replicator, while this is designed by ARM. This can cause confusion to a user selecting the driver. Cleanup all references to make it explicitly clear. This patch : 1) Adds a new compatible string for the same, retaining the old one for compatibility. 2) Changes the Kconfig symbol (since this is not part of any defconfigs) CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR 3) Improves the help message in the Kconfig. 4) Changes the name of the driver : coresight-replicator-qcom => coresight-dynamic-replicator Cc: Pratik Patel <pratikp@codeaurora.org> Cc: Ivan T. Ivanov <ivan.ivanov@linaro.org> Cc: Mathieu Poirier <mathieu.poirier@linaro.org> Cc: devicetree@vger.kernel.org Cc: Rob Herring <robh+dt@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> --- Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- drivers/hwtracing/coresight/Kconfig | 10 +++++----- drivers/hwtracing/coresight/Makefile | 2 +- drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- 4 files changed, 10 insertions(+), 8 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt index fcbae6a..f77329f 100644 --- a/Documentation/devicetree/bindings/arm/coresight.txt +++ b/Documentation/devicetree/bindings/arm/coresight.txt @@ -34,7 +34,9 @@ its hardware characteristcs. - Embedded Trace Macrocell (version 4.x): "arm,coresight-etm4x", "arm,primecell"; - - Qualcomm Configurable Replicator (version 1.x): + - Coresight programmable Replicator (version 1.x): + "arm,coresight-dynamic-replicator", "arm,primecell"; + OR "qcom,coresight-replicator1x", "arm,primecell"; - System Trace Macrocell: diff --git a/drivers/hwtracing/coresight/Kconfig b/drivers/hwtracing/coresight/Kconfig index 8d55d6d..e50ea56 100644 --- a/drivers/hwtracing/coresight/Kconfig +++ b/drivers/hwtracing/coresight/Kconfig @@ -70,13 +70,13 @@ config CORESIGHT_SOURCE_ETM4X for instruction level tracing. Depending on the implemented version data tracing may also be available. -config CORESIGHT_QCOM_REPLICATOR - bool "Qualcomm CoreSight Replicator driver" +config CORESIGHT_DYNAMIC_REPLICATOR + bool "Programmable CoreSight Replicator driver" depends on CORESIGHT_LINKS_AND_SINKS help - This enables support for Qualcomm CoreSight link driver. The - programmable ATB replicator sends the ATB trace stream from the - ETB/ETF to the TPIUi and ETR. + This enables support for dynamic CoreSight replicator link driver. + The programmable ATB replicator allows independent filtering of the + trace data based on the traceid. config CORESIGHT_STM bool "CoreSight System Trace Macrocell driver" diff --git a/drivers/hwtracing/coresight/Makefile b/drivers/hwtracing/coresight/Makefile index 433d590..c7638d4 100644 --- a/drivers/hwtracing/coresight/Makefile +++ b/drivers/hwtracing/coresight/Makefile @@ -14,6 +14,6 @@ obj-$(CONFIG_CORESIGHT_SOURCE_ETM3X) += coresight-etm3x.o coresight-etm-cp14.o \ coresight-etm3x-sysfs.o obj-$(CONFIG_CORESIGHT_SOURCE_ETM4X) += coresight-etm4x.o \ coresight-etm4x-sysfs.o -obj-$(CONFIG_CORESIGHT_QCOM_REPLICATOR) += coresight-replicator-qcom.o +obj-$(CONFIG_CORESIGHT_DYNAMIC_REPLICATOR) += coresight-replicator-qcom.o obj-$(CONFIG_CORESIGHT_STM) += coresight-stm.o obj-$(CONFIG_CORESIGHT_CPU_DEBUG) += coresight-cpu-debug.o diff --git a/drivers/hwtracing/coresight/coresight-replicator-qcom.c b/drivers/hwtracing/coresight/coresight-replicator-qcom.c index 0a3d15f..b7e44d1 100644 --- a/drivers/hwtracing/coresight/coresight-replicator-qcom.c +++ b/drivers/hwtracing/coresight/coresight-replicator-qcom.c @@ -186,7 +186,7 @@ static struct amba_id replicator_ids[] = { static struct amba_driver replicator_driver = { .drv = { - .name = "coresight-replicator-qcom", + .name = "coresight-dynamic-replicator", .pm = &replicator_dev_pm_ops, .suppress_bind_attrs = true, }, -- 2.7.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
[parent not found: <1497278211-5001-2-git-send-email-suzuki.poulose-5wv7dgnIgG8@public.gmane.org>]
* Re: [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming [not found] ` <1497278211-5001-2-git-send-email-suzuki.poulose-5wv7dgnIgG8@public.gmane.org> @ 2017-06-13 16:55 ` Mathieu Poirier 2017-06-13 17:56 ` Suzuki K Poulose 2017-06-18 14:04 ` Rob Herring 0 siblings, 2 replies; 6+ messages in thread From: Mathieu Poirier @ 2017-06-13 16:55 UTC (permalink / raw) To: Suzuki K Poulose Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Pratik Patel, Ivan T . Ivanov, devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland On Mon, Jun 12, 2017 at 03:36:40PM +0100, Suzuki K Poulose wrote: > The Linux coresight drivers define the programmable ATB replicator as > Qualcom replicator, while this is designed by ARM. This can cause confusion > to a user selecting the driver. Cleanup all references to make it > explicitly clear. This patch : > > 1) Adds a new compatible string for the same, retaining the old one for > compatibility. > 2) Changes the Kconfig symbol (since this is not part of any defconfigs) > CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR > 3) Improves the help message in the Kconfig. > 4) Changes the name of the driver : > coresight-replicator-qcom => coresight-dynamic-replicator > > Cc: Pratik Patel <pratikp-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> > Cc: Ivan T. Ivanov <ivan.ivanov-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > Cc: Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> > Signed-off-by: Suzuki K Poulose <suzuki.poulose-5wv7dgnIgG8@public.gmane.org> Hi Suzuki, > --- > Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- > drivers/hwtracing/coresight/Kconfig | 10 +++++----- > drivers/hwtracing/coresight/Makefile | 2 +- > drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- > 4 files changed, 10 insertions(+), 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt > index fcbae6a..f77329f 100644 > --- a/Documentation/devicetree/bindings/arm/coresight.txt > +++ b/Documentation/devicetree/bindings/arm/coresight.txt > @@ -34,7 +34,9 @@ its hardware characteristcs. > - Embedded Trace Macrocell (version 4.x): > "arm,coresight-etm4x", "arm,primecell"; > > - - Qualcomm Configurable Replicator (version 1.x): > + - Coresight programmable Replicator (version 1.x): > + "arm,coresight-dynamic-replicator", "arm,primecell"; > + OR > "qcom,coresight-replicator1x", "arm,primecell"; Rob, what's your view on keeping the old binding around? We could simply change the two occurences we find in the DTs (Juno and 410c) to the new name and be done with the old one. > > - System Trace Macrocell: > diff --git a/drivers/hwtracing/coresight/Kconfig b/drivers/hwtracing/coresight/Kconfig > index 8d55d6d..e50ea56 100644 > --- a/drivers/hwtracing/coresight/Kconfig > +++ b/drivers/hwtracing/coresight/Kconfig > @@ -70,13 +70,13 @@ config CORESIGHT_SOURCE_ETM4X > for instruction level tracing. Depending on the implemented version > data tracing may also be available. > > -config CORESIGHT_QCOM_REPLICATOR > - bool "Qualcomm CoreSight Replicator driver" > +config CORESIGHT_DYNAMIC_REPLICATOR > + bool "Programmable CoreSight Replicator driver" > depends on CORESIGHT_LINKS_AND_SINKS > help > - This enables support for Qualcomm CoreSight link driver. The > - programmable ATB replicator sends the ATB trace stream from the > - ETB/ETF to the TPIUi and ETR. > + This enables support for dynamic CoreSight replicator link driver. > + The programmable ATB replicator allows independent filtering of the > + trace data based on the traceid. > > config CORESIGHT_STM > bool "CoreSight System Trace Macrocell driver" > diff --git a/drivers/hwtracing/coresight/Makefile b/drivers/hwtracing/coresight/Makefile > index 433d590..c7638d4 100644 > --- a/drivers/hwtracing/coresight/Makefile > +++ b/drivers/hwtracing/coresight/Makefile > @@ -14,6 +14,6 @@ obj-$(CONFIG_CORESIGHT_SOURCE_ETM3X) += coresight-etm3x.o coresight-etm-cp14.o \ > coresight-etm3x-sysfs.o > obj-$(CONFIG_CORESIGHT_SOURCE_ETM4X) += coresight-etm4x.o \ > coresight-etm4x-sysfs.o > -obj-$(CONFIG_CORESIGHT_QCOM_REPLICATOR) += coresight-replicator-qcom.o > +obj-$(CONFIG_CORESIGHT_DYNAMIC_REPLICATOR) += coresight-replicator-qcom.o > obj-$(CONFIG_CORESIGHT_STM) += coresight-stm.o > obj-$(CONFIG_CORESIGHT_CPU_DEBUG) += coresight-cpu-debug.o > diff --git a/drivers/hwtracing/coresight/coresight-replicator-qcom.c b/drivers/hwtracing/coresight/coresight-replicator-qcom.c > index 0a3d15f..b7e44d1 100644 > --- a/drivers/hwtracing/coresight/coresight-replicator-qcom.c > +++ b/drivers/hwtracing/coresight/coresight-replicator-qcom.c While at it, I would also change the name of the file. Thanks, Mathieu > @@ -186,7 +186,7 @@ static struct amba_id replicator_ids[] = { > > static struct amba_driver replicator_driver = { > .drv = { > - .name = "coresight-replicator-qcom", > + .name = "coresight-dynamic-replicator", > .pm = &replicator_dev_pm_ops, > .suppress_bind_attrs = true, > }, > -- > 2.7.4 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming 2017-06-13 16:55 ` Mathieu Poirier @ 2017-06-13 17:56 ` Suzuki K Poulose 2017-06-18 14:04 ` Rob Herring 1 sibling, 0 replies; 6+ messages in thread From: Suzuki K Poulose @ 2017-06-13 17:56 UTC (permalink / raw) To: Mathieu Poirier Cc: linux-arm-kernel, linux-kernel, Pratik Patel, Ivan T . Ivanov, devicetree, Rob Herring, Mark Rutland On 13/06/17 17:55, Mathieu Poirier wrote: > On Mon, Jun 12, 2017 at 03:36:40PM +0100, Suzuki K Poulose wrote: >> The Linux coresight drivers define the programmable ATB replicator as >> Qualcom replicator, while this is designed by ARM. This can cause confusion >> to a user selecting the driver. Cleanup all references to make it >> explicitly clear. This patch : >> >> 1) Adds a new compatible string for the same, retaining the old one for >> compatibility. >> 2) Changes the Kconfig symbol (since this is not part of any defconfigs) >> CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR >> 3) Improves the help message in the Kconfig. >> 4) Changes the name of the driver : >> coresight-replicator-qcom => coresight-dynamic-replicator >> >> Cc: Pratik Patel <pratikp@codeaurora.org> >> Cc: Ivan T. Ivanov <ivan.ivanov@linaro.org> >> Cc: Mathieu Poirier <mathieu.poirier@linaro.org> >> Cc: devicetree@vger.kernel.org >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: Mark Rutland <mark.rutland@arm.com> >> Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> > > Hi Suzuki, > >> --- >> Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- >> drivers/hwtracing/coresight/Kconfig | 10 +++++----- >> drivers/hwtracing/coresight/Makefile | 2 +- >> drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- >> 4 files changed, 10 insertions(+), 8 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt >> index fcbae6a..f77329f 100644 >> --- a/Documentation/devicetree/bindings/arm/coresight.txt >> +++ b/Documentation/devicetree/bindings/arm/coresight.txt >> @@ -34,7 +34,9 @@ its hardware characteristcs. >> - Embedded Trace Macrocell (version 4.x): >> "arm,coresight-etm4x", "arm,primecell"; >> >> - - Qualcomm Configurable Replicator (version 1.x): >> + - Coresight programmable Replicator (version 1.x): >> + "arm,coresight-dynamic-replicator", "arm,primecell"; >> + OR >> "qcom,coresight-replicator1x", "arm,primecell"; > > Rob, what's your view on keeping the old binding around? We could simply change > the two occurences we find in the DTs (Juno and 410c) to the new name and be > done with the old one. ... >> --- a/drivers/hwtracing/coresight/coresight-replicator-qcom.c >> +++ b/drivers/hwtracing/coresight/coresight-replicator-qcom.c > > While at it, I would also change the name of the file. Sure, will do. Thanks Suzuki ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming 2017-06-13 16:55 ` Mathieu Poirier 2017-06-13 17:56 ` Suzuki K Poulose @ 2017-06-18 14:04 ` Rob Herring 2017-06-20 16:44 ` Mathieu Poirier 1 sibling, 1 reply; 6+ messages in thread From: Rob Herring @ 2017-06-18 14:04 UTC (permalink / raw) To: Mathieu Poirier Cc: Suzuki K Poulose, linux-arm-kernel, linux-kernel, Pratik Patel, Ivan T . Ivanov, devicetree, Mark Rutland On Tue, Jun 13, 2017 at 10:55:28AM -0600, Mathieu Poirier wrote: > On Mon, Jun 12, 2017 at 03:36:40PM +0100, Suzuki K Poulose wrote: > > The Linux coresight drivers define the programmable ATB replicator as > > Qualcom replicator, while this is designed by ARM. This can cause confusion > > to a user selecting the driver. Cleanup all references to make it > > explicitly clear. This patch : > > > > 1) Adds a new compatible string for the same, retaining the old one for > > compatibility. > > 2) Changes the Kconfig symbol (since this is not part of any defconfigs) > > CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR > > 3) Improves the help message in the Kconfig. > > 4) Changes the name of the driver : > > coresight-replicator-qcom => coresight-dynamic-replicator > > > > Cc: Pratik Patel <pratikp@codeaurora.org> > > Cc: Ivan T. Ivanov <ivan.ivanov@linaro.org> > > Cc: Mathieu Poirier <mathieu.poirier@linaro.org> > > Cc: devicetree@vger.kernel.org > > Cc: Rob Herring <robh+dt@kernel.org> > > Cc: Mark Rutland <mark.rutland@arm.com> > > Signed-off-by: Suzuki K Poulose <suzuki.poulose@arm.com> > > Hi Suzuki, > > > --- > > Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- > > drivers/hwtracing/coresight/Kconfig | 10 +++++----- > > drivers/hwtracing/coresight/Makefile | 2 +- > > drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- > > 4 files changed, 10 insertions(+), 8 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt > > index fcbae6a..f77329f 100644 > > --- a/Documentation/devicetree/bindings/arm/coresight.txt > > +++ b/Documentation/devicetree/bindings/arm/coresight.txt > > @@ -34,7 +34,9 @@ its hardware characteristcs. > > - Embedded Trace Macrocell (version 4.x): > > "arm,coresight-etm4x", "arm,primecell"; > > > > - - Qualcomm Configurable Replicator (version 1.x): > > + - Coresight programmable Replicator (version 1.x): > > + "arm,coresight-dynamic-replicator", "arm,primecell"; > > + OR > > "qcom,coresight-replicator1x", "arm,primecell"; > > Rob, what's your view on keeping the old binding around? We could simply change > the two occurences we find in the DTs (Juno and 410c) to the new name and be > done with the old one. Juno uses the Qcom string? We should keep the old string. You can switch the dts files, but the driver should support the old name. Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming 2017-06-18 14:04 ` Rob Herring @ 2017-06-20 16:44 ` Mathieu Poirier [not found] ` <CANLsYkzDMyhX_L5Aaf2p9+ze_qA=FCe+GdjEZcz03ZXaHXb46Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Mathieu Poirier @ 2017-06-20 16:44 UTC (permalink / raw) To: Rob Herring Cc: Suzuki K Poulose, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pratik Patel, Ivan T . Ivanov, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mark Rutland On 18 June 2017 at 08:04, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: > On Tue, Jun 13, 2017 at 10:55:28AM -0600, Mathieu Poirier wrote: >> On Mon, Jun 12, 2017 at 03:36:40PM +0100, Suzuki K Poulose wrote: >> > The Linux coresight drivers define the programmable ATB replicator as >> > Qualcom replicator, while this is designed by ARM. This can cause confusion >> > to a user selecting the driver. Cleanup all references to make it >> > explicitly clear. This patch : >> > >> > 1) Adds a new compatible string for the same, retaining the old one for >> > compatibility. >> > 2) Changes the Kconfig symbol (since this is not part of any defconfigs) >> > CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR >> > 3) Improves the help message in the Kconfig. >> > 4) Changes the name of the driver : >> > coresight-replicator-qcom => coresight-dynamic-replicator >> > >> > Cc: Pratik Patel <pratikp-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> >> > Cc: Ivan T. Ivanov <ivan.ivanov-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >> > Cc: Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >> > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> > Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> >> > Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> >> > Signed-off-by: Suzuki K Poulose <suzuki.poulose-5wv7dgnIgG8@public.gmane.org> >> >> Hi Suzuki, >> >> > --- >> > Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- >> > drivers/hwtracing/coresight/Kconfig | 10 +++++----- >> > drivers/hwtracing/coresight/Makefile | 2 +- >> > drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- >> > 4 files changed, 10 insertions(+), 8 deletions(-) >> > >> > diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt >> > index fcbae6a..f77329f 100644 >> > --- a/Documentation/devicetree/bindings/arm/coresight.txt >> > +++ b/Documentation/devicetree/bindings/arm/coresight.txt >> > @@ -34,7 +34,9 @@ its hardware characteristcs. >> > - Embedded Trace Macrocell (version 4.x): >> > "arm,coresight-etm4x", "arm,primecell"; >> > >> > - - Qualcomm Configurable Replicator (version 1.x): >> > + - Coresight programmable Replicator (version 1.x): >> > + "arm,coresight-dynamic-replicator", "arm,primecell"; >> > + OR >> > "qcom,coresight-replicator1x", "arm,primecell"; >> >> Rob, what's your view on keeping the old binding around? We could simply change >> the two occurences we find in the DTs (Juno and 410c) to the new name and be >> done with the old one. > > Juno uses the Qcom string? We should keep the old string. You can switch > the dts files, but the driver should support the old name. When we first started working on CoreSight programmable replicators were available but the documentation wasn't public. As such when I saw Qualcomm's design I mistakenly thought it was a custom IP block and came up with a compatible string that reflected that reality. Fast forward 3 years the documentation is available and Juno has used the same IP block in their design. Suzuki's patch rectifies history by changing the programmable replicator naming convention to what it should have been from the start. That being said, we can keep the old compatible string around but it won't change anything. CoreSight devices are discovered on the AMBA bus and don't use the compatible string - drivers are probed based on AMBA IDs laid out in the drivers and device IDs found in HW ID registers. In light of the above let me know what you want to do. Thanks, Mathieu > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CANLsYkzDMyhX_L5Aaf2p9+ze_qA=FCe+GdjEZcz03ZXaHXb46Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming [not found] ` <CANLsYkzDMyhX_L5Aaf2p9+ze_qA=FCe+GdjEZcz03ZXaHXb46Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> @ 2017-06-22 3:21 ` Rob Herring 0 siblings, 0 replies; 6+ messages in thread From: Rob Herring @ 2017-06-22 3:21 UTC (permalink / raw) To: Mathieu Poirier Cc: Suzuki K Poulose, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pratik Patel, Ivan T . Ivanov, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mark Rutland On Tue, Jun 20, 2017 at 11:44 AM, Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: > On 18 June 2017 at 08:04, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote: >> On Tue, Jun 13, 2017 at 10:55:28AM -0600, Mathieu Poirier wrote: >>> On Mon, Jun 12, 2017 at 03:36:40PM +0100, Suzuki K Poulose wrote: >>> > The Linux coresight drivers define the programmable ATB replicator as >>> > Qualcom replicator, while this is designed by ARM. This can cause confusion >>> > to a user selecting the driver. Cleanup all references to make it >>> > explicitly clear. This patch : >>> > >>> > 1) Adds a new compatible string for the same, retaining the old one for >>> > compatibility. >>> > 2) Changes the Kconfig symbol (since this is not part of any defconfigs) >>> > CORESIGHT_QCOM_REPLICATOR => CORESIGHT_DYNAMIC_REPLICATOR >>> > 3) Improves the help message in the Kconfig. >>> > 4) Changes the name of the driver : >>> > coresight-replicator-qcom => coresight-dynamic-replicator >>> > >>> > Cc: Pratik Patel <pratikp-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org> >>> > Cc: Ivan T. Ivanov <ivan.ivanov-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>> > Cc: Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> >>> > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>> > Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> >>> > Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> >>> > Signed-off-by: Suzuki K Poulose <suzuki.poulose-5wv7dgnIgG8@public.gmane.org> >>> >>> Hi Suzuki, >>> >>> > --- >>> > Documentation/devicetree/bindings/arm/coresight.txt | 4 +++- >>> > drivers/hwtracing/coresight/Kconfig | 10 +++++----- >>> > drivers/hwtracing/coresight/Makefile | 2 +- >>> > drivers/hwtracing/coresight/coresight-replicator-qcom.c | 2 +- >>> > 4 files changed, 10 insertions(+), 8 deletions(-) >>> > >>> > diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt >>> > index fcbae6a..f77329f 100644 >>> > --- a/Documentation/devicetree/bindings/arm/coresight.txt >>> > +++ b/Documentation/devicetree/bindings/arm/coresight.txt >>> > @@ -34,7 +34,9 @@ its hardware characteristcs. >>> > - Embedded Trace Macrocell (version 4.x): >>> > "arm,coresight-etm4x", "arm,primecell"; >>> > >>> > - - Qualcomm Configurable Replicator (version 1.x): >>> > + - Coresight programmable Replicator (version 1.x): >>> > + "arm,coresight-dynamic-replicator", "arm,primecell"; >>> > + OR >>> > "qcom,coresight-replicator1x", "arm,primecell"; >>> >>> Rob, what's your view on keeping the old binding around? We could simply change >>> the two occurences we find in the DTs (Juno and 410c) to the new name and be >>> done with the old one. >> >> Juno uses the Qcom string? We should keep the old string. You can switch >> the dts files, but the driver should support the old name. > > When we first started working on CoreSight programmable replicators > were available but the documentation wasn't public. As such when I > saw Qualcomm's design I mistakenly thought it was a custom IP block > and came up with a compatible string that reflected that reality. > Fast forward 3 years the documentation is available and Juno has used > the same IP block in their design. Suzuki's patch rectifies history > by changing the programmable replicator naming convention to what it > should have been from the start. > > That being said, we can keep the old compatible string around but it > won't change anything. CoreSight devices are discovered on the AMBA > bus and don't use the compatible string - drivers are probed based on > AMBA IDs laid out in the drivers and device IDs found in HW ID > registers. > > In light of the above let me know what you want to do. Well, if drivers don't use the string, then there is nothing to keep around. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-06-22 3:21 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1497278211-5001-1-git-send-email-suzuki.poulose@arm.com>
2017-06-12 14:36 ` [PATCH 01/12] coresight replicator: Cleanup programmable replicator naming Suzuki K Poulose
[not found] ` <1497278211-5001-2-git-send-email-suzuki.poulose-5wv7dgnIgG8@public.gmane.org>
2017-06-13 16:55 ` Mathieu Poirier
2017-06-13 17:56 ` Suzuki K Poulose
2017-06-18 14:04 ` Rob Herring
2017-06-20 16:44 ` Mathieu Poirier
[not found] ` <CANLsYkzDMyhX_L5Aaf2p9+ze_qA=FCe+GdjEZcz03ZXaHXb46Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-06-22 3:21 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).