All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hanjun Guo <guohanjun@huawei.com>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Will Deacon <will.deacon@arm.com>,
	Catalin Marinas <catalin.marinas@arm.com>
Cc: linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	Ganapatrao Kulkarni <gkulkarni@caviumnetworks.com>,
	Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>,
	Shannon Zhao <shannon.zhao@linaro.org>,
	Steve Capper <steve.capper@linaro.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Robert Richter <rrichter@cavium.com>,
	Hanjun Guo <hanjun.guo@linaro.org>
Subject: [PATCH v3 03/12] acpi, numa: remove duplicate NULL check
Date: Sat, 23 Jan 2016 17:39:18 +0800	[thread overview]
Message-ID: <1453541967-3744-4-git-send-email-guohanjun@huawei.com> (raw)
In-Reply-To: <1453541967-3744-1-git-send-email-guohanjun@huawei.com>

From: Hanjun Guo <hanjun.guo@linaro.org>

The argument "header" for acpi_table_print_srat_entry()
is always checked before the function is called, it's
duplicate to check it again, remove it.

Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org>
---
 drivers/acpi/numa.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c
index e34b5d0..2de6068 100644
--- a/drivers/acpi/numa.c
+++ b/drivers/acpi/numa.c
@@ -125,9 +125,6 @@ EXPORT_SYMBOL(acpi_map_pxm_to_online_node);
 static void __init
 acpi_table_print_srat_entry(struct acpi_subtable_header *header)
 {
-	if (!header)
-		return;
-
 	switch (header->type) {
 	case ACPI_SRAT_TYPE_CPU_AFFINITY:
 		{
-- 
1.9.1



WARNING: multiple messages have this Message-ID (diff)
From: guohanjun@huawei.com (Hanjun Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 03/12] acpi, numa: remove duplicate NULL check
Date: Sat, 23 Jan 2016 17:39:18 +0800	[thread overview]
Message-ID: <1453541967-3744-4-git-send-email-guohanjun@huawei.com> (raw)
In-Reply-To: <1453541967-3744-1-git-send-email-guohanjun@huawei.com>

From: Hanjun Guo <hanjun.guo@linaro.org>

The argument "header" for acpi_table_print_srat_entry()
is always checked before the function is called, it's
duplicate to check it again, remove it.

Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org>
---
 drivers/acpi/numa.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c
index e34b5d0..2de6068 100644
--- a/drivers/acpi/numa.c
+++ b/drivers/acpi/numa.c
@@ -125,9 +125,6 @@ EXPORT_SYMBOL(acpi_map_pxm_to_online_node);
 static void __init
 acpi_table_print_srat_entry(struct acpi_subtable_header *header)
 {
-	if (!header)
-		return;
-
 	switch (header->type) {
 	case ACPI_SRAT_TYPE_CPU_AFFINITY:
 		{
-- 
1.9.1

WARNING: multiple messages have this Message-ID (diff)
From: Hanjun Guo <guohanjun@huawei.com>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Will Deacon <will.deacon@arm.com>,
	Catalin Marinas <catalin.marinas@arm.com>
Cc: <linux-acpi@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>,
	Ganapatrao Kulkarni <gkulkarni@caviumnetworks.com>,
	Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>,
	Shannon Zhao <shannon.zhao@linaro.org>,
	"Steve Capper" <steve.capper@linaro.org>,
	Mark Rutland <mark.rutland@arm.com>,
	"Robert Richter" <rrichter@cavium.com>,
	Hanjun Guo <hanjun.guo@linaro.org>
Subject: [PATCH v3 03/12] acpi, numa: remove duplicate NULL check
Date: Sat, 23 Jan 2016 17:39:18 +0800	[thread overview]
Message-ID: <1453541967-3744-4-git-send-email-guohanjun@huawei.com> (raw)
In-Reply-To: <1453541967-3744-1-git-send-email-guohanjun@huawei.com>

From: Hanjun Guo <hanjun.guo@linaro.org>

The argument "header" for acpi_table_print_srat_entry()
is always checked before the function is called, it's
duplicate to check it again, remove it.

Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org>
---
 drivers/acpi/numa.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa.c
index e34b5d0..2de6068 100644
--- a/drivers/acpi/numa.c
+++ b/drivers/acpi/numa.c
@@ -125,9 +125,6 @@ EXPORT_SYMBOL(acpi_map_pxm_to_online_node);
 static void __init
 acpi_table_print_srat_entry(struct acpi_subtable_header *header)
 {
-	if (!header)
-		return;
-
 	switch (header->type) {
 	case ACPI_SRAT_TYPE_CPU_AFFINITY:
 		{
-- 
1.9.1

  parent reply	other threads:[~2016-01-23  9:44 UTC|newest]

Thread overview: 100+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-23  9:39 [PATCH v3 00/12] ACPI NUMA support for ARM64 Hanjun Guo
2016-01-23  9:39 ` Hanjun Guo
2016-01-23  9:39 ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 01/12] acpi, numa: Use pr_fmt() instead of printk Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 02/12] acpi, numa: Replace ACPI_DEBUG_PRINT() with pr_debug() Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` Hanjun Guo [this message]
2016-01-23  9:39   ` [PATCH v3 03/12] acpi, numa: remove duplicate NULL check Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 04/12] acpi, numa: introduce ACPI_HAS_NUMA_ARCH_FIXUP Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23 10:25   ` Robert Richter
2016-01-23 10:25     ` Robert Richter
2016-01-23 10:25     ` Robert Richter
2016-01-24  4:56     ` Hanjun Guo
2016-01-24  4:56       ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 05/12] arm64, acpi, numa: NUMA support based on SRAT and SLIT Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-25 10:21   ` Robert Richter
2016-01-25 10:21     ` Robert Richter
2016-01-25 10:21     ` Robert Richter
2016-01-27  7:12     ` Hanjun Guo
2016-01-27  7:12       ` Hanjun Guo
2016-01-27  7:12       ` Hanjun Guo
2016-01-27 14:01       ` Robert Richter
2016-01-27 14:01         ` Robert Richter
2016-01-27 14:01         ` Robert Richter
2016-01-28  3:16         ` Hanjun Guo
2016-01-28  3:16           ` Hanjun Guo
2016-01-28  3:16           ` Hanjun Guo
2016-02-01 18:09   ` Robert Richter
2016-02-01 18:09     ` Robert Richter
2016-02-01 18:09     ` Robert Richter
2016-02-02 11:30     ` Hanjun Guo
2016-02-02 11:30       ` Hanjun Guo
2016-02-02 17:00       ` Lorenzo Pieralisi
2016-02-02 17:00         ` Lorenzo Pieralisi
2016-03-02 14:10     ` Matthias Brugger
2016-03-02 14:10       ` Matthias Brugger
2016-03-02 14:10     ` Matthias Brugger
2016-03-02 14:10       ` Matthias Brugger
2016-03-02 14:08   ` Matthias Brugger
2016-03-02 14:08     ` Matthias Brugger
2016-03-10  9:50     ` Hanjun Guo
2016-03-10  9:50       ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 06/12] acpi, numa: Enable ACPI based NUMA on ARM64 Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-29 16:37   ` Robert Richter
2016-01-29 16:37     ` Robert Richter
2016-01-29 16:37     ` Robert Richter
2016-01-23  9:39 ` [PATCH v3 07/12] acpi, numa: move acpi_numa_slit_init() to common place Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 08/12] arm64, numa: rework numa_add_memblk() Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-25  9:34   ` Robert Richter
2016-01-25  9:34     ` Robert Richter
2016-01-25  9:34     ` Robert Richter
2016-01-27  6:20     ` Hanjun Guo
2016-01-27  6:20       ` Hanjun Guo
2016-01-27  6:20       ` Hanjun Guo
2016-03-09 12:27   ` Robert Richter
2016-03-09 12:27     ` Robert Richter
2016-03-09 12:27     ` Robert Richter
2016-03-10 10:10     ` Hanjun Guo
2016-03-10 10:10       ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 09/12] x86, acpi, numa: cleanup acpi_numa_processor_affinity_init() Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 10/12] acpi, numa: move bad_srat() and srat_disabled() to common place Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 11/12] acpi, numa: remove unneeded acpi_numa=1 Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39 ` [PATCH v3 12/12] acpi, numa: reuse acpi_numa_memory_affinity_init() Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-23  9:39   ` Hanjun Guo
2016-01-25 10:26   ` Robert Richter
2016-01-25 10:26     ` Robert Richter
2016-01-25 10:26     ` Robert Richter
2016-01-27  6:15     ` Hanjun Guo
2016-01-27  6:15       ` Hanjun Guo
2016-01-27  6:15       ` Hanjun Guo
2016-01-27 14:18       ` Robert Richter
2016-01-27 14:18         ` Robert Richter
2016-01-27 14:18         ` Robert Richter
2016-01-28  2:48         ` Hanjun Guo
2016-01-28  2:48           ` Hanjun Guo
2016-01-28  2:48           ` Hanjun Guo
2016-01-28 13:31           ` Robert Richter
2016-01-28 13:31             ` Robert Richter
2016-01-28 13:31             ` Robert Richter

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=1453541967-3744-4-git-send-email-guohanjun@huawei.com \
    --to=guohanjun@huawei.com \
    --cc=Lorenzo.Pieralisi@arm.com \
    --cc=catalin.marinas@arm.com \
    --cc=gkulkarni@caviumnetworks.com \
    --cc=hanjun.guo@linaro.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=rjw@rjwysocki.net \
    --cc=rrichter@cavium.com \
    --cc=shannon.zhao@linaro.org \
    --cc=steve.capper@linaro.org \
    --cc=will.deacon@arm.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.