All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
To: Mathieu Poirier
	<mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
	catalin.marinas-5wv7dgnIgG8@public.gmane.org,
	will.deacon-5wv7dgnIgG8@public.gmane.org,
	alexander.shishkin-VuQAYsv1563Yd54FQh9/CA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	peter.lachner-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	norbert.schulz-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	keven.boell-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	yann.fouassier-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	laurent.fert-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	kaixu.xia-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	zhang.chunyan-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org
Subject: Re: [PATCH] coresight: moving to new "hwtracing" directory
Date: Thu, 19 Mar 2015 23:27:41 +0100	[thread overview]
Message-ID: <20150319222741.GA24999@kroah.com> (raw)
In-Reply-To: <1426802522-25743-1-git-send-email-mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>

On Thu, Mar 19, 2015 at 04:02:02PM -0600, Mathieu Poirier wrote:
> Keeping drivers related to HW tracing on ARM, i.e coresight,
> under "drivers/coresight" doesn't make sense when other
> architectures start rolling out technologies of the same
> nature.
> 
> As such creating a new "drivers/hwtracing" directory where all
> drivers of the same kind can reside, reducing namespace
> pollution under "drivers/".
> 
> Signed-off-by: Mathieu Poirier <mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> ---
>  MAINTAINERS                                        |    2 +-
>  arch/arm/Kconfig.debug                             |    2 +-
>  arch/arm64/Kconfig.debug                           |    2 +-
>  drivers/Makefile                                   |    2 +-
>  drivers/coresight/Kconfig                          |   61 -
>  drivers/coresight/Makefile                         |   11 -
>  drivers/coresight/coresight-etb10.c                |  527 ------
>  drivers/coresight/coresight-etm-cp14.c             |  591 ------
>  drivers/coresight/coresight-etm.h                  |  251 ---
>  drivers/coresight/coresight-etm3x.c                | 1932 --------------------
>  drivers/coresight/coresight-funnel.c               |  258 ---
>  drivers/coresight/coresight-priv.h                 |   63 -
>  drivers/coresight/coresight-replicator.c           |  137 --
>  drivers/coresight/coresight-tmc.c                  |  822 ---------
>  drivers/coresight/coresight-tpiu.c                 |  207 ---
>  drivers/coresight/coresight.c                      |  720 --------
>  drivers/coresight/of_coresight.c                   |  200 --
>  drivers/hwtracing/coresight/Kconfig                |   61 +
>  drivers/hwtracing/coresight/Makefile               |   11 +
>  drivers/hwtracing/coresight/coresight-etb10.c      |  527 ++++++
>  drivers/hwtracing/coresight/coresight-etm-cp14.c   |  591 ++++++
>  drivers/hwtracing/coresight/coresight-etm.h        |  251 +++
>  drivers/hwtracing/coresight/coresight-etm3x.c      | 1932 ++++++++++++++++++++
>  drivers/hwtracing/coresight/coresight-funnel.c     |  258 +++
>  drivers/hwtracing/coresight/coresight-priv.h       |   63 +
>  drivers/hwtracing/coresight/coresight-replicator.c |  137 ++
>  drivers/hwtracing/coresight/coresight-tmc.c        |  822 +++++++++
>  drivers/hwtracing/coresight/coresight-tpiu.c       |  207 +++
>  drivers/hwtracing/coresight/coresight.c            |  720 ++++++++
>  drivers/hwtracing/coresight/of_coresight.c         |  200 ++
>  30 files changed, 5784 insertions(+), 5784 deletions(-)
>  delete mode 100644 drivers/coresight/Kconfig
>  delete mode 100644 drivers/coresight/Makefile
>  delete mode 100644 drivers/coresight/coresight-etb10.c
>  delete mode 100644 drivers/coresight/coresight-etm-cp14.c
>  delete mode 100644 drivers/coresight/coresight-etm.h
>  delete mode 100644 drivers/coresight/coresight-etm3x.c
>  delete mode 100644 drivers/coresight/coresight-funnel.c
>  delete mode 100644 drivers/coresight/coresight-priv.h
>  delete mode 100644 drivers/coresight/coresight-replicator.c
>  delete mode 100644 drivers/coresight/coresight-tmc.c
>  delete mode 100644 drivers/coresight/coresight-tpiu.c
>  delete mode 100644 drivers/coresight/coresight.c
>  delete mode 100644 drivers/coresight/of_coresight.c
>  create mode 100644 drivers/hwtracing/coresight/Kconfig
>  create mode 100644 drivers/hwtracing/coresight/Makefile
>  create mode 100644 drivers/hwtracing/coresight/coresight-etb10.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm-cp14.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm3x.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-funnel.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-priv.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-replicator.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tmc.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tpiu.c
>  create mode 100644 drivers/hwtracing/coresight/coresight.c
>  create mode 100644 drivers/hwtracing/coresight/of_coresight.c

This looks good to me, but please create your patches with -M option to
git, so it just moves the code, and we can easily see that it is a move
and nothing else changed.

thanks,

greg k-h

WARNING: multiple messages have this Message-ID (diff)
From: gregkh@linuxfoundation.org (Greg KH)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] coresight: moving to new "hwtracing" directory
Date: Thu, 19 Mar 2015 23:27:41 +0100	[thread overview]
Message-ID: <20150319222741.GA24999@kroah.com> (raw)
In-Reply-To: <1426802522-25743-1-git-send-email-mathieu.poirier@linaro.org>

On Thu, Mar 19, 2015 at 04:02:02PM -0600, Mathieu Poirier wrote:
> Keeping drivers related to HW tracing on ARM, i.e coresight,
> under "drivers/coresight" doesn't make sense when other
> architectures start rolling out technologies of the same
> nature.
> 
> As such creating a new "drivers/hwtracing" directory where all
> drivers of the same kind can reside, reducing namespace
> pollution under "drivers/".
> 
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
>  MAINTAINERS                                        |    2 +-
>  arch/arm/Kconfig.debug                             |    2 +-
>  arch/arm64/Kconfig.debug                           |    2 +-
>  drivers/Makefile                                   |    2 +-
>  drivers/coresight/Kconfig                          |   61 -
>  drivers/coresight/Makefile                         |   11 -
>  drivers/coresight/coresight-etb10.c                |  527 ------
>  drivers/coresight/coresight-etm-cp14.c             |  591 ------
>  drivers/coresight/coresight-etm.h                  |  251 ---
>  drivers/coresight/coresight-etm3x.c                | 1932 --------------------
>  drivers/coresight/coresight-funnel.c               |  258 ---
>  drivers/coresight/coresight-priv.h                 |   63 -
>  drivers/coresight/coresight-replicator.c           |  137 --
>  drivers/coresight/coresight-tmc.c                  |  822 ---------
>  drivers/coresight/coresight-tpiu.c                 |  207 ---
>  drivers/coresight/coresight.c                      |  720 --------
>  drivers/coresight/of_coresight.c                   |  200 --
>  drivers/hwtracing/coresight/Kconfig                |   61 +
>  drivers/hwtracing/coresight/Makefile               |   11 +
>  drivers/hwtracing/coresight/coresight-etb10.c      |  527 ++++++
>  drivers/hwtracing/coresight/coresight-etm-cp14.c   |  591 ++++++
>  drivers/hwtracing/coresight/coresight-etm.h        |  251 +++
>  drivers/hwtracing/coresight/coresight-etm3x.c      | 1932 ++++++++++++++++++++
>  drivers/hwtracing/coresight/coresight-funnel.c     |  258 +++
>  drivers/hwtracing/coresight/coresight-priv.h       |   63 +
>  drivers/hwtracing/coresight/coresight-replicator.c |  137 ++
>  drivers/hwtracing/coresight/coresight-tmc.c        |  822 +++++++++
>  drivers/hwtracing/coresight/coresight-tpiu.c       |  207 +++
>  drivers/hwtracing/coresight/coresight.c            |  720 ++++++++
>  drivers/hwtracing/coresight/of_coresight.c         |  200 ++
>  30 files changed, 5784 insertions(+), 5784 deletions(-)
>  delete mode 100644 drivers/coresight/Kconfig
>  delete mode 100644 drivers/coresight/Makefile
>  delete mode 100644 drivers/coresight/coresight-etb10.c
>  delete mode 100644 drivers/coresight/coresight-etm-cp14.c
>  delete mode 100644 drivers/coresight/coresight-etm.h
>  delete mode 100644 drivers/coresight/coresight-etm3x.c
>  delete mode 100644 drivers/coresight/coresight-funnel.c
>  delete mode 100644 drivers/coresight/coresight-priv.h
>  delete mode 100644 drivers/coresight/coresight-replicator.c
>  delete mode 100644 drivers/coresight/coresight-tmc.c
>  delete mode 100644 drivers/coresight/coresight-tpiu.c
>  delete mode 100644 drivers/coresight/coresight.c
>  delete mode 100644 drivers/coresight/of_coresight.c
>  create mode 100644 drivers/hwtracing/coresight/Kconfig
>  create mode 100644 drivers/hwtracing/coresight/Makefile
>  create mode 100644 drivers/hwtracing/coresight/coresight-etb10.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm-cp14.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm3x.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-funnel.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-priv.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-replicator.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tmc.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tpiu.c
>  create mode 100644 drivers/hwtracing/coresight/coresight.c
>  create mode 100644 drivers/hwtracing/coresight/of_coresight.c

This looks good to me, but please create your patches with -M option to
git, so it just moves the code, and we can easily see that it is a move
and nothing else changed.

thanks,

greg k-h

WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <gregkh@linuxfoundation.org>
To: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: linux@arm.linux.org.uk, catalin.marinas@arm.com,
	will.deacon@arm.com, alexander.shishkin@linux.intel.com,
	linux-kernel@vger.kernel.org, peter.lachner@intel.com,
	norbert.schulz@intel.com, keven.boell@intel.com,
	yann.fouassier@intel.com, laurent.fert@intel.com,
	linux-api@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	kaixu.xia@linaro.org, zhang.chunyan@linaro.org
Subject: Re: [PATCH] coresight: moving to new "hwtracing" directory
Date: Thu, 19 Mar 2015 23:27:41 +0100	[thread overview]
Message-ID: <20150319222741.GA24999@kroah.com> (raw)
In-Reply-To: <1426802522-25743-1-git-send-email-mathieu.poirier@linaro.org>

On Thu, Mar 19, 2015 at 04:02:02PM -0600, Mathieu Poirier wrote:
> Keeping drivers related to HW tracing on ARM, i.e coresight,
> under "drivers/coresight" doesn't make sense when other
> architectures start rolling out technologies of the same
> nature.
> 
> As such creating a new "drivers/hwtracing" directory where all
> drivers of the same kind can reside, reducing namespace
> pollution under "drivers/".
> 
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
>  MAINTAINERS                                        |    2 +-
>  arch/arm/Kconfig.debug                             |    2 +-
>  arch/arm64/Kconfig.debug                           |    2 +-
>  drivers/Makefile                                   |    2 +-
>  drivers/coresight/Kconfig                          |   61 -
>  drivers/coresight/Makefile                         |   11 -
>  drivers/coresight/coresight-etb10.c                |  527 ------
>  drivers/coresight/coresight-etm-cp14.c             |  591 ------
>  drivers/coresight/coresight-etm.h                  |  251 ---
>  drivers/coresight/coresight-etm3x.c                | 1932 --------------------
>  drivers/coresight/coresight-funnel.c               |  258 ---
>  drivers/coresight/coresight-priv.h                 |   63 -
>  drivers/coresight/coresight-replicator.c           |  137 --
>  drivers/coresight/coresight-tmc.c                  |  822 ---------
>  drivers/coresight/coresight-tpiu.c                 |  207 ---
>  drivers/coresight/coresight.c                      |  720 --------
>  drivers/coresight/of_coresight.c                   |  200 --
>  drivers/hwtracing/coresight/Kconfig                |   61 +
>  drivers/hwtracing/coresight/Makefile               |   11 +
>  drivers/hwtracing/coresight/coresight-etb10.c      |  527 ++++++
>  drivers/hwtracing/coresight/coresight-etm-cp14.c   |  591 ++++++
>  drivers/hwtracing/coresight/coresight-etm.h        |  251 +++
>  drivers/hwtracing/coresight/coresight-etm3x.c      | 1932 ++++++++++++++++++++
>  drivers/hwtracing/coresight/coresight-funnel.c     |  258 +++
>  drivers/hwtracing/coresight/coresight-priv.h       |   63 +
>  drivers/hwtracing/coresight/coresight-replicator.c |  137 ++
>  drivers/hwtracing/coresight/coresight-tmc.c        |  822 +++++++++
>  drivers/hwtracing/coresight/coresight-tpiu.c       |  207 +++
>  drivers/hwtracing/coresight/coresight.c            |  720 ++++++++
>  drivers/hwtracing/coresight/of_coresight.c         |  200 ++
>  30 files changed, 5784 insertions(+), 5784 deletions(-)
>  delete mode 100644 drivers/coresight/Kconfig
>  delete mode 100644 drivers/coresight/Makefile
>  delete mode 100644 drivers/coresight/coresight-etb10.c
>  delete mode 100644 drivers/coresight/coresight-etm-cp14.c
>  delete mode 100644 drivers/coresight/coresight-etm.h
>  delete mode 100644 drivers/coresight/coresight-etm3x.c
>  delete mode 100644 drivers/coresight/coresight-funnel.c
>  delete mode 100644 drivers/coresight/coresight-priv.h
>  delete mode 100644 drivers/coresight/coresight-replicator.c
>  delete mode 100644 drivers/coresight/coresight-tmc.c
>  delete mode 100644 drivers/coresight/coresight-tpiu.c
>  delete mode 100644 drivers/coresight/coresight.c
>  delete mode 100644 drivers/coresight/of_coresight.c
>  create mode 100644 drivers/hwtracing/coresight/Kconfig
>  create mode 100644 drivers/hwtracing/coresight/Makefile
>  create mode 100644 drivers/hwtracing/coresight/coresight-etb10.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm-cp14.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-etm3x.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-funnel.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-priv.h
>  create mode 100644 drivers/hwtracing/coresight/coresight-replicator.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tmc.c
>  create mode 100644 drivers/hwtracing/coresight/coresight-tpiu.c
>  create mode 100644 drivers/hwtracing/coresight/coresight.c
>  create mode 100644 drivers/hwtracing/coresight/of_coresight.c

This looks good to me, but please create your patches with -M option to
git, so it just moves the code, and we can easily see that it is a move
and nothing else changed.

thanks,

greg k-h

  parent reply	other threads:[~2015-03-19 22:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-19 22:02 [PATCH] coresight: moving to new "hwtracing" directory Mathieu Poirier
     [not found] ` <1426802522-25743-1-git-send-email-mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2015-03-19 22:27   ` Greg KH [this message]
2015-03-19 22:27     ` Greg KH
2015-03-19 22:27     ` Greg KH
     [not found]     ` <20150319222741.GA24999-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2015-03-20  0:21       ` Mathieu Poirier
2015-03-20  0:21         ` Mathieu Poirier
2015-03-20  0:21         ` Mathieu Poirier
     [not found]         ` <CANLsYkx07_xYcXYbj=TwvUpaDRo9=ppLqsg69e2ju2nO-MArwA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-03-20  7:17           ` Greg KH
2015-03-20  7:17             ` Greg KH
2015-03-20  7:17             ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150319222741.GA24999@kroah.com \
    --to=gregkh-hqyy1w1ycw8ekmwlsbkhg0b+6bgklq7r@public.gmane.org \
    --cc=alexander.shishkin-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=catalin.marinas-5wv7dgnIgG8@public.gmane.org \
    --cc=kaixu.xia-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=keven.boell-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=laurent.fert-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=mathieu.poirier-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=norbert.schulz-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=peter.lachner-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=will.deacon-5wv7dgnIgG8@public.gmane.org \
    --cc=yann.fouassier-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=zhang.chunyan-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.