diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2017-05-23 13:03:26 -0400 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2017-05-29 05:27:25 -0400 |
commit | 1d7f2cdd56d45254be73301d08c65efa8041d3f3 (patch) | |
tree | 354d6ee00de3794f10e544c0a366dfd9c9323dfb /drivers/pnp | |
parent | 2a56e9195dfb59548cc8b32e8e8d0eef4b65f7eb (diff) |
PNP / ACPI: join strings back for better maintenance
Simply join string literals back for better maintenance and debugging.
No functional changes intended.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pnp')
-rw-r--r-- | drivers/pnp/pnpacpi/rsparser.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/drivers/pnp/pnpacpi/rsparser.c b/drivers/pnp/pnpacpi/rsparser.c index d0358cfb316e..39af2a0a270e 100644 --- a/drivers/pnp/pnpacpi/rsparser.c +++ b/drivers/pnp/pnpacpi/rsparser.c | |||
@@ -149,8 +149,8 @@ static int vendor_resource_matches(struct pnp_dev *dev, | |||
149 | uuid_len == sizeof(match->data) && | 149 | uuid_len == sizeof(match->data) && |
150 | memcmp(uuid, match->data, uuid_len) == 0) { | 150 | memcmp(uuid, match->data, uuid_len) == 0) { |
151 | if (expected_len && expected_len != actual_len) { | 151 | if (expected_len && expected_len != actual_len) { |
152 | dev_err(&dev->dev, "wrong vendor descriptor size; " | 152 | dev_err(&dev->dev, |
153 | "expected %d, found %d bytes\n", | 153 | "wrong vendor descriptor size; expected %d, found %d bytes\n", |
154 | expected_len, actual_len); | 154 | expected_len, actual_len); |
155 | return 0; | 155 | return 0; |
156 | } | 156 | } |
@@ -204,9 +204,8 @@ static acpi_status pnpacpi_allocated_resource(struct acpi_resource *res, | |||
204 | * one interrupt, we won't be able to re-encode it. | 204 | * one interrupt, we won't be able to re-encode it. |
205 | */ | 205 | */ |
206 | if (pnp_can_write(dev)) { | 206 | if (pnp_can_write(dev)) { |
207 | dev_warn(&dev->dev, "multiple interrupts in " | 207 | dev_warn(&dev->dev, |
208 | "_CRS descriptor; configuration can't " | 208 | "multiple interrupts in _CRS descriptor; configuration can't be changed\n"); |
209 | "be changed\n"); | ||
210 | dev->capabilities &= ~PNP_WRITE; | 209 | dev->capabilities &= ~PNP_WRITE; |
211 | } | 210 | } |
212 | } | 211 | } |
@@ -347,8 +346,8 @@ static __init void pnpacpi_parse_ext_irq_option(struct pnp_dev *dev, | |||
347 | if (p->interrupts[i] < PNP_IRQ_NR) | 346 | if (p->interrupts[i] < PNP_IRQ_NR) |
348 | __set_bit(p->interrupts[i], map.bits); | 347 | __set_bit(p->interrupts[i], map.bits); |
349 | else | 348 | else |
350 | dev_err(&dev->dev, "ignoring IRQ %d option " | 349 | dev_err(&dev->dev, |
351 | "(too large for %d entry bitmap)\n", | 350 | "ignoring IRQ %d option (too large for %d entry bitmap)\n", |
352 | p->interrupts[i], PNP_IRQ_NR); | 351 | p->interrupts[i], PNP_IRQ_NR); |
353 | } | 352 | } |
354 | } | 353 | } |
@@ -949,8 +948,9 @@ int pnpacpi_encode_resources(struct pnp_dev *dev, struct acpi_buffer *buffer) | |||
949 | case ACPI_RESOURCE_TYPE_EXTENDED_ADDRESS64: | 948 | case ACPI_RESOURCE_TYPE_EXTENDED_ADDRESS64: |
950 | case ACPI_RESOURCE_TYPE_GENERIC_REGISTER: | 949 | case ACPI_RESOURCE_TYPE_GENERIC_REGISTER: |
951 | default: /* other type */ | 950 | default: /* other type */ |
952 | dev_warn(&dev->dev, "can't encode unknown resource " | 951 | dev_warn(&dev->dev, |
953 | "type %d\n", resource->type); | 952 | "can't encode unknown resource type %d\n", |
953 | resource->type); | ||
954 | return -EINVAL; | 954 | return -EINVAL; |
955 | } | 955 | } |
956 | resource++; | 956 | resource++; |