From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Joe Perches <joe@perches.com>
Cc: Jiri Kosina <trivial@kernel.org>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Vinod Koul <vinod.koul@intel.com>,
Matt Porter <mporter@kernel.crashing.org>,
Nicolas Ferre <nicolas.ferre@atmel.com>,
linux-kernel@vger.kernel.org, Benoit Parrot <bparrot@ti.com>,
Alexandre Bounine <alexandre.bounine@idt.com>,
linux-nvdimm@lists.01.org, dmaengine@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-media@vger.kernel.org
Subject: Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Date: Mon, 7 Mar 2016 09:08:00 -0700 [thread overview]
Message-ID: <20160307160800.GA9185@linux.intel.com> (raw)
In-Reply-To: <dcdebc08a17023fac93595312281cb8e91185668.1457162650.git.joe@perches.com>
On Fri, Mar 04, 2016 at 11:46:32PM -0800, Joe Perches wrote:
> Since commit 3cab1e711297 ("lib/vsprintf: refactor duplicate code
> to special_hex_number()") %pa uses have been ouput with a 0x prefix.
>
> These 0x prefixes in the formats are unnecessary.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/dma/at_hdmac_regs.h | 2 +-
> drivers/media/platform/ti-vpe/cal.c | 2 +-
> drivers/nvdimm/pmem.c | 2 +-
> drivers/rapidio/devices/rio_mport_cdev.c | 4 ++--
> drivers/rapidio/devices/tsi721.c | 8 ++++----
> 5 files changed, 9 insertions(+), 9 deletions(-)
<>
> diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
> index 8d0b546..eb619d1 100644
> --- a/drivers/nvdimm/pmem.c
> +++ b/drivers/nvdimm/pmem.c
> @@ -172,7 +172,7 @@ static struct pmem_device *pmem_alloc(struct device *dev,
>
> if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size,
> dev_name(dev))) {
> - dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n",
> + dev_warn(dev, "could not reserve region [%pa:0x%zx]\n",
> &pmem->phys_addr, pmem->size);
> return ERR_PTR(-EBUSY);
> }
For the pmem part:
Acked-by: Ross Zwisler <ross.zwisler@linux.intel.com>
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
WARNING: multiple messages have this Message-ID (diff)
From: ross.zwisler@linux.intel.com (Ross Zwisler)
To: linux-arm-kernel@lists.infradead.org
Subject: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Date: Mon, 7 Mar 2016 09:08:00 -0700 [thread overview]
Message-ID: <20160307160800.GA9185@linux.intel.com> (raw)
In-Reply-To: <dcdebc08a17023fac93595312281cb8e91185668.1457162650.git.joe@perches.com>
On Fri, Mar 04, 2016 at 11:46:32PM -0800, Joe Perches wrote:
> Since commit 3cab1e711297 ("lib/vsprintf: refactor duplicate code
> to special_hex_number()") %pa uses have been ouput with a 0x prefix.
>
> These 0x prefixes in the formats are unnecessary.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/dma/at_hdmac_regs.h | 2 +-
> drivers/media/platform/ti-vpe/cal.c | 2 +-
> drivers/nvdimm/pmem.c | 2 +-
> drivers/rapidio/devices/rio_mport_cdev.c | 4 ++--
> drivers/rapidio/devices/tsi721.c | 8 ++++----
> 5 files changed, 9 insertions(+), 9 deletions(-)
<>
> diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
> index 8d0b546..eb619d1 100644
> --- a/drivers/nvdimm/pmem.c
> +++ b/drivers/nvdimm/pmem.c
> @@ -172,7 +172,7 @@ static struct pmem_device *pmem_alloc(struct device *dev,
>
> if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size,
> dev_name(dev))) {
> - dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n",
> + dev_warn(dev, "could not reserve region [%pa:0x%zx]\n",
> &pmem->phys_addr, pmem->size);
> return ERR_PTR(-EBUSY);
> }
For the pmem part:
Acked-by: Ross Zwisler <ross.zwisler@linux.intel.com>
WARNING: multiple messages have this Message-ID (diff)
From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Joe Perches <joe@perches.com>
Cc: Nicolas Ferre <nicolas.ferre@atmel.com>,
Vinod Koul <vinod.koul@intel.com>,
Dan Williams <dan.j.williams@intel.com>,
Benoit Parrot <bparrot@ti.com>,
Ross Zwisler <ross.zwisler@linux.intel.com>,
Jiri Kosina <trivial@kernel.org>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Matt Porter <mporter@kernel.crashing.org>,
Alexandre Bounine <alexandre.bounine@idt.com>,
linux-arm-kernel@lists.infradead.org, dmaengine@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-nvdimm@lists.01.org
Subject: Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Date: Mon, 7 Mar 2016 09:08:00 -0700 [thread overview]
Message-ID: <20160307160800.GA9185@linux.intel.com> (raw)
In-Reply-To: <dcdebc08a17023fac93595312281cb8e91185668.1457162650.git.joe@perches.com>
On Fri, Mar 04, 2016 at 11:46:32PM -0800, Joe Perches wrote:
> Since commit 3cab1e711297 ("lib/vsprintf: refactor duplicate code
> to special_hex_number()") %pa uses have been ouput with a 0x prefix.
>
> These 0x prefixes in the formats are unnecessary.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/dma/at_hdmac_regs.h | 2 +-
> drivers/media/platform/ti-vpe/cal.c | 2 +-
> drivers/nvdimm/pmem.c | 2 +-
> drivers/rapidio/devices/rio_mport_cdev.c | 4 ++--
> drivers/rapidio/devices/tsi721.c | 8 ++++----
> 5 files changed, 9 insertions(+), 9 deletions(-)
<>
> diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
> index 8d0b546..eb619d1 100644
> --- a/drivers/nvdimm/pmem.c
> +++ b/drivers/nvdimm/pmem.c
> @@ -172,7 +172,7 @@ static struct pmem_device *pmem_alloc(struct device *dev,
>
> if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size,
> dev_name(dev))) {
> - dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n",
> + dev_warn(dev, "could not reserve region [%pa:0x%zx]\n",
> &pmem->phys_addr, pmem->size);
> return ERR_PTR(-EBUSY);
> }
For the pmem part:
Acked-by: Ross Zwisler <ross.zwisler@linux.intel.com>
WARNING: multiple messages have this Message-ID (diff)
From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Joe Perches <joe@perches.com>
Cc: Nicolas Ferre <nicolas.ferre@atmel.com>,
Vinod Koul <vinod.koul@intel.com>,
Dan Williams <dan.j.williams@intel.com>,
Benoit Parrot <bparrot@ti.com>,
Ross Zwisler <ross.zwisler@linux.intel.com>,
Jiri Kosina <trivial@kernel.org>,
Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
Matt Porter <mporter@kernel.crashing.org>,
Alexandre Bounine <alexandre.bounine@idt.com>,
linux-arm-kernel@lists.infradead.org, dmaengine@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
linux-nvdimm@ml01.01.org
Subject: Re: [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses
Date: Mon, 7 Mar 2016 09:08:00 -0700 [thread overview]
Message-ID: <20160307160800.GA9185@linux.intel.com> (raw)
In-Reply-To: <dcdebc08a17023fac93595312281cb8e91185668.1457162650.git.joe@perches.com>
On Fri, Mar 04, 2016 at 11:46:32PM -0800, Joe Perches wrote:
> Since commit 3cab1e711297 ("lib/vsprintf: refactor duplicate code
> to special_hex_number()") %pa uses have been ouput with a 0x prefix.
>
> These 0x prefixes in the formats are unnecessary.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/dma/at_hdmac_regs.h | 2 +-
> drivers/media/platform/ti-vpe/cal.c | 2 +-
> drivers/nvdimm/pmem.c | 2 +-
> drivers/rapidio/devices/rio_mport_cdev.c | 4 ++--
> drivers/rapidio/devices/tsi721.c | 8 ++++----
> 5 files changed, 9 insertions(+), 9 deletions(-)
<>
> diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
> index 8d0b546..eb619d1 100644
> --- a/drivers/nvdimm/pmem.c
> +++ b/drivers/nvdimm/pmem.c
> @@ -172,7 +172,7 @@ static struct pmem_device *pmem_alloc(struct device *dev,
>
> if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size,
> dev_name(dev))) {
> - dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n",
> + dev_warn(dev, "could not reserve region [%pa:0x%zx]\n",
> &pmem->phys_addr, pmem->size);
> return ERR_PTR(-EBUSY);
> }
For the pmem part:
Acked-by: Ross Zwisler <ross.zwisler@linux.intel.com>
next prev parent reply other threads:[~2016-03-07 16:08 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-05 7:46 [TRIVIAL PATCH] treewide: Remove unnecessary 0x prefixes before %pa extension uses Joe Perches
2016-03-05 7:46 ` Joe Perches
2016-03-05 7:46 ` Joe Perches
2016-03-05 7:46 ` Joe Perches
2016-03-05 10:02 ` Mauro Carvalho Chehab
2016-03-05 10:02 ` Mauro Carvalho Chehab
2016-03-05 10:02 ` Mauro Carvalho Chehab
2016-03-05 10:02 ` Mauro Carvalho Chehab
2016-03-07 9:52 ` Nicolas Ferre
2016-03-07 9:52 ` Nicolas Ferre
2016-03-07 9:52 ` Nicolas Ferre
2016-03-07 9:52 ` Nicolas Ferre
2016-03-07 12:39 ` Bounine, Alexandre
2016-03-07 12:39 ` Bounine, Alexandre
2016-03-07 14:50 ` Vinod Koul
2016-03-07 14:50 ` Vinod Koul
2016-03-07 14:50 ` Vinod Koul
2016-03-07 14:50 ` Vinod Koul
2016-03-07 16:08 ` Ross Zwisler [this message]
2016-03-07 16:08 ` Ross Zwisler
2016-03-07 16:08 ` Ross Zwisler
2016-03-07 16:08 ` Ross Zwisler
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=20160307160800.GA9185@linux.intel.com \
--to=ross.zwisler@linux.intel.com \
--cc=alexandre.bounine@idt.com \
--cc=bparrot@ti.com \
--cc=dmaengine@vger.kernel.org \
--cc=joe@perches.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-nvdimm@lists.01.org \
--cc=mchehab@osg.samsung.com \
--cc=mporter@kernel.crashing.org \
--cc=nicolas.ferre@atmel.com \
--cc=trivial@kernel.org \
--cc=vinod.koul@intel.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.