From: Dan Carpenter <dan.carpenter@oracle.com>
To: Eduardo Valentin <eduardo.valentin@ti.com>
Cc: gregkh@linuxfoundation.org, devel@driverdev.osuosl.org,
Nishanth Menon <nm@ti.com>, Benoit <b-cousson@ti.com>,
linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
Santosh Shilimkar <santosh.shilimkar@ti.com>,
linux-omap@vger.kernel.org
Subject: Re: [PATCH 4/8] staging: rename omap-thermal driver to ti-soc-thermal
Date: Mon, 18 Mar 2013 22:17:10 +0300 [thread overview]
Message-ID: <20130318191710.GV9138@mwanda> (raw)
In-Reply-To: <1363618756-15851-5-git-send-email-eduardo.valentin@ti.com>
On Mon, Mar 18, 2013 at 10:59:12AM -0400, Eduardo Valentin wrote:
> Because this driver will support also OMAP derivatives,
> this patch does a big rename inside this driver, so it
> better fits its usage.
>
It would be better to do a minimal move patch which just renames the
files and updated the Makefile. The following patches would rename
the functions. That makes it easier to review.
> Cc: Santosh Shilimkar <santosh.shilimkar@ti.com>
> Cc: Benoit <b-cousson@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
>
> Signed-off-by: Eduardo Valentin <eduardo.valentin@ti.com>
Put a --- line here after the Signed-off-by line so that "git am"
doesn't include the diff stat in the changelog.
regards,
dan carpenter
next prev parent reply other threads:[~2013-03-18 19:17 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-18 14:59 [PATCH 0/8] staging: [omap,ti-soc]-thermal: fixes and renaming Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 1/8] staging: omap-thermal: fix return value Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 16:39 ` Dan Carpenter
2013-03-18 17:20 ` Eduardo Valentin
2013-03-18 17:20 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 2/8] staging: omap-thermal: use spin_lock_irqsave inside IRQ handler Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 19:16 ` Dan Carpenter
2013-03-18 19:38 ` Eduardo Valentin
2013-03-18 19:38 ` Eduardo Valentin
2013-03-18 19:58 ` Dan Carpenter
2013-03-18 14:59 ` [PATCH 3/8] staging: omap-thermal: rename bg_ptr to bgp Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 4/8] staging: rename omap-thermal driver to ti-soc-thermal Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 19:17 ` Dan Carpenter [this message]
2013-03-18 14:59 ` [PATCH 5/8] staging: ti-soc-thermal: make unexported functions local Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 6/8] staging: ti-soc-thermal: split writable data from readonly data Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 7/8] stating: ti-soc-thermal: use sizeof(*pointer) while allocating Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-18 14:59 ` [PATCH 8/8] staging: ti-soc-thermal: fix several kernel-doc warnings and error Eduardo Valentin
2013-03-18 14:59 ` Eduardo Valentin
2013-03-19 14:36 ` [PATCH 0/8] staging: [omap,ti-soc]-thermal: fixes and renaming Eduardo Valentin
2013-03-19 14:36 ` Eduardo Valentin
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=20130318191710.GV9138@mwanda \
--to=dan.carpenter@oracle.com \
--cc=b-cousson@ti.com \
--cc=devel@driverdev.osuosl.org \
--cc=eduardo.valentin@ti.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=nm@ti.com \
--cc=santosh.shilimkar@ti.com \
/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.