diff options
author | Colin Ian King <colin.king@canonical.com> | 2016-01-05 05:57:11 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-01-05 07:40:05 -0500 |
commit | 45288978859119c019eff93f0cb838a0de100bcc (patch) | |
tree | a9aa4d1270af3cc3b9695401016a5a9f8feede70 /drivers/acpi/pci_irq.c | |
parent | 0971686954f9cdde502c7bce4794c5ad53ea2e66 (diff) |
ACPI, PCI, irq: remove redundant check for null string pointer
source is decleared as a 4 byte char array in struct acpi_pci_routing_table
so !prt->source is a redundant null string pointer check. Detected with
smatch:
drivers/acpi/pci_irq.c:134 do_prt_fixups() warn: this array is probably
non-NULL. 'prt->source'
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi/pci_irq.c')
-rw-r--r-- | drivers/acpi/pci_irq.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drivers/acpi/pci_irq.c b/drivers/acpi/pci_irq.c index c9336751e5e3..d30184c7f3bc 100644 --- a/drivers/acpi/pci_irq.c +++ b/drivers/acpi/pci_irq.c | |||
@@ -131,9 +131,6 @@ static void do_prt_fixups(struct acpi_prt_entry *entry, | |||
131 | quirk = &prt_quirks[i]; | 131 | quirk = &prt_quirks[i]; |
132 | 132 | ||
133 | /* All current quirks involve link devices, not GSIs */ | 133 | /* All current quirks involve link devices, not GSIs */ |
134 | if (!prt->source) | ||
135 | continue; | ||
136 | |||
137 | if (dmi_check_system(quirk->system) && | 134 | if (dmi_check_system(quirk->system) && |
138 | entry->id.segment == quirk->segment && | 135 | entry->id.segment == quirk->segment && |
139 | entry->id.bus == quirk->bus && | 136 | entry->id.bus == quirk->bus && |