On 09/24/2013 03:11 PM, Mark Salter wrote:
On Tue, 2013-09-24 at 14:45 -0600, al.stone@linaro.org wrote:
From: Al Stone ahs3@redhat.com
This patch just makes it painfully obvious when the RSDP cannot be found.
Signed-off-by: Al Stone al.stone@linaro.org
drivers/acpi/acpica/tbxface.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/acpi/acpica/tbxface.c b/drivers/acpi/acpica/tbxface.c index ad11162..41aebe6 100644 --- a/drivers/acpi/acpica/tbxface.c +++ b/drivers/acpi/acpica/tbxface.c @@ -137,6 +137,7 @@ acpi_initialize_tables(struct acpi_table_desc * initial_table_array, if (!rsdp_address) { return_ACPI_STATUS(AE_NOT_FOUND); }
- WARN_ON(rsdp_address == 0);
Isn't the WARN_ON unreachable here? Or maybe you wanted to remove the preceding error return.
--Mark
Argh. Right you are. This part of the patch should be ignored, then. This is a leftover from some previous debug code that should not have been submitted. Sorry about the noise...